fråga om php classer

Tjena.
Jag har en liten fråga gällande classer. Det känns som att jag tänker lite fel när jag kodar och tänkte höra hur ni gör.

Jag kör som exempel med PDO. Jag har skapat en singleton class, static database för att skapa ett PDO objekt och har också skapat olika metoder för att hämta data etc.

Sedan har jag en annan class, som exempel customer:


I classen customer så hämtar jag en instance av classen database som ska sköta alla databas anrop. Sedan skapar jag metoder i den classen som är till för classen, som exempel get_all_customers().

Gör jag rätt, eller kan man sköta det på ett snyggare sätt?

Egentligen kallas detta för ett antimönster.

"The real pros" rekommenderar inte att man använder singetons. Det förstör poängen med objektorienterad programmering.

Det rekommenderas att man skapar sin __construct typ såhär:

Hur många som verkligen har energin att göra så är en annan fråga, men att använda singletons är ett antimönster

Det låter som att du försöker skapa en ORM i mina ögon. Skulle tipsa dig om att använda en befintlig ORM istället. Det finns naturligtvis en massa lösa ORM:er som Doctrine så du inte behöver använda ett ramverk om du inte vill.

Annars skulle jag tipsa om Laravel:
http://laravel.com/docs/eloquent

PhalconPHP för prestanda:
http://phalconphp.com/en/

Eller om du vill ha något lite mer minimalistiskt så finns Silex:
http://silex.sensiolabs.org/

men att inte köra databas instancen som singleton är ju inte så smart, då skapar men flera anslutningar till databasen istället för att använda den öppna anslutningen.

Eller har jag fel?

Principen är att databasinstansen skapas utanför din Customer-klass. Sen bifogar du den till din Customer-construct() - och till andra klassinstanser som du skapar och som kan ha nytta av den. Så den kan återanvändas.

Objekt skapar inte kopior som andra variabler, utan använder exakt samma objekt. Det är en direkt referens, så ingen dubbel anslutning skapas.

Använder man en gammal version av PHP får man dock skapa en direkt referens genom &-tecknet

Ok tackar.
Sista fråga som jag har nu är:

Om jag har en user klass, ska jag göra mina databas-anslutningar direkt från den klassen, eller ha en modell-klass som user klassen ärver istället.

Har sätt vissa göra så för att separera alla db connections från själva klassen. Istället skapar dem funktioner i modell klassen som exempel deleteUser(23); för att göra en query i själva user klassen.

Vad tror ni om det

"The gang of four" är ganska duktiga har jag hört

De använder "Singleton".

Det finns flera olika designmönster, alla har sina fördelar och nackdelar, i slutändan handlar det om vad som är mest effektivt för dig.

Eloquent som någon nämnde innan är en ActiveRecord implementation.

Det du beskriver ovan låter som en Data Mapper implementation, du har en Mapper som håller databasen och själva objektet frånskilda.

User
username
password

UserMapper
insert
update
delete

User -> UserMapper <-- Databasen

Man kan inte säga rakt av att en singleton alltid är en dålig approach. Det beror helt på hur resten av din kod ser ut och hur du ska använda den. Mycket av kritiken mot singletons kommer ifrån att det kan vara svårt att skriva automatiska tester för sådan kod, vilket ses som väldigt viktigt idag.

Sen kan du alltid använda en singleton oavsett om du använder den globalt eller om du skickar med instansen som parameter till klasserna som använder den. Däremot är det oftast en bra idé att inte tillåta klasser att ha en externa (globala) referenser i och med att det är en strategi som kan ställa till det ordentligt om man inte håller tungan rätt i mun. Då är det alltså oftast bättre att skicka med allt som klassen/dess metoder behöver som parametrar istället.


Det där är svårt att inte tolka fel. Objekt som man kopierar och skickar runt i olika variabler är samma objekt, men instansierar du flera objekt av en klass är det naturligtvis olika objekt.

Visst. Men hämta inte fram instansen direkt i klasser utan mata klasser med de beroenden de har. Typ vad Strategy (ockås GoF) går ut på .

Genom att ha MySingleton::getInstance() lite överallt har man skapat massa beroenden till just MySingleton, vilket är dålig OO. Koda mot abstraktioner (interface), inte implementationer.