Datenbankverbindung und Verwaltungsmethoden objektorientiert

Creamroyal

Sprache: PHP , SQL

Problem:
Ich versuche seit längerer Zeit Verwaltungsmethoden für MySQL-Abfragen, Updates usw. zu realisieren. Da keine Ausgabe erfolgt ist
meine Methode sicherlich falsch. Ich habe die Methoden deklariert in der Klasse z.B. in folgender Struktur:

Code

public function query_assoc( $sql )
{
  mysql_fetch_assoc(mysql_query($sql));
}

Dies funktioniert aber nicht, da keine Ausgabe erfolgt. (Der SQL-Befehl im Anwendungsprogramm ist einwandfrei.)
Nun, früher wurde mir schonmal gesagt, dass man so ein Problem mit Hilfe einer zweiten Klasse lösen kann. Nun wollte ich
fragen wie ich das realisieren kann, da ich noch nicht so der Profi bin.

Wäre dankbar für jede Hilfe[/code]

Dodo

ich denke du willst eher das hier:

Code

public function query_assoc( $sql )
{ return mysql_fetch_assoc(mysql_query($sql));
}

Bandit

Ist zwar noch nicht an PHP5-OOP angepasst, aber vielleicht hilft dir das weiter:

Code

<?
   /********************************************************************
   *
   * File:              class.mysql.php
   *
   * Kurzbeschreibung:  Eine Klasse zum MySQL-Handling
   *
   * Erstellungsdatum:  17.01.2004
   *
   * Letzte Änderung:
   *   ********************************************************************/


   class CDb
   {
     var $connection;
     //---------------------------------------------------------------
     function Db()
     {
        $this->connection = 0;
     }
     //---------------------------------------------------------------
     function connect($hostName, $userName, $pw)
     {
       $this->connection = @mysql_connect($hostName, $userName, $pw);


       return $this->connection;
     }
     //---------------------------------------------------------------
     function connect_db($hostName, $userName, $pw, $db_name)
     {
       $this->connection = @mysql_connect($hostName, $userName, $pw);


       if ($this->connection)
       {
          $value = mysql_select_db($db_name, $this->connection);
          if ($value == false)
          {
             db_close($this->connection);
             $this->connection = 0;
          }
       }
       return $this->connection;
     }
     //---------------------------------------------------------------
     function db_close()
     {
        if ($this->connection)
        {
           mysql_close($this->connection);
           $this->connection = 0;
        }
     }
     //---------------------------------------------------------------
     function select_db($dbName)
     {
        $result = false;


        if ($this->connection)
           $result = mysql_select_db($dbName, $this->connection);


        return $result;
     }
     //---------------------------------------------------------------
     function query($query="")
     {
        $result = false;


        if (strlen ($query) && $this->connection)
             $result = @mysql_query ($query);


        return $result;
     }
     //---------------------------------------------------------------
     function num_fields($value=0)
     {
        $result = 0;


        if ($this->connection && $value)
           $result = @mysql_num_fields ($value);


         return $result;
     }
     //---------------------------------------------------------------
     function num_rows($value)
     {
        $result = 0;


        if ($this->connection && $value)
           $result = @mysql_num_rows ($value);


         return $result;
     }
     //---------------------------------------------------------------
     function has_error()
     {
       $result = true;


       if ($this->connection)
         $result = @mysql_errno();


       return $result;
     }
     //---------------------------------------------------------------
     function error_text()
     {
       $result = false;


       if ($this->connection)
         $result = @mysql_error();


       return $result;
     }
     //---------------------------------------------------------------
     function data_seek($value=0, $position=-1)
     {
       $result = false;

       if ($value && $position >= 0 && $this->connection)
         $result = mysql_data_seek($value, $position);


       return $result;
     }
     //---------------------------------------------------------------
     function fetch_array($value=0)
     {
       $result = false;


       if ($value && $this->connection)
         $result =  @mysql_fetch_array($value);


       return $result;
     }
     //---------------------------------------------------------------
     function fetch_object($value=0)
     {
       $result = false;


       if ($value && $this->connection)
         $result =  @mysql_fetch_object($value);


       return $result;
     }

     //---------------------------------------------------------------
     function result($value=0, $num=-1)
     {
       $result = false;


       if ($value && $num >= 0 && $this->connection)
          $result =  @mysql_result($value, $num);


       return $result;
     }
     //---------------------------------------------------------------
     function inserted_id()
     {
       $result = false;


       if ($this->connection)
         $result = mysql_insert_id();


       return $result;
     }
     //---------------------------------------------------------------
     function free_query($query)
     {
       $result = false;


       if ($this->connection)
         $result = mysql_free_query ($query);


       return $result;
     }
     //---------------------------------------------------------------
     function check_error()
     {
       if ($this->connection)
       {
         if (mysql_errno ())
           die ("Datenbankfehler: " . mysql_error());
       }
     }

 }
?>

Alles anzeigen

Außerdem ist so eine verschachtelung

Code

mysql_fetch_assoc(mysql_query($sql));

eine mittlere katastophe

Creamroyal

Zitat von bandit600
Außerdem ist so eine verschachtelung
Code
mysql_fetch_assoc(mysql_query($sql));
eine mittlere katastophe

Erstmal danke an euch beiden für die Hilfe. Aber was gibt's an solchen Verschachtelungen auszusetzen?

Dodo

Hats geklappt mit dem return?
denke nämlich du wilst das was aus fetch() ausgelesen wird auch irgendwo speichern und weiterverwenden

jojo87

Zitat von Creamroyal

Erstmal danke an euch beiden für die Hilfe. Aber was gibt's an solchen Verschachtelungen auszusetzen?

Zum Beispiel erschwert es die Fehlersuche bei Fehlverhalten des Scriptes.

Gruß,
Jojo

Creamroyal

Zitat von Dodo

Hats geklappt mit dem return?
denke nämlich du wilst das was aus fetch() ausgelesen wird auch irgendwo speichern und weiterverwenden

Jo das mit dem return hat geklappt. Zieht eine Verschachterlung auch Sicherheitsbedenken mit sich (oder erschwert
es nur die Analyse von Fehlern usw.?)

Nunja, ich wollte gerne eine Gildensoftware schreiben. Prozedual ist das kein Thema, allerdings wollte ich das gerne objektorientiert
gestalten, damit ich mich so weiterentwickeln kann und dazu lerne.

Dodo

Dann gib einfach die Verschachtelung weg?

Code

public function query_assoc( $sql )
{ $daten = mysql_query($sql);
  $ausgelesen = mysql_fetch_assoc($daten);
  return $ausgelesen;
}

Creamroyal

Zitat von Dodo

Dann gib einfach die Verschachtelung weg?

Code

public function query_assoc( $sql )
{ $daten = mysql_query($sql);
  $ausgelesen = mysql_fetch_assoc($daten);
  return $ausgelesen;
}

Dann bleibt doch die Erschwerung der Fehlersuche erhalten oder irre ich mich da jetzt?

Edit: Ich habe einen verallgemeinernden Weg (sofern dieser korrekt ist) gefunden:

Code

public function query( $sql, $type ) {
  $query = mysql_query($sql);
  return $result = $type($query);
}

Diese Methode soll flexibel sein, sprich es können verschiedene Mysql-Methoden eingesetzt werden (mysql_fetch_assoc, mysql_fetch_array, mysql_num_rows usw.)
Wäre diese Lösung ein guter Weg oder würdet ihr mir davon abraten?

Bandit

Es ist nicht eine der dümmsten ideen, nach mysql_query einen eventuellen fehler abzufangen, alles andere wäre eine schlechte programmierung.

Dodo

Mach mal diesen hier

Code

public function query( $sql, $type ) {
  $query = mysql_query($sql) OR die(mysql_error());
  return $result = $type($query);
}

Bandit

Eine klasse, die mein script bei einem fehler abbricht, würde ich nicht benutzen!

Code

public function query( $sql, $type ) 
{ 
  $result = false;

  $query = mysql_query($sql);

  if (! mysql_errno())
   $result = $type($query);


  return $result;   
}

Alles anzeigen

No0ob

Mal ganz doof gefragt:
Was ist denn $type($query); ?

Creamroyal

Zitat von No0ob

Mal ganz doof gefragt:
Was ist denn $type($query); ?

$type soll die Art des Befehls bestimmen sprich

z.B. grob gesagt:

$type = mysql_fetch_assoc;
dann lautet die Syntax = mysql_fetch_assoc($query);

Andersrum z.B. auch:

$type = mysql_num_rows;
dann lautet die Syntax = mysql_num_rows($query);

No0ob

Ok, das hab ich mir gedacht. Der Codestil hat mich nur etwas verwirrt.
Normalerweise sollte man sowas ueber Treiber loesen, sodass du dann Sub-Klassen fuer die verschiedenen Verbindungstypi definierst und dann einbindest.
Es kommt eventuell vor, dass eine Funktion andere Parameter erwartet, je nachdem welche Datenbank du hast.

Tags