Discussion :

[absot]

Bonjour, je me pose une question sur comment bien faire.

Je ne sais pas si cela porte un nom mais concrètement, si on prend l'exemple que je souhaite connaître obtenir les concerts d'un artiste, dans quel modèle dois-je implémenter cette fonction?

Dans le modèle Artiste ou Concert?

J'ai toujours entendu qu'il fallait se dire "à qui est-ce que je demande ça?"

Mais de ma faible expérience dans le développement, je ne sais pas trop quoi faire parce qu'il y a des avantages et des inconvénients dans ces deux cas de figures.

Je m'en remet donc à vous et à vos expériences.

[grunk]

J'ai presque envie de dire les deux mais ça va surtout dépendre de ton besoin

Si tu n'as jamais besoin de retourner autre chose que les concerts d'un artiste une méthode dans artiste me semble appropriée. meme pas besoin de retourner un tableau d'objet un tableau extrait de la bdd est largement suffisant.

Code :

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

1
2
3
4
5
6
7
class Artist
{
	public function getConcert()
	{
              return $tabConcert
	}
}

Par contre si tu as besoin de retourner tous les concerts de plusieurs artistes il peut ête intéressant de mettre cette méthode dans concert.
Par exemple si $artist est un objet Artist turetourne les concert d'une seule personne si c'est un tableau d'id ou d'objet tu retourner tous les concerts de toutes ses personnes plutôt que de devoir appeler x fois getConcert() sur les instances d'artiste

Code :

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

1
2
3
4
5
6
class Concert
{
	public function getFor($artist)
	{
	}
}

Mais siono pas de règle particulière (enfin j'en ai pas en tout cas)

[absot]

J'essaie de faire en sorte de retourner des objets ou tableaux d'objets dans le cas où je veux plusieurs objets (valeurs extraites d'une BDD).

Je me dis que le plus important est le résultat et je pensais qu'il y avait une façon de faire précise mais apparemment non.

[glen1789]

Bonjour,

Je ne sais pas si cela porte un nom mais concrètement, si on prend l'exemple que je souhaite connaître obtenir les concerts d'un artiste, dans quel modèle dois-je implémenter cette fonction?

Vu ta demande, moi je dirais qu'il y a bien deux bonnes solutions:

Code :

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

1
2
3
4
5
6
7
class Artist
{
	public function getConcerts()
	{
              return $tabConcert;
	}
}

Sur ce point je suis d'accord avec grunk.

Code :

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

1
2
3
4
5
6
class Concert
{
	public static function getFor($artist)
	{
	}
}

Ici je dirais qu'il faut passer par une méthode de classe car retourner une liste de concerts ne concerne pas l'instance de la classe.


Après au niveau du meilleur choix, chacun à ses préférences, mais le plus simple est d'imaginer la manière la plus simple et logique pour toi de procéder au moment de l'utilisation des classes:

Te parait-il plus clair et lisible d'écrire:

Code :

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

1
2
$artist = Artist::get($id_artist);
$liste_concerts = $artist->getConcerts();

ou:

Code :

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

1
2
$artist = Artist::get($id_artist);
$liste_concerts = Concert::getFor($artist);

La différence se fera au niveau de ce que tu considères comme le plus lisible, personnellement (j’écris aussi ce type de classes), j'ai opté pour la version 1, mais encore une fois, c'est personnel...