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

---------------
I mean, true, a cancer will probably destroy its host organism. But what about the cells whose mutations allow them to think outside the box, and replicate and expand beyond their wildest dreams by throwing away the limits imposed by overbearing genetic r
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