Discussion :

[laurentSc]

Bonsoir,

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

livres.php :

Code php :

Sélectionner tout - Visualiser dans une fenêtre à part

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

avec la classe LivreManager :

Code php :

Sélectionner tout - Visualiser dans une fenêtre à part

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

( ! ) 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 ?

[Séb.]

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

Pense à l'initialiser.

[laurentSc]

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 ?

[Séb.]

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)

[laurentSc]

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 ?

[Séb.]

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

[ypcman]

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

Code PHP :

Sélectionner tout - Visualiser dans une fenêtre à part

return count($this->livres);

[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é ?

[ypcman]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

foreach( $livres as $livre )

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

private $livres = [];

ou

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

private $livres = array();

[laurentSc]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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...

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

private $livres = [];

ou

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

private $livres = array();

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

[ypcman]

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.