commutateur cas d'erreur: le cas des expressions doit être l'expression constante

Mon instruction de cas switch fonctionne parfaitement bien hier. Mais lorsque j'exécute le code, ce matin, eclipse m'a donné une erreur en soulignant le cas des déclarations de couleur rouge et dit: cas des expressions doit être constante, elle est constante, 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 tous soulignés en rouge.

Pouvez-vous donner la définition de R.id.playbtn etc.? Tout est statique et définitive?
Probablement vous avez supprimé/modifié votre mise en page et les codes n'existent plus, ou quelque chose comme ça...
La classe R est généralement généré par l'IDE/outils de dev, donc il est généralement correcte pour la version d'Android utilisée.
mon R. id.* sont tous beaux et exister dans la gen classe d'android.. et c'est aussi dans la page principale.

InformationsquelleAutor HeartlessArchangel | 2012-02-01

266

Dans un projet Android, les constantes de la ressource en classe R sont déclarées comme ceci:
```
public static final int main=0x7f030004;
```
Cependant, comme de l'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 n'est plus de la compilation.

La solution pour cela est simple: Convertir l'instruction switch dans un 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 convertir rapidement une switch déclaration à un if-else de résultats en utilisant la suivante:

Dans Eclipse

Déplacez votre curseur à l' switch mot-clé et appuyez sur Ctrl + 1, puis sélectionnez

Convertir 'switch' à 'if-else'.

Dans Android Studio

Déplacez votre curseur à l' switch mot-clé et appuyez sur Alt + Entrée, puis sélectionnez

Remplacer "switch" avec des "si".
- j'ai changer mon cas switch déclaration à autre chose-si la déclaration.. Il a juste obtenu, je me demande-je créer un nouveau projet android et utilisé un interrupteur instruction de cas et le travail bien..
- Il se peut que votre premier projet est à l'aide d'une bibliothèque de projet et de votre nouveau projet n'est pas.
- je ne comprends pas que désolé im vraiment un débutant ici.. pouvez-vous m'expliquer
- Votre réponse était ce dont j'avais besoin. Je viens de convertir un projet d'application pour un projet de bibliothèque et a été d'obtenir un peu de ces erreurs, et n'avait pas remarqué que le mot-clé final avait été abandonné à partir de la constante de déclarations. Après la lecture de votre réponse, vu tout de suite que la vieille déclaration de ma constante btnSave était: public static final int btnSave=0x7f0a001f; tandis que celui de la bibliothèque a été: public static int btnSave=0x7f0a001f;
- Au moins eclipse vous permettra de convertir de l'interrupteur de si/d'autre automatiquement. cliquez sur l'interrupteur à clé. ensuite, appuyez sur ctrl-1
- Bon conseil! Je vais l'ajouter à ma réponse.
- Alors, pourquoi switch spécifiquement ne permet pas que?
- Le compilateur a besoin de l'expression à être connu au moment de la compilation. Sans le final mot-clé, une variable peut être modifiée lors de l'exécution.
- Notez que final n'est pas suffisant. Si la valeur de la final variable est créée au moyen d'une méthode, il ne sera malheureusement pas de travail non plus.
- Un merci spécial pour les raccourcis auto convertir commutateur de cas/si/d'autre!
InformationsquelleAutor Benito Bertoli
51

En décochant "Est la Bibliothèque" dans les Propriétés du projet a travaillé pour moi.
- Important de nettoyer et de se reconstruire après...
- Clic droit sur le nom de votre projet. Puis cliquez sur propriétés -> Android. Sur la partie inférieure droite de la fenêtre contextuelle est une section intitulée "Bibliothèque". En vertu de l', si la "Bibliothèque est" l'option est cochée, décochez-la si vous ne voulez pas que votre projet soit un projet de bibliothèque. Puis nettoyer et reconstruire. Si vous voulez qu'il soit un projet de bibliothèque, alors vous devez changer votre commutateur à un si d'autre conditionnelle comme il est indiqué ailleurs.
- Il y a des raisons pour lesquelles un projet de bibliothèque est marqué par "Bibliothèque". Ce n'est pas une solution adéquate au problème - il brisera votre projet Android structure en faisant ce qui doit être bibliothèques à se comporter comme des applications régulières.
InformationsquelleAutor rick
11

Solution qui peut être fait de cette façon:
1. Juste attribuer la valeur à Entier
2. Faire variable à final
Exemple:
```
public static final int cameraRequestCode = 999;
```
Espère que cela va vous aider.
- La "finale" a fait le tour.
InformationsquelleAutor Hiren Patel
8

R. id.*, depuis ADT 14 ne sont pas plus a déclaré que final static int de sorte que vous ne pouvez pas utiliser l'interrupteur cas de construire. Vous pourrait utiliser en cas de clause else au lieu.
- oui, j'ai lu que sur tools.android.com, j'ai aussi essayé de créer un nouveau projet et utilisé le code ci-dessus et il fonctionne très bien.. comment est-ce?
- tools.android.com/recent/buildchangesinrevision14 voir le "Projet de Bibliothèque de Relooker" section
- Pourquoi ont-ils fait ce changement n'a aucun sens.
InformationsquelleAutor Blackbelt
8

Simple solution de ce problème est :

Cliquez sur le commutateur puis appuyez sur CTL+1, Il va changer votre commutateur de if-else instruction de bloc, et permettra de résoudre votre problème

InformationsquelleAutor Pir Fahim Shah

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.

Les énumérations sont découragés en Android en raison de leur mémoire de météorisation; et c'est la principale raison pour laquelle ils ne sont jamais utilisés dans PSBA - et la raison pour laquelle vous voyez ints partout.
stackoverflow.com/questions/5143256/...

InformationsquelleAutor pablisco

3

C'était en me jetant à cette erreur quand j'ai l'aide de l'interrupteur dans une fonction avec des variables déclarées dans ma classe:
```
private void ShowCalendar(final Activity context, Point p, int type) 
{
    switch (type) {
        case type_cat:
            break;

        case type_region:
            break;

        case type_city:
            break;

        default:
            //sth
            break;
    }
}
```
Le problème a été résolu quand j'ai déclaré final pour les variables dans le début de la classe:
```
final int type_cat=1, type_region=2, type_city=3;
```
- enum est une meilleure alternative à int dans ce cas. L'appelant de la méthode ne sera pas en mesure d'appeler la fonction avec un type non valide.
- j'ai des types int de sorte que son ok si j'utilise ints. Cependant, je voudrais savoir un exemple avec le protocole enum 😀
- i have specific int types so its ok if i use ints N'a pas vraiment de sens. Concernant enum exemple: docs.oracle.com/javase/tutorial/java/javaOO/enum.html
- je veux dire que la prochaine int type de variable dans la fonction sera toujours un de ces 3 types afin de ne pas casser quelque chose merci pour l'enum exemple 🙂
- i mean that the incoming int variable type in the function will always be one of these 3 types so it won't break anything C'est votre hypothèse. Quelqu'un d'autre peut faire appel à la fonction incorrecte avec un nombre arbitraire. Avec enum, vous n'avez pas besoin de supposer, elle est imposée par la langue.
- hmm ok, votre argument est true si quelqu'un ne sait pas comment appeler la fonction ou passer tort d'entrée int mais quelqu'un pourrait mettre dans la branche par défaut retourne false si il change le retour de la fonction à valeur Booléenne. de toute façon je suppose que c'est une meilleure façon pour appeler une fonction avec enum variables prédéfinies par l'utilisateur.
InformationsquelleAutor aimiliano
2

Je tiens à mentionner que, je suis tombé sur la même situation lorsque j'ai essayé d'ajouter une bibliothèque dans mon projet. Tout d'un coup, toutes les instructions switch commencé à montrer des erreurs!

Maintenant j'ai essayé de retirer la bibliothèque, j'ai ajouté, même alors, il n'a pas de travail.
comment jamais "quand j'ai nettoyé le projet" toutes les erreurs vient de sonner !

InformationsquelleAutor Muhammad Riyaz

Vous devez vous connecter pour publier un commentaire.