Hjælp til strukturing/optimering?

Nu er det selvfølgelig lidt svært at se ud fra det eksempel, men sådan her ser min aktuelle klasse ud.

PHP kode 

 class database
 {
     private $connection_libray;
     private $db_hostname, $db_username, $db_password, $db_database;
     private $db_link;
     private $error_array = array();
 
     function __construct()
     {
         if(class_exists("MySQLi") || function_exists("mysql_connect"))
         {
             if(class_exists("MySQLi"))
                 $this->connection_libray = "MySQLi";
             else
                 $this->connection_libray = "MySQL";
         }
         else
         {
             die("Error: Your server haven't support for MySQL or MySQLi. You need one of them if you wish to use this libray.");    
         }
     }
     
     function connect()
     {
         switch($this->connection_libray)
         {
             case 'MySQLi':
                 print "Bruger MySQLi";
                 break;
             case 'MySQL':
                 print "Bruger MySQL";
                 MySQL_connect();
                 break;
         }
     }
     
     private function MySQL_connect()
     {
         $this->db_link = @mysql_connect($this->db_hostname, $this->db_username, $this->db_password);
         
         if(@mysql_select_db($this->db_database, $this->db_link))
         {
             print "Blev oprettet.";
         }
     }
 }

Det giver et lidt bedre billede af hvad jeg gør.

I længden ville al den if, else programmering nok tage en del af serverens tid. Hvad med at lade brugeren selv bestemme hvilken database han vil bruge, og så opbygge to klasser. En til hver af database mulighederne og så måske arver fra en database klasse?

Det er nemlig det jeg tænkte på, for der vil komme en del switch og jeg kunne godt ønske at der måske var en mere effektiv måde. Men jeg aner ikke om det er langsomere at bruge switch i forhold til if-else, da jeg aldrig har haft brug for at overveje det.

Det gør sig også gældne med udskrivning af fejl. Måske skulle jeg lave noget custom error handling for at få mindre if-else.

custom error handling er en god idé.

Men, if-sætningers performance-krav er altså ikke så slemt, at du skal opbygge hele din arkitektur ud fra hvor mange af dem du har. Bare hav det en smule i baghovedet mens du koder

MySQLi er ikke mere sikkert end MySQL driveren i den forstand at man kan lave de samme dumhedsfejl med begge drivere.

MySQLi er en integreret driver til PHP hvor MySQL drivere er en udvidelse. MySQLi driverens force ifht. MySQL er muligheden for prepared statements og hvis man kigger den vej, er det mere sikkert, da MySQLi laver autoescaping på alt det "farlige". Det gør den ikke ved alm. statements.

Den mest omtalte forskel der er på de to drivere, er deres integration. Da MySQL er et modul/en udvidelse til PHP sige det at være langsommere til at afvikle set i forhold til MySQLi, da denne driver er udviklet specifikt til PHP og kommunikationen med databasen. Om der er hold i det ved jeg ikke. Men jeg ved at prepared statements er en super idé og fungerer optimalt for mit vedkommende.

Selvom der ikke rigtig er nogen grund til det, er det en udemærket idé at udvikle to forskellige komponenter til hhv. MySQL og MySQLi. Det giver rig mulighed for at lære hvordan man laver abstraktionsklasser mm.
Men det er dumt at lave et interface paradigm der beskriver strukturen for en MySQL klasse og et andet interface der beskriver strukturen for en MySQLi klasse. Det skal være mere generelt, så det kan bruges flere steder. Så kan interfacet måske hedde "DatabaseInterface".

...og nu gider jeg ikke skrive mere.

Det vil såmend fungere fint nok, skulle jeg mene.

Selvom jeg nok ville validere hvilken forbindelse der skulle bruges, og derefter forbinde, i ét hug. Istedet for først validere, så smide ind i en variabel, og så validere variablen, og så forbinde. Det er alligevel nogle led man skærer bort

Nu er det selvfølgelig lidt svært at se ud fra det eksempel, men sådan her ser min aktuelle klasse ud.

 class database
 {
     private $connection_libray;
     private $db_hostname, $db_username, $db_password, $db_database;
     private $db_link;
     private $error_array = array();
 
     function __construct()
     {
         if(class_exists("MySQLi") || function_exists("mysql_connect"))
         {
             if(class_exists("MySQLi"))
                 $this->connection_libray = "MySQLi";
             else
                 $this->connection_libray = "MySQL";
         }
         else
         {
             die("Error: Your server haven't support for MySQL or MySQLi. You need one of them if you wish to use this libray.");    
         }
     }
     
     function connect()
     {
         switch($this->connection_libray)
         {
             case 'MySQLi':
                 print "Bruger MySQLi";
                 break;
             case 'MySQL':
                 print "Bruger MySQL";
                 MySQL_connect();
                 break;
         }
     }
     
     private function MySQL_connect()
     {
         $this->db_link = @mysql_connect($this->db_hostname, $this->db_username, $this->db_password);
         
         if(@mysql_select_db($this->db_database, $this->db_link))
         {
             print "Blev oprettet.";
         }
     }
 }

Det giver et lidt bedre billede af hvad jeg gør.

Ja det har jeg skam også overvejet, men jeg har læst mig til at MySQLi skulle være mere sikkert, så derfor synes jeg at jeg ville bygge en klasse der så brugte det sikreste at bruge uden at brugeren skal tænke over det.

Ah, ja. Med den indfaldsvinkel er min fremgangsmåde nok næppe ønskelig.

Isåfald synes jeg, din fremgangsmåde er fin. Du skal som Jakob er inde på, være opmærksom på hvor ofte du bruger if-sætninger - men jeg tvivler på det er noget der sænker den her klasse.

Det er nemlig det jeg tænkte på, for der vil komme en del switch og jeg kunne godt ønske at der måske var en mere effektiv måde. Men jeg aner ikke om det er langsomere at bruge switch i forhold til if-else, da jeg aldrig har haft brug for at overveje det.

Det gør sig også gældne med udskrivning af fejl. Måske skulle jeg lave noget custom error handling for at få mindre if-else.

Godt så. Så vil jeg prøve at arbejde videre med det, så må vi jo se hvad der kommer ud af det

Jeg fordeler point i morgen, så hvis der kommer andre så kan de stadigvæk nå at få indført et ord.

Jeg ville nok vælge en anden løsning end hele tiden at tjekke:

abstract class MySQLInterface {
    public abstract function connect($host, $user, $pass);
    public abstract function query($sql);
}

class MySQL extends MySQLInterface {
    private $concrete;

    public function __construct() {
        if(class_exists("MySQLi")) {
            $concrete = new MySQLiImplementation();
        } else if function_exists("mysql_connect")) {
            $concrete = new MySQLOldImplementation();
        } else {
            die ("No MySQL");
        }
    }

    public function connect($host, $user, $pass) {
        return $concrete->connect($host, $user, $pass);
    }

    public function query($sql) {
        return $concrete->query($sql);
    }
}

public class MySQLiImplementation extends MySQLInterface {
    public function connect($host, $user, $pass) {
        return mysqli_connect($host, $user, $pass);
    }
    ...
}

public class MySQLOldImplementation extends MySQLInterface {
    public function connect($host, $user, $pass) {
        return mysql_connect($host, $user, $pass);
    }
    ....
}

På den måde skal du oprette et objekt af MySQL klassen som vælger at oprette et objekt af enten MySQLiImplementation eller MySQLOldImplementation klassen som den delegerer al arbejdet til.