variable cannot be resolved

variable cannot be resolved - Java - Programmation

Marsh Posté le 26-03-2009 à 18:17:56    

Slt, je suis en train de me faire une calculette toute simple en java et normalement elle est finie sauf que j'ai 4 fois l'erreur "choix cannot be resolved" à la ligne 55 et la ligne 55 est mon while, je vous montre mon code:
 

Code :
  1. import java.util.Scanner;
  4. public class sdz1 {
  6. /**
  7.  * @param args
  8.  */
  9. public static void main(String[] args) {
  10.  // TODO Auto-generated method stub
  12. Scanner lire = new Scanner (System.in);
  13. do{
  14.    double premier;
  15.    double second;
  16.    int choix;
  17.    double res;
  18.    System.out.println("Saisissez un premier nombre" );
  19.    premier = lire.nextDouble();
  20.    System.out.println("Saisissez un deuxieme nombre" );
  21.    second = lire.nextDouble();
  22.    System.out.print("1 pour ajouter /n" +
  23.                 "2 pour soustraire /n" +
  24.                "3 pour multiplier /n" +
  25.                "4 pour diviser /n" +
  26.                "5 pour quitter" );
  27.                        
  28.    choix = lire.nextInt();
  29.    switch (choix){
  30.      
  31.       case '1':
  32.       res = (premier+second);
  33.       System.out.println("Le resultat est " + res);
  34.       break;
  35.      
  36.       case '2':
  37.       res = (premier-second);
  38.       System.out.println("Le resultat est " + res);
  39.       break;
  40.      
  41.       case '3':
  42.       res = (premier*second);
  43.       System.out.println("Le resultat est " + res);
  44.       break;
  45.      
  46.       case '4':
  47.       res = (premier/second);
  48.       System.out.println("Le resultat est " + res);
  49.       break;
  50.      
  51.       Default:
  52.        System.out.println("Erreur de programmation" );
  53.           }
  54.        }
  55. while ( (choix ==1) || (choix ==2) || (choix ==3) || (choix ==4) );
  56. }
  57. }


 
J'ai essayé en mettant choix =='1' mais ça ne change rien, j'ai peut-être mal pensé en programmant et je devrais changer quelques trucs?

Reply

Marsh Posté le 26-03-2009 à 18:17:56   

Reply

Marsh Posté le 26-03-2009 à 18:42:58    

choix est un int donc dans tes case, n'entoure pas les valeurs de single quote:
case 1:
...
case 2:
...
etc


Message édité par souk le 26-03-2009 à 18:43:11
Reply

Marsh Posté le 26-03-2009 à 18:47:45    

Je viens d'enlever les guillements et ça ne change rien..  :/

Reply

Marsh Posté le 26-03-2009 à 20:27:12    

choix est créé dans le do{}while, donc sa scope de définition est entre { et }, ce qui n'inclut pas la condition dans le while [:spamafote]

 

Donc au niveau de la dite condition, choix n'existe pas, d'où le message d'erreur.

 

Accessoirement, ce code est très moche, pourquoi répéter 4 fois de suite System.out.println("Le resultat est " + res);? Pourquoi mettre 4 cas à la condition finale alors que 2 suffisent? Pourquoi forcer l'utilisateur à se chopper un message "erreur de programmation" quand il veut quitter? Pourquoi ne pas dire à l'utilisateur comment quitter, d'ailleurs? Et pourquoi toutes ces parenthèses sans intérêt à travers le code?


Message édité par masklinn le 26-03-2009 à 20:33:27

---------------
Stick a parrot in a Call of Duty lobby, and you're gonna get a racist parrot. — Cody
Reply

Marsh Posté le 26-03-2009 à 20:34:52    

Je viens de refaire mon programme et voici mon code final:
 

Code :
  1. import java.util.Scanner;
  4. public class sdz1 {
  6. /**
  7.  * @param args
  8.  */
  9. public static void main(String[] args) {
  10.  // TODO Auto-generated method stub
  12. Scanner lire = new Scanner (System.in);
  13. int choix;
  14. do{
  15.    double premier;
  16.    double second;
  17.    double res;
  18.    System.out.println("Saisissez un premier nombre" );
  19.    premier = lire.nextDouble();
  20.    System.out.println("Saisissez un deuxieme nombre" );
  21.    second = lire.nextDouble();
  22.    System.out.println("1 pour ajouter, 2 pour soustraire, 3 pour multiplier, 4 pour diviser ou 5 pour quitter" );
  23.                        
  24.    choix = lire.nextInt();
  25.    switch (choix){
  26.      
  27.       case 1:
  28.       res = (premier+second);
  29.       System.out.println("Le resultat est " + res);
  30.       break;
  31.      
  32.       case 2:
  33.       res = (premier-second);
  34.       System.out.println("Le resultat est " + res);
  35.       break;
  36.      
  37.       case 3:
  38.       res = (premier*second);
  39.       System.out.println("Le resultat est " + res);
  40.       break;
  41.      
  42.       case 4:
  43.       res = (premier/second);
  44.       System.out.println("Le resultat est " + res);
  45.       break;
  46.           }
  47.        }
  48. while ( (choix ==1) || (choix ==2) || (choix ==3) || (choix ==4) );
  49. }
  50. }


 
Maintenant je vais l'améliorer de façon à ce que l'utilisateur puisse choisiri autant de nombre qu'il veut parce que là il ne peut agir qu'avec deux nombres seulement..

Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed