geen submenu

Hoezo half Engels en half Nederlands?
Parent en ParentID lijkt mij toch juist logische benamingen? In mijn ogen is dit juist vragen om om problemen.

Je hoeft geen onderscheid te maken tussen de UL's als de buitenste een class moet hebben, immers:
- je zou dit altijd als argument mee kunnen geven aan de functie ($class=''), en als deze parameter niet-leeg is zou je een bijbehorende property weg kunnen schrijven naar de (buitenste) UL-tag
- je zou de buitenste UL-tag buiten de functie kunnen trekken, je hoeft eigenlijk alleen maar een UL-tag te openen als je de recursie induikt (een element children / subelementen heeft)
- je zou een omvattende div kunnen maken die een class heeft en op die manier alles stylen; en dit doe je in het bovenstaande fragment al, dus waarom heb je dan die art-hmenu nodig?

We kunnen ook niet echt zien wat er misgaat, omdat we de inhoud van $menuData niet kennen.

En mogelijk:
- zit er iets in de verbanden van de records mis zodat het geen goede boomstructuur betreft
- gaat er iets mis met de HTML-output (hoe ziet de uiteindelijke HTML code er uit?)
- zit er iets mis in de CSS-opmaak zodat niet alles goed weergegeven wordt?

Voor het oplossen van deze puzzel hebben we meer informatie nodig, zodat we kunnen nabootsen wat er misgaat.

Een iets andere aanpak lost al het bovenstaande op: start enkel een UL als je subitems hebt, en duik dan de recursie in. Dus bijvoorbeeld als volgt:

<?php
// items uit database - parent 0 = root van de te bouwen boom
$myItems = array(
    1 => array('parent' => 0, 'data' => '1 een'),
    2 => array('parent' => 1, 'data' => '1.1 een punt een'),
    3 => array('parent' => 2, 'data' => '1.1.1 een punt een punt een'),
    4 => array('parent' => 1, 'data' => '1.2 een punt twee'),
    5 => array('parent' => 0, 'data' => '2 twee'),
    6 => array('parent' => 5, 'data' => '2.1 twee punt een'),
    7 => array('parent' => 0, 'data' => '3 drie'),
);

// initiele boom
$myTree = array(
    0 => array(
        'data'      => 'root',
        'children'  => array(),
    ),
);

// bouw boom mbv items
foreach ($myItems as $id => $data) {
    $myTree[$id] = array(
        'data'      => $data['data'],
        'children'  => array(),
    );
    $myTree[$data['parent']]['children'][] = $id;
}

// weergavefunctie
function displayTree($tree, $index=0) {
    foreach ($tree[$index]['children'] as $child) {
        ?><li><?php echo $tree[$child]['data']; ?><?php
            if (count($tree[$child]['children']) > 0) {
                ?><ul><?php
                    displayTree($tree, $child);
                ?></ul><?php
            }
        ?></li><?php
    }
}

// weergave
if (count($myItems) > 0) {
    ?><ul class="whatever"><?php
        displayTree($myTree);
    ?></ul><?php
}
?>

Dit levert:

<ul class="whatever">
    <li>1 een<ul>
        <li>1.1 een punt een<ul>
            <li>1.1.1 een punt een punt een</li>
        </ul></li>
        <li>1.2 een punt twee</li>
    </ul></li>
    <li>2 twee<ul>
        <li>2.1 twee punt een</li>
    </ul></li>
    <li>3 drie</li>
</ul>

Thomas van den Heuvel, bedankt voor je reactie.
Maar vooral bedankt voor de uitleg. En voor het nieuwe alternatief dat je mij gaf.
Ik vroeg me alleen af, of dat ik door het begin niet gelijk gelimiteerd ben.
(zie de code onderaan)
Want ik wil juist zo open mogelijk blijven.
Ik heb bewust het menu in een database gezet om niet steeds in de code het menu aan te moeten passen. Dit biedt mij in de toekomst meer vrijheid.

-Arieën-
Half Engels half Nederlands komt door dat ik in het Nederlands ben begonnen met het opzetten van de database. Ik heb al wel meer code die met de database te maken heeft, en ik had geen zien om ook die code aan te passen. Gewoon om het werkende deel werkend te houden en me te kunnen focussen op dit onderdeel.

<?php
// items uit database - parent 0 = root van de te bouwen boom
$myItems = array(
    1 => array('parent' => 0, 'data' => '1 een'),
    2 => array('parent' => 1, 'data' => '1.1 een punt een'),
    3 => array('parent' => 2, 'data' => '1.1.1 een punt een punt een'),
    4 => array('parent' => 1, 'data' => '1.2 een punt twee'),
    5 => array('parent' => 0, 'data' => '2 twee'),
    6 => array('parent' => 5, 'data' => '2.1 twee punt een'),
    7 => array('parent' => 0, 'data' => '3 drie'),
);

Uiteraard trek je de data uit je database voor het bouwen van $myItems, het bovenstaande was enkel een test-set om de werking te illustreren :).

Als je tijdens het bouwen van deze set het stramien:

$myItems[<record id>] = array(
    'parent' => <parent id>,
    'data'   => '<label tekst>',
    // en wat je hier nog meer wilt toevoegen, of breng dit in 'data' onder
);

hanteert dan werkt de rest "vanzelf".

Frits van Leeuwen op 02/02/2020 00:12:49

Half Engels half Nederlands komt door dat ik in het Nederlands ben begonnen met het opzetten van de database. Ik heb al wel meer code die met de database te maken heeft, en ik had geen zien om ook die code aan te passen. Gewoon om het werkende deel werkend te houden en me te kunnen focussen op dit onderdeel.

Het kan geen kwaad om in hetzelfde stramien te werken qua naamgevingen. Dan kunnen andere mensen ook direct aan je code sleutelen of mee-debuggen zonder de hele applicatie door te worstelen. Het ergste voorbeeld wat ik ooit tegen kwam was iemand die zijn code in het Esperanto had benoemd. Als je dat niet kent, en dingen lijken op elkaar, dan ga je ze echt door elkaar halen.

Thoma's van den Heuvel, Ik probeer jouw code nu, maar daarvoor moet ik deze code om te zetten naar wat ik nodig heb. Maar ik kom er niet uit. Wat doe ik fout?

// items uit database - parent 0 = root van de te bouwen boom
$myItems = array(
    1 => array('parent' => 0, 'data' => '1 een'),
    2 => array('parent' => 1, 'data' => '1.1 een punt een'),
    3 => array('parent' => 2, 'data' => '1.1.1 een punt een punt een'),
    4 => array('parent' => 1, 'data' => '1.2 een punt twee'),
    5 => array('parent' => 0, 'data' => '2 twee'),
    6 => array('parent' => 5, 'data' => '2.1 twee punt een'),
    7 => array('parent' => 0, 'data' => '3 drie'),
);

$query="SELECT menu_id, menu_tekst, menu_waarde, menu_sub_van, menu_link, menu_reg_date FROM tabel_menu ORDER BY menu_sub_van, menu_waarde";
$result = mysqli_query($connect, $query);

$myItems = array(
    'parent' => array(),
    'data' => array()
);

foreach ( mysqli_fetch_assoc($result) as $row )
{
   $myItems = array (
		$row['menu_id'] => array(
			'parent' => $row['menu_sub_van'],
			'data' => $row['menu_tekst'],
		),
	);
}

Die hardcoded array moet je overigens wel vertalen naar een set data uit je database. Nu overschrijf je met de array voor je structuur je bestaande data.

- Arien - Ik snap je niet helemaal.
De array die Thomas van den Hoeven in zijn code had geplaatst moet natuurlijk worden omgebouwd naar mijn database en informatie. Daarom heb ik bij de query geen * gebruikt, maar alle velden genoemd die ik heb.
De query is goed, en die wil ik gebruiken voor het opbouwen van een menu. Maar deze code gaat gewoon fout.
Ik krijg een aantal regels die dit zeggen:
Warning: Illegal string offset 'menu_sub_van' in /.../menu.php on line 13
Warning: Illegal string offset 'menu_tekst' in /.../menu.php on line 14

Je kan moeilijk een database in een praktisch werkend voorbeeld plaatsen. Dus daarom heeft Thomas wat data hardcoded in een array geplaatst. Maar in de echte praktijk is een database natuurlijk veel handiger ;-)

Dus vergeet die array $myItems uiteindelijk, en haal alles uit de database op.

Trouwens, je zit nu wel zijn voorbeeld nog eens volledig de vernieling in te helpen. Want hij heeft de als voorbeeld gemaakte $myItem, en de vaste $myTree, en die laatste noem jij ook vrolijk $myItems. Dan kan je verwachten dat de code opeens heel vreemd gaat doen.

Er is in ieder geval een verschil tussen myItems en myTree.

myItems zijn in wezen de records uit je database die je tijdelijk in een array onderbrengt.
myTree is vervolgens de structuur die je bouwt door het doorlopen van myItems.

Je zou de twee natuurlijk ook kunnen combineren door on-the-fly de boom te bouwen en enkel hierin de informatie weg te schrijven die je uit de database opvraagt. Daarin ben je helemaal vrij. Maar het kan in sommige gevallen handig zijn om data en structuur gescheiden te houden. Stel dat je de data wilt (her)gebruiken om meer bomen te kunnen bouwen oid.

Anyway, ik ben nog nooit de vorm "foreach ( mysqli_fetch_assoc($result) as $row )" tegengekomen en weet ook niet zeker of dat gaat werken.

Een overweging om mee te nemen in je ontwerp is de volgende: normaal gesproken zullen items op volgorde aangemaakt worden. Dus het kan dan nooit voorkomen dat een child id een lager nummer heeft dan de parent. Maar dit zou dus over tijd kunnen veranderen op het moment dat je met elementen in de boom gaat schuiven. Bij het bouwen van de boom is het in ieder geval belangrijk dat je een zodanige volgorde hanteert dat je altijd de parents ophaalt voor de children, anders wordt het bouwen van de boom nogal problematisch :p. Daartoe wordt meestal extra "meta data" opgeslagen over de boom. Hiertoe wordt meestal een redundant veld "depth" of "level" of wat dan ook opgeslagen. Als je in je query je resultaten sorteert op dit veld dan garandeert dit dat de items voldoende op volgorde staan om de boom te kunnen bouwen. Daarnaast komt het ook voor dat je kunt schuiven met elementen op een zeker niveau in de boom, dus ook een volgorde veld, "order" ofzo, kan (zeer) handig zijn.

Anyhow, als dit allemaal (nog) niet op jouw situatie van toepassing is (maar het is nog steeds verstandig om te overwegen om zoiets in te bouwen), dan zou je gewoon met een while-loop je boom kunnen bouwen zoals je nu doet, zonder al deze sorteer-criteria (maar expliciet een ORDER BY toevoegen kan natuurlijk nooit kwaad). Kijk hierbij goed naar de opbouw van myTree. Twee onderdelen zijn redelijk cruciaal:
- bij het aanmaken van elk element maak je bij dit element ook een children array aan, dit zit niet in jouw bovenstaande loop?
- tegelijkertijd voeg je het huidige element toe als child van de parent, laat dit even bezinken :p, op deze manier klik je de elementen aan elkaar als boom en om deze reden is het dus belangrijk dat je de items in de goede volgorde ophaalt, anders bestaan parents mogelijk nog niet in myTree

Met enige aanpassingen in jouw code zou je met behulp van mijn voorbeeld dit alles redelijk eenvoudig aan de praat moeten krijgen.