Parameter must be an array or an object that implements Countable

Bonsoir,

j'obtiens le message d'erreur donné dans le titre et je ne parviens pas à l'éliminer...

livres.php :

Code:

---

1 2 3

$livres = $livreManager->getLivres(); for($i=0; $i < count($livres);$i++) : etc

---

avec la classe LivreManager :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

class LivreManager extends Model implements Countable { private $livres; private $count = 0;   public function count() { return ++$this->count; }   public function getLivres(){ return $this->livres; } ...

---

Ca me génère

Citation:

---

( ! ) Warning: count(): Parameter must be an array or an object that implements Countable in C:\projets\dvp\e2\biblio\livres.php on line 2

---

Ayant cette erreur, j'ai ajouté à la classe LivreManager le implements Countable et la méthode count() (lignes 5 à 7) mais ça ne suffit pas. Comment faire ?

C'est la propriété $livres qui n'implémente pas Countable.

Pense à l'initialiser.

Merci, j'ai rajouté l'init de la propriété à la classe LivreManager et je n'ai plus l'erreur :
private $livres = [];.

Quel rapport entre l'init de cette propriété et l'implémentation de l'interface, STP ?

Tu fais un foreach sur $livres, variable venant de la propriété du même nom de $livresManager.

Si la propriété n'est pas valorisée c'est comme si tu faisais un foreach sur NULL.

L'implementation de Countable ne sert à rien ici. Ça pourrait servir si tu faisais count($livresManager)

Merci pour l'explication.

C'est pas un foreach mais une boucle for ;) .

Par contre juste avant la boucle, il y a une init : $livres = $livreManager->getLivres();. Pourquoi ça suffit pas ?

Pardon, c'est pas le foreach qui pose problème mais le count sur un élément non initialisé / non tableau

::getLivres retourne la propriété ::$livres, si $livres n'est pas Countable ça bloque, un tableau est Countable

Intuitant que $livres est un tableau de livres
Ligne 6 de la déclaration de classe :

Code:

---

return count($this->livres);

---

Merci ; ici, ça fonctionne car justement, le seul endroit où on utilise count porte sur $this->livres mais si on voulait utiliser count sur une autre propriété ?

Citation:

---

Envoyé par laurentSc

Merci ; ici, ça fonctionne car justement, le seul endroit où on utilise count porte sur $this->livres mais si on voulait utiliser count sur une autre propriété ?

---

Tu pourrais mais, à mon sens, ça n'aurait guère de sens ... car tu cherches à connaitre la taille du tableau $livres.
Mais admettons que tu crées des objets LivreManager par auteur et que tu ajoutes donc un attribut $auteur. rien ne t'interdit de définir count comme le nombre de lettres du nom de l'auteur avec

Code:

---

return sizeof($this->auteur);

---

En revanche, je trouve que tu te compliques bien la vie(le code surtout) car pour balayer touts les valeurs d'un tableau, "foreach" est vraiment plus élégant :

Code:

---

foreach( $livres as $livre )

---

Enfin, tu pourrais aussi, dans la classe, déclarer que $livres est un tableau

Code:

---

private $livres = [];

---

ou

Code:

---

private $livres = array();

---

Citation:

---

Envoyé par ypcman

En revanche, je trouve que tu te compliques bien la vie(le code surtout) car pour balayer touts les valeurs d'un tableau, "foreach" est vraiment plus élégant :

Code:

---

foreach( $livres as $livre )

---

---

D'accord avec toi mais ce code n'est pas de moi : il vient d'une formation que je fais pour réviser les concepts PHP/POO/MVC...

Citation:

---

Envoyé par ypcman

Enfin, tu pourrais aussi, dans la classe, déclarer que $livres est un tableau

Code:

---

private $livres = [];

---

ou

Code:

---

private $livres = array();

---

---

c'est ce que j'ai fait (cf post #3)

J'ai écrit trop vite.
Seb a raison, l'implémentation de count() pour la classe LivreManager ne sert à rien puisque ton count() ne concerne pas un objet de la classe LivreManager comme $livreManager mais $livre.
Si tu voulais faire count($livreManager), alors tu devrais implémenter count pour ta classe, mais dans ton cas, c'est inutile. En revanche, déclarer $livre comme array le rend countable et supprime l'erreur.