检索数据库表信息OOP [重复]

Question

This question already has an answer here:

• PHP, Fatal error: Call to undefined method, why? 1 answer

I am trying to retrieve my database rows using OOP I keep getting an error though. I feel like I am missing something simple to make this work. My code is below:

Class CommentFactory {      
    public function getImage(PDO $conn){
        $stmt2 = $conn->prepare('SELECT * FROM `pictures`');
        $stmt2->execute();

        return $stmt2;
    }
}


$statement = new CommentFactory();
$statement->getImage($conn);
while ($idOfComment = $statement->fetch(PDO::FETCH_ASSOC)) {
    echo $idOfComment['photo_id'];          
}

The error I am getting is:

Fatal error: Call to undefined method CommentFactory::fetch() in /var/www/CommentSystem/includes/Classes.php on line 29

I just recently began trying to program in OOP so my understanding is still vague.

</div>

Answer 1

You are only returning a value from the method getImage(), therefore you should do:

Class CommentFactory {

    protected $conn;

    public function getImage($conn){
        $stmt2 = $conn->prepare('SELECT * FROM `pictures`');
        $stmt2->execute();

        return $stmt2;
    }
}


$statement = new CommentFactory();
$values = $statement->getImage($conn);
while ($idOfComment = $values->fetch(PDO::FETCH_ASSOC)){
    echo $idOfComment['photo_id'];

}

Edit: However, as engvrdr noted, this is not the best you can do. You could the method to return the array, furthermore, you can pass the connection in the constructor as shown here:

Class CommentFactory {

    protected $conn;

    public function __construct($conn) {
      $this->con = $conn;
      }

    public function getImage(){
        $stmt2 = $this->conn->prepare('SELECT * FROM `pictures`');
        $stmt2->execute();
        return $values->fetchAll(PDO::FETCH_ASSOC);
    }
}


$statement = new CommentFactory($conn);
foreach ($statement->getImage() as $Image){
    echo $Image['photo_id'];    
}