nonstop loop

Start med at rette:

if ($sub_keys = array_keys($all_cats,$id)) {

- til:

if ($sub_keys == array_keys($all_cats,$id)) {

$sub_cats[0] = $cat_id;

Det virker ikke umiddelbart smart i en rekursiv funktion at overskrive med en fast nøgle. Hvad er grunden til dette?

-> nielle: Jeg vil ikke tjekke om de er ens, men om array_keys returnerer noget

-> wicez: Jeg forstår godt hvad du mener, men hvordan skal jeg ellers bevare id'et for kategorien?

Jeg havde forestillet mig flg resultat
print_r($cat_tree);

Udskriver:

[0] => Rod
[1] => Array
    (
    [0] => 11
    [1] => Array
        (
        [0] => 18
        [1] => 14
        )
    [2] => 19
[2] = 12

Manglede lige en parentes:
[0] => Rod
[1] => Array
    (
    [0] => 11
    [1] => Array
        (
        [0] => 18
        [1] => 14
        )
    [2] => 19
    )
[2] = 12

I såfald bør dit tjek se sådan her ud:

$sub_keys = array_keys($all_cats,$id)
if ($sub_keys !== false) {

foreach ($sub_keys as $sub_id) {
            $sub_cats[$cat_id][] = getsubcatsdeep($all_cats,$sub_id);
        }

I stedet for

$sub_cats[0] = $cat_id;
        foreach ($sub_keys as $sub_id) {
            $sub_cats[] = getsubcatsdeep($all_cats,$sub_id);
        }

Ville jeg gøre

Jeg kan ikke umiddelbart se nogen stor forskel mellem de to følgende linier:

1. while ($sql_array = mysql_fetch_array($sql)) {

2. if ($sub_keys = array_keys($all_cats,$id)) {

Så hvorfor mener du lige at linie 2 skal omskrives?

Hvis det er mig du spørger, så kan jeg ikke lige se hvad din linje 1. reference går ud på?

Mht. mit råd om at erstatte:

if ($sub_keys = array_keys($all_cats,$id)) {

- med:

$sub_keys = array_keys($all_cats,$id);
if ($sub_keys !== false) {

- så er pointen at PHP sagtens kan finde ud af at fortolke ting som true og false selv om de er noget helt andet - f.eks. tallene 0 og 1. Med sammenligningsoperatoren !== sikre man sig imidlertid imod at noget "ser ud til at være" false (f.eks. 1), men ikke er det i virkeligheden.

Det er jeg skam med på, og jeg er ganske enig, i de tilfælde hvor man rent faktisk kan forvente en string eller integer return,. Men det er bare ikke lige tilfældet her. Her returneres enten et array eller null, så det har i virkeligheden ikke nogen betydning her.
Jeg sætter virkelig pris på jeres hjælp, men de småting i nævner er ikke årsagen til at scriptet loop'er lige lovligt meget :)

Faktisk skal det måske lige siges, at jeg rent faktisk ikke ved 100% at scriptet looper i evighed. Men scriptet bliver i hvert fald stoppet på serveren

Det er vel kun naturligt at vi kaster os over de mest iøjenfaldende ting først. ;^)

Man, kan du ikke lige starte med at udskrive de tre grundlæggede arrays, $ass_tree, $ass_cats og $subcat_to, ser de ud som du ville forvente det? Post dem lige her sådan at vi også kan følge med.

Jeg regner med at du mente $root_cats i stedet for $subcats_to :)
Men her er de - alle som forventet:
Array
(
    [11] => 0
    [12] => 0
    [14] => 18
    [18] => 11
    [19] => 11
)
ass_cats:
Array
(
    [11] => Array
        (
            [0] => 11
            [id] => 11
            [1] => Beklædning
            [cat] => Beklædning
            [2] => 0
            [subcat_to] => 0
        )

    [12] => Array
        (
            [0] => 12
            [id] => 12
            [1] => Motorcykler
            [cat] => Motorcykler
            [2] => 0
            [subcat_to] => 0
        )

    [14] => Array
        (
            [0] => 14
            [id] => 14
            [1] => speciel
            [cat] => speciel
            [2] => 18
            [subcat_to] => 18
        )

    [18] => Array
        (
            [0] => 18
            [id] => 18
            [1] => trøjer
            [cat] => trøjer
            [2] => 11
            [subcat_to] => 11
        )

    [19] => Array
        (
            [0] => 19
            [id] => 19
            [1] => bukser
            [cat] => bukser
            [2] => 11
            [subcat_to] => 11
        )

)
root_cats:
Array
(
    [11] => 0
    [12] => 0
)

Den øverste var $ass_tree!
sorry

Jeg kan informere om, at selv hvis funktionen laves om til følgende stoppes scriptet:

function getsubcatsdeep($all_cats,$id) {
    if ($sub_keys = array_keys($all_cats,$id)) {
        #$sub_cats[0] = $cat_id;
        #foreach ($sub_keys as $sub_id) {
        #    $sub_cats[] = getsubcatsdeep($all_cats,$sub_id);
        #}
        return $sub_cats;
    } else return $id;
}

...Men hvis funktionen ikke køres forløber scriptet fint.
???

Så det er vidst ikke som først antaget en evighedsløkke

Jeg kigger på det i morgen :^)

Men, jo det er helt sikkert en evighedsløkke. Ved at udkommentere de 4 linjer fjerner du jo netop det rekursive, og så går den glat igennem. Det er helt sikker også nok at fjerne selve kaldet:

function getsubcatsdeep($all_cats,$id) {
    if ($sub_keys = array_keys($all_cats,$id)) {
        $sub_cats[0] = $cat_id;
        foreach ($sub_keys as $sub_id) {
            // $sub_cats[] = getsubcatsdeep($all_cats,$sub_id);
        }
        return $sub_cats;
    } else return $id;
}

Det jeg var inde på er jo netop, at det stadig ikke virkede, selvom jeg udkommenterede linierne.

Undskyld, læste åbenbart ikke grundigt nok hvad du skrev. :^|

Det er da helt ok... Er selv ved at være lidt træt :)

Men tak for opmærksomheden indtil videre!

Skal din funktion ikke snare se sådan her ud?

function getsubcatsdeep($all_cats, $id) {
    if ($sub_keys = array_keys($all_cats, $id)) {
        $sub_cats[0] = $id;
        foreach ($sub_keys as $sub_id) {
            $sub_cats[] = getsubcatsdeep($all_cats, $sub_id);
        }
        return $sub_cats;
    } else
        return $id;
}

Jo.... variablen $id hed fra starten $cat_id. Havde åbenbart ikke ømdøbt det hele. Men det er stadig ikke problemet.

Det går også i spin hvis du udkommentere dette?

function getsubcatsdeep($all_cats, $id) {
    if ($sub_keys = array_keys($all_cats, $id)) {
        $sub_cats[0] = $id;
        //foreach ($sub_keys as $sub_id) {
        //    $sub_cats[] = getsubcatsdeep($all_cats, $sub_id);
        //}
        return $sub_cats;
    } else
        return $id;
}

Hvis det er tilfældet, hvad sker der så hvid ud udkommentere her?

$cat_tree[0] = 'rod';
foreach($root_cats as $cat_id => $value) {
    // $cat_tree[] = getsubcatsdeep($ass_tree,$cat_id);
}

Hehe... Nej, så køres scriptet fint.
Følgende fucker også op.

function getsubcatsdeep($all_cats, $id) {
    $sub_keys = array_keys($all_cats, $id);
    #if (!$sub_keys) {
    #    $sub_cats[0] = $cat_id;
    #    foreach ($sub_keys as $sub_id) {
    #        $sub_cats[] = getsubcatsdeep($all_cats,$sub_id);
    #    }
    #    return 'ko';//$sub_cats;
    #} else
    return $id;
}

Altså må det jo være følgende linie:
$sub_keys = array_keys($all_cats, $id);

Men hvad der er galt fatter jeg virkelig ikke.

Ser der det samme hvis du ændre:

cat_tree[0] = 'rod';
foreach($root_cats as $cat_id => $value) {
    // $cat_tree[] = getsubcatsdeep($ass_tree,$cat_id);
}

- til:

cat_tree[0] = 'rod';
foreach($root_cats as $cat_id => $value) {
    $sub_keys = array_keys($ass_tree,$cat_id);
}

?

virker fint

Sjovt, for det er ellers den samme kode som bliver udført i begge tilfælde. :^|

Ja... Det er fucked up... Men jeg lover dig... Jeg har virkelig dobbelt tjekket flere gange... Der ER en forskel.

Det her driver mig virkelig til vanvid. Er der ikke en, der gider prøve på en anden server?

Dette script er i virkeligheden et udpluk af et større script på 1200+ linier.
Jeg prøvede at køre dette script alene, og nu virker det - eller det bliver i hvert fald kørt i den fulde udstrækning. Resultat afviger en anelse fra det ønskede, men det retter jeg i morgen.
Er der nogen, der har en forklaring??

Jeg vil lige skynde mig at sige, at funktionen faktisk returnerer præcist det ønskede:
function getsubcatsdeep($all_cats, $id) {
    if ($sub_keys = array_keys($all_cats, $id)) {
        $sub_cats[0] = $id;
        foreach ($sub_keys as $sub_id) {
            $sub_cats[] = getsubcatsdeep($all_cats,$sub_id);
        }
        return $sub_cats;
    } else return $id;
}
$cat_tree[0] = 'rod';
foreach($root_cats as $cat_id => $value) {
    $cat_tree[] = getsubcatsdeep($ass_tree,$cat_id);
}
-------
Returnerer:
-------
Array
(
    [0] => rod
    [1] => Array
        (
            [0] => 11
            [1] => Array
                (
                    [0] => 18
                    [1] => 14
                )

            [2] => 19
        )

    [2] => 12
)

Men hvorfor hulen virker det ikke fra den "store" fil????

Hehe... Da jeg flyttede funktionen til filen, med nogle af de andre funktioner, virker det også pludselig fra den før omtalte "store" fil.

Måske virker dette bedre:

function getsubcatsdeep($all_cats, $id) {
    if ($sub_keys = array_keys($all_cats, $id)) {
        $sub_cats = array();
        $sub_cats[0] = $id;
        foreach ($sub_keys as $sub_id) {
            $sub_cats[] = getsubcatsdeep($all_cats,$sub_id);
        }
        return $sub_cats;
    } else return $id;
}

Jeg er stadig interesseret i en forklaring, hvis der skulle være en der sidder inde med den.
Læg et svar i to, så deler vi alle tre.
I har trods alt forsøgt godt og grundigt :)

Mit bud er at et eller andet i getsubcatsdeep() ødelækker noget for noget længere nede. Jeg kunne f.eks. foreestille mig at det er $sub_cats som bliver mishandlet når du bare uden vider smider ind på 0' plads. Det er derfor at jeg foreslår at du startere med at initialisere den med:

$sub_cats = array();

Jeg ved dog ikke om det er det, men jeg har på den anden side ikke noget bedre bud.

... og et svar :^)

Jeg har vist ikke fortjent nogen points i denne sammenhæng ;o) Ellers tak.