Problème pour arrêter mon programme

Bonsoir,

Quelqu'un peut il me dire si on peut créer une boucle dans un listener de button...peut on appeller l'action d'un autre button dans un button?
Bref, je ne suis pas expert mais je cherche à comprendre pour résoudre mon problème. J'ai tenté des boucles dans createQuestion() mais a priori ce n'est pas cela...après dans le button next....je suis perdu. Si vous pouvez me lancer des pistes de travail svp...Je vous en remercie d'avance.



Le code est le suivant:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 75 76 77 78 79 80 81 82 83 84 85 86 87 88 89 90 91 92 93 94 95 96 97 98 99 100 101 102 103 104 105 106 107 108 109 110 111 112 113 114 115 116 117 118 119 120 121 122 123 124 125 126 127 128 129 130 131 132 133 134 135 136 137 138 139 140 141 142 143 144 145 146 147 148 149 150 151 152 153 154 155 156 157 158 159 160 161 162 163 164 165 166 167 168 169 170 171 172 173 174 175 176 177 178 179 180 181 182 183 184 185 186 187 188 189 190 191 192 193 194 195 196 197 198 199 200 201 202 203 204 205 206 207 208 209 210 211 212 213 214 215 216 217 218 219 220 221 222 223 224 225 226 227 228 229 230 231 232 233 234 235 236 237 238 239 240 241 242 243 244 245 246 247 248 249 250 251 252 253 254 255

package performyourgame.quizz.prepabac;   /*  * Author: Romuald Grandin  * Date: 07/09/2015  *  *  * Programme: L'objectif de cette application est de répondre à différentes questions sous forme de quiz dans un temps imparti.  */   import java.awt.Color; import java.awt.Font; import java.awt.Graphics; import java.awt.Image; import java.awt.event.ActionEvent; import java.awt.event.ActionListener; import java.util.ArrayList; import java.util.Random;   import javax.swing.ImageIcon; import javax.swing.JButton; import javax.swing.JFrame; import javax.swing.JOptionPane; import javax.swing.JPanel; import javax.swing.JTextField;   public class Main { // ====================DECLARATION ET // INITIALISATION===========================//   static private JTextField question = new JTextField(40); static private JTextField reponse = new JTextField(40); static private int index, score, total; static private JButton start = new JButton("START"); static private JButton next = new JButton("NEXT"); static private JButton validate = new JButton("VALIDER"); static private JButton stop = new JButton("STOP"); static private JButton closeFrame = new JButton("QUITTER"); static private ArrayList<Integer> indexesAlreadyTaken = new ArrayList<>(); static private String preguntas, reponseAnswer, reponseClavier; static private long endTime, startTime; static String[][] Objects = getData();   public static void main(String[] args) { // =====================MISE EN FORME DE LA // FENETRE==========================// final JFrame frame = new JFrame(); frame.setVisible(true); frame.setTitle("QUIZ PREPA BAC"); frame.setSize(800, 600); frame.setResizable(true); frame.setLocationRelativeTo(null); frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); // =================================================================// // ======================FOND ECRAN============================// JPanel content = new ImagePanel(new ImageIcon("design.jpeg").getImage()); // =================================================================// // ======================AJOUT PANNEAU DANS LA // FENETRE==============================// frame.setContentPane(content); // =================================================================// // ======================CHOIX TAILLE ET POLICE DU // JLABEL==============================// Font f = new Font("Serif", Font.PLAIN, 36); // par exemple question.setFont(f); reponse.setFont(f); Font ft = new Font("Serif", Font.PLAIN, 40); validate.setFont(ft); closeFrame.setFont(ft); stop.setFont(ft); start.setFont(ft); next.setFont(ft); // =================================================================// // ============MISE EN FORME DES BOUTONS MISE A ZERO ET // QUITTER============================// question.setBackground(Color.white); question.setBorder(null); reponse.setBackground(Color.white); reponse.setBorder(null); next.setEnabled(false); // =======================================================================================// // ======================AJOUT DES ELEMENTS AU // PANNEAU============================// content.add(question); content.add(reponse); content.add(start); content.add(next); content.add(validate); content.add(stop); content.add(closeFrame); // =================================================================// // =====================AFFICHAGE ICONE COIN DE // FENETRE=======================// frame.setIconImage(new ImageIcon("fGris.jpeg").getImage()); // =================================================================// frame.validate(); // =================================================================// // ==========EVENEMENT EN CAS DE CLIC SUR BOUTON // START============================// start.addActionListener(new ActionListener() { public void actionPerformed(ActionEvent e) { if (e.getSource() == start) { startTime = System.currentTimeMillis();   reponse.setText(""); createQuestion(); } } }); // ===============================================================================// // ==========EVENEMENT EN CAS DE CLIC SUR BOUTO NEXT // ============================// next.addActionListener(new ActionListener() { public void actionPerformed(ActionEvent e) { if (e.getSource() == next) { validate.setEnabled(true); reponse.setText(""); if(total == Objects.length ){ stop.getActionCommand(); } else{ createQuestion(); next.setEnabled(false); }   } } }); // ===============================================================================// // ===============================================================================// // ==========EVENEMENT EN CAS DE CLIC SUR BOUTON // STOP============================// stop.addActionListener(new ActionListener() { public void actionPerformed(ActionEvent e) { if (e.getSource() == stop) {   endTime = System.currentTimeMillis();   question.setText(String.format( "Tu as terminé !! Ton score est de << %d/%d >>.", score, total)); reponse.setText(String .format("Il vous a fallu environ %d secondes pour répondre aux %d questions.", getElapsedTimeInSeconds(), total));   } }   }); // ===============================================================================// // ==========EVENEMENT EN CAS DE CLIC SUR BOUTON // QUITTER============================// closeFrame.addActionListener(new ActionListener() { public void actionPerformed(ActionEvent e) { if (e.getSource() == closeFrame) { frame.dispose(); } } }); // ===============================================================================// // ==================EVENEMENT EN CAS DE CLIC SUR BOUTON // VALIDER==============// validate.addActionListener(new ActionListener() { public void actionPerformed(ActionEvent e) { if (e.getSource() == validate) { start.setEnabled(false); next.setEnabled(false);   question.setText("Cliquez sur NEXT pour continuer ou STOP si vous vous arrêtez!!"); reponseClavier = reponse.getText();   if (reponseClavier.equalsIgnoreCase(reponseAnswer)) { reponse.setText("Bonne réponse!! Tu es un champion! "); score++; total++; } else { reponse.setText(String.format( "Dommage ! La bonne réponse était: %s. ", reponseAnswer)); total++; } next.setEnabled(true); validate.setEnabled(false); } } }); }   // ============================================================================ // // protected static int getElapsedTimeInSeconds() { long timeElapsed = (endTime - startTime); return (int) (timeElapsed / 1000); }   // ==================================================================== // static String[][] getData() {   String[][] data = { { "Qui mord?", "Crocodile" }, { "Qui fait cui cui?", "Canari" }, { "Qui est Willy?", "Baleine" }, { "Quel est ce fruit?", "Orange" } }; return data; }   // =================================================================//   // ====================== FONCTION TIMER ========================== // // =======================FONCTION CREATION QUESTION AU // HASARD========================// public static void createQuestion() {   start.setEnabled(false); reponse.setText("");   do { Random random = new Random(); index = random.nextInt(Objects.length);   preguntas = Objects[index][0]; reponseAnswer = Objects[index][1]; } while (indexesAlreadyTaken.contains(index));   indexesAlreadyTaken.add(index);   question.setText(String.format("%s", preguntas)); };     // ===============================================================================// }   /*  * Classe image background  */ class ImagePanel extends JPanel {   private static final long serialVersionUID = 1L; private Image img;   public ImagePanel(String img) { this(new ImageIcon(img).getImage()); }   public ImagePanel(Image img) { this.img = img; }   public void paintComponent(Graphics g) { g.drawImage(img, 0, 0, null); } /* * ==================================================== */ }

---

J'ai oublié de préciser mon problème :lol:

Quand j'arrive à la dernière question, mon programme bug...car il cherche une nouvelle question...mais le tableau est fini. Du coup, j'aimerai donner le score obtenu et le temps réalisé.

Mille merci pour votre aide

Salut,

Citation:

---

Envoyé par copinmux

Bonsoir,
...si on peut créer une boucle dans un listener de button...

---

Oui dans l'absolu. Mais il faut savoir que Swing s'exécute dans un thread spécifique (appelé l'Event Dispatch Thread, ou EDT). Un thread ne peut exécuter une seule instruction à la fois. Comme ce thread sert à l'affichage et au traitement de l'interaction homme-machine (souris, clavier, etc., donc entre autres les actions des boutons), si tu fais une boucle dans une action, cela bloque l'affichage et le reste l'interaction. Il faut donc éviter de faire des boucles, ou des traitements longs en général, dans une action, parce que pendant l'exécution de cette boucle toute l'application est figée.

Citation:

---

Envoyé par copinmux

peut on appeller l'action d'un autre button dans un button?

---

Oui, soit directement en invoquant l'action, soit en simulant un click sur le bouton, mais devoir le faire témoigne souvent d'une mauvaise conception de l'IHM. Chaque bouton doit avoir sa fonction indépendante. Si 2 boutons ont un traitement similaire, on peut aussi appeler ce même traitement dans les deux actions (une méthode unique appelée dans les deux actions : c'est d'ailleurs ce que tu fais avec ta méthode createQuestion).

Citation:

---

Envoyé par copinmux

Bref, je ne suis pas expert mais je cherche à comprendre pour résoudre mon problème. J'ai tenté des boucles dans createQuestion() mais a priori ce n'est pas cela...après dans le button next....je suis perdu. Si vous pouvez me lancer des pistes de travail svp...Je vous en remercie d'avance.

---

Ici ta boucle sert à sélectionner une question qui n'a pas déjà été posée. Au début, quand il y a peu de questions déjà posée, la boucle va vite s'arrêter, mais vers la fin, elle peut boucler plusieurs fois avant de trouver une question qui n'a pas déjà été posée, et cela peut bloquer plus ou moins légèrement l'affichage. Cette boucle s'exécute très très vite, donc l'impact peut être négligeable, mais en théorie il ne faudrait pas faire comme ça, parce que le risque existe.
Tu peux procéder de manière beaucoup plus simple. Déjà, je commencerai par faire une classe Question, avec des attributs pour un intitullé, les réponses à proposer, la bonne réponse...
Ensuite tu mets toutes les questions dans une liste, et tu mélanges la liste avec Collections.shuffle(list). Ensuite, tu n'a plus qu'a parcourir la liste du début à la fin : le bouton start et le bouton next font exactement la même chose : prendre une question situé à un index donné (un attribut), et incrémenter l'index. Pour arrête le jeu, on compare l'index à la fin de la liste.

Exemple de principe :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 75 76 77 78 79 80 81 82 83 84 85 86 87 88 89 90 91 92 93 94 95 96 97 98 99 100 101 102 103 104 105 106 107 108 109 110

public class Exemple extends JPanel {   private final static List<Question> QUESTIONS = Arrays.asList( new Question("question 1","reponse 1"), new Question("question 2","reponse 2") );   private List<Question> questions; private int index=0; private Question questionPosee;   private JButton boutonStart; private JButton boutonNext; private JButton boutonValider;   public Exemple() { super();   questions = creerQuestions();   boutonStart = addButton("Start", e-> poserPremiereQuestion()); boutonValider = addButton("Valider", e-> validerReponse()); boutonNext = addButton("Next", e-> poserQuestion());   // ces boutons sont inaccessibles au début boutonNext.setEnabled(false); boutonValider.setEnabled(false);   }     private void validerReponse() { // ici tu compares la réponse if ( true /*questionPosee.getReponse().equals(reponseTextfield.getText())*/ ) { // bonne reponse (incrémenter un score, ou ce que tu veux) nouvelleQuestion(); } else { // mauvaise réponse // par exemple : si nombre de réponses fausses > 3... appeler nouvelleQuestion() } }   private void nouvelleQuestion() { boutonValider.setEnabled(false); if ( index<questions.size() ) { boutonNext.setEnabled(true); } }     private void poserPremiereQuestion() { boutonStart.setEnabled(false); poserQuestion(); }   private void poserQuestion() { boutonNext.setEnabled(false); if (index<questions.size()) { setQuestion(questions.get(index)); index++; boutonValider.setEnabled(true); } }   private void setQuestion(Question question) { questionPosee=question; // ici tu affectes le texte de ta question (questionTextField.setText(questionPosee.get....)) // tu effaces la réponse précédente }     private JButton addButton(String texte, ActionListener actionListener) { JButton button = new JButton(texte); add(button); button.addActionListener(actionListener); return button; }     private List<Question> creerQuestions() { List<Question> questions = new ArrayList<>(QUESTIONS); // créer une copie de la liste Collections.shuffle(questions); return questions; }     public static void main(String[] args) {     JFrame frame = new JFrame("Démo"); frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);     frame.getContentPane().add(new Exemple());   frame.pack();; frame.setLocationRelativeTo(null); frame.setVisible(true);   }   private static class Question {   public Question(String string, String string2) { // TODO Auto-generated constructor stub } }   }

---

Merci Joel pour tous ces conseils et pistes de travail.

Je m'affaire à cela. Par contre, j'ai repris ta base de proposition. La façon de faire me perturbe mais je vois que le code sera plus flexible. Je vais m'adapter. Cela me sera très formateur. Par contre, sur la déclaration de List...eclipse me donne cela: "the type List is not generic; it cannot be parameterized with arguments <Exemple.Question>"

et pour
super();
question = creerQuestions();

il m'affiche: "the left-hand side of an assignment must be a variable"

Du coup, je ne sais pas comment le corriger pour que je puisse m'attaquer aux codes. En tout cas, un grand merci pour ton aide et ton temps.

Citation:

---

Envoyé par copinmux

Par contre, sur la déclaration de List...eclipse me donne cela: "the type List is not generic; it cannot be parameterized with arguments <Exemple.Question>"

---

importer java.util.List pas java.awt.List !

Merci pour ce conseil tchize....cela fonctionne ;) merci beaucoup