switch case statement error : les expressions de cas doivent être des expressions constantes

Question

Mon énoncé sur le switch-case fonctionne parfaitement hier. Mais lorsque j'ai exécuté le code plus tôt ce matin, eclipse m'a donné une erreur en soulignant les déclarations de cas en couleur rouge et dit : les expressions de cas doivent être des expressions constantes, il est constant Je ne sais pas ce qui s'est passé. Voici mon code ci-dessous :

public void onClick(View src)
    {
        switch(src.getId()) {
        case R.id.playbtn:
            checkwificonnection();
            break;

        case R.id.stopbtn:
            Log.d(TAG, "onClick: stopping srvice");
            Playbutton.setImageResource(R.drawable.playbtn1);
            Playbutton.setVisibility(0); //visible
            Stopbutton.setVisibility(4); //invisible
            stopService(new Intent(RakistaRadio.this,myservice.class));
            clearstatusbar();
            timer.cancel();
            Title.setText(" ");
            Artist.setText(" ");
            break;

        case R.id.btnmenu:
            openOptionsMenu();
            break;
        }
    }

Tous les R.id.int sont soulignés en rouge.

Answer 1

Dans un projet Android classique, les constantes de la classe de ressources R sont déclarées comme suit :

public static final int main=0x7f030004;

Cependant, à partir d'ADT 14, dans un projet de bibliothèque, ils seront déclarés comme ceci :

public static int main=0x7f030004;

En d'autres termes, les constantes ne sont pas définitives dans un projet de bibliothèque. Par conséquent, votre code ne compilerait plus.

La solution à ce problème est simple : Convertissez l'instruction switch en une instruction if-else.

public void onClick(View src)
{
    int id = src.getId();
    if (id == R.id.playbtn){
        checkwificonnection();
    } else if (id == R.id.stopbtn){
        Log.d(TAG, "onClick: stopping srvice");
        Playbutton.setImageResource(R.drawable.playbtn1);
        Playbutton.setVisibility(0); //visible
        Stopbutton.setVisibility(4); //invisible
        stopService(new Intent(RakistaRadio.this,myservice.class));
        clearstatusbar();
        timer.cancel();
        Title.setText(" ");
        Artist.setText(" ");
    } else if (id == R.id.btnmenu){
        openOptionsMenu();
    }
}

http://tools.Android.com/tips/non-constant-fields

Vous pouvez rapidement convertir un switch à un if-else en utilisant les éléments suivants :

Dans Eclipse
Déplacez votre curseur sur le switch et appuyez sur Ctrl + 1 puis sélectionnez

Convertir le "switch" en "if-else".

Dans Android Studio
Déplacez votre curseur sur le switch et appuyez sur Alt + Enter puis sélectionnez

Remplacez "switch" par "if".

Answer 2

Décocher "Is Library" dans les propriétés du projet a fonctionné pour moi.

Answer 3

R.id.*, depuis ADT 14 ne sont plus déclarés comme final static int donc vous ne pouvez pas utiliser dans la construction switch case. Vous pouvez utiliser la clause if else à la place.

Answer 4

Une solution simple pour ce problème est :

Cliquez sur le commutateur puis appuyez sur CTL+1, Cela changera votre interrupteur en un bloc d'instructions if-else, et résoudra votre problème.

Answer 5

Comment au sujet de cette solution pour garder l'interrupteur belle place d'un if-else:

private enum LayoutElement {
    NONE(-1),
    PLAY_BUTTON(R.id.playbtn),
    STOP_BUTTON(R.id.stopbtn),
    MENU_BUTTON(R.id.btnmenu);

    private static class _ {
        static SparseArray<LayoutElement> elements = new SparseArray<LayoutElement>();
    }

    LayoutElement(int id) {
        _.elements.put(id, this);
    }

    public static LayoutElement from(View view) {
        return _.elements.get(view.getId(), NONE);
    }

}

Donc, dans votre code, vous pouvez le faire:

public void onClick(View src) {
    switch(LayoutElement.from(src)) {
    case PLAY_BUTTTON:
        checkwificonnection();
        break;

    case STOP_BUTTON:
        Log.d(TAG, "onClick: stopping srvice");
        Playbutton.setImageResource(R.drawable.playbtn1);
        Playbutton.setVisibility(0); //visible
        Stopbutton.setVisibility(4); //invisible
        stopService(new Intent(RakistaRadio.this,myservice.class));
        clearstatusbar();
        timer.cancel();
        Title.setText(" ");
        Artist.setText(" ");
        break;

    case MENU_BUTTON:
        openOptionsMenu();
        break;
    }
}

Les énumérations sont statiques alors cela aura un impact très limité. La seule fenêtre de préoccupation serait le double de recherche impliqués (d'abord sur le plan interne SparseArray et, plus tard, sur l'interrupteur de la table)

Cela dit, cette enum peut aussi être utilisée pour extraire les éléments dans un fluide manière, en cas de besoin pour garder une référence à l'id... mais c'est une histoire pour une autre fois.