Discussion :

[tioseb]

Je travail sur un projet de gestion d'inventaire dans un hôpital.
Je reprend le travail qu'un développeur à commencé mais je ne comprend pas sa class pouvez vous me m'aider à la comprendre en y mettant des commentaire si vous la comprennez merci d'avance.

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
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
Class outils
{
 
 
 
// Fonction qui affiche un champs de table indexé dans une liste déroulante,
// une connection au serveur et à la base de données est exigée
 
	function combo_liste($table,$champ,$index,$valeur,$affiche="") 
	{
		$this->table=$table;
		$this->champ=$champ;
		$this->index=$index;
		$this->valeur=$valeur;
		$this->affiche=$affiche;
 
		$sql="Select * from ".$this->table." order by ".$this->index;
		if($resultat=mysql_query($sql))
		{
			if($nbligne=mysql_numrows($resultat))
			{
				$i=0; 
				while ($i < $nbligne)
				{
					$option = mysql_result($resultat,$i,$this->champ);
					if ($this->affiche!="")
					{
						$val_affiche = mysql_result($resultat,$i,$this->affiche);
						$value = $val_affiche;
					}
					else
					{	
						$value=$option;
					}
					if($this->valeur!=$value)
					{
						$liste_resultat .= '<OPTION value="'.$value.'">'.$option.'</OPTION>'; 
					}
					else
					{
						$liste_resultat .= '<OPTION selected value="'.$value.'">'.$option.'</OPTION>'; 	
					}
					$i++; 
				}
				return $liste_resultat;
			} 
		}
	}
}

[Mr N.]

Deux remarques :
- Dans un hopital, je ne vois pas ce que viens faire une classe dénommée Outil. En général, les classes utilitaires (comprendre: fourre-tout) ne sont pas terribles en terme de conception.
- Au lieu d'essayer de comprendre le code d'un autre, pourquoi ne ferais-tu pas le tien ? Au moins deux raisons à celà : 1) tu passeras autant de temps à reécrire à ta sauce qu'à comprendre ce bout de code, 2) si il y a un bug, tu seras plus apte à le detecter sur ton code que sur celui de quelqu'un dont tu ne comprend pas une ligne.

[jeff_!]

salut
j'ai pas trop regarder mais ca c'est pas bon

Code :

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

mysql_numrows

Code :

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

mysql_num_rows

sinon cette classe sert juste a afficher une liste deroulante

[siddh]

- Dans un hopital, je ne vois pas ce que viens faire une classe dénommée Outil. En général, les classes utilitaires (comprendre: fourre-tout) ne sont pas terribles en terme de conception.

???
Euh pourquoi il aurais pas droit d'avoir une toolbox ?
En général ce genre de classes ne contiennent que des methodes statiques, ce qui est tres facile à l'emploi

[Mr N.]

Ah mais je ne dis pas le contraire. Mais pour préserver le principe de cohésion, il vaut mieux les limiter (personnelement je les éradique)... Mais bon c'est comme toute conception, c'est une histoire de goût et d'habitude...