Discussion :

[Invité]

Bonjour
Je me posais la question si c'est deux codes sont égaux:

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
	public List<Media> searchByField(int fieldCode, String value) {
 
		LinkedList<Media> listeDeRecherche = new LinkedList<Media>();		
 
		for(Media mediaL : medialist){	
			if(MediaFields.getFieldType(fieldCode) == Integer.class){
				if((Integer) mediaL.getFieldValue(fieldCode) == Integer.parseInt(value)){
					listeDeRecherche.add(mediaL);
				}
			}
			else{
				if(mediaL.getFieldValue(fieldCode).toString().toLowerCase().contains(value.toLowerCase())){
					listeDeRecherche.add(mediaL);
				}
			}
		}
		return listeDeRecherche;

et :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
public List<Media> searchByField(int fieldCode, String value) {
 
		LinkedList<Media> listeDeRecherche = new LinkedList<Media>();
 
		for(Media mediaL : medialist){				
			if(MediaFields.getFieldType(fieldCode) == Integer.class && (Integer) mediaL.getFieldValue(fieldCode) == Integer.parseInt(value)){
				listeDeRecherche.add(mediaL);		
			}		
			else if(mediaL.getFieldValue(fieldCode).toString().toLowerCase().contains(value.toLowerCase())){
					listeDeRecherche.add(mediaL);
			}
 
		}
		return listeDeRecherche;

Mon doute se pose véritablement sur le :

Code :

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

if(MediaFields.getFieldType(fieldCode) == Integer.class && (Integer) mediaL.getFieldValue(fieldCode) == Integer.parseInt(value))

car le and avec au départ l'affectation de la classe integer.class puis l'utilisation de cette meme classe me laisse un peu perplexe.

Bref n'hésité pas à m'indiqué d'autres erreurs potentielles

[michon]

Dans le détail de ton code je ne comprend pas trop ton problème.

Mais :

Code :

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

1
2
3
4
5
if(aa){
      if(bb){
            //faire qq chose
      }
}

et en effet équivalent à

Code :

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

1
2
3
if(aa && bb){
      //faire qq chose
}

Et personnellement je trouve la deuxième solution beaucoup plus esthétique. Après je ne saurais pas te dire si il y a une différence de performances entre les deux manières de faire.

[Invité]

Oui mon problème est triviale :-)
Je sais pas pourquoi j'ai eu un doute avec les conditions d'égalité
Merci quand même