OOP aan het leren

Probeer de namen anders te doen van de function of class.
Ik zelf vind het irritant als je 2 dezelfde namen achter elkaar hebt: $login->login.

Verder zou je best ook meerdere functions kunnen maken.
Zoals je dat deed met die query kan je ook makkelijk in functie zetten. Staat netter want dan krijg je meer overzicht.

Verder ob_start en ob_end_flush hoef je volgens mij niet eens echt te gebruiken, kan iemand mij uitleggen waarom hij dat wel zou moeten, zover ik zie doe je dit alleen als je fout hebt en waardoor de mensen die niet kunnen zien want ze worden dan gelijk doorgestuurd.?

kan je niet gewoon een constructor gebruiken ofzo (is niet heel goed in OOP)
zoiets:

class Login { 
    var $naam; 
    var $wachtwoord; 
     
    function _constructor() { 
        $this->naam = $_POST['login']; 
        $this->wachtwoord = $_POST['wachtwoord']; 
         
        $l = "SELECT * 
                FROM user 
                WHERE naam=". $this->naam ." && 
                wachtwoord=". $this->wachtwoord ." 
            "; 
        $query = mysql_query($l); 
        $sql = mysql_fetch_array($query); 
        $result = mysql_num_rows($query); 
     
        if($result == "1") { 
            $_SESSION['login'] == $this->naam; 
            $_SESSION['ingelogd'] == TRUE; 
            header("Location: http://www.unitedwar.nl/ingame.php"); 
        } 
    } 
}

zodat je met:

$login = new login;

gelijk al de functie: _constructor uitvoert

Verder is het niet mooier en beter als je gewoon een database class maakt in de config.

Zodat je dan simpel weg korte code krijg in het script om dingen uit te voeren, o ja nog slimmer is om ook gelijk proberen mysqli te leren.

Dit aangezien in php 6 mysql eruit wordt gehaald en omdat mysqli veel sneller werkt dan mysql in Object Oriënterend Programmering, Object Orientated Programming (oop).

NielsB het is __construct()

1. een classe is niet gelijk OOP
2. laat $_GET en $_POST zo veel mogelijk buiten classes, geef ze mee als argumenten, dan kun je deze class voor meerdere doeleinden (her)gebruiken.
3. niet alles selecteren, je hebt niets nodig, dus haal alleen het id op.
4. zet niet de naam in een sessie, maar het id.
5. niet de naam en wachtwoord opslaan in de classe.
6. de 'var' namespace is voor PHP 4, protected, private en public zijn voor PHP 5 (static ook nog, maar deze kan bij een van de andere drie)

Citaat

NielsB het is __construct()

ik denk dat je @ dragontje124 bedoelt

Darsstar edit: inderdaad, foutje, sorry niels.

Oke, daar gaan we!

- Maak geen onnodig gebruik van ob_start(): Dit is alleen maar sloom, wanneer je goed programmeert heb je ob_start(); niet nodig om errors te voorkomen.
- Geef je variabele geen nietszeggende namen. $1 zegt niks.
- Als je uitvoer van de functionaliteit wilt scheiden, doe het dan goed: Een header is toch echt uitvoer!
- Gebruik nooit dezelfde functienaam als klassenaam, dan wordt de functie namelijk als de constructor gezien (De hoofdletter voorkomt dat denk ik in het voorbeeld).
- Encrypt je wachtwoorden!
- Al eerder gezegd houd de invoer gescheiden van de functionaliteit.
- MySQLi werd al genoemd: Mijn aanrader is een PDO klasse (evt. een uitbreiding daarop).
- '$sql = mysql_fetch_array($query);' Deze code gebruik je verder niet meer. Waarom staat hij er dan nog?
- Ook al eerder gezegd: Go PHP 5! Functies en variabelen zijn tegenwoordig public, private of protected.

Niet negatief bedoeld, maar dit is nog lang niet OO.

Alle kritiek is bedoeld als opbouwende kritiek, vind een eigen klasse juist een leuk initiatief.

Misschien kan je ook eens PDO gebruiken

Edit by Killingdevil:
Mijn reactie niet gelezen ofzo?:p

Je kan beter gewoon pdo en oop gelijke tijd leren.
Dit omdat je voor pdo heel andere wijze van connecten moet doen.

Dan is het beter als je gelijke tijd dit leert, dan maak jezelf ook makkelijker voor het geval dat je eindelijk alles snapt en je het opeens anders moet doen.