[PHP]Adodb connectie door classes heen

[DDemolition]

Hoi,

Ik heb me rot zitten zoeken naar goede raad hoe om te gaan met een database connectie. Ik ben bezig met het schrijven van een CMS class (met bijbehorend paneel).
Voor de db connectie gebruik ik adodb.
Het is nu zo dat elke losse klasse zijn connectie los regelt, wat resulteert in meerdere connecties per pagina. (dus bijvoorbeeld al bij elke knoppenreeks)

Ik zou het connectie object aan de functie Execute($db) mee kunnen geven, maar hier wil ik de programmeur die de klasses oproept niet mee pesten en het dus onzichtbaar houden.

Dit voorbeeldje haalt een reeks knoppen op, maar er is ook een class om inhoud weer te geven.

PHP:

$submenu = new Buttons();
//$submenu->LoginHandler($loginhandler);
$submenu->AddCondition('WHERE', 'childof', $iMain);
$submenu->AddCondition('WHERE', 'level', "2");
$submenu->AddSorting('sort', 'asc');
$rs = $submenu->Execute();
while($rs->EOF == false)
       {
       $id = $rs->GetValue('id');
       $rs->MoveNext();
       }

PHP:

$db = NewADOConnection($gaConfig['DBType']);
$db->Connect($gaConfig['DBHost'], $gaConfig['DBUser'], $gaConfig['DBPassword'], $gaConfig['DBName']);

Ik had al zitten denken om de connectie te openen tijdens het initialiseren van alle classes, maar wat is de netste manier om te openen en uiteindelijk te sluiten?

[DDemolition]

Schopperdeschop

Misschien om de programmeurs met een geweten wakker te maken: het kan wel makkelijk, maar hoe zouden jullie het doen om het netjes te houden?

[Megamind]

Ik snap het niet helemaal.... Maar als je een DB class maakt, dan kan je toch gewoon per pagina 1x die klass aanmaken, een query maken, uitlezen etc, en daarna gewoon weer een nieuwe query setten en weer uitlezen etc.

code:

$db = new db_driver();
$db->set_query("SELECT * FROM");
$rows = $db->get_rows();
$db->free();

etcetcetc

De class opent natuurlijk de DB connectie, en sluit deze evt weer.

[ Voor 9% gewijzigd door Megamind op 10-05-2007 20:24 ]

[MBV]

Waarom maak je geen singleton? http://www.tml.tkk.fi/~pnr/GoF-models/html/Singleton.html
Je maakt een static functie die je aanroept als klasse::get(), zodra je een database-connectie nodig hebt.

Als je met verschillende db-servers een verbinding moet maken, dan kan je zoiets doen:

PHP:

class dbCon
{
  //static
  function get($dbname)
  {
    static $connections = array();
    if (isset($connections[$dbname])) return &$connections[$dbname];
    else {
      $connections[$dbname] = new dbCon($dbname);
      return &$connections[$dbname];
    }
  }
  
  //constructor in PHP4-stijl
  function dbCon(dbname)
  {
    //maak hier je database aan
  }
  function ~dbCon($dbname)
  {
    //hier vrijgeven
  }
}

maak geen aparte functie voor het vrijgeven, zou ik zeggen. Dat gaat altijd een keer fout. Gewoon een destructor.

Vergeet dan trouwens niet dat je database-connectie stuk is nadat de sessie verder is gegaan. Als je een object met een database-verbinding als member in $_SESSION zet, dan doet hij het op de volgende pagina niet meer. Daar moet je iets slims op verzinnen, moet wel lukken als je toch met singletons werkt enzo

@megamind: stel je nu eens voor dat je aan het begin van de pagina nog niet weet welke database je gaat aanspreken? Ik ken een paar applicaties waarbij dat het geval is. Dan denk ik aan objecten met overerving e.d. Als je het gewoon in je klasse kan houden, is dat veel makkelijker. Als je het op andere manieren doet, moet je het continue gaan doorgeven aan constructors (of aan functies, zoals mijn voorganger deed )

[Megamind]

MBV schreef op donderdag 10 mei 2007 @ 20:49:
@megamind: stel je nu eens voor dat je aan het begin van de pagina nog niet weet welke database je gaat aanspreken? Ik ken een paar applicaties waarbij dat het geval is. Dan denk ik aan objecten met overerving e.d. Als je het gewoon in je klasse kan houden, is dat veel makkelijker. Als je het op andere manieren doet, moet je het continue gaan doorgeven aan constructors (of aan functies, zoals mijn voorganger deed )

Jou voorbeeld is toch gewoon hetzelfde als ik bedoel Ik doelde je DB klasse een klasse maken, welke je aanspreekt voor je begint te werken met een DB.

Of zie ik jou voorbeeld fout?

[DDemolition]

Ah ok,

Het idee is dus redelijk simpel maar briljant!

Simpel gezegd kijkt de classe of de singleton leeg is, zoja:
- bouw het op
zo nee:
- return zichzelf met de db connectie
http://en.wikipedia.org/wiki/Singleton_pattern

In php kan ik toch geen klasse object in een $_SESSION stoppen of begrijp ik het verkeerd?

[MBV]

waarom niet? Dat is gewoon een array. Je moet er wel rekening mee houden dat ze serialized worden:

http://php.net/serialize
http://php.net/oop5.magic (PHP5 specifiek, behalve dan __sleep en __wakeup)

[daniëlpunt]

DDemolition schreef op donderdag 10 mei 2007 @ 23:18:
Ah ok,

Het idee is dus redelijk simpel maar briljant!

Simpel gezegd kijkt de classe of de singleton leeg is, zoja:
- bouw het op
zo nee:
- return zichzelf met de db connectie
http://en.wikipedia.org/wiki/Singleton_pattern

In php kan ik toch geen klasse object in een $_SESSION stoppen of begrijp ik het verkeerd?

Zo gaat het ongeveer ja, de laatste 'zo nee' moet weg.
Het maakt niet uit of de instantie al aangemaakt is of niet, hij word gereturnt.

[ Voor 6% gewijzigd door daniëlpunt op 11-05-2007 13:46 ]

[DDemolition]

super-muffin schreef op vrijdag 11 mei 2007 @ 13:45:
[...]

Zo gaat het ongeveer ja, de laatste 'zo nee' moet weg.
Het maakt niet uit of de instantie al aangemaakt is of niet, hij word gereturnt.

Ik heb al wat meer praktische voorbeelden gevonden op internet. Ik snap de werking ervan nu beter.
Ik ga het morgen eventjes uitwerken en kijk of het werkt. Heb nu in de ado classe echo "connect"; gezet bij elke database connect. Wanneer er maar één staat is het goed (dit zijn er nu nog een stuk of 5)

[DDemolition]

Geweldig!
Default.class.php

PHP:

    class DB
        {
        function get_db()
            {
            static $db;
            if (!isset($db))
                {
                require('Config.inc.php');
                $db = NewADOConnection($gaConfig['DBType']);
                $db->Connect($gaConfig['DBHost'], $gaConfig['DBUser'], $gaConfig['DBPassword'], $gaConfig['DBName']);
                }
            return $db;
            }
        }

en in de class zelf:
Modules/Buttons/Buttons.class.php

PHP:

            $db = DB::get_db();
//          $db = NewADOConnection($gaConfig['DBType']);
//          $db->Connect($gaConfig['DBHost'], $gaConfig['DBUser'], $gaConfig['DBPassword'], $gaConfig['DBName']);
            $rs = $db->Execute($SQL);

Netjes programmeren FTW!

Nu nog even kijken hoe ik het doe met php5 en php4 compatibliteit...

[MBV]

wat een ranzige vorm van indenting gebruik jij

Maar dat is inderdaad het idee. Je zou kunnen overwegen om met references ( return &$db) te werken, en ik zou je aanraden om in een klasse waar veel queries voorkomen get_db alleen in de constructor te zetten (en eventueel in __wakeup).

Volgens mij werkt dit al in PHP4, ik werk nog niet met PHP5. Op mijn werk zijn ze nog niet over (en gaat niet gebeuren ook, PHP wordt uitgefaseerd om VB.net te gebruiken ) dus dat mag je zelf uitproberen

[ Voor 26% gewijzigd door MBV op 12-05-2007 15:33 ]

[DDemolition]

MBV schreef op zaterdag 12 mei 2007 @ 15:31:
wat een ranzige vorm van indenting gebruik jij

Is waarschijnlijk nog wat blijven hangen vanuit vb.net en kom er niet van af...

Er staan genoeg dingen over PHP5, maar dan moet ik een PHP5 versie uitbrengen voor alleen die klasse...

[ Voor 18% gewijzigd door DDemolition op 14-05-2007 16:39 ]

[MBV]

DDemolition schreef op maandag 14 mei 2007 @ 16:37:
[...]

Is waarschijnlijk nog wat blijven hangen vanuit vb.net en kom er niet van af...

Er staan genoeg dingen over PHP5, maar dan moet ik een PHP5 versie uitbrengen voor alleen die klasse...

Ah, dat verklaart al een heleboel...

En over PHP5: wil je dat wel of niet? Omschrijven zou triviaal moeten zijn.

[DDemolition]

MBV schreef op maandag 14 mei 2007 @ 16:40:
[...]

Ah, dat verklaart al een heleboel...

En over PHP5: wil je dat wel of niet? Omschrijven zou triviaal moeten zijn.

Ja dat wil ik wel ja.. zal het toch eens op een php5 testomgeving zetten.