Problème de code en Java

Version imprimable

Bonjour,
alors voila j'essai de coder mais j'ai de nombreux problèmes.

Est ce que quelqu'un est en mesure de me dire pourquoi ces lignes sont fausses:

Code:

1
2
3
if (cat[i][4]=1]){ T2[i][3]= masculin;}
            else if {T2[i][3]=féminin;}
           if (cat[i][3]== 1994 || cat[i][3]==1995){T2[i][3]=junior;}

le début du programme étant :

Code:

1
2
3
4
5
6
7
8
9
10
11
public static String[][] classement(String[] Tnom, int[][] cat, int nbboxeurs) {
        String masculin; String junior; String cadet; String minime; String senior; String poussin;
        cat = new int[nbboxeurs][5];
        Tnom = new String[nbboxeurs];
        String[][] T2 = new String[nbboxeurs][4];
        int i;
        for (i = 0; i < nbboxeurs; i++) {
            T2[i][0]=Tnom[i];
            if (cat[i][4]=1]){ T2[i][3]= masculin;}
            else if {T2[i][3]=féminin;}
           if (cat[i][3]== 1994 || cat[i][3]==1995){T2[i][3]=junior;}

Merci beaucoup

12/11/2011, 01h11
plegat

Salut,

Citation:

Envoyé par felixismad

Est ce que quelqu'un est en mesure de me dire pourquoi ces lignes sont fausses:

Oui.
Parce que tu formates ton code n'importe comment, ce qui ne te permet pas de voir que tu fermes des crochets sans les avoir ouverts...
Parce que tu fais un test d'égalité avec "=" au lieu de "==".
Parce que tu fais un test d'égalité entre un int et une string... ah non, au temps pour moi, ce n'est pas un test (voir plus loin)
Parce que tu utilises des variables non déclarées (il est déclaré ou "féminin"?)
Parce que tu utilises des variables non initialisées. Non, pas "des", toutes en fait! ;)
Parce que n'encadres pas chaque terme de tes tests entre parenthèses.
Parce que tu oublies le test après le "else if".
...

Bref, euh... revois tes cours de Java, je sens que tu as de grosses lacunes là...

Et n'oublie pas de mettre tes chaines entre guillemets! ;)

alors j'ai suivi ton conseil, je me suis penché à nouveau sur mes cours...
J'ai tout corrigé, mais sans grand étonnement ça ne fonctionne pas!
Mes lignes ne sont pas fausses mais je n'ai pas la sortie attendue!

Si jamais tu as le temps voici mon code:
je t'ai mis la réponse de Netbeans ci dessous:
Voici mon code:

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
/* 
* To change this template, choose Tools | Templates 
* and open the template in the editor. 
*/ 
package javaapplication2; 
/** 
* 
* @author Degrave Félix 
*/ 
public class JavaApplication2 { 
public static String[][] classement(String[] Tnom, int[][] cat, int nbboxeurs) { 
cat = new int[nbboxeurs][5]; 
Tnom = new String[nbboxeurs]; 
String[][] T2 = new String[nbboxeurs][4]; 
int i; 
for (i = 0; i <= nbboxeurs; i++) { 
T2[i][0] = Tnom[i]; 
if (cat[i][4] == 1) { 
T2[i][3] = "masculin"; 
} else { 
T2[i][3] = "féminin"; 
} 
if (cat[i][3] == 1994 || cat[i][3] == 1995) { 
T2[i][3] = "junior"; 
} 
if (cat[i][3] == 1996 || cat[i][3] == 1997) { 
T2[i][3] = "cadet"; 
} 
if (cat[i][3] == 1998 || cat[i][3] == 1999) { 
T2[i][3] = "minime"; 
} 
if (cat[i][3] == 2000 || cat[i][3] == 2001) { 
T2[i][3] = "benjamin"; 
} 
if (cat[i][3] >= 2002) { 
T2[i][3] = "poussin"; 
} 
if (cat[i][3] <= 1993) { 
T2[i][3] = "senior"; 
} 
if (T2[i][3] == "poussin" || T2[i][3] == "benjamin") { 
if (cat[i][5] > 15 && cat[i][5] <= 18) { 
T2[i][3] = "entre15et18kginclu"; 
} 
if (cat[i][5] > 18 && cat[i][5] <= 21) { 
T2[i][3] = "entre18et21kginclu"; 
} 
if (cat[i][5] > 21 && cat[i][5] <= 24) { 
T2[i][3] = "entre21et24kginclu"; 
} 
if (cat[i][5] > 24 && cat[i][5] <= 27) { 
T2[i][3] = "entre24et27kginclu"; 
} 
if (cat[i][5] > 27 && cat[i][5] <= 30) { 
T2[i][3] = "entre27et30kginclu"; 
} 
if (cat[i][5] > 30 && cat[i][5] <= 33) { 
T2[i][3] = "entre30et33kginclu"; 
} 
if (cat[i][5] > 33 && cat[i][5] <= 36) { 
T2[i][3] = "entre33et36kginclu"; 
} 
if (cat[i][5] > 36 && cat[i][5] <= 39) { 
T2[i][3] = "entre36et39kginclu"; 
} 
if (cat[i][5] > 39 && cat[i][5] <= 42) { 
T2[i][3] = "entre39et42kginclu"; 
} 
if (cat[i][5] > 42 && cat[i][5] <= 45) { 
T2[i][3] = "entre42et45kginclu"; 
} 
if (cat[i][5] > 45 && cat[i][5] <= 48) { 
T2[i][3] = "entre45et48kginclu"; 
} 
if (cat[i][5] > 48 && cat[i][5] <= 51) { 
T2[i][3] = "entre48et51kginclu"; 
} 
if (cat[i][5] > 51 && cat[i][5] <= 54) { 
T2[i][3] = "entre51et54kginclu"; 
} 
if (cat[i][5] > 54 && cat[i][5] <= 57) { 
T2[i][3] = "entre54et57kginclu"; 
} 
if (cat[i][5] > 57 && cat[i][5] <= 60) { 
T2[i][3] = "entre57et60kginclu"; 
} 
if (cat[i][5] > 60 && cat[i][5] <= 64) { 
T2[i][3] = "entre60et64kginclu"; 
} 
if (cat[i][5] > 64 && cat[i][5] <= 68) { 
T2[i][3] = "entre64et68kginclu"; 
} 
if (cat[i][5] > 68 && cat[i][5] <= 72) { 
T2[i][3] = "entre68et72kginclu"; 
} 
} 
if (T2[i][3] == "minime") { 
if (cat[i][5] > 30 && cat[i][5] <= 33) { 
T2[i][3] = "entre30et33kginclu"; 
} 
if (cat[i][5] > 33 && cat[i][5] <= 36) { 
T2[i][3] = "entre33et36kginclu"; 
} 
if (cat[i][5] > 36 && cat[i][5] <= 39) { 
T2[i][3] = "entre36et39kginclu"; 
} 
if (cat[i][5] > 39 && cat[i][5] <= 42) { 
T2[i][3] = "entre39et42kginclu"; 
} 
if (cat[i][5] > 42 && cat[i][5] <= 45) { 
T2[i][3] = "entre42et45kginclu"; 
} 
if (cat[i][5] > 45 && cat[i][5] <= 48) { 
T2[i][3] = "entre45et48kginclu"; 
} 
if (cat[i][5] > 48 && cat[i][5] <= 51) { 
T2[i][3] = "entre48et51kginclu"; 
} 
if (cat[i][5] > 51 && cat[i][5] <= 54) { 
T2[i][3] = "entre51et54kginclu"; 
} 
if (cat[i][5] > 54 && cat[i][5] <= 57) { 
T2[i][3] = "entre54et57kginclu"; 
} 
if (cat[i][5] > 57 && cat[i][5] <= 60) { 
T2[i][3] = "entre57et60kginclu"; 
} 
if (cat[i][5] > 60 && cat[i][5] <= 64) { 
T2[i][3] = "entre60et64kginclu"; 
} 
if (cat[i][5] > 64 && cat[i][5] <= 68) { 
T2[i][3] = "entre64et68kginclu"; 
} 
if (cat[i][5] > 68 && cat[i][5] <= 72) { 
T2[i][3] = "entre68et72kginclu"; 
} 
if (cat[i][5] > 72 && cat[i][5] <= 76) { 
T2[i][3] = "entre72et76kginclu"; 
} 
if (cat[i][5] > 76) { 
T2[i][3] = "plusde76kg"; 
} 
} 
if (T2[i][3] == "cadet") { 
if (cat[i][5] > 42 && cat[i][5] <= 45) { 
T2[i][3] = "entre42et45kginclu"; 
} 
if (cat[i][5] > 45 && cat[i][5] <= 48) { 
T2[i][3] = "entre45et48kginclu"; 
} 
if (cat[i][5] > 48 && cat[i][5] <= 51) { 
T2[i][3] = "entre48et51kginclu"; 
} 
if (cat[i][5] > 51 && cat[i][5] <= 54) { 
T2[i][3] = "entre51et54kginclu"; 
} 
if (cat[i][5] > 54 && cat[i][5] <= 57) { 
T2[i][3] = "entre54et57kginclu"; 
} 
if (cat[i][5] > 57 && cat[i][5] <= 60) { 
T2[i][3] = "entre57et60kginclu"; 
} 
if (cat[i][5] > 60 && cat[i][5] <= 64) { 
T2[i][3] = "entre60et64kginclu"; 
} 
if (cat[i][5] > 64 && cat[i][5] <= 68) { 
T2[i][3] = "entre64et68kginclu"; 
} 
if (cat[i][5] > 68 && cat[i][5] <= 72) { 
T2[i][3] = "entre68et72kginclu"; 
} 
if (cat[i][5] > 72 && cat[i][5] <= 76) { 
T2[i][3] = "entre72et76kginclu"; 
} 
if (cat[i][5] > 76 && cat[i][5] <= 80) { 
T2[i][3] = "entre762et80kginclu"; 
} 
if (cat[i][5] > 80 && cat[i][5] <= 86) { 
T2[i][3] = "entre80et86kginclu"; 
} 
if (cat[i][5] > 86) { 
T2[i][3] = "plusde86kg"; 
} 
} 
if (T2[i][3] == "junior" || T2[i][3] == "senior") { 
if (cat[i][5] > 45 && cat[i][5] <= 48) { //ligne 190// 
T2[i][3] = "entre45et48kginclu"; 
} 
if (cat[i][5] > 48 && cat[i][5] <= 51) { 
T2[i][3] = "entre48et51kginclu"; 
} 
if (cat[i][5] > 51 && cat[i][5] <= 54) { 
T2[i][3] = "entre51et54kginclu"; 
} 
if (cat[i][5] > 54 && cat[i][5] <= 57) { 
T2[i][3] = "entre54et57kginclu"; 
} 
if (cat[i][5] > 57 && cat[i][5] <= 60) { 
T2[i][3] = "entre57et60kginclu"; 
} 
if (cat[i][5] > 60 && cat[i][5] <= 64) { 
T2[i][3] = "entre60et64kginclu"; 
} 
if (cat[i][5] > 64 && cat[i][5] <= 69) { 
T2[i][3] = "entre64et69kginclu"; 
} 
if (cat[i][5] > 69 && cat[i][5] <= 75) { 
T2[i][3] = "entre69et75kginclu"; 
} 
if (cat[i][5] > 75 && cat[i][5] <= 81) { 
T2[i][3] = "entre75et81kginclu"; 
} 
if (cat[i][5] > 81 && cat[i][5] <= 91) { 
T2[i][3] = "entre812et91kginclu"; 
} 
if (cat[i][5] > 91) { 
T2[i][3] = "plusde91kg"; 
} 
} 
} 
return T2; 
} 
/** 
* @param args the command line arguments 
*/ 
public static void main(String[] args) { 
String[] Tnom = {"Morgan"}; 
int nbboxeurs = 1; 
int[][] cat = {{2, 11, 2000, 1, 42}}; 
String [][] T9= classement(Tnom, cat, nbboxeurs); //ligne 238// 
System.out.println(T9); 
// TODO code application logic here 
} 
}

voici la sortie quand je fais tourner le programme:

Citation:

run:
Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: 5
at javaapplication2.JavaApplication2.classement(JavaApplication2.java:190)
at javaapplication2.JavaApplication2.main(JavaApplication2.java:238)
Java Result: 1
BUILD SUCCESSFUL (total time: 0 seconds)

12/11/2011, 20h31
felixismad

j'ai corriger l'erreur sur le tableau, j'avais oublié que la case 0 existait!
d'autre idée ? ça ne fonctionne pas
12/11/2011, 20h48
plegat

Citation:

Envoyé par felixismad

ça ne fonctionne pas

on va aller loin avec ça...

Alors, déjà, je me répète.... les balises [ CODE ] stp!!!

Ensuite précise où ça ne fonctionne pas.

Et avant de répondre, réfléchit aussi au pourquoi tu redéclares les paramètres de ta méthode "classement" dans le corps de ta méthode...
Ca devrait mieux fonctionner ensuite...

Remarques:
1) comparaison d'égalité entre strings, ça se fait avec "equals", et pas avec "=="
2) prend l'habitude de mettre chaque terme de tes tests entre parenthèses (je me répète...). Là ça passe parce que le ">" est prioritaire sur le "||", mais un jour tu te planteras...
3) ton algo est très bourrin... et pas classe! :mrgreen:
13/11/2011, 10h47
felixismad

oui ça ne fonctionne pas, je ne sais pas ou ça bug malheureusement.

Voici ma sortie:

Citation:

run:
[[Ljava.lang.String;@19821f
BUILD SUCCESSFUL (total time: 0 seconds)

J'ai modifié les erreurs de tableau, pour les equals si je change, ça me dis que la ligne est fausse.

Code:

if (T2[i][1] equals "poussin" || T2[i][1] equals "benjamin") {
13/11/2011, 13h30
plegat

Citation:

Envoyé par felixismad

oui ça ne fonctionne pas, je ne sais pas ou ça bug malheureusement.

Normal.
Ca ne bug pas. :mouarf:

Par contre ça ne fait pas ce que tu voulais, ça je comprends. Maintenant, explique ce que tu veux faire, et explique ce que tu as fait, et normalement tu devrais comprendre.

Citation:

Envoyé par felixismad

J'ai modifié les erreurs de tableau, pour les equals si je change, ça me dis que la ligne est fausse.
"
if (T2[i][1] equals "poussin" || T2[i][1] equals "benjamin") {
"

euuuh... c'est comme ça que tu appliques une méthode sur un objet??? Y'aurait pas des points et des parenthèses à mettre par hasard?

Dernier conseil, vu que je (et les autres aussi) ne suis pas là pour t'apprendre à programmer en java, ni pour débugger ton code, et que là c'est tout de même énorme comme style d'erreur, je ne saurais que trop te conseiller d'aller lire un bon cours Java pour acquérir les bases.