Je dois envoyer ce soir par mail un programme java simulant un jeu pierre/feuille/ciseaux qui demande à l'utilisateur de saisir un choix "p,f ou c" et l'ordinateur indique "je joue ..."
Chaque partie se déroule en 5 points gagnants et à la fin l'ordinateur demande si l'utilisateur veut rejouer "O" ou "N".
L'ordinateur doit afficher combien de parties l'ordinateur et l'utilisateur ont gagné.
Est-ce que vous pouvez me dire si celui-ci est bien ?
public class Jeu{
public static void main (String [] args){
int j = 0;
int o = 0;
int s =0;
int k = 0;
int d = 0;
int e = 0;
do {
s=0;
k=0;
Console.start();
while (k
System.out.println(“Faites votre choix (p, f ou c)”) ;
String n = console.readString() ;
if (n==“p”){
j=1;
}else if (n==“f”){
j=2;
}else if (n==“c”){
j=3;
}
double a=Math.random();
if(a
System.out.println(“Je joue pierre”);
o=1 ;
}else if(a=>1/3 && a
System.out.println(“Je joue feuille”);
o=2;
}else{
System.out.println(“Je joue ciseaux”);
o==3 ;
}
if(j==1&&o==3){
k++ ;
}else if (j==2&&o==1){
k++ ;
}else if (j==3&&o==2){
k++ ;
}else if (j !=0){
s++ ;
}
}
if (s==5){
System.out.println(“l’ordinateur a gagné”) ;
d++ ;
}else{
System.out.println(“vous avez gagné”) ;
e++ ;
}
System.out.println(“Voulez-vous rejouer ? (O/N) ”)
String r=console.readString() ;
}while (r==N) ;
}
}
# GCJ -o tonjava tonfichier.java && java tonjava
Posté par NeoX . Évalué à 5.
tu compile ton programme,
tu l'execute...
et tu vois si ca tourne...
# Conseil
Posté par Fabimaru (site web personnel) . Évalué à 7.
Même pour les compteurs je mets des noms lisibles.
C'était un message de l'association française des mecs qui supportent ton code de merde! (oups, je me laisse emporter)
[^] # Re: Conseil
Posté par snt . Évalué à 3.
Et puis tu rajoutera quelques contrôles de saisie, parce que là, ça m'a l'air un peu léger.
Et y'a pas un truc en java à propos des comparaisons de chaines ? genre utiliser equals plutôt que == ?
[^] # Re: Conseil
Posté par Benoît Bâlon (site web personnel) . Évalué à 3.
if ( "Ma chaine".equals( seDechaine ) )
{
System.out.println("Hello world !");
}
# Commentaire
Posté par Tiwaz . Évalué à 3.
- pas de vérification de l'entrée utilisateur : Si on se trompe dans ce que l'on entre, ton programme a un problème.
- C'est du Java, et pas un seul objet à l'horizon
- Dernière ligne : (r=="N"), et pas (r==N). De plus ,même problème que précédemment.
- d et e ne serve à rien, tu ne les affiches et utilisent jamais.
- Comme dit précédemment, le nom des variables pour plus de lisibilité, est très mal choisis.
- Tes "if" imbriqué serait avantageusement remplacé par un switch/case à mon avis.
- Moi, j'aime l'économie de variable. A mon avis, ton programme n'a besoin que de 4 variables maximum.
# .
Posté par Sycho . Évalué à 1.
Pour ce qui est des variables d et e, j'avais juste oublier d'écrire : "System.out.println("l'ordinateur a gagné "+e+" parties") ...
Ensuite, en réponse au premier commentaire me demandant de compiler le programme et de le tester, j'aurai bien aimé si seulement j'avais réussi à télécharger java sur mon ordinateur
[^] # Re: .
Posté par liberforce (site web personnel) . Évalué à 4.
http://www.siteduzero.com/tutoriel-3-10601-programmation-en-(...)
# Erreurs
Posté par s[e]th & h[o]lth (site web personnel) . Évalué à 2.
Par exemple, un match nul donne un point à l'ordinateur, etc...
Suivre le flux des commentaires
Note : les commentaires appartiennent à celles et ceux qui les ont postés. Nous n’en sommes pas responsables.