PHP OOP class i class??

Din login-class skal nedarve fra Mysqldb:

class Login extends Mysqldb {
  // osv
}

Så kan du kalde metoderne i Mysqldb fra Login :)

Jeg vil snarere antage at Login skal have en instans af Mysqldb.

Det var også en mulighed; at lave en instans af Mysqldb i Login-contructoren. Jeg ser dog intet problem i at extende Mysqldb.

Jeg ser 2 problemer ved det:
* Login - Mysqldb er ikke en ægte "is a"
* Login bliver meget tightly coupled med MySQL

Send en Mysqldb med over i Login constructor og gem den i et felt er bedre.

Hvad er den bedst løsning så.

Skal jeg oprette mysql forbindelse i alle mine nye class?? Kan jeg ikke have min mysql funktioner et sted
Og så benytte dem i alle mine kommende class??

Tænker mere på nemheden ved at man kun et sted skal indskrive data login oplysninger??

jeg ville også lave en instans af mysql-klassen i login-constructoren - det virker mest logisk

hvis du skal bruge mysql-funktioner i en ny class opretter du bare en instans af mysql-class'en

okay jeg er helt ny inden for programmering i OOP så hvordan lave man en instans??

Her er et uddrag af min class mysqldb:
class Mysqldb {

    function dbconnect($host, $user, $pass, $db){
         
        if($this->hostname_logon == "") {
            $this->hostname_logon = $host;
        }
       
        if($this->database_logon == "") {
            $this->database_logon = $db;
        }
       
        if($this->username_logon == "") {
            $this->username_logon = $user;
        }
       
        if($this->password_logon == "") {
            $this->password_logon = $pass;
        }
       
        // Try to make a connection to the server
        if (!$this->connection = @mysql_connect($this->hostname_logon,$this->username_logon,$this->password_logon,true)){
            $this->mailerror($this->errorCode=mysql_errno(), $this->errorMsg=mysql_error());
            return false;
        }
       
        // Now select the database
        if (!@mysql_select_db($this->database_logon,$this->connection)){
            $this->mailerror($this->errorCode=mysql_errno(), $this->errorMsg=mysql_error());
            @mysql_close($this->connection);
            return false;
        }
           
        return true;
    }
   
    function dbquery($query) {
     
        ++$this->queryCounter;
       
        if(!$this->resultSet = @mysql_query($query,$this->connection)) {
            $this->mailerror($this->errorCode=mysql_errno(), $this->errorMsg=mysql_error());

            return false;
        }
       
        return $this->resultSet;
    }

}

så hvis jeg skulle bruge noget fra denne i min anden class login hvordan gør jeg så det??

F.eks. sådan:

class Login {
  private $db;

  function __construct() {
    $this->db = new Mysqldb();
  }

  function checkLogin($username, $password) {
    $result = $this->db->dbquery("SELECT * FROM users WHERE username='".$username."' && password='".$password."'");
  }
}

mange tak for hjælpen... hvem vil have point???

Jeg ville lave:
- en abstrakt klasse Anydb
- en konkret klasse Mysqldb som arver fra denne
- en factory klasse som producerer Anydb'er
- en constructor i Login med et argument (som så skal være en Anydb)
- et kald af den cosntructor med en Anydb fra factory klassen som argument

Men måske er det en lille smule overkill hvis du først er igang med at lære OOP.

Men man hardcoder ikke en database vendor specific klasse i noget business logic funktionalitet.

springer over...

Jeg tænker at det ville være smart at bruge et Singleton pattern her eftersom det er database kontakt.
Det vil i betyde at du kun kan oprette et objekt af din handler.
Jeg smider her et udsnit af min egen Singleton Mysql Connector.

Det er så meningen at du kalder på klassens getInstance() som returnerer et et objekt af klassen. Derefter kan du udbygge funktioner på klassen som du kan kalde på det objekt du får tilbage.

$instance = dbHandler::getInstance();
$instance->someFunction();



class dbHandler {
        private $dbuser = '';
        private $dbPass = '';
        private $dbHost = '';
        private $dbName = '';
    private static $Instance;

    private function __construct()     {
        mysql_connect($this->dbHost, $this->dbuser, $this->dbPass);
                mysql_select_db($this->dbName) or die(throw new MysqlException());
        }

        /**
        * Returns an object of the database connection
        * @return Mysql Connection object
        */
    public static function getInstance()    {
        if (!self::$Instance)
        {
            self::$Instance = new dbHandler();
        }
        return self::$Instance;
    }
}

Aldrig singleton for database klasse som indeholder en database connection.

Den dag du har brug for to database forbindelser er du toast.

Singleton er glimrende for en factory som producerer database klasse instanser.

Arne, jeg har svært ved at forstå præcis hvad du mener. Jeg finder det oplagt at bruge Singleton i sådan en situation.
At gå over og bruge en factory patttern synes jeg er overkill set i lyset af at kun er behov for MySQL her, skulle der også være mulighed for oracle eller andre databaser kunne jeg forstå hvor du ville hen.

Og egentlig forstår jeg heller ikke hvad du mener med
"Den dag du har brug for to database forbindelser er du toast"

Men nok om det, det må være op til trådstarter at afgøre hvilken løsning der passer bedst.

Hvis man kan se ind i fremtiden og ved at man aldrig vil skifte database, så er der ikke nogen grund til at forberede koden på andre databaser. Men det rækker mine evner ikke til.

Hvis du har din database connection i en singleton, saa har du kun mulighed for 1 database connection. Hvis du har brug for 2, saa har du et problem.

Singleton med database connection er et meget kendt anti-pattern. P.g.a. PHP's execution model er det ikke så slemt som i ASP.NET og Java EE. Men det er stadig skidt.

Det er formentligt ikke saa relevant for spoerger. Men der kan jo ogsaa vaere andre brugere.

Arne_v:
Jeg kan godt se det problem du nævner, men finder det alligevel også brugbart at bruge en singleton for at sikre, at der altid kun findes én instans af det givne forbindelse.

Hvordan ville du sikre det uden en singleton? Kontrollere det i factory klassen eller hvordan?

Det er ren nysgerrighed. Jeg har heller aldrig været ude for, at skulle have en ny forbindelse.

Hvad formål er der ved at sikre at der kun er en connection?

Du skal bruge to connections hvis:
- du skal lave en ny query inden du er færdig med at hente alle data fra den første query
- du skal lave en opdatering udenfor transaktion mens du har en transaktion i gang på den oprindelige query
- bruger to forskellige databaser

Jeg tænker på, man vil vel ikke køre f.eks. en mysql_connect() hver gang man skal kalde databasen.

Man er vel, typisk, kun interesseret i at have én forbindelse til databasen, for at forhindre unødigt overhead.

Hvad ville være den logiske måde, at sørge for det på, hvis ikke med en singleton?
Man kunne selvfølgelig lave det i et registry eller kode noget i dens 'parent', det virker bare ikke som den nemme løsning.

Der er ikke noget som forhindrer en i at constructe sin connection wrapper øverst i siden, sende den med over i diverse funktions/metode kald og close den til sidst.

Når det ikke er en singleton har man muligheden af at lave en instans mere hvis man vil.

Hvis det er vigtigere at minimere antallet af database connections fremfor latency på side generering, så kan man også vælge at betale prisen for flere mysql_connect og mysql_close par.

Og så er der the dark horse muligheden nemlig at skifte fra mysql_connect til mysql_pconnect.

Ah ja, mysql_pconnect, den brugte jeg for mange år siden efterhånden, kører stadig fint :)

Men jeg kan godt se logikken i dit svar, tak :)

mysql_pconnect er en undervurderet mulighed i PHP.

Der er jo nok en grund til at 99% af alle ASP.NET og Java EE løsninger bruger en tilsvarende teknik.

Indrømmet p.g.a. den måde eller måske mere præcist udtrykt de mange forskellige måder PHP kan køres på er der masser af faldgruber. Men den slags kan jo løses.

Tid at få afsluttet her?

En sidste spørgsmål inden jeg lukker. Hvilken af disse bøger er bedste når jeg skal lære OOP??

http://www.amazon.co.uk/gp/product/143022925X/ref=oss_product
http://www.amazon.co.uk/PHP-Objects-Patterns-Practice-2nd/dp/1590599098/ref=sr_1_2?ie=UTF8&qid=1294902028&sr=1-2

Er det ikke 3. og 2. udgave af samme bog?

Man maa antage at 3. udgave er bedre end 2. udgave!

takker for svaret... vil du have point??

gerne

men jeg er jo ikke den eneste som har vaeret med i traaden

okay hvis der er andre som vil have del i point... så skal i ligge et svar i dag inden kl. 24 TAk