Discussion :

[FCL31]

Bonsoir.

Question peut-etre bête.

J'ai un code pour avoir un nombre random entre 1 et b (qui est passé en paramètre) :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
    public int randomNumber(int b){
        int result;
        int a = 1;
        result  = a + (int)(Math.random()*b);
        return result;
    }

Mais etant donné que a est supérieur à 0, je ne comprend pas pourquoi ce code arrive me renvoyer 0.

Merci d'avance pour vos lumières.

[thelvin]

Hello,

Ben là il faut que (int)(Math.random()*b) soit égal à -1,
ce qui veut dire que Math.random()*b doit être entre -1 et -2,
et comme Math.random() est toujours positif, ça veut dire que b doit être négatif.
Accessoirement, Math.random() est toujours inférieur à 1, donc b doit être inférieur à -1, ce qui veut dire -2 ou moins... Mais il n'y a pas de borne inférieure.

[FCL31]

Merci pour ta réponse.

Sauf que comme tu viens de me le faire remarquer, b que je passe en paramètre est toujours supérieur à 0.

D’ailleurs, il ne prend que quatre valeurs possible : 2, 3, 9 ou 100.

D'où mon étonnement et ma question.

[Népomucène]

Question peut-etre bête.

Réponse encore plus bête simple, pourquoi ne pas utiliser ce qui existe :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
Random random = new Random();
int pseudoRandomNumberTwo = random.nextInt(b) + 1; //Un nombre aléatoire entre 1 et b.

Question traitée dans la
https://java.developpez.com/faq/java...rage-aleatoire

[joel.drigo]

Voire encore plus simple :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

int randomNumber = ThreadLocalRandom.current().nextInt(1, b)

Mais la question qui nous occupe, celle de @FCL31, est pourquoi la méthode retournerait 0, ce qui semble impossible à première vue si on ne passe pas un argument inférieur ou égale à -1.

Pour répondre à ça, @FCL31, il nous faudrait voir l'appel de la méthode et le code qui te fait dire que ça retourne 0.

[FCL31]

Voici une parti de code qui devrait etre suffivante :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
                RadioButton chk = (RadioButton)t1.getToggleGroup().getSelectedToggle(); // Cast object to radio button                // 
                if ("Niveau Difficile".equals(chk.getText())) {
                    diff = 100;
                    nbrOperateur = 3;
                    tbResult.setText("");
                }
                else{
                    diff = 9;
                    nbrOperateur = 2;
                    tbResult.setText("");
                }
 
                int nbr1 = randomNumber(diff);
                int nbr2 = randomNumber(diff);

aprés, un simple

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

System.out.println(nbr1 + ", " + nbr2);

me permet de voir le résultat dans la console

[joel.drigo]

Cela me rend bien perplexe, mais si j'en crois le code que tu nous montres, il s'agit d'une interface JavaFX (dis moi si je me trompe). Alors, ne voyant pas tout le code, je peux envisager un cas :

le System.out.println(nbr1 + ", " + nbr2); n'est pas invoqué dans le même thread, et que là, tu as simplifié en écrivant int nbr1 = randomNumber(diff);, que nbr1 est un attribut d'un objet accessible depuis le thread de JavaFX, et le thread qui fait le System.out.println, et cet attribut n'est ni déclaré volatile ni son accès synchronisé. Il y a une optimisation Java qui fait que la valeur d'une variable affectée dans un thread n'est pas immédiatement disponible dans un autre, la valeur 0 n'étant donc pas le retour de la méthode randomNumber, mais la valeur initiale de l'attribut.

[FCL31]

Effectivement j'utilise bien JavaFX.

Etant quand même débutant, il est fort probable qu'il y ai un problème de thread (même si je ne sais pas comment l'avoir éventuellement fais).

Comment puis-je résoudre le "problème" ??

Voici le code de ma classe (le code est pas top mais il fonctionne pour moi ^^)

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
package game;
 
import javafx.beans.value.ChangeListener;
import javafx.beans.value.ObservableValue;
import javafx.event.EventHandler;
import javafx.geometry.Insets;
import javafx.scene.Parent;
import javafx.scene.control.Alert;
import javafx.scene.control.Button;
import javafx.scene.control.Label;
import javafx.scene.control.RadioButton;
import javafx.scene.control.TextField;
import javafx.scene.control.Toggle;
import javafx.scene.control.ToggleGroup;
import javafx.scene.input.MouseEvent;
import javafx.scene.layout.GridPane;
import javafx.scene.layout.HBox;
import javafx.scene.layout.VBox;
import javafx.scene.paint.Color;
import javafx.scene.text.Font;
 
 
/**
 *
 * @author Florent
 */
public class Calcul extends Parent  {
 
 
     public Calcul(){
         CalculGUI();
    }
 
 
    public GridPane CalculGUI(){
        /*
        Initialisation du Pane principal de la classe
        */
        GridPane calculPane = new GridPane();
        calculPane.setHgap(30.0);
        calculPane.setVgap(30.0);
        calculPane.setPadding(new Insets(50));
        calculPane.setMinWidth(200);
 
 
        /*
        Initialisation d'un Pane secondaire dans lequel se trouve 
        un groupe de bouton radio pour le choix du niveau de difficulté.
        Avec peronnalisation des boutons radio via du CSS
        */
        VBox calculLevel = new VBox(2);
        final ToggleGroup rbGroupLevel = new ToggleGroup();
        RadioButton rbEasyLevel = new RadioButton("Niveau Facile");
        RadioButton rbDifficultLevel = new RadioButton("Niveau Difficile");
        rbDifficultLevel.setStyle("-fx-font: 24 arial;");
        rbEasyLevel.setStyle("-fx-font: 24 arial;");
        rbEasyLevel.setToggleGroup(rbGroupLevel);
        rbDifficultLevel.setToggleGroup(rbGroupLevel);
        calculLevel.getChildren().addAll(rbEasyLevel,rbDifficultLevel);
 
        /*
        Initialisation des différents éléments visuel avec utilisation du CSS
        */
        Label calcul = new Label("Opèration : ");
        calcul.setStyle("-fx-font: 24 arial;");
        Label lblLigneCalcul = new Label("");
        Label lblTextResult = new Label("");
        lblLigneCalcul.setStyle("-fx-font: 24 arial;");
        Label resultat = new Label("Rèponse :");
        resultat.setStyle("-fx-font: 24 arial;");
        Label lblMsgChoixLevel = new Label("Veuillez choisir votre niveau de difficulté");
        lblMsgChoixLevel.setStyle("-fx-font: 25 arial;");
 
        Label lblAfficheResult = new Label("");
        lblAfficheResult.setStyle("-fx-font: 20 arial;");
        lblAfficheResult.setVisible(false);
 
        TextField tbResult = new TextField();
        tbResult.setPrefHeight(50);
        tbResult.setFont(Font.font ("Verdana", 20));
        /*
        Initialisation d'un Pane secondaire dans lequel se trouve 
        un groupe de boutons : "Valider", "Solution" et "Autre calcul"
        */
        HBox boutons = new HBox();
        Button btnValider = new Button("Valider");
        btnValider.setPrefSize(100, 50);
        Button btnSolution = new Button("Solution");
        btnSolution.setPrefSize(100, 50);
        Button btnAutreCalcul = new Button("Autre Calcul");
        btnAutreCalcul.setPrefSize(100, 50);
        boutons.getChildren().addAll(btnValider, btnSolution, btnAutreCalcul);
 
        /*
        'Acrochage' des différents éléments graphique dans le Pane principal
        */
        calculPane.setConstraints(calculLevel, 1,0);
        calculPane.setConstraints(calcul, 0,1);
        calculPane.setConstraints(lblLigneCalcul, 1,1);
        calculPane.setConstraints(resultat, 0,2);
        calculPane.setConstraints(tbResult, 1,2);
        calculPane.setConstraints(boutons, 1,3);
        calculPane.setConstraints(lblTextResult, 1,4);
        calculPane.setConstraints(lblAfficheResult, 1,5);
        calculPane.setConstraints(lblMsgChoixLevel, 1,6);
 
        calculPane.getChildren().add(calculLevel);
        calculPane.getChildren().add(calcul);
        calculPane.getChildren().add(lblLigneCalcul);
        calculPane.getChildren().add(resultat);
        calculPane.getChildren().add(tbResult);
        calculPane.getChildren().add(boutons);
        calculPane.getChildren().add(lblTextResult);
        calculPane.getChildren().add(lblMsgChoixLevel);
        calculPane.getChildren().add(lblAfficheResult);
 
 
        /*
        Evenements en fonction du choix de difficulté au nibeau des boutons radio
        */
        rbGroupLevel.selectedToggleProperty().addListener(new ChangeListener<Toggle>()
        {
        @Override
        public void changed(ObservableValue<? extends Toggle> ov, Toggle t, Toggle t1)
            {
                lblMsgChoixLevel.setVisible(false);
                lblAfficheResult.setVisible(false);
                lblTextResult.setVisible(false);
                tbResult.setDisable(false);
 
 
                int diff;
                int nbrOperateur;
                int nbrTemp = 0; 
                int resultat;
                RadioButton chk = (RadioButton)t1.getToggleGroup().getSelectedToggle(); // Cast object to radio button
 
 
                if ("Niveau Difficile".equals(chk.getText())) {
                    diff = 100;
                    nbrOperateur = 3;
                    tbResult.setText("");
                }
                else{
                    diff = 9;
                    nbrOperateur = 2;
                    tbResult.setText("");
                }
 
                int nbr1 = randomNumber(diff);
                int nbr2 = randomNumber(diff);
                int operator = randomNumber(nbrOperateur);
 
 
                if ("Niveau Facile".equals(chk.getText())) {
                    if (nbr1 < nbr2) {
                        nbr1 = nbrTemp;
                        nbr1 = nbr2;
                        nbr2 = nbrTemp;
                    }
                }
 
 
                Calcul myOperation = new Calcul();
 
 
                String operateur;
                switch (operator) {
                    case 1:
                        operateur = "+";
                        resultat = myOperation.Add(nbr1,nbr2);
                        break;
                    case 2:
                        operateur = "-";
                        resultat = myOperation.Sous(nbr1,nbr2);
                        break;
                    case 3:
                        operateur = "x";
                        resultat = myOperation.Multi(nbr1,nbr2);
                        break;
                    default:
                        throw new AssertionError();
                }
 
                String ope = (nbr1 + " " + operateur + " " + nbr2);
 
                lblLigneCalcul.setText(ope);
 
 
                String res = Integer.toString(resultat);
 
 
                btnValider.setOnMouseClicked(new EventHandler<MouseEvent>() {
                    @Override
                    public void handle(MouseEvent event) {
                        String msg;
                        String txtColor;
                        if ("".equals(tbResult.getText())) {
                            Alert alert = new Alert(Alert.AlertType.INFORMATION);
                            alert.setTitle("");
                            alert.setHeaderText(null);
                            alert.setContentText("Veuillez saisir une réponse");
                            alert.showAndWait();
                        }else{
                            if (tbResult.getText().equals(res)) {
                                msg = "Bravo";
                                txtColor = "#00ff00";
                            }else{
                                msg = "Dommage";
                                txtColor = "#ff0000";
                                lblAfficheResult.setText("La solution était : " + res);
                                lblAfficheResult.setVisible(true);  
                            }
                                lblTextResult.setText(msg);
                                lblTextResult.setStyle("-fx-font: 60 arial;");
                                lblTextResult.setTextFill(Color.web(txtColor));
                                rbEasyLevel.setSelected(false);
                                rbDifficultLevel.setSelected(false);
                                lblMsgChoixLevel.setVisible(true);
                                lblTextResult.setVisible(true);
                                tbResult.setDisable(true);
                        }
 
 
                    }
                });
 
                /*
                Evenement sur clic du bouton permettant d'affichier la solution
                */
                btnSolution.setOnMouseClicked(new EventHandler<MouseEvent>() {
                    @Override
                    public void handle(MouseEvent event) {
                        String msg;
                        if (lblLigneCalcul.getText().isEmpty()) {
                            msg = "Je ne peux pas donner de solution qi il n'y a pas de question !!";
                        }else{
                            msg = "La solution était : " + res;
                        }
 
 
                        lblAfficheResult.setText(msg);
                        lblAfficheResult.setVisible(true);
                        rbEasyLevel.setSelected(false);
                        rbDifficultLevel.setSelected(false);
                        lblMsgChoixLevel.setVisible(true);
                        tbResult.setDisable(true);
 
 
                    }
                });
 
                /*
                Evenement sur clic du bouton permettant de faire un nouveau calcul
                */
                btnAutreCalcul.setOnMouseClicked(new EventHandler<MouseEvent>() {
                    @Override
                    public void handle(MouseEvent event) {
                        rbEasyLevel.setSelected(false);
                        rbDifficultLevel.setSelected(false);
                        lblMsgChoixLevel.setVisible(true);
                        tbResult.setDisable(true);
                        lblLigneCalcul.setText("");
                        lblMsgChoixLevel.setVisible(true);
                        lblAfficheResult.setVisible(false);
                        lblTextResult.setVisible(false);
                    }
                });
            }
        });
        return calculPane;
    }
 
 
    /*
    Methode qui retourne de chiffres aléatoire
    */
    public int randomNumber(int b){
        int result;
        int a = 1;
        result  = a + (int)(Math.random()*b);
        return result;
    }
 
    /*
    Méthode de calcul d'addition
    */
    public int Add(int a, int b){
        int result;
        result = a + b;
        return result;
    }
 
    /*
    Méthode de calcul de soustraction
    */
    public int Sous(int a, int b){
        int result;
        result = a - b;
        return result;
    }
 
    /*
    Méthode de calcul de miltiplication
    */
    public int Multi(int a, int b){
        int result;
        result = a * b;
        return result;
    }
 
}

Tu as le code complet de la classe.

Merci de ton aide.

[joel.drigo]

Ton problème n'a rien à voir avec un problème de thread, et c'est justement pour le voir qu'il aurait fallut nous montrer ton code dès le départ :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
if (nbr1 < nbr2) {
                        nbr1 = nbrTemp;
                        nbr1 = nbr2;
                        nbr2 = nbrTemp;
                    }

nbrTemp vaut 0 ! Donc si nbr1 < nbr2, tu fais nbr2=0 ! C'est probablement çà que tu voulais écrire :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
if (nbr1 < nbr2) {
                        nbrTemp=nbr1;
                        nbr1 = nbr2;
                        nbr2 = nbrTemp;
                    }

D'ailleurs, on déclarant nbrTemp à ce moment (ça ne sert à rien de le déclarer ailleurs), tu n'aurais probablement pas eu le souci :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
if (nbr1 < nbr2) {
                        int nbrTemp=nbr1;
                        nbr1 = nbr2;
                        nbr2 = nbrTemp;
                    }

[FCL31]

Oh PUT@:&# (oui, on parle comme ça dans le sud ^^), je suis vrai .

Je savais bien que c’était un truc tout con.

'il aurait fallut nous montrer ton code dès le départ

Oui, effectivement.

Merci beaucoup Joel et developpez.net