Code spaghetti, visibilité du code

Version imprimable

Bonjour, je dois faire du code vraiment de base, que je vais devoir traduire en assembleur et en plein de language. Pricipalement je veux faire des pluriels sur des quantités. Voici mon code :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 if (Total_Pieds == 1 && Total_Pouces > 0) { System.out.print(Total_Pieds + " pied "); } if (Total_Pieds == 1 && Total_Pouces == 0) { System.out.println(Total_Pieds + " pied"); } if (Total_Pieds > 1 && Total_Pouces > 0) { System.out.print(Total_Pieds + " pieds "); } if (Total_Pieds > 1 && Total_Pouces == 0) { System.out.println(Total_Pieds + " pieds"); } if (Total_Pieds == 0 && Total_Pouces == 0 || Total_Pouces == 1) { System.out.println(Total_Pouces + " pouce"); } if (Total_Pouces > 1) { System.out.println(Total_Pouces + " pouces"); }
Tout fonctionne comme je le veux et comme cela doit fonctionner, comme certains cas ou 0 pouce, ou bien, 0 pied ne doit pas apparaitre.

EX : 1 pied / 2 pieds / 2 pieds 1 pouce / 0 pouce / 11 pouces / ....

Est-ce que il y a une facon plus esthétique et simple de vérifier ces conditions sans avoir a faire un spaghetti de code avec plein de IF. Je peux toujours transformer cela en fonction, mais bon, j'hésite et je ne peux pas utiliser des fonctions trop avancés de java sinon cela va foirer en assembleur.

Pour ce qui est des println et print, c'est normal, car je dois absolument terminer l'affichage de la ligne avec aucun espace vide.

Merci de votre aide !

17/01/2013, 06h55
druzy

Un moment donnée, même si tu dois faire une fonction, si tu dois différencier les cas, tu devras forcement passer par ce genre de truc. Sinon il y a le case of mais ça revient un peu au même.

PS : J'ai mis case of au lieu de switch case. (habitude de premier language appris)
17/01/2013, 10h26
Lician

Hello!

Tu peux effectivement passer par un switch case...
Ça sera plus propre et ça t'évitera la succession des if ou if else.

Surtout que dans ton cas les tests sont tous de même "types" donc oui... Switch case! ;)

Bon courage

La logique de construction de ton message n'est pas apparent dans ton code. Pourtant cela devrait être le cas !
Je te propose une solution qui mes un peu plus en valeur la logique de construction :

Code:

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
package snippet;
 
public class Snippet {
	private static final String SPACE = " ";
 
	public static void main(String[] args) {
		print(0,0);
		print(0,1);
		print(0,2);
		print(1,0);
		print(2,0);
		print(2,1);
		print(2,2);
	}
 
	public static void print(int totalPied, int totalPouce) {
		StringBuilder message = new StringBuilder();
		if (totalPied >= 1) {
			// Debut partie Pied
			message.append(totalPied);
			message.append(SPACE);
			message.append("pied");
			if (totalPied > 1) {
				message.append("s");
			}
			// Fin partie Pied
			if (totalPouce >= 0) {
				message.append(SPACE);
			}
		}
		if (totalPouce >= 1 || totalPied <= 0) {
			// Debut partie Pouce
			message.append(totalPouce);
			message.append(SPACE);
			message.append("pouce");
			if (totalPouce > 1) {
				message.append("s");
			}
			// Fin partie Pouce
		}
		System.out.println(message);
	}
}

Les deux parties que j'ai mise en valeur avec des commentaire pourrait-être factoriser ou extraite. Pour voir seulement la logique de fusion d'affichage :

Code:

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
public static void print(int totalPied, int totalPouce) {
		StringBuilder message = new StringBuilder();
		if (totalPied >= 1) {
			printPied(totalPied, message);
			if (totalPouce >= 0) {
				message.append(SPACE);
			}
		}
		if (totalPouce >= 1 || totalPied <= 0) {
			printPouce(totalPouce, message);
		}
		System.out.println(message);
	}
 
	private static void printPouce(int totalPouce, StringBuilder message) {
		message.append(totalPouce);
		message.append(SPACE);
		message.append("pouce");
		if (totalPouce > 1) {
			message.append("s");
		}
	}
 
	private static void printPied(int totalPied, StringBuilder message) {
		message.append(totalPied);
		message.append(SPACE);
		message.append("pied");
		if (totalPied > 1) {
			message.append("s");
		}
	}

Cela me semble plus simple à prendre ou reprendre en main.

Cordialement,
Patrick Kolodziejczyk.

Allez, amusons nous, mais c'est de la masturbation intellectuelle

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
 
    if (Total_Pieds >0){
        System.out.print(Total_Pieds + " pied");
        if (Total_Pieds >1)
           System.out.print("s"); // pluriel
        if (Total_Pouces > 0)
           System.out.print(" "); // espace
 
    }
    if (Total_Pouces >0 || Total_Pieds == 0){
       System.out.print(Total_Pouces + " pouce");
       if (Total_Pouces > 1) 
           System.out.print("s");
    }
    System.out.println();

17/01/2013, 16h17
metraon

Merci à tous ! Je vais regarder cela et changer mon code même ci cela ne concerne que l'affichage, je préfère que mon code soit élégant et visible ! Je saute dans la documentation et la traduction maintenant ;)

Merci encore !