Frage zu OOP

  • Frage zu OOP

    Hi,

    ich habe eine Klasse DB:

    PHP-Quellcode

    1. class DB {
    2. function connect() {
    3. include ("configs.php");
    4. $connection = mysql_connect($server,$user,$pass) or die(mysql_error());
    5. mysql_select_db($db,$connection) or die(mysql_error());
    6. }
    7. function query($query) {
    8. $ergebnis=mysql_query($query,$connection) or die(mysql_error());
    9. }
    10. function close() {
    11. //mysql_free_result($ergebnis);
    12. mysql_close($connection);
    13. }
    14. }
    Alles anzeigen


    Meine Klasse Board greift auf DB zu:

    PHP-Quellcode

    1. class Board {
    2. var $Subboard = array();
    3. var $Themen = array();
    4. var $Antworten = array();
    6. function show() {
    7. $DB = new DB();
    8. $DB->connect();
    9. //$DB->query("SELECT subboard,threads,answers FROM board");
    10. $ergebnis=mysql_query("SELECT subboard,threads,answers FROM board",$connection);
    11. while ($res=mysql_fetch_assoc($ergebnis)) {
    12. array_push($this->Subboard,$res[subboard]);
    13. array_push($this->Themen,$res[threads]);
    14. array_push($this->Antworten,$res[answers]);
    15. }
    16. foreach($this->Subboard as $var) {
    17. echo "Subboard: " . $var . "<br>";
    18. }
    19. foreach($this->Themen as $var) {
    20. echo "Themen: " . $var . "<br>";
    21. }
    22. foreach($this->Antworten as $var) {
    23. echo "Antworten: " . $var . "<br>";
    24. }
    25. $DB->close();
    26. }
    28. function add() {
    29. $DB->connect();
    30. mysql_query("INSERT INTO board (subboard,answers,hits) VALUES ('allgemein','5','20')",$connection) or die(mysql_error());
    31. $DB->close();
    32. }
    33. }
    Alles anzeigen


    Allerdings kann $DB->querey() nichts bewirken, da $connection schon verfallen ist. Wie kann ich das ganze als Klassenattriubut anglegen, so dass ich immer drauf zugreifen kann?

    Einfach so schreiben geht ja nicht wirklcih:

    var $connection= mysql_connect($server,$user,$pass) or die(mysql_error());

    Oder doch?

    Danke!

    KageMurai
    Instant URL - just add Gamers!

  • Meiner Meinung nach kannst du auf die Variable $connection gar nicht zugreifen, weil sie nur innerhalb der Methode connect besteht.

    Du hast doch ausserdem auch extra eine query_method in deiner DB-Klasse, warum benutzt du die dann nicht?! Zusätlzihc brauchst die die Verbindungsvariable, wenn du sowieso nur eine Datenbankverbindung aufrufst gar nicht speichern. Da mysql_query, automatisch eine bestehende Connection nimmt.

    Wenn du das aber alles 100% korrekt machen wollen würdest müsstest du das so machen (PHP5 vorrausgesetzt):

    Quellcode

    1. class DB {
    2. private $connection;
    4. public function connect() {
    5. include ("configs.php");
    6. $this->connection = mysql_connect($server,$user,$pass) or die(mysql_error());
    7. mysql_select_db($db,$this->connection) or die(mysql_error());
    8. }
    9. public function query($query) {
    10. return mysql_query($query,$this->connection) or die(mysql_error());
    11. }
    12. public function close() {
    13. mysql_close($this->connection);
    14. }
    15. }
    16. }
    Alles anzeigen


    Und innerhalb der Boardklasse:

    Quellcode

    1. class Board {
    2. (...)
    4. $DB = new DB();
    5. $DB->connect();
    6. $ergebnis = $DB->query("SELECT subboard,threads,answers FROM board");
    8. while ($res=mysql_fetch_assoc($ergebnis)) {
    9. array_push($this->Subboard,$res[subboard]);
    10. array_push($this->Themen,$res[threads]);
    11. array_push($this->Antworten,$res[answers]);
    12. }
    13. foreach($this->Subboard as $var) {
    14. echo "Subboard: " . $var . "<br>";
    15. }
    16. foreach($this->Themen as $var) {
    17. echo "Themen: " . $var . "<br>";
    18. }
    19. foreach($this->Antworten as $var) {
    20. echo "Antworten: " . $var . "<br>";
    21. }
    22. $DB->close();
    23. }
    24. }
    Alles anzeigen

  • Erstmal vielen dank für die Antworten!

    Hat mir sehr geholen!

    Es gibt allerdings ein weiteres kleines Problem:

    Ich rufe mit $ergebnis = $DB->query("SELECT subboard,threads,answers FROM board"); die Funktion query auf. Sie wird ausgeführt und der erste Parameter wird richtig übergeben (habe ihn mir ausgeben lasse). Er lautet wirklich: SELECT subboard,threads,answers FROM board. Dann weißt die Funktion query() einfach das Ergebnis der Abfrage an die Variable $ergebnis zurück. Wieso wird $ergebnis aber der Wert 1 zugewiesen?

    Danek!

    KageMurai
    Instant URL - just add Gamers!