Discussion :

[mariek]

Bonjour,
Je voudrais connaître votre avis sur une question concernant les classes et les méthodes.
J'ai une classe cl_image, avec une méthode getAllImages qui va chercher une liste de résultats.
Selon vous, que vaut-il mieux faire, et est-ce cela a une incidence (notamment niveau performances) ?
1) faire une méthode getAllImages qui va prendre toutes les infos sur les images (nom, id, taille,etc, tout ce qui est stocké dans la base concernant cette image)
ex :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
function getAllImages () {
 
	$allImages = array();
	$sql = "select image_id, image_comment, image_taille from images";
	$req = mysql_query($sql);
	while($row=mysql_fetch_object($req)) {
		$allImages[$row->image_id]["comment"] = $row->image_comment;
		$allImages[$row->image_id]["taille"]= $row->image_taille;
	}
 
	return $allImages;
}

2) Ou bien plutôt utiliser getAllImages pour aller chercher juste un ID et ensuite utiliser une méthode pour aller chercher des infos à partir de cet ID ?
Ex:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
function getAllImages () {
 
	$allImages = array();
	$sql = "select image_id from images";
	$req = mysql_query($sql);
	while($row=mysql_fetch_object($req)) {
		$allImages[] = $row->image_id;
	}
 
	return $allImages;
}
function getImageInfo ($image_id) {
	$imageInfo= array();
 
	$sql = "select image_taille, image_comment from images where image_id=".$image_id;
	$req = mysql_query($sql);
 
	$row = mysql_fetch_object($req);
	$imageInfo["taille"] 	= $row->image_taille;
	$imageInfo["comment"] = $row->image_comment;
 
	return $imageInfo;
}

Merci pour votre participation.

Mariek

[MaitrePylos]

Dans ta deuxième solution, la méthode all, n'est pas tout a faits correcte, tu oublies l'appel à getImagesInfos

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
function getAllImages () {
 
	$allImages = array();
	$sql = "select image_id from images";
	$req = mysql_query($sql);
	while($row=mysql_fetch_object($req)) {
		$allImages[] = $this->getImageInfo($row->image_id);
	}
 
	return $allImages;
}

Pour ta question de performance, disons que tu as 100 photos :

Vaux-t-il mieux faire 1 appel à la db et avoir toutes les infos d'un coup ou faire 100 appels à la db pour avoir 1 info à la fois

Par contre dans ton deuxième expemple, est que tu pourras récupérer 1 info à la fois, ce qui pourras être intéressant dans certain cas.

[mariek]

Merci pour ta réponse.
Dans la 2e version, je n'appelle pas la 2e méthode dans la 1e, mais dans mon script php, de cette manière :

Code :

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

1
2
3
4
5
6
7
 
$AllImages = $cl_images->getAllImages();
 
foreach($AllImages as $image_id) {
     $curr_icon = $ps_icon->getIconInfo($row->icon_id);
     [...]
}

Tu penses qu'il vaut mieux l'appeler dans la première méthode ?
Mais tu réponds à ma question : il vaut mieux taper dans la base une fois que 100 fois, tu as raison. merci !

Pour récupérer un seul résultat, il suffit d'ajouter un paramètre :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
// $cond : string condition (par exemple : " where image_id=5")
function getAllImages ($cond="") {
 
	$allImages = array();
	$sql = "select image_id, image_comment, image_taille from images".$cond;
	$req = mysql_query($sql);
	while($row=mysql_fetch_object($req)) {
		$allImages[$row->image_id]["comment"] = $row->image_comment;
		$allImages[$row->image_id]["taille"]= $row->image_taille;
	}
 
	return $allImages;
}

Je pense même utiliser 3 paramètres, un pour la condition, un autre pour l'ordre et un 3e pour la limite, comme ça ma méthode est complètement souple et je peux l'utiliser pour des listes paginées et ordonnées.

[MaitrePylos]

Personnellement, je n'aime pas trop ta façon de vouloir faire.

Je préfère avoir plusieurs méthodes, qui soit bien clair e qui ne font qu'un chose, que d'avoir des méthodes, qui vont faire a peu près tout.

Question de clarté !

[mariek]

Personnellement, je n'aime pas trop ta façon de vouloir faire.

Je préfère avoir plusieurs méthodes, qui soit bien clair e qui ne font qu'un chose, que d'avoir des méthodes, qui vont faire a peu près tout.

Question de clarté !

Je débute en POO, je n'ai pas forcément de bons réflexes. J'ai tendance à vouloir compresser un maximum, par exemple
Est-ce que en POO, il vaut mieux avoir plusieurs méthodes qui font à peu près la même chose, mais pas complètement (par exemple mes deux méthodes de l'exemple 2) ?

[MaitrePylos]

Tu dois surtout éviter des situation qui pourrais compromettre la sécurité.

Et oui je préfère deux méthodes.

[mariek]

Merci pour tous ces éclaicissements !