PHP OO Error con foreach PDO

[ILuzbel]

Hola a todos estoy intentando acceder a una noticia via get imprimiendo en la url la id de la noticia y el titulo.

en el index hice un enlace de este modo:

Código PHP:

 <p><a href="noticia.php?id=<?php echo $d['id_notice']; ?>&title=<?php echo str_replace(" ", "-", $d['title']); ?>"><?php echo $d['title']; ?></a></p>  

he creado esta funcion

Código PHP:

      public function notice(){
        self::set_names();
        $sql="SELECT * FROM notices WHERE id=? AND title=?";
        $stmt=$this->dbh->prepare($sql);
        if($stmt->execute(array($_GET["id"], $_GET["title"]))){
            while ($row = $stmt->fetch()){
                $this->n[]=$row;
            }
            return $this->n;
            $this->dbh=null;
        }
    } 
   

y la he llamado asi a noticia.php:

Código PHP:

 $obj= new Work();
$datos=$obj->notice();
 
    <div id="body">
        <div id="wrapper">
        <?php
        foreach ($datos as $d){
            ?>
        <p><?php echo $d['title']; ?></p>
        <?php
        }
        ?>
        </div>
    </div>
    <div id="footer">
    </div>  

El problema es que cuando entro en la noticia por ejemplo:

http://localhost/PDO/noticia.php?id=1&title=Lorem-Ipsum

me da este error

Warning: Invalid argument supplied for foreach() in C:\xampp\htdocs\PDO\noticia.php on line 12

he leido que es porque la variable datos no es un array, pero no logro arreglarlo.

Alguien sabe como solucionarlo?

Gracias de antemano

[maycolalvarez]

no entiendo para que usas una variable de la clase ($this->n), cuando solo necesitas una variable para construir el array a devolver, puede que ese sea el problema, que $this->n no exista:

Código PHP:

  $n = Array(); //creamos el array local
while ($row = $stmt->fetch()){
     $n[]=$row; //seteamos un nuevo elemento del array
}
 return $n; //devolvemos el array
 $this->dbh=null; // esta linea NUNCA se ejecutará, debe ir ANTES del Return 
   

de igual forma, ¿no es mejor usar fetchAll?: http://www.php.net/manual/en/pdostatement.fetchall.php

[ILuzbel]

Iniciado por maycolalvarez

no entiendo para que usas una variable de la clase ($this->n), cuando solo necesitas una variable para construir el array a devolver, puede que ese sea el problema, que $this->n no exista:

Código PHP:

  $n = Array(); //creamos el array local
while ($row = $stmt->fetch()){
     $n[]=$row; //seteamos un nuevo elemento del array
}
 return $n; //devolvemos el array
 $this->dbh=null; // esta linea NUNCA se ejecutará, debe ir ANTES del Return 
   

de igual forma, ¿no es mejor usar fetchAll?: http://www.php.net/manual/en/pdostatement.fetchall.php

..::EDITADO::..
Gracias por la respuesta, lo he resuelto, no funcionaba porque me equvoque al definir $n como array
Bueno no use fetchAll porque todavia estoy aprendiendo PDO
Gracias +1