Application Java si/recommandation de cas

Question

J'écris une application pour un cours de Java. Je suis un débutant complet et je sors juste du matériel que j'ai appris du cours et du web. L'application montre un certain comportement et je ne suis pas sûr de ce qui le cause. L'application est basée sur l'interface graphique et effectue des calculs sur les entrées utilisateur. Pour la section d'écouteur d'action, j'ai un ensemble de Si des déclarations telles que: « si ce bouton faire ce si ce bouton le faire »

en enfilade comme ça. Il semble que l'application exécute TOUTES les instructions if au lieu d'exécuter celle qui correspond au bouton pressé.

Serais-je mieux d'utiliser une structure de boîtier/commutateur pour ce genre de chose?

Je peux poster mon code si nécessaire, je suis nouveau sur ce site et je ne suis pas sûr si cette chose est acceptable.

Answer 1

Vous ne devriez probablement pas avoir un seul écouteur d'actions - en créer un distinct pour chaque contrôle aide à garder votre code plus lisible. Cependant, si vos contrôles partagent beaucoup de code (le contenu avant ou après les instructions if), il peut être logique de le faire de cette façon. Dans ce cas, il devrait être if/elseif. De plus, faire de vos écouteurs des classes à part entière (plutôt que des classes internes anonymes) peut aussi aider à réutiliser le code (le contenu de chaque instruction if va dans chaque sous-classe). Cela peut être au-delà de ce que vous avez appris jusqu'à présent.

Edit: (réponse plus directe à votre question réelle)

En ce qui concerne votre problème, le cas de ne doit pas tout exécuter à moins que la réponse soit drs9222 que dit que vous utilisiez un point-virgule à la fin de votre si est correct, ou votre instruction if teste la mauvaise chose.

Vous pouvez simplement publier ce que vous testez, mais vous devez comparer votre objet "Button" connu à event.getSource() en utilisant == ou .equals. Puisque chaque objet Button que vous comparez est différent, un seul doit être exécuté.

Answer 2

Sans voir le code je ne peux pas dire avec certitude, mais quelque chose que j'ai toujours l'habitude de faire était d'utiliser if et non else if - si plusieurs de mes conditions étaient remplies, alors tout le code associé a été exécuté. Si vous souhaitez exécuter uniquement la première partie, utilisez if-else pour vous assurer qu'aucune des instructions suivantes ne sera exécutée.

Answer 3

Jusqu'à ce que je vois votre code, je vais devoir deviner mais étant donné votre nouveauté admise que vous pouvez écrire votre si des déclarations comme celle-ci

if (condition); 
{ 
    ... 
} 

au lieu de comme celui-ci

if (condition) 
{ 
    ... 
}