Forum PHP.pl

Cześć,
stworzyłem pewną funkcję która pobiera dane z bazy i na ich podstawie buduje menu, jak zrobić żeby zwracała mi kod gdy użyję jej np. tak:

[PHP] pobierz, plaintext 
$top = Menu();
[PHP] pobierz, plaintext 

funkcja:

[PHP] pobierz, plaintext 
function Menu ()
{
 
$db['name']="zs4tychy";
$db['prefix']="new_";
$page = 1;
$link = mysql_connect("".$db['host']."", "".$db['user']."", "".$db['pass']."") or die ("Nie można się połączyć");
mysql_select_db ("".$db['name']."") or die ("Nie mozna wybrać bazy danych");
 
$query = "SELECT * FROM ".$db['prefix']."menu WHERE  submenu='0' AND page='".$page."'  ORDER BY position DESC";
$result = mysql_query ($query) or die ("Zapytanie zakończone niepowodzeniem1");
 
while($rek = mysql_fetch_array($result))
{
$return = '<ul>';
$return .= '<li><a href="'.$rek['link'].'">'.$rek['name'].'</a></li>';
if ($rek['parent'] == TRUE)
{
        $query = "SELECT * FROM ".$db['prefix']."menu WHERE pid='".$rek['id']."' AND submenu='1' AND page='".$page."'  ORDER BY position DESC";
        $result = mysql_query ($query) or die ("Zapytanie zakończone niepowodzeniem2");
        if ($rek['parent'] == TRUE)
        {
        while($rek = mysql_fetch_array($result))
        {
            $return .= '<ul>';
            $return .= '<li><a href="'.$rek['link'].'">'.$rek['name'].'</a></li>';
            if ($rek['parent'] == TRUE)
            {
            $query = "SELECT * FROM ".$db['prefix']."menu WHERE pid='".$rek['id']."' AND submenu='2' AND page='".$page."'  ORDER BY position DESC";    
            $result = mysql_query ($query) or die ("Zapytanie zakończone niepowodzeniem3");
            while($rek = mysql_fetch_array($result))
            {
                $return .= '<ul>';
                $return .= '<li><a href="'.$rek['link'].'">'.$rek['name'].'</a></li>';
                $return .= '</ul>';            
            }
 
            }
            $return .= '</ul>';
        }
        }
}        
$return .= '</ul>';    
}
 
mysql_free_result($result); 
mysql_close($link);
return $return;
}
[PHP] pobierz, plaintext 

Tak mi zwraca tylko 1 pozycję...

Ten post edytował patryk9200 27.08.2009, 19:33:56

Powiem tylko 2 słowa: nadpisywanie zmiennych

Wywalasz do bazy wiele zapytań, ale skrypt szybko działa bo w każdej pętli do zmiennej $result i $rek wpisujesz wyniki kolejnych podzapytań oraz przebiegów pętli, co sprawia, że dochodzi do najgłębszego podzapytania, wykonuje je i we wszystkich nadrzędnych do niego WHILE następuje wyjście z pętli. Wystarczy, że zmienne $result i $rek ponazywasz jak tam chcesz, byle na różnych poziomach zagłębień były inne i wszystko zadziała jak trzeba.

Ten post edytował thek 27.08.2009, 00:39:26

[PHP] pobierz, plaintext 
$result = mysql_query ($query) or die ("Zapytanie zakończone niepowodzeniem1");
 
$return = '';
while($rek = mysql_fetch_array($result))
{
$return .= '<ul>';
 
[PHP] pobierz, plaintext 

Oraz to co powiedział, kolega wyżej.

Ten post edytował wookieb 27.08.2009, 07:49:03

eee
co to jest?

[PHP] pobierz, plaintext 
$link = mysql_connect("".$db['host']."", "".$db['user']."", "".$db['pass']."") or die ("Nie można się połączyć");
[PHP] pobierz, plaintext 

nie można tak?

[PHP] pobierz, plaintext 
$link = mysql_connect($db['host'], $db['user'], $db['pass']) or die ("Nie można się połączyć");
[PHP] pobierz, plaintext 

?

edit:
nadpisywania nie komentuje, nie będę cię dołował, masz to @up (IMG:style_emoticons/default/winksmiley.jpg)

Ten post edytował Pawel_W 27.08.2009, 08:23:24

Czyszczenie $result nie jest wymagane (IMG:style_emoticons/default/smile.gif) To zmienna widoczna tylko w zasięgu funkcji, więc przy każdym nowym jej wywołaniu jest tworzona na nowo, a więc automatycznie się "resetuje".
Ja się tylko zastanawiam jaki jest sens Tworzenia tylu list... Nie lepiej było zrobić otwarcie listy PRZED pętlą a jej zamknięcie PO? W tej chwili bowiem, nawet gdy nie ma się nic więcej niż menu główne, to tworzy coś takiego:

[HTML] pobierz, plaintext 
<ul>
<li><a href="link">nazwa</a></li>
</ul>
<ul>
<li><a href="link">nazwa</a></li>
</ul>
<ul>
<li><a href="link">nazwa</a></li>
</ul> 
[HTML] pobierz, plaintext 

Ten post edytował thek 27.08.2009, 08:25:00

Zainteresuj się opcją http://nospor.pl/grupowanie-wynikow-n35.html ~nospora, zamiast tworzyć wiele zapytań.

Przy okazji... Jeśli miałbym się bawić w budowanie wielokrotnie zagnieżdżonych menu to raczej bym tabelę w bazie zmienił. Jedna tabela z polami: id_węzła, nazwa, root (domyślnie 0) i inne jakie Ci potrzebne. Dlaczego?
Bo wszystko co ma root = 0 jest elementem głównego menu, a to co ma tam liczbę jest submenu podpiętym do pola o id równym id_węzła. Można więc zrobić nie wiem jak głębokiee zagnieżdżenia. Co prawda odbywa się to kosztem pisania kodu nieco bardziej skomplikowanego (rekurencja), ale kto powiedział, że życie jest łatwe? (IMG:style_emoticons/default/tongue.gif)

Ten post edytował thek 27.08.2009, 09:17:45

jeśli chodzi o tabelę root, to taką samą funkcję spełnia submenu, domyślnie ma wartość 0, a menu jest tak skonstruowane, że moge zrobić maksymalnie 3 zagnieżdżenia dlatego submenu może mieć tylko wartości 1,2 lub 3. tabela pid to id rodzica submenu.

poprawiłem trochę funkcje bo jak ktoś wcześniej zwrócił uwagę zawierała błąd, próbowałem zrobić tak, że każdy stopień miał swoje zmienne z których na koniec wartości zwracała funkcja, ale mi to nie działało.
Teraz mam coś takiego:

[PHP] pobierz, plaintext 
function Menu ()
{
$db['host']="sql.nazwa.pl";
$db['user']="z";
$db['pass']="n";
$db['name']="y";
$db['prefix']="new_";
$page = 1;
$link = mysql_connect("".$db['host']."", "".$db['user']."", "".$db['pass']."") or die ("Nie można się połączyć");
mysql_select_db ("".$db['name']."") or die ("Nie mozna wybrać bazy danych");
 
$query = "SELECT * FROM ".$db['prefix']."menu WHERE  submenu='0' AND page='".$page."'  ORDER BY position DESC";
$result = mysql_query ($query) or die ("Zapytanie zakończone niepowodzeniem");
 
while($rek = mysql_fetch_array($result))
{
return  '<ul>';
 
if ($rek['parent'] == TRUE)
{
$poziom1= '<li><span class="dir">'.$rek['name'].'</span><ul>';
        if ($rek['parent'] == TRUE)
        {
        return   '<li><span class="dir">'.$rek['name'].'</span><ul>';
        $query = "SELECT * FROM ".$db['prefix']."menu WHERE pid='".$rek['id']."' AND submenu='1' AND page='".$page."'  ORDER BY position DESC";
        $result = mysql_query ($query) or die ("Zapytanie zakończone niepowodzeniem");
        while($rek = mysql_fetch_array($result))
        {
            if ($rek['parent'] == TRUE)
            {
            return '<li><span class="dir">'.$rek['name'].'</span><ul>';
            $query = "SELECT * FROM ".$db['prefix']."menu WHERE pid='".$rek['id']."' AND submenu='2' AND page='".$page."'  ORDER BY position DESC";    
            $result = mysql_query ($query) or die ("Zapytanie zakończone niepowodzeniem");
            while($rek = mysql_fetch_array($result))
            {
                return '<li><a href="'.$rek['link'].'">'.$rek['name'].'</a></li>';        
            }
            return '</ul></li>';
            } else {return '<li><a href="'.$rek['link'].'">'.$rek['name'].'</a></li>';}
        }
        return '</ul></li>';
        } else {return '<li><a href="'.$rek['link'].'">'.$rek['name'].'</a></li>';}
$poziom1e= '</ul></li>';        
}    else
{
$poziom1 '<li><a href="'.$rek['link'].'">'.$rek['name'].'</a></li>';
}    
return $poziom1.$poziom1e;
return = '</ul>';    
}
 
mysql_free_result($result); 
mysql_close($link);
}
[PHP] pobierz, plaintext 

ale nie wiem dlaczego ale żadne pętle sie nie wykonują, strukturę bazy mam taką:

id pid page link name submenu parent position


1 1 1 # Treść 0 0 1


2 1 1 # Artykuły 1 1 1


3 2 1 # Najnowsze 2 1 1


4 1 1 newsy.php News 0 0 1

Proszę poprawić tytuł na bardziej sensowny.

bo return konczy działanie funkcji...

Jak byk napisałem... zmień nie tylko $result, ale i $rek... Co pętlę bowiem wiersze masz wpisywane do $rek... Je też musisz odróżnić od siebie bo one także się nadpisują.
Ja w przykładzie nie mam tabeli root. To tylko jedna kolumna na podstawie jakiej ustalam jaki węzeł z jakim węzłem się łączy. Nie potrzebuję do tego 3 kolumn (IMG:style_emoticons/default/smile.gif)
1 główna 0
2 Forum 0
3 Galeria 0
4 Galeria Ani 3
5 Galeria Basi 3
Jak widzisz taki układ oznacza dla mnie tyle, że węzły 4 i 5 są dziećmi węzłe 3 i mam z nich dla węzła 3 zrobić submenu. W ten sposób mogę zagnieżdżać na dowolną głębokość submenu, tworząc z rekordów drzewo. Czyli mam o wiele mniejszym kosztem objętości bazy to co Ty o ile dobrze widzę.

Return jak wspomniano kończy działanie funkcji więc nic nie zwrócisz w zasadzie poza tagiem otwierającym listę (IMG:style_emoticons/default/winksmiley.jpg)

napisałem funkcję od nowa i teraz zwraca mi tylko "1" i nic więcej co jest źle?

[PHP] pobierz, plaintext 
function Menu ()
{
$db['host']="";
$db['user']="";
$db['pass']="";
$db['name']="";
$db['prefix']="new_";
$page = 1;
 
$menuStart = '<div><div id="ContentBelka">'.$title.'</div><div id="ContentKontener">
<div style="height:258px">
<ul id="nav" class="dropdown dropdown-vertical" style="width:140px; z-index: 90;">';
 
$link = mysql_connect("".$db['host']."", "".$db['user']."", "".$db['pass']."") or die ("Nie można się połączyć");
mysql_select_db ("".$db['name']."") or die ("Nie mozna wybrać bazy danych");
$query = "SELECT * FROM ".$db['prefix']."menu WHERE submenu='0' AND page='".$page."' ORDER BY position DESC";
$result = mysql_query ($query) or die ("Zapytanie zakończone niepowodzeniem");
while($rek1 = mysql_fetch_array($result))
{
    if ($rek1['parent'] == TRUE)
    {    
        $menu1 = '<li><span class="dir">'.$rek1['name'].'</span><ul>';
        $query = "SELECT * FROM ".$db['prefix']."menu WHERE pid='".$rek1['id']."' AND page='".$page."' ORDER BY position DESC";
        $result = mysql_query ($query) or die ("Zapytanie zakończone niepowodzeniem");
        while($rek2 = mysql_fetch_array($result))
        {
        if ($rek2['parent'] == TRUE)
        {
            $menu2 = '<li><span class="dir">'.$rek2['name'].'</span><ul>';
            $query = "SELECT * FROM ".$db['prefix']."menu WHERE pid='".$rek2['id']."' AND page='".$page."' ORDER BY position DESC";
            $result = mysql_query ($query) or die ("Zapytanie zakończone niepowodzeniem");
            while($rek3 = mysql_fetch_array($result))
            {
                $menu3 =    '<li><a href="'.$rek3['link'].'">'.$rek3['name'].'</a></li>';
            }
            $menu2e = '</ul></li>';
        } else
        { 
            $menu2 .= '<li><a href="'.$rek2['link'].'">'.$rek2['name'].'</a></li>';
        }
 
        }
        $menu1e = '</ul></li>';    
    } else
    { $menu1 .= '<li><a href="'.$rek1['link'].'">'.$rek1['name'].'</a></li>';
    }
}
 
$menuEnd = '</ul></div></div>';
$t = $menuStart.$menu1.$menu2.$menu3.$menu2e.$menu1e.$menuEnd;
return 
mysql_free_result($result);
mysql_close($link);
 
}
[PHP] pobierz, plaintext 

czy dobrze zrobiłem z tymi zmiennymi?

No a CO ZWRACASZ?

no dobra dałem

[PHP] pobierz, plaintext 
return $t;
[PHP] pobierz, plaintext 

i zwróciło mi coś takiego:

[HTML] pobierz, plaintext 
<div id="ContentBelka">Menu</div><div id="ContentKontener">
<div style="height: 258px;">
<ul id="nav" class="dropdown dropdown-vertical" style="width: 140px; z-index: 90;"><li><span class="dir">Tre��</span><ul><li><span class="dir">Tre��</span><ul><li><a href="newsy.php">News</a></li></ul></li></ul></li></ul></div></div>
 
[HTML] pobierz, plaintext 

a tablela w bazie wygląda tak:
id pid page link name submenu parent position


1 1 1 # Treść 0 1 1


2 1 1 # Artykuły 1 1 1


3 2 1 # Najnowsze 2 0 1


4 1 1 newsy.php News 0 0 1

powinna mi się taka struktura budować:

[HTML] pobierz, plaintext 
<ul id="nav" class="dropdown dropdown-horizontal">
<li><a href="./">Strona głowna</a></li>
    <li><span class="dir">Strony tematyczne</span>
        <ul>
            <li><a href="./">Historia</a></li>
            <li><a href="./">Matematyka</a></li>
            <li><span class="dir">Angielski</span>
                <ul>
                    <li><a href="./">począkujacy</a></li>
                    <li><a href="./">John</a></li>
                    <li><a href="./">Michael</a></li>
                    <li><a href="./">Peter</a></li>
                    <li><a href="./">Sarah</a></li>
                </ul>
            </li>
            <li><a href="./">Clients</a></li>
            <li><a href="./">Testimonials</a></li>
            <li><a href="./">Press</a></li>
            <li><a href="./">FAQs</a></li>
        </ul>
    </li>
.....
[HTML] pobierz, plaintext 

widać ze nie dociera do ciebie co pisali koledzy wyżej więc powstawiaj sobie echo w każdą pętlę, która wyświetli ci aktualna zawartość menu, aż dotrzesz do rozwiązania.

Błąd 1: czemu nadal i namiętnie nadpisujesz zmienną $result?
Błąd 2: czemu w bezsensowny sposób tworzysz menu PO wykonaniu wszystkich pętli? W pierwszym swoim kodzie z łączeniem na bieżąco kodu w zmiennej $return było robione to tak jak chciałeś.

Na dogłębne szukanie błędów nie mam teraz głowy, bo za śpiący jestem (IMG:style_emoticons/default/winksmiley.jpg)