Mon script est-il correcte ?

Version imprimable

Bonjour a tous :),

comme je débute en java , je m'exerce a des petits exercices.

Mon but ici est de simuler un distributeur de pizza par exemple.
donc je supposes qu'on accepte 3 pieces de 1 euros de 2 euros et de 5 euros(même si ca n'existe pas).

que pensez vous de mon script ? il y a t'il des choses que je peux améliorer ?

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
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
 
import java.util.Scanner;
 
 
public class sdz1 {
 
	public static void main (String[] ars)
	{
		int pizza = 8; //le prix de la pizza
		byte piece1EU, piece2EU, piece5EU;
		byte piece;
		int totalrecu = 0; //on initialise le prix total a 0 euro
		int monnais=0;
 
		Scanner sc= new Scanner(System.in);//instancer la class scanner importer
 
		do
		{
 
		System.out.println("Quelle pièce introduisez vous ?");
		piece = sc.nextByte();
 
		switch (piece)
		{
		case 1:
			piece1EU = piece;
			System.out.println("vous avez introduit 1 piece de 1 euro");
			break;
 
		case 2:
			piece2EU = piece;
			System.out.println("vous avez introduit 1 piece de 2 euros");
			break;
 
		case 5:
			piece5EU = piece;
			System.out.println("vous avez introduit 1 piece de 5 euros");
			break;
 
		default:
		    piece=0;
			System.out.println("Votre piece n'est pas accepté!!");
		}
 
		totalrecu = totalrecu + piece;
		System.out.println("votre totat est de " + totalrecu +"euros");
 
		}while (totalrecu < pizza); //si la somme total n'est pas suppérieur a 8 euro on recommence
 
		System.out.println("La somme total a été atteint");
 
		monnais = totalrecu - pizza;
 
		if(monnais > 0)
		{
			System.out.println("je vous rends la monnaie de la somme de :" + monnais);
		}
		else 
		{
			System.out.print("prix exacte");
		}	
	}
 
 
}

merci pour votre réponse

prenez l'habitude de mettre votre code dans un package, et non pas dans le package par défaut:

Code:

package monpackage;

un nom de classe doit commencer par une majuscule suivant les conventions

Code:

public class Sdz1

ces variables sont inutiles

Code:

byte piece1EU, piece2EU, piece5EU;

respectez les conventions java pour l'indentation
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 do { //...... switch (piece) { case 1: piece1EU = piece; //..... } //.... System.out.println("votre totat est de " + totalrecu +"euros"); } while (totalrecu < pizza); //si la somme total n'est pas suppérieur a 8 euro on recommence
les variables doivent avoir un nom qui suit le convention lowerCamelCase:

Code:

int totalRecu = 0; //on initialise le prix total a 0 euro

merci de ta réponse tchize

Code:

1
2
 
package monpackage;

je n'ai pas encore pencher sur le package ?

qu'est ce que cela veut dire ??

c'est vrai que j'ai homis de mettre une Majuscule sur le nom de la class, c'est aussi vrai en poo php

pourquoi ces variables sont inutiles ??

Code:

1
2
 
byte piece1EU, piece2EU, piece5EU;

pourant je les utilisent en variables.

Code:

1
2
 
est ce que cela change a quelque chose lorsque je vais ajouter une nouvelle fonctionnalité, le distributeur va utiliser les ces variables pour défnire quelle piece va t'il rendre ?

mais peut être que j'ai mal compris

il est vrai que je n'ai pas respecté l'indentation dans la boucle switch

donc en résumer cela donne:

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
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
 
import java.util.Scanner;
 
 
public class DistribPizza{
 
	public static void main (String[] ars)
	{
		int pizza = 8; //le prix de la pizza
		byte piece1EU, piece2EU, piece5EU;
		byte piece;
		int totalrecu = 0; //on initialise le prix total a 0 euro
		int monnais=0;
 
		Scanner sc= new Scanner(System.in);//instancer la class scanner importer
 
		do
		{
		   System.out.println("Quelle pièce introduisez vous ?");
		   piece = sc.nextByte();
 
		   switch (piece)
		   {
		      case 1:
			    piece1EU = piece;
			    System.out.println("vous avez introduit 1 piece de 1 euro");
			    break;
 
		      case 2:
			    piece2EU = piece;
			    System.out.println("vous avez introduit 1 piece de 2 euros");
			  break;
 
		      case 5:
			    piece5EU = piece;
			    System.out.println("vous avez introduit 1 piece de 5 euros");
			    break;
 
		      default:
		        piece=0;
			    System.out.println("Votre piece n'est pas accepté!!");
		   }
 
 
		  totalrecu = totalrecu + piece;
		  System.out.println("votre totat est de " + totalrecu +"euros");
 
		}while (totalrecu < pizza); //si la somme total n'est pas suppérieur a 8 euro on recommence
 
 
		System.out.println("La somme total a été atteint");
 
		monnais = totalrecu - pizza;
 
		if(monnais > 0)
		{
			System.out.println("je vous rends la monnaie de la somme de :" + monnais);
		}
		else 
		{
			System.out.println("prix exacte");
			System.out.println("Envoyer la pizza!");
		}
 
 
 
 
	}
 
 
}

29/01/2011, 23h38
tchize_

Citation:

Envoyé par neufrdb

je n'ai pas encore pencher sur le package ?

qu'est ce que cela veut dire ??

ca veux dire que ta classe est dans les package du même nom. Sans non, elle est dans le package par défaut, qui n'est par reconnu par les jar, par exemple.

Citation:

pourquoi ces variables sont inutiles ??

Parce que vous ne les utilisez pas et qu'elles ont toujours la même valeur: 1, 2 et 5 respectivement.[/QUOTE]
30/01/2011, 10h33
neufrdb
merci de toutes ces précisions :ccool:

j'ai compris que je doit toujours ranger la class dans mon package donc je l'écrit au tout début:
Code:

1 2 package DistribPizza;
je serais que si les variables on ne les change jamais de valeur alors il n'est pas necessaire des les déclarées.

je souhaiterais maintenant faire une boucle qui permet de l'exécuter a chaque fois qu'on met un piece dans le distrib il calclule la somme et envoye la pizza

la boucle do while est t'elle adéquate pour mon problème?

cela donnerai:
Code:

1 2 3 tant que une des pieces a été introduit //traitement de la demande
est correcte de tapez ceci:
Code:

1 2 3 4 5 6 do { // .... System.out.println("Pizza a 8 euro, je rends la monnaie"); // traitement de la demande } while ( (piece1EU || piece2EU ||piece5EU) > 1 )
présenter comme ceci cette boucle est t'elle infinie, que chaque fois j'aurai ma pizza, il pourra en fournire d'autre ??

en php j'avais l'autorisation de mettre (empty) en java peut t'on le faire ?
Code:

1 2 (piece1EU != empty)

j'ai finalement utiliser la boucle while pour faire une boucle infinie

voici le scipt

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
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
 
package Test; //on l'indique pour ne pas qu'il appartiennte par le package default;
import java.util.Scanner; 
 
 
public class Test{
 
	public static void main (String[] ars)
	{
		int pizza = 8; //le prix de la pizza
		byte piece;
		byte piece1EU, piece2EU, piece5EU; //je n'ai pas retirer cette ligne sinon cela fait une erreur de compilation
		int totalrecu = 0; //on initialise le prix total a 0 euro
		int monnais = 0;
		byte envoyerPizza = 0;
 
		Scanner sc= new Scanner(System.in);//instancer la class scanner importer
 
		while (true)
		{
			totalrecu = 0; //intialisation a 0 du total recu à 0
			  do{
		        	System.out.println("Pizza a 8 euros, je rend la monnaie");
 
		    		do
		    		{
		    		   System.out.println("Quelle pièce introduisez vous ?");
		    		   piece = sc.nextByte();
 
		    		   switch (piece)
		    		   {
		    		      case 1:
		    			    piece1EU = piece;
		    			    System.out.println("vous avez introduit 1 piece de 1 euro");
		    			    break;
 
		    		      case 2:
		    			    piece2EU = piece;
		    			    System.out.println("vous avez introduit 1 piece de 2 euros");
		    			  break;
 
		    		      case 5:
		    			    piece5EU = piece;
		    			    System.out.println("vous avez introduit 1 piece de 5 euros");
		    			    break;
 
		    		      default:
		    		        piece=0;
		    			    System.out.println("Votre piece n'est pas accepté!!");
		    		   }
 
 
		    		  totalrecu = totalrecu + piece;
		    		  System.out.println("votre totat est de " + totalrecu +"euros");
 
		    		}while (totalrecu < pizza); //si la somme total n'est pas suppérieur a 8 euro on recommence
 
 
		    		System.out.println("La somme total a été atteint");
 
		    		monnais = totalrecu - pizza;
 
		    		if(monnais > 0)
		    		{
		    			System.out.println("je vous rends la monnaie de la somme de :" + monnais);
		    		}
		    		else 
		    		{
		    			System.out.println("prix exacte");
		    			System.out.println("Envoyer la pizza!");
		    		}
 
		        } while( envoyerPizza >1);
 
		}
 
 
 
	}
 
 
}

je n'ai trouvé que cette solution pour faire une boucle infinie

maintenant si j'efface

Code:

1
2
 
int piece1EU, piece2EU, piece3EU

j'ai cette erreur de compilation

Code:

1
2
3
4
5
6
7
 
Exception in thread "main" java.lang.Error: Unresolved compilation problems: 
	piece1EU cannot be resolved
	piece2EU cannot be resolved
	piece5EU cannot be resolved
 
	at Test.Test.main(Test.java:31)

:cry:

01/02/2011, 11h13
michon
En fait je pense que vous avez mal saisit ce que vous à dit Tchize...

Il n'est pas inutile de déclarer une variable constante (qui ne change pas de valeur) ! Au contraire, si elle est utilisé, il FAUT la déclarer.

Ce que Tchize à voulu vous dire ici, c'est que vous affectez constamment les valeurs 1,2,5 à vos variables 1EU,2EU,5EU, ce qui est inutile. Autant leur affecter leur valeur directement à la déclaration :

Code:

int piece1EU = 1;

Mais, ce que vous n'avez pas saisit, est que vous ne lisez JAMAIS le contenu de ces variables. Vous y stocké les valeurs 1,2,5, mais elles ne sont jamais utilisé.

Vous pouvez donc supprimer ces 3 variables de votre code, et non pas uniquement leur déclaration. Car il est alors normal que le compilateur ne soit pas d'accords... Pensez à supprimer les affectations présentes dans votre boucle "switch case" :
Code:

1 2 3 4 piece1EU = piece; piece2EU = piece; piece5EU = piece;
Le compilo ne râlera plus pour cela...