Problem mit OOP

Diese Seite verwendet Cookies. Durch die Nutzung unserer Seite erklären Sie sich damit einverstanden, dass wir Cookies setzen. Weitere Informationen

  • Problem mit OOP

    Hi

    ich habe leider noch riesige Probleme mit der OOP. Liegt zum Teil an der fehlenden Übung zum anderen daran, dass ich den Sinn wohl nocht nicht ganz durchschaut habe. Jedenfalls habe ich eine MySQLi-Klasse gefunden:

    Quellcode

    1. class DB
    2. {
    4. /**
    5. * @var <str> The mode to return results, defualt is MYSQLI_BOTH, use setFetchMode() to change.
    6. */
    7. public $fetchMode = MYSQLI_BOTH;
    8. /**
    9. * @desc Creates the MySQLi object for usage.
    10. *
    11. * @param <arr> $db Required connection params.
    12. */
    13. public function __construct($db) {
    14. $this->mysqli = new mysqli($db['host'], $db['user'], $db['pass'], $db['table']);
    16. if (mysqli_connect_errno())
    17. {
    18. printf("<b>Connection failed:</b> %s\n", mysqli_connect_error());
    19. exit;
    20. }
    21. }
    23. /**
    24. * @desc Simple preparation to clean the SQL/Setup result Object.
    25. *
    26. * @param <str> SQL statement
    27. * @return <bln|null>
    28. */
    29. public function query($SQL)
    30. {
    32. $SQL = $this->mysqli->real_escape_string($SQL);
    33. $this->result = $this->mysqli->query($SQL);
    35. if ($this->result == true)
    36. {
    37. return true;
    38. }
    39. else
    40. {
    41. printf("<b>Problem with SQL:</b> %s\n", $this->SQL);
    42. exit;
    43. }
    44. }
    45. // und so weiter
    46. }
    Alles anzeigen


    Diese binde ich per extend in eine zweite Klasse ein und möchte von dieser Klasse Chat auf die Funktion query() zugreifen.

    Quellcode

    1. class Chat extends DB {
    3. public $chat_id;
    4. public $last_id;
    5. public $db;
    7. public function __construct($config) {
    8. $db = parent::__construct($config);
    9. }
    11. public static function getLines($chat_id, $last_id) {
    13. $select = $this->db->query("SELECT * FROM `lines` JOIN `user` ON lines_author = user_id WHERE lines_chat = '".$chat_id."' AND lines_id > '".$last_id."' ORDER BY lines_id") or die(mysqli_error);
    14. }
    15. }
    Alles anzeigen


    Aufruf:

    Quellcode

    1. $config = array();
    2. $config['host'] = 'localhost';
    3. $config['user'] = 'root';
    4. $config['pass'] = '';
    5. $config['table'] = 'project';
    6. $chat = new Chat($config);
    8. echo $chat->getLines(1, 0);



    Nur erhalte ich immer Fehlermeldungen. Die letzte ist jetzt:

    Fatal error: Using $this when not in object context in C:\xampp\htdocs\project\Chat.class.php on line 15

    An dieser Stelle komme ich leider wirklich nicht mehr weiter, denn egal wie ich die Variable anspreche erhalte ich immer Fehlermeldungen, dass die Variable nicht definiert ist. Bitte klärt mich auf. Gerne auch Meinungen zum Code bis jetzt, da ich überzeugt bin, dass einige Programmierer kurz vor dem Herzinfarkt waren, als sie meinen Code gesehen haben^^

    Danke schonmal

  • Wenn ich mich nicht vollkommen irre, dann liegt es daran, dass die Methode vererbt wird und nicht die db-Klasse als Variable der Chatklasse erhält. Also einfach $this->query('Anfrage'); ... Den SQL-String würde ich aber aus performanten Gründen in Single Quotes schreiben und anstatt SELECT * die einzelnen Spalten wirklich definieren;)

    Wenn du noch Probleme mit der OOP hast, solltest du villeicht keine Klassen aus dem Internet übernehmen, sondern lieber selber schreiben ;)

  • Warum ist die getLines Funktion statisch?

    Ich habe keine Ahnung mehr warum^^ Ist entfernt. Auch den Rat von JaffaCoder hab ich befolgt. Neue Meldung:

    Notice: Undefined property: Chat::$SQL in C:\xampp\htdocs\project\DB.class.php on line 116
    Problem with SQL:

    Komischerweise wird auch keine SQL-Fehlermeldung ausgegeben. Heißt das, dass die Verbindung schon nicht aufgebaut wird?

    PS: Der SELECT ist noch nicht fertig, das * bleibt da nicht

  • Kaotisch würd ich dazu sagen. Willst du wirklich jedes mal in jeder deiner Klassen wo du eine Datenbank braucht mit extends die Datenbank Klasse einbinden?
    Was ist wenn mal auch andere Klassen eingebunden werden solln du kannst nur immer bei extends eine Klasse angeben.

    Zuerstmal würd ich dir generell zu neueren moderenen Methode PDO für den Datenbank zugriff raten.
    Am besten ruft du die Datenbank statisch auf so kanst du überall und in jeder Klasse darauf zugreifen ohne sie vorher einbinden zu müssen.

    Weiters würd ich Singleton bei einer Datenbank Klasse empfehlen und auch hier sind verschiedene Verbindungen möglich je nachdem wie die Klasse geschrieben wurde.

    Hier mal schnell ein Ungetestetes Beispiel einer Singlton Datenbank Klasse passierend auf PDO.
    (Wer Fehler findet darf sie hier Posten sowie die Verbesserungs vorschläge sind Natürlich immer Willkommen.)

    Die Config

    Quellcode

    1. <?php defined('SYSPATH') or die('No direct access allowed.');
    3. return array
    4. (
    5. 'default' => array
    6. (
    7. 'hostname' => 'localhost', # Datenbank Server
    8. 'database' => 'dbname', # Datenbank Name
    9. 'username' => 'user', # Benutzename
    10. 'password' => 'pass', # Password
    11. 'persistent' => FALSE,
    13. 'prefix' => '', # Tabellen Prefix
    14. 'charset' => 'utf8', # Zeichensatz
    15. 'caching' => FALSE,
    16. ),
    17. 'db1' => array
    18. (
    19. 'hostname' => 'localhost', # Datenbank Server
    20. 'database' => 'dbname', # Datenbank Name
    21. 'username' => 'user', # Benutzename
    22. 'password' => 'pass', # Password
    23. 'persistent' => FALSE,
    25. 'prefix' => '', # Tabellen Prefix
    26. 'charset' => 'utf8', # Zeichensatz
    27. 'caching' => FALSE,
    28. ),
    30. );
    Alles anzeigen


    Die zugehörige Datenbank Klasse

    Quellcode

    1. <?php
    2. /**
    3. * Datenbank Klasse
    4. * @version 1.2
    5. * @author Splasch
    6. * Filename: dbpdo.php
    7. * Verbindung über die PDO herstellen
    8. **/
    10. final class Dbpdo
    11. {
    12. protected static $db_obj = null; # Datenbank Objekt PDO
    13. private $db_file = 'dbconfig.php'; # Configuration Datei
    14. public $error; = ''; # Fehler Varibale
    15. private $anfragen; # Sql Anfragen zählen
    17. /**
    18. * PDO verbindung aufbauen
    19. * Verbindungsdaten abfragen
    20. */
    21. private function __construct($instance)
    22. {
    23. $config = $this->get_config($instance);
    25. $db = "mysql:dbname=".$config['database'].";host=".$config['hostname'];
    26. try
    27. {
    28. self::$db_obj[$instance] = new PDO($db, $config['username'], $config['password'],array
    29. (
    30. PDO::ATTR_PERSISTENT => false,
    31. PDO::ATTR_ERRMODE => PDO::ERRMODE_WARNING
    32. #PDO::ERRMODE_EXCEPTION
    33. #PDO::MYSQL_ATTR_USE_BUFFERED_QUERY => true
    34. )
    35. );
    36. $this->exec("SET CHARACTER SET ".$config['charset']);
    37. }
    38. catch (PDOException $e)
    39. {
    40. $this->error.='Fehler beim Öffnen der Datenbank: ' . $e->getMessage();
    41. echo $this->error; #Fehler ausgabe
    42. }
    44. }
    46. /**
    47. * get_config
    48. * Konfigurations Daten (Verbindungsdaten)
    49. * Standard einstellung Default
    50. * @param instance (string)
    51. */
    52. private function get_config($instance = 'default')
    53. {
    54. if(is_file($this->db_file))
    55. {
    56. $config = include $this->db_file;
    57. return $config[$instance];
    58. }
    59. $this->error.='Konfiguration Datei nicht gefunden.<br>';
    60. return false
    61. }
    63. /**
    64. * Eine Instanz des PDO Objektes übergeben
    65. * @return Objekt $db_obj
    66. */
    67. public static function get_db($instance = 'default')
    68. {
    69. if(!isset (self::$db_obj[$instance]))
    70. {
    71. self::$db_obj[$instance] = new Dbpdo($instance);
    72. }
    73. return self::$db_obj[$instance];
    74. }
    76. /**
    77. * exec
    78. * Auch exec Anfragen mit zählen
    79. * @param sql (string)
    80. **/
    81. public function exec($sql)
    82. {
    83. $this->anfragen ++;
    84. return parent::exec($sql); // Funktion der Eltern Klasse zurückgeben
    85. }
    87. /**
    88. * execute
    89. * Auch execute Anfragen mit zählen
    90. * Nach prepare
    91. * @param params (array)
    92. **/
    93. public function execute($params = array())
    94. {
    95. $this->anfragen ++;
    96. return parent::execute($params);
    97. }
    99. /**
    100. * get_anfragen
    101. * Gesamt Sql anfragen zurück geben
    102. **/
    103. public function get_anfragen()
    104. {
    105. return $this->anfragen;
    106. }
    108. } // Ende Klasse
    109. ?>
    Alles anzeigen


    Mfg Splasch

    Dieser Beitrag wurde bereits 6 mal editiert, zuletzt von splasch ()