[php] probleem met variabelen in geneste functie

[tombo_inc]

beste mensen,

ik heb een functie met daarin een paar andere functies. nu heb ik echter een probleem met het aanspreken van bepaalde variabelen. mijn structuur is als volgt:

PHP:

$woei = "hallee";

function groot() {
$blaat = "uhuh";
    function klein() {
        global $blaat,$woei;
        echo($blaat); // dit werkt niet
        echo($woei); // dit werkt wel
                        // $blaat heb ik nodig in deze functie
    }

          function klein2() {
          //andere meuk
         }
}

ik weet niet hoe ik dit op moet lossen. ik hoop dat jullie me kunnen helpen.

[ Voor 10% gewijzigd door tombo_inc op 13-04-2005 17:17 ]

[Brakkie]

Wat is het nut van een functie binnen een andere functie? Doe dat gewoon niet dan ben je waarschijnlijk ook wel van dit probleem af.

[tombo_inc]

ik heb dit in mijn geval gewoon nodig. dus als je een oplossing weet dan moet je het zeggen en anders niet.

Modbreak:

Iets aardiger mag ook wel. Brakkie probeert je alleen te helpen; zijn suggestie lost inderdaad gewoon je probleem op. Als dit voor jou niet voldoet, dan kun je ook gewoon zeggen waarom niet; sterker nog, dat hoort in je startpost thuis.

[ Voor 54% gewijzigd door NMe op 13-04-2005 17:49 ]

[AtleX]

Je zit met scopes, groot() kan uit de bovenliggende scope variabelen halen met global, maar de bovenliggende scope van klein() is groot() die kan dus nooit bij de variabele $woei.

Globals zijn eigenlijk best vies, ze zorgen ervoor dat je code niet kan hergebruiken en verminderen de duidelijkheid over afkomst van variabelen. Je kan beter functie parameters gebruiken, die zijn daar tenslotte voor gemaakt.

Ook nested functions zijn in 99% van de gevallen totaal overbodig, je kan beter eens kijken naar de logische opbouw van je code om dit soort constructies te omzeilen.

Tombo_inc schreef op woensdag 13 april 2005 @ 17:41:
ik heb dit in mijn geval gewoon nodig. dus als je een oplossing weet dan moet je het zeggen en anders niet.

Wat vriendelijker kan ook wel

[ Voor 77% gewijzigd door AtleX op 13-04-2005 18:21 ]

[Creepy]

Daarnaast: Je kan vaak ook prima gebruik maken van parameters en return values. Mocht je met return values niet uitkomen (stel dat je 2 variabelen wilt "returnen" uit een functie) dan kan je ook nog eens kijken naar references.

Het gebruik van globale variabelen hiervoor is vaak niet de mooiste oplossing. Het kan zelfs verwarrend gaan werken omdat je niet meer precies weet welke variabele nu waar wordt aangepast.

[mbravenboer]

Merkwaardige reacties hier . Allereerst gaat het helemaal niet om globale variabelen: de topicstarter wil een variabele uit een omringende functie gebruiken in een geneste functie. Hier is helemaal niets mis en het is zelfs een stijl die wordt aangemoedigd in veel andere (functionele) talen.

Jullie raden de topicstarter aan om een taal feature niet te gebruiken omdat deze niet nodig is. Waarom bestaat de taal feature dan eigenlijk? Moeten geneste functies dan maar gewoon uit PHP gehaald worden? Als een taal geneste functies support, verwacht je ook dat je die normaal mag gebruiken ...

Het echte probleem is echter dat de hele taal feature brak geimplementeerd is. Normaal gesproken hebben geneste functies in een taal lexical scope, wat betekent dat ze de variabelen van de functies waarin ze zich bevinden kunnen gebruiken. Dit verwacht de topicstarter dus ook, en terecht!

De taal feature moet dus inderdaad maar weg, maar niet omdat het concept van geneste functies zinloos is. Geneste functies in PHP zijn gewoon niet goed geimplementeerd.

Zie ook: (even zoeken op nested functions)
http://c2.com/cgi/wiki?PhpProsAndCons

[tombo_inc]

Tombo_inc schreef op woensdag 13 april 2005 @ 17:41:
ik heb dit in mijn geval gewoon nodig. dus als je een oplossing weet dan moet je het zeggen en anders niet.

Modbreak:

Iets aardiger mag ook wel. Brakkie probeert je alleen te helpen; zijn suggestie lost inderdaad gewoon je probleem op. Als dit voor jou niet voldoet, dan kun je ook gewoon zeggen waarom niet; sterker nog, dat hoort in je startpost thuis.

ja srry maar ik zit al de hele dag aan dit probleem te rotzooien en het werkt niet. daarom ben ik een beetje geirriteerd en dan zit je niet op zulke reacties te wachten. ik weet zelf ook wel dat ik het op die manier op kan lossen maar dan werkt mijn programma niet meer. ik zou natuurlijk geen dingen gaan vragen als ik ze ook niet nodig heb. nogmaals mijn excuses

@atlex.
ik kan juist wel bij $woei (zit twee lagen erboven) en niet bij $blaat die er maar een laag erboven zit

[ Voor 7% gewijzigd door tombo_inc op 13-04-2005 18:21 ]

[AtleX]

mbravenboer schreef op woensdag 13 april 2005 @ 18:08 iets over scopes enzo

GOTO bestaat ook in aardig wat talen, en overal wordt het gebruik afgeraden . Verder is mijn stukje over scopes exact gelijk en jou stukje over lexical scopes. klein() kan niet buiten de scope van groot() en kan dus niet bij $woei

Tombo_inc schreef op woensdag 13 april 2005 @ 18:19:
[...]

@atlex.
ik kan juist wel bij $woei (zit twee lagen erboven) en niet bij $blaat die er maar een laag erboven zit

Sorry, glad verkeerd gelezen

Kijk eens met error_reporting(E_ALL) of dat iets zinnigs oplevert?

[ Voor 57% gewijzigd door AtleX op 13-04-2005 18:25 ]

[tombo_inc]

heb ik al, er zijn geen fouten. hij doet het gewoon niet. maar wat dus het doel is uiteindelijk voor de mensen die het willen weten ik wil dus waardes hebben in de parent functie die ik kan gebruiken in de child functies, zonder dat ik ze steeds apart op moet gaan halen in een child.

[AtleX]

Tombo_inc schreef op woensdag 13 april 2005 @ 18:30:
heb ik al, er zijn geen fouten. hij doet het gewoon niet. maar wat dus het doel is uiteindelijk voor de mensen die het willen weten ik wil dus waardes hebben in de parent functie die ik kan gebruiken in de child functies, zonder dat ik ze steeds apart op moet gaan halen in een child.

Kan je dan niet een function parameter meegeven?

PHP:

<?
$woei = "hallee";

function groot() {
$blaat = "uhuh";
    function klein($param1) {
        echo($param1); // dit werkt niet
    }

          function klein2() {
          //andere meuk
         }
}
?>

[ Voor 3% gewijzigd door AtleX op 13-04-2005 18:34 ]

[Brakkie]

Wanneer je een functie binnen een andere functie definieert valt de nieuw gedefinieerde functie niet binnen de locale scope van de functie waarbinnen hij gedefinieerd is maar in de globale scope.

$blaat is alleen bekend binnen de functie waarbinnen hij gedefineerd is en valt dus niet onder de globale scope.

Deze info heb ik trouwens uit de comments van php.net

[ Voor 4% gewijzigd door Brakkie op 13-04-2005 18:39 ]

[ACM]

mbravenboer schreef op woensdag 13 april 2005 @ 18:08:
Allereerst gaat het helemaal niet om globale variabelen: de topicstarter wil een variabele uit een omringende functie gebruiken in een geneste functie. Hier is helemaal niets mis en het is zelfs een stijl die wordt aangemoedigd in veel andere (functionele) talen.

Maar php is geen functionele taal
In php wordt normaliter een stel parameters aan de functie-aanroep meegegeven alvorens verder te werken. De functie is overigens niet genest, alleen het aanmaken van de functie gebeurt pas bij de eerste aanroep van de buitenste functie.
Bij een tweede call naar de buitenste functie zul je een error krijgen die je verteld dat de functie al bestaat, en je kan ook de binnenste functies na de aanroep buiten je functie gebruiken...

PHP:

<?
        function outer()
        {
                function inner()
                {}
        }

        outer();
        outer();
?>
Fatal error: Cannot redeclare inner() (previously declared in /home/acm/temp/functies.php:6) in
 /home/acm/temp/functies.php on line 4

<?
        function outer()
        {
                function inner()
                {
                        echo "inner";
                }
        }

        outer();
        inner(); // inner komt op het scherm
?>

Jullie raden de topicstarter aan om een taal feature niet te gebruiken omdat deze niet nodig is. Waarom bestaat de taal feature dan eigenlijk? Moeten geneste functies dan maar gewoon uit PHP gehaald worden? Als een taal geneste functies support, verwacht je ook dat je die normaal mag gebruiken ...

Zoals dus gezegd, het is slechts een definitie die pas uitgevoerd wordt na de aanroep van de buitenste functie. Het heeft wel wat van de enorm wazige scoping van php met o.a. for-loops en if's enzo:

PHP:

if(true)
{
   $a = 1;
}

echo $a; // echo'd 1
for($i = 0; $i < 10; $i++)
{}
echo $i; // echo'd 10

In php mag dat.

Het echte probleem is echter dat de hele taal feature brak geimplementeerd is. Normaal gesproken hebben geneste functies in een taal lexical scope, wat betekent dat ze de variabelen van de functies waarin ze zich bevinden kunnen gebruiken. Dit verwacht de topicstarter dus ook, en terecht!

De "geneste" functie komt zelfs in de normale globale functie-scope...

De taal feature moet dus inderdaad maar weg, maar niet omdat het concept van geneste functies zinloos is. Geneste functies in PHP zijn gewoon niet goed geimplementeerd.

Ach juist doordat het overal al zo los mee is, kan de functionaliteit op zich best nuttig zijn. Waar je bij andere omgevingen via Reflection functionaliteit dynamisch kan toevoegen kan je in php dmv de definitie van functies in andere functies evt nog wat leuks doen. Je zou het onder de noemer "registreren van functies" kunnen gooien ofzo.

[tombo_inc]

ja dat is idd een interessant verhaal maar nou weet ik nog steeds niet hoe ik het oplos :p(zonder dan argumenten mee te geven)

[ACM]

Niet. Als je goed gelezen had had je dat ook kunnen weten nu.

Dit is gewoon niet mogelijk met php's brakke scoping en hoewel de definitie van je binnenste functie in een andere zit, wordt hij gewoon als een normale functie met een eigen scope opgebouwd. Het enige wat de ene met de andere functie heeft te maken is de plaats en het tijdstip van declaratie, meer niet!

Wat jij wil lijkt me prima met functie-parameters en eventueel object georienteerd werken te doen.

[Huppie]

Je zou het altijd op kunnen lossen door $blaat ook te globaliseren?

PHP:

<?
$woei = "hallee";

function groot() {
$blaat = "uhuh";
$GLOBALS['blaat'] = $blaat;
    function klein() {
        global $blaat,$woei;
        echo($blaat); // dit werkt niet
        echo($woei); // dit werkt wel
                        // $blaat heb ik nodig in deze functie
    }

          function klein2() {
          //andere meuk
         }
}

groot();
klein();
?>

Dat is wel compleet ranzig hoor imo

[tombo_inc]

ja idd dat zou kunnen, maar ik denk dat ik dan toch maar voor de parameter methode ga. en ik zit al in een object te werken eigenlijk. want die parent functie is een object methode waarin ik een ander bestand include. in dat bestand zitten de (child) functies.

[drm]

Wat is eigenlijk de reden dat je in eerste instantie af zou willen zien van een parameter?

[tombo_inc]

dat past niet in mijn structuur. de functies spugen alleen code uit en worden gewoon aangeroepen. de waardes die ik in deze functies nodig heb zijn config waardes. die kan ik dan allemaal los uit de db halen in iedere functie maar ik kan ze ook 1x uit de db halen en dan de array met config waardes global maken zodat ze in iedere functie beschikbaar zijn. ik kan nu die waardes ook wel via argumenten meegeven, maar dat leek mee niet zo handig. maar ik zal nu dus wel moeten, of ik moet in iedere functie de configs los ophalen maar dat kost me weer onnodige serverload en extra code regels. dus ik denk dat de parameter methode het beste is.

[drm]

Met configuratievariabelen (constanten, eigenlijk) kun je ze het beste gewoon (altijd) global maken, dan zou ik er ook niet zo gauw voor kiezen om ze als parameter mee te geven.

[tombo_inc]

ja dat was ook mijn bedoeling maar dat gaat in dit geval dus niet (op een nette manier). (ik haal de configs uit de db)