Return gibt nichts zurück!

[Juka]

Hey Community, ich habe irgendwie ein kleines Problem.
Meine Funktion gibt kein Return zurück. Hier mal zu meinem Code:

    public function getPosts($topic_id) {
        $topic_id = $this->mysqli->real_escape_string($topic_id);
        $result = $this->mysqli->query("SELECT * FROM ncms_forum_posts WHERE `topic_id` = '".$topic_id."'");
        if($result->num_rows > 0) {
            $forumPosts = array();
            while($row = $result->fetch_assoc()) {
                $forumPosts[] = $row;
            }
            return $forumPosts;
        } else {
            return "Es hat noch niemand zu diesem Post einen Beitrag geschrieben!";
        }
    } 

Nunja, sonst bekomme ich folgende Fehlermeldung zurück, als mein Return:
Warning: Invalid argument supplied for foreach() in C:\xampp\htdocs\templates\post.tpl.php on line 52

Ist ja klar, weil es NULL ist bzw. keine Posts hat.
Wenn ich nun ein Post erstelle funktioniert alles, aber ich will falls kein Post vorhanden ist das folgendes zurückgegeben wird:
"Es hat noch niemand zu diesem Post einen Beitrag geschrieben!"

- Juka

[nikosch]

Ist ja klar, weil es NULL ist bzw. keine Posts hat.

Falsch. Is ja klar, weil Du versuchst, über die Fehlermeldung zu iterieren. Insofern guckst Du in den falschen Codeteil.

[Juka]

Falsch. Is ja klar, weil Du versuchst, über die Fehlermeldung zu iterieren. Insofern guckst Du in den falschen Codeteil.

Zeile 52:

<?php foreach($Forum->getPosts($_GET['id']) as $post) { ?>

[nikosch]

Ja.

[Juka]

Ja.

Ich verstehe nicht genau, wo Du nun hinaus willst. :S

#EDIT - Ich habe mal var_dump() auf $Forum->getPosts($_GET['id']) gemacht, nun bekomme ich meinen Return. Wie mach ich das genau mit der Foreach Schleife?

[Chris.]

Ich verstehe nicht genau, wo Du nun hinaus willst. :S

• Wenn du MySQLi verwendest, dann verwende doch direkt Prepared Statements.
• Der folgende Rückgabewert ist ein String und kein Array, weshalb foreach damit nichts anfangen kann:
  
  

  return "Es hat noch niemand zu diesem Post einen Beitrag geschrieben!"; 

[tr0y]

Warum antwortet ein Postings-Getter mit einem String wenn man von ihm eigentlich erwartet ein Array, das optional Posts enthält, zurückzugeben ?

Ich würde die ganze Methode überdenken, wenn ich du wär.

[lstegelitz]

Ich verstehe nicht genau, wo Du nun hinaus willst. :S

#EDIT - Ich habe mal var_dump() auf $Forum->getPosts($_GET['id']) gemacht, nun bekomme ich meinen Return. Wie mach ich das genau mit der Foreach Schleife?

1. Die "getPosts" Methode sollte IMMER ein Array zurückliefern (leer oder gefüllt).
2. Nachdem die "getPosts" aufgerufen wurde, wird der Rückgabewert überprüft. Wenn "leer" => Meldung, wenn gefüllt => foreach Schleife.

[Günni]

Oder einfach . . .

public function getPosts($topic_id) {
 // Code . . .
 else{
      return array("Es hat noch niemand zu diesem Post einen Beitrag geschrieben!");
     }
} 

Gruß

[jprangenberg]

Oder einfach . . .

public function getPosts($topic_id) {
 // Code . . .
 else{
      return array("Es hat noch niemand zu diesem Post einen Beitrag geschrieben!");
     }
} 

Gruß

Nein. Die Methode getPosts() gibt die Posts eines Forum-Threads zurück - und zwar nur Posts eines Forums-Threads. Keine Messages.

Bei dem Aufruf deiner Methode kann man sich nie sicher sein was der Rückgabewert nun genau sein wird.

[tr0y]

Warum antwortet ein Postings-Getter mit einem String wenn man von ihm eigentlich erwartet ein Array, das optional Posts enthält, zurückzugeben ?

Ich würde die ganze Methode überdenken, wenn ich du wär.

1. Die "getPosts" Methode sollte IMMER ein Array zurückliefern (leer oder gefüllt).
2. Nachdem die "getPosts" aufgerufen wurde, wird der Rückgabewert überprüft. Wenn "leer" => Meldung, wenn gefüllt => foreach Schleife.

Nein. Die Methode getPosts gibt die Posts eines Forum-Threads zurück - und zwar nur Posts eines Forums. Keine Messages. Eine Methode hat genau eine Aufgabe!

[lstegelitz]

Würde ich nicht so machen... die Methode heisst "getPosts" und das ist ihre Aufgabe. Ein leeres Array signalisiert von ganz alleine, das noch keine Posts gemacht wurden - ein Array mit einem Eintrag hingegen suggeriert, das tatsächlich ein Post vorhanden ist.

edit: zu spät

[jprangenberg]

Hups, ich habe den Post von lstegelitz gekonnt überlesen! Sorry!

[Gast]

// auch mal ne frage
Ok, getPosts gibt mir ein array, leer oder gefüllt, wenn sie erfogreich durchgelaufen ist.
falls es aber4 zu irgendwelchen problemen kommt, im aufruf der methode(falsche argumente/falsche(r) datentyp/anzahl) oder im erfassen der Posts umstellen der schnittstellen, wathever, wie kann ich dann nen fehler zurückgeben? ne exeption werfen?

auch wenn sich das jetzt sehr akademisch anhört,. so kämpfe ich doch ab und an mit sowas.

[nikosch]

Exception werden oder gleich einen Iterator, kein Array zurückgeben, der dann auch eine Meldung enthalten kann.