[PHP] Na contruct lege waarde

[Erhnam]

Ik ben mij een beetje aan het verdiepen in het oob concept van PHP. Wat ik graag wil is dat een waarde van een sessie wordt gebruikt en daarna in de functies kan worden aangeroepen. Ik krijg echter altijd een lege waarde terug. Ook als de $_SESSION['username'] gevuld is. Wat doe ik verkeerd?

code:

<?php
class ACL
{
    public $username;

    function __construct()
    {
        if (isset($_SESSION['username'])) {
            $this->username = $_SESSION['username'];
        }

    }

    function getUserName()
    {
        return $this->username;
    }

}

?>

[BeRtjh]

De constructor in php is een vaste methode met de naam __construct, niet __constructor. Heb je tevens gecontroleerd of session_start(); aangeroepen wordt?

[InZane]

Begin eens hier te lezen: http://www.php.net/manual/en/language.oop5.decon.php

[Biersteker]

Misschien een leuke om wat php oop skill op te pakken.
YouTube: PHP OOP Experiment: Text based CLI game
Wel CLI trouwens.

[telefoontoestel]

Is dit voorbeeld puur voor lering ende vermaak of heb je daar ook echt een doel mee? Het lijkt mij dan namelijk overbodig om je $_SESSION waarde in de construct te plaatsen aangezien het een superglobal is.

[Erhnam]

Ja want ik wil het op een centrale plek kunnen aanpassen, stel dat een COOKIE een alternatieve wens is. Door het als GLOBAL los aan te roepen vanuit iedere pagina geeft meer werk.

[NMe]

Dan doe je dat niet door je class $_SESSION uit te laten lezen maar door de benodigde waarde door te geven aan de class door middel van een parameter. Je class zou geen enkel benul moeten hebben van waar de waarde vandaan komt, juist omdat het later inderdaad zomaar eens uit $_COOKIE zou kunnen komen. Als dat zo is wil je je applicatie zelf aan moeten passen, niet dit soort classes. Als je vervolgens vanuit meerdere plaatsen in je code diezelfde constructor aanroept moet je ofwel nadenken over of dat wel het juiste model is en als je daarvan overtuigd bent zou je er nog een class tussen kunnen stoppen die dit gedrag voor je opvangt.

Overigens vind ik het ook een beetje dubieus om je class "ACL" te noemen. Er zijn natuurlijk uitzonderingen, maar meestal als je moeite hebt met het verzinnen van een naam die je je in de fysieke wereld kan voorstellen is de class meestal niet goed opgezet. Op basis van de ene functie die ik zie in je startpost vermoed ik dat je een security context probeert te bouwen, dus SecurityContext zou zomaar een betere naam kunnen zijn.

[Erhnam]

Bedankt voor de heldere uitleg! SecurityContext dekt de lading beter af. Ik ga nog eens goed nadenken over een goede opzet/structuur. Het idee was meer afkomstig uit een torial van:

http://code.tutsplus.com/...er-login-system--net-3461

Hierin wordt in de contruct de waarde vanuit een Sessie gehaald en gebruikt binnen andere functies van de Class.

[NMe]

Ik zie dat je de __constructor daar ook vandaan hebt. Alleen al daarom zou ik die site niet meer vertrouwen.

[Voutloos]

quote: http://code.tutsplus.com/...er-login-system--net-3461

When you call new ACL() in PHP5, the PHP interpreter runs the __constructor() method. However, when you run the same code in PHP4, the interpreter runs ACL(). By providing a method named the same as the class, we make the class PHP4 compatible.

PHP4 Oude shit. Noem aub alle constructors gewoon __construct() en gebruik nooit meer een methodname gelijk aan de class name.

En als je oop wil leren: Dat kan uberhaupt pas een beetje sinds PHP5 en daarmee is een PHP4 compatible tutorial echt een no-go. En werkelijk niemand zou er slechter van woorden als dergelijk oude tutorials vh internet zou verdwijnen.

[NMe]

Voutloos schreef op donderdag 17 april 2014 @ 15:32:
[...]

En als je oop wil leren: Dat kan uberhaupt pas een beetje sinds PHP5 en daarmee is een PHP4 compatible tutorial echt een no-go. En werkelijk niemand zou er slechter van woorden als dergelijk oude tutorials vh internet zou verdwijnen.

Amen.