Discussion :

[Jimalexp]

Salut,

Je travaille sur un exercice portant sur la gestion d'un catalogue de livres. On demande notamment d'écrire une méthode pour rechercher un livre stocké dans une table.

Je me suis rendu compte que la recherche possède un tronc commun qui compare tout simplement une chaîne au titre ou au code isbn.

On pourrais créer une méthode commune comme ci-dessous mais quelque chose me dis que cette solution n'est pas vraiment élégante.

Auriez-vous une alternative à proposer ?

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
public Livre rechercherLivre( String chaineRecherche, String typeRecherche )
	{
		String chaineComparaison = "";
		Livre instanceLivre = null;
 
		if( typeRecherche == null )
			return null;
 
		if( chaineRecherche == "" )
			return null;
 
		for( int i = 0; i < tailleTableau; i++ )
		{
			if( typeRecherche == "Titre" )
				chaineComparaison = tableauLivres[ i ].getTitre();
			else if( typeRecherche == "ISBN" )
				chaineComparaison = tableauLivres[ i ].getIsbn();
 
			if( chaineComparaison == chaineRecherche )
				instanceLivre = tableauLivres[ i ];
 
			return instanceLivre;
		}
 
		return null;
	}

[KiLVaiDeN]

Salut,

Ce qui serait plus élégant et rapide, serait que tu crées deux HashMap, avec pour clé d'une part la clé ISBN pour l'une, et le titre pour l'autre.

Ainsi, tu accéderais directement au livre sans constamment parser ton tableau, ce qui peut être assez lent.

Est-ce que l'exercice se limite à l'usage d'un tableau ?

Sinon pour répondre plus particulièrement à la question, il est bien entendu de bonne pratique de séparer les différents types de recherche en plusieurs méthodes voire plusieurs classes.

Je te conseille pour améliorer la qualité de ton code, de séparer les processus métier ( ici recherche par titre ou clé ISBN ) et d'utiliser des constantes plutôt que de directement avoir des chaines de caractères dans ton code.

A+

[Jimalexp]

Salut,

Ce qui serait plus élégant et rapide, serait que tu crées deux HashMap, avec pour clé d'une part la clé ISBN pour l'une, et le titre pour l'autre.

Ainsi, tu accéderais directement au livre sans constamment parser ton tableau, ce qui peut être assez lent.

Est-ce que l'exercice se limite à l'usage d'un tableau ?

Je serais tenté d'utiliser des HashMaps mais l'exercice demande bien un tableau.

Sinon pour répondre plus particulièrement à la question, il est bien entendu de bonne pratique de séparer les différents types de recherche en plusieurs méthodes voire plusieurs classes.

Donc, il vaudrais mieux viser une surcharge de méthode, même si le code est très similaire. Bon, je vais mettre ces vielles habitudes que j'ai pris avec le C de côté.

Je te conseille pour améliorer la qualité de ton code, de séparer les processus métier ( ici recherche par titre ou clé ISBN ) et d'utiliser des constantes plutôt que de directement avoir des chaines de caractères dans ton code.

C'est noté pour les constantes. Pourrais-tu donner plus d'explications sur la notion de processus métier ? Est-ce que tu entends par là qu'une méthode dois agir de manière "unitaire" et seulement chercher une information précise au lieu de jouer le role d'un couteau suisse ? Je dois avouer que c'est un peu flou dans mon esprit.

[KiLVaiDeN]

Plus tu sépares tes traitements, plus facile seront les évolutions et la maintenance.

Par exemple, ta méthode "couteau-suisse" pourrait vite devenir une usine à gaz si on te demande d'ajouter d'autres paramètres de recherche, ou bien si pour une raison X ou Y des processus qui te parraissaient similaires au départ, deviennent subtilement distincts.

Par exemple, on peut te dire que la recherche par titre doit pouvoir te retourner une collection de Livre si tu ne donnes qu'une partie du titre, alors que la recherche par ISBN doit directement t'afficher le livre correspondant à ce code unique.

A+

[Jimalexp]

Merci pour tes explications. C'est beaucoup plus clair.

a+