Un Commutateur de Java problème : le cas des expressions doivent être des expressions constantes

J'ai un problème dans mon switch/case déclaration. Le message d'erreur indique : "Cas des expressions doivent être des expressions constantes". Je comprends l'erreur, et je peux le résoudre à l'aide de Si mais quelqu'un peut-il me dit pourquoi le cas de l'expression doit être une constante dans un switch/case.
Un exemple de code de mon erreur :

public boolean onOptionsItemSelected(MenuItem item) {
    int idDirectory = ((MenuItem) findViewById(R.id.createDirectory)).getItemId();
    int idSuppression = ((MenuItem) findViewById(R.id.recycleTrash)).getItemId();
    int idSeeTrash = ((MenuItem) findViewById(R.id.seeTrash)).getItemId();

    switch (item.getItemId()) {
    case idDirectory:
        createDirectory(currentDirectory);
        break;
    case idSuppression:
        recycleTrash();
        break;
    case idSeeTrash:
        seeTrash();
        break;
    }

    return super.onOptionsItemSelected(item);
}

Merci pour votre explication!!!

  • Pourquoi êtes-vous à l'aide de findViewById().getItemId()? Si vous avez déjà l'id, vous n'avez pas besoin d'obtenir de l'élément pour obtenir l'id!
  • instructions de commutation sont une mauvaise odeur... je vous recommande de regarder dans l'utilisation du polymorphisme ou de l'adaptateur afin de parvenir à une meilleure recherche de code. En outre, vous pouvez éviter de tels problèmes tout à fait.
  • Je suis d'accord, spécialement lorsque la condition se répète assez souvent. Si vous l'utilisez rarement, il peut être ok. Aussi, je l'utilise parfois avec le polymorphisme pour créer l'initiale de béton instance.
  • Depuis Dimitri utilisant Android, son MenuItem objets sont susceptibles défini en xml et (à ma connaissance) ne peut pas être facilement sous-classé, sans renoncer à la simplicité de MenuInflater.inflate().
InformationsquelleAutor Dimitri | 2011-01-20
  1. 54

    De sorte qu'il peut être évalué lors de la phase de compilation ( statiquement vérifier )

    Voir: http://docs.oracle.com/javase/specs/jls/se7/html/jls-14.html#jls-14.11 pour une définition formelle de la switch.

    En outre, il peut vous aider à mieux comprendre comment que switch est transformé en bytecode:

    class Switch {
  void x(int n ) {
    switch( n ) {
      case 1: System.out.println("one"); break;
      case 9: System.out.println("nine"); break;
      default:  System.out.println("nothing"); break;
    }
  }
}

    Et après la compilation: 

    C:\>javap -c Switch
Compiled from "Switch.java"
class Switch extends java.lang.Object{
Switch();
  Code:
   0:   aload_0
   1:   invokespecial   #1; //Method java/lang/Object."<init>":()V
   4:   return

void x(int);
  Code:
   0:   iload_1
   1:   lookupswitch{ //2
                1: 28;
                9: 39;
                default: 50 }
   28:  getstatic       #2; //Field java/lang/System.out:Ljava/io/PrintStream;
   31:  ldc     #3; //String one
   33:  invokevirtual   #4; //Method java/io/PrintStream.println:(Ljava/lang/String;)V
   36:  goto    58
   39:  getstatic       #2; //Field java/lang/System.out:Ljava/io/PrintStream;
   42:  ldc     #5; //String nine
   44:  invokevirtual   #4; //Method java/io/PrintStream.println:(Ljava/lang/String;)V
   47:  goto    58
   50:  getstatic       #2; //Field java/lang/System.out:Ljava/io/PrintStream;
   53:  ldc     #6; //String nothing
   55:  invokevirtual   #4; //Method java/io/PrintStream.println:(Ljava/lang/String;)V
   58:  return

}

    Voir que la ligne marquée comme 1: 

     1:   lookupswitch{ //2
            1: 28;
            9: 39;
            default: 50 }

    Il évalue la valeur et passe à une autre ligne. Par exemple, si la valeur est 9 il saute à l'instruction 39: 

       39:  getstatic       #2; //Field java/lang/System.out:Ljava/io/PrintStream;
   42:  ldc     #5; //String nine
   44:  invokevirtual   #4; //Method java/io/PrintStream.println:(Ljava/lang/String;)V
   47:  goto    58

    Qui saute à l'instruction 58 :

       58:  return

    Tout cela ne serait pas possible si elle a été évaluée de façon dynamique. C'est pourquoi.

    • Merci mec, c'est une belle réponse. Donc, si la compilation avec javap, je peux voir le bytecode généré??
    • Vous êtes les bienvenus. javap est un autre outil ( comme javadoc ou jar ) qui est la valeur par défaut de java dissambler. Lorsqu'elle est invoquée avec -c vous verrez ces instructions.
    • Ok. Mais je pensais, pensez vous que le bytecode généré dans la JVM est le même dans la Machine Dalvik (Android). Je suis en train de travailler sur Android comme vous pouvez le voir
    • En fait... c'est pas la même. Dalvik a ses propres codes, mais le commutateur travail sera le même dans les deux. Regardez bien: en.wikipedia.org/wiki/Dalvik_(logiciel)
    InformationsquelleAutor OscarRyz
  2. 13

    dans eclipse IDE est simple ,dans le commutateur de phrase CTRL + 1 et convertir switch phrase - si-sinon la phrase http://tools.android.com/tips/non-constant-fields

    InformationsquelleAutor ingyesid
  3. 3

    La idDirectory et d'autres, doivent être une constante et non une variable déclarée. Switch ne fonctionnera pas dans ce cas, vous devez passer à if-else construire.

    MODIFIER je vois ce que l'OP voulait dire. C'est juste comment commutateur fonctionne en langage java.

    • C'est exactement votre problème. Si vous ne pouvez pas contourner cela, avec des constantes il suffit d'utiliser un peu plus complexe si-alors-sinon-si-bla-bla de l'installation.
    • -1 Pour répondre à ce que l'OP sait déjà: je comprends l'erreur, et je peux le résoudre à l'aide de Si mais quelqu'un peut-il me dit pourquoi le cas de l'expression doit être une constante dans un switch/case
    InformationsquelleAutor fastcodejava