[PHP] OOP, lijkt dit nog ergens op ?

[Mafioso]

Goedo avond allemaal, vandaag verveelde me ik een beetje (terwijl ik nog zooooooo veel moet doen). Maar ik was eens aan het kijken naar OOP in PHP. En het was allemaal wel interesant, en leek me wel handig. Ik heb nogteeds niet helemaal begrepen hoe het allemaal werkt. Maar na wat artikeltje ben ik maar gewoon zelf wat gaan knutselen, een beetje looz eigenlijk.

De class die ik aan het maken ben is voor een inlog systeem, dat leek me wel handig omdat je dan in een kort manier op een boel verschillende pages makkelijk terug kan vallen op het usersyteem.

Anyway hier is wat ik beetje heb lopen brouwen (nog lang niet af, en erg basic), maar m'n vraag is nu of ik wel goed bezig ben, is het wel OOP ?, wat moet r anders?

PHP:

<?
class User {

    // Vars
    var $Username;
    var $Password;
    var $Errors = array(
                        "NoSuchUser" => "Sorry, de gebruikte username bestaat niet.",
                        "" => ""
                        );
    var $ErrorFormat = "%s";

    // Function setID()
    function setID ($ID) {
        $this->ID = $ID;
    }

    function getID () {
        return $this->ID;
    }

    function setUserName ($UserName) {
        $this->UserName = $UserName;
    }

    function getUserName () {
        if (!IsSet($this->UserName)) $this->getUserData();
        return $this->UserName;
    }

    function setRealName ($RealName) {
        $this->RealName = $RealName;
    }

    function getRealName () {
        if (!IsSet($this->RealName)) $this->getUserData();
        return $this->RealName;
    }

    function setEmail ($Email) {
        $this->Email = $Email;
    }

    function getEmail () {
        if (!IsSet($this->Email)) $this->getUserData();
        return $this->Email;
    }

    function getUserData () {
        $SQL = "SELECT username, email, realname FROM users WHERE id = '".$this->getID()."'";
        $Query = mysql_query($SQL);
        $UserData = mysql_fetch_array($Query);
        $this->setUserName($UserData["username"]);
        $this->setRealName($UserData["realname"]);
        $this->setEmail($UserData["email"]);
    }

    // Function Login()
    function Login ($Username,$Password) {
        $SQL = "SELECT id FROM users WHERE username = '$Username' AND password = MD5('$Password') ";
        $Query = mysql_query($SQL);
        $NumRows = mysql_num_rows($Query);
        If ($NumRows == 1) {
            $UserID = mysql_result($Query,0);
            $this->setID($UserID);
            return True;
        } else {
            return False;
        } // End - If ($NumRows == 1)
    } // End - function Login($Username,$Password)

}

$user = new User;
if ($user->Login($username,$password)) {
  echo "Goedzo, en welkom <a href=mailto:".$user->getEmail().">".$user->getUserName()." (".$user->getRealName().")"."</a>";
} else {
  echo "STOUT !!!!!";
}
?>

[Tomatrix]

In PHP kan je niet OO programmeren, het is er te beperkt voor. Als je echt wil OO-en ga dan een taal als Java of C# leren.

[Mafioso]

Op zaterdag 23 februari 2002 01:16 schreef Tomatrix het volgende:
In PHP kan je niet OO programmeren, het is er te beperkt voor. Als je echt wil OO-en ga dan een taal als Java of C# leren.

Tsja sorry, maar dat is niet helemaal m'n bedoeling

Nouja vraag ff veranderen : Doe ik dit allemaal goed, is het dus (zover dat kan met PHP) een beetje OO ?

[Tomatrix]

Oh, ok.
Ik zou ff het e-mail adres checken op correctheid of er een aparte e-mail klasse van make.
Ook zou ik je SQL en connectie statements in een aparte klasse zetten zodat je snel van database kan wisselen.

En tsja, om het echt mooi te maken...

[mbravenboer]

Ik vind het eigenlijk al wel een aardig begin . Je kunt inderdaad nog wat meer zaken scheiden in aparte klassen, maar je maakt in ieder geval geen grote fouten of rare constructies. Ga zo door dus .

[Mafioso]

Maar is het ook nuttig om dit verder te bouwen?, met een paar normale functions kom je r ook wel. En al die get en set functions die ik hier heb gemaakt, het lijkt allemaal zo loos, maar dat zal later nog wel echt handig zijn.

[Johannes]

Op zaterdag 23 februari 2002 01:42 schreef Mafioso het volgende:
Maar is het ook nuttig om dit verder te bouwen?, met een paar normale functions kom je r ook wel. En al die get en set functions die ik hier heb gemaakt, het lijkt allemaal zo loos, maar dat zal later nog wel echt handig zijn.

Het voordeel van al die getters en setters is dat je de complete interne representatie van een class kan veranderen(om snelheidsredenen bijv., maar dit is niet zo relevant in PHP), zonder dat je ook maar één stukje code buiten de class zelf hoeft te veranderen. Dit is juist één van de voordelen van OOP.

[Feyd-Rautha]

Ik zie niet direct zware fouten ofzo, maar wat ik wel mis (maar dat zal waarschijnlijk eerder aan PHP liggen dan aan u), dat zijn de access modifiers.
Eén van de principes van OOP is 'data hiding'. Dwz dat bepaalde members niet toegankelijk mogen zijn buiten de class. Ik denk in uw voorbeeld aan 'naam', 'passwoord' enz. Die zouden moeten 'private' zijn en dan uw getters en setters 'public'. Nu hebben ze waarschijnlijk allemaal dezelfde access modifier (het zal wel 'public' zijn).

Wat ik ook niet zou doen is, die array met foutmeldingen als class-member maken. Nu heeft ieder object die je van die class instantieert geheugen gereserveert voor die array, en die is voor ieder object toch hetzelfde. Die kan je dus best static maken.

[whoami]

Whoops.
Mijn broer heeft blijkbaar niet uitgelogd. Die post van -Avalanche- is eignelijk van mij.

[Glimi]

Op zaterdag 23 februari 2002 09:57 schreef -Avalanche- het volgende:
Ik zie niet direct zware fouten ofzo, maar wat ik wel mis (maar dat zal waarschijnlijk eerder aan PHP liggen dan aan u), dat zijn de access modifiers.
Eén van de principes van OOP is 'data hiding'. Dwz dat bepaalde members niet toegankelijk mogen zijn buiten de class. Ik denk in uw voorbeeld aan 'naam', 'passwoord' enz. Die zouden moeten 'private' zijn en dan uw getters en setters 'public'. Nu hebben ze waarschijnlijk allemaal dezelfde access modifier (het zal wel 'public' zijn).
Wat ik ook niet zou doen is, die array met foutmeldingen als class-member maken. Nu heeft ieder object die je van die class instantieert geheugen gereserveert voor die array, en die is voor ieder object toch hetzelfde. Die kan je dus best static maken.

Volgens mij ondersteund PHP beide niet Echter dat static zou je kunnen opvangen via een include. Maar dit lijkt me ontzettend smerig, dus je het het beste zo laten

[whoami]

Op zaterdag 23 februari 2002 10:42 schreef Glimi het volgende:

[..]

Volgens mij ondersteund PHP beide niet Echter dat static zou je kunnen opvangen via een include. Maar dit lijkt me ontzettend smerig, dus je het het beste zo laten

Daarmee ook dat PHP niet de geschikte tool is om OOP aan te leren. Ga dan idd eerder voor C#, Java, Smalltalk, ... zoals eerder al vermeld was.

[chem]

ja, ziet er wel goed uit hoor

zoals eerder gezegd: verwerk je DB etc. ook in een aparte class.

Je zult zien dat naarmate je classes beter worden, je projecten bij elkaar kan c/p-en... door hergebruik van je classes

[Mafioso]

Op zaterdag 23 februari 2002 11:00 schreef chem het volgende:Je zult zien dat naarmate je classes beter worden, je projecten bij elkaar kan c/p-en... door hergebruik van je classes

Ja dat leek mij ook, wel erg handig natuurlijk, het kost je in het begin misschien wat meer tijd, maar dat bespaar je later.

En wat betreft die db access zooi, het was ook mij plan om daar een aparte class voor te maken/zoeken, ik dacht dat r in een artikel over OOP @ phpbuilder wel een goede stond.

[edit]Hmmm ik zie dat artikel ff niet meer, kweet niet meer waar die stond .. nouja ik vind hem wel weer.

[-RenE-]

Op zaterdag 23 februari 2002 13:03 schreef Mafioso het volgende:

[..]


[edit]Hmmm ik zie dat artikel ff niet meer, kweet niet meer waar die stond .. nouja ik vind hem wel weer.

Kijk een naar ADODB. Dit is een zeer uitgebreide DB class.
http://php.weblogs.com/ADODB

Alternatief is Metabase, die gebruikt wordt in BinaryCloud. De laatste opzet is ook zeer de moeite waard.

Andere tip voor gestructureerd programmeren in php in Fusebox voor PHP (http://groups.yahoo.com/group/PHP-fusebox/) en voor algemene info over de Fusebox methode van programmeren (www.fusebox.org).

Succes met programmeren