Exception in thread "AWT-EventQueue-0" java.lang.ArrayIndexOutOfBoundsException:

**gani_95** · 24/08/2017, 13h39

Salut Tout le monde aidez moi

.
Ma premier classe est la bases et ses fonctions.
En general je fait une application de gestion mais le problem est que je ne peut modifier un ligne, mais l'ajout ce passe bien, mais a la modification apres selection de la ligne une exception apparet dans les logs;Exception in thread "AWT-EventQueue-0" java.lang.ArrayIndexOutOfBoundsException: 6 ...

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
public class MaBase {
 
    Connection connection;
    Statement statement;
    String SQL;
 
    String url;
    String username;
    String password;
    Socket client;
    int Port;
    String Host;
 
 
  public MaBase(String url, String username, String password, String Host, int Port) {
 
 
  this.url = url;
        this.username = username;
        this.password = password;
        this.Host = Host;
 
   this.Port = Port;
    }
 
 
 public Connection connexionDatabase() {
 
 
  try {
            Class.forName("com.mysql.jdbc.Driver");
 
  connection = DriverManager.getConnection(url, username, password);
        } catch (Exception e) 
{System.err.println(e);//
        }
        return connection;
    }
 
 
   public Connection closeconnexion() {
 
        try {
            connection.close();
        } catch (Exception e) {System.err.println(e);//
        }
        return connection;
    }
 
    public ResultSet exécutionQuery(String sql) {
        connexionDatabase();
        ResultSet resultSet = null;
        try {
            statement = connection.createStatement();
            resultSet = statement.executeQuery(sql);
            System.out.println(sql);
        } catch (SQLException ex) {System.err.println(ex);//
        }
        return resultSet;
    }
 
//update query
 
 public String queryUpdate(String nomTable, String[] nomColonne, String[] contenuTableau, String état) {
 
        connexionDatabase();
        int i;
        SQL = "UPDATE " + nomTable + " SET ";
 
        for (i = 0; i <= nomColonne.length - 1; i++) {
            SQL += nomColonne[i] + "='" + contenuTableau[i] + "'";
            if (i < nomColonne.length - 1) {
                SQL += ",";
            }
        }
 
        SQL += " WHERE " + état;
        return this.exécutionUpdate(SQL);
 
    }

Puis l'interface utilisateur

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
public class Articles extends javax.swing.JFrame {
 
    ResultSet rs;
    MaBase db;
 
    public Articles() {
         try{
            db = new MaBase(
                new connection().HOST_DB, 
                new connection().USERNAME_DB, 
                new connection().PASSWORD_DB, 
                new connection().IPHOST, 
                new connection().PORT);
        }
        catch(Exception e){
            System.err.println("OOOGOOgggggggggggggggg");
        }
 
        initComponents();
        table();
        actualiser();
        remplirfournisseurs();
        remplirlisttype();
 
 
    }
 
    /*
    REMPLISSAGE DU TABLEAU
    */
     public void table() {
        String t[] = {"*"};
        rs = db.querySelect(t, "articles");
        try{
           Tblart.setModel(new tablemodel(rs)); 
 
        }
        catch(Exception e){
            System.err.println("Errreur a la tabbble mmodel !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!");
        }
        nbr.setText(String.valueOf(Tblart.getRowCount()));
    }
 
//L'erreur vien de la je pense :)
 
 private void jButton3ActionPerformed(java.awt.event.ActionEvent evt) {                                         
       if (txtcode.getText().equals("") ||
               txtlib.getText().equals("") || 
               taille.getText().equals("")  || 
               quantite.getText().equals("") ) {
            JOptionPane.showMessageDialog(this, "SVP entrer les informations complete");
        } else {
                  String[] colon = {"code","libelle","taille","type","fourni","policier","quantite"};
           String[] inf = {
               txtcode.getText(),
                txtlib.getText(),
                taille.getText(),
                combtype.getSelectedItem().toString(),
                combfour.getSelectedItem().toString(),
 
                quantite.getText()
            };
            String id = String.valueOf(Tblart.getValueAt(Tblart.getSelectedRow(), 0));
            System.out.println(db.queryUpdate("Articles", colon, inf, "id='" + id + "'"));
            table();
            actualiser();
       }
 
 
//table selection rows
  private void TblartMouseClicked(java.awt.event.MouseEvent evt) {                                    
 
        txtcode.setText(String.valueOf(Tblart.getValueAt(Tblart.getSelectedRow(), 1)));
        txtlib.setText(String.valueOf(Tblart.getValueAt(Tblart.getSelectedRow(), 2)));
        taille.setText(String.valueOf(Tblart.getValueAt(Tblart.getSelectedRow(), 3)));
        combtype.setSelectedItem(String.valueOf(Tblart.getValueAt(Tblart.getSelectedRow(), 4)));
        combfour.setSelectedItem(String.valueOf(Tblart.getValueAt(Tblart.getSelectedRow(), 5)));
        quantite.setText(String.valueOf(Tblart.getValueAt(Tblart.getSelectedRow(), 6)));
    } 
    }

L'erreur est la suivante :

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
Exception in thread "AWT-EventQueue-0" java.lang.ArrayIndexOutOfBoundsException: 6
    at cnx.MaBase.queryUpdate(MaBase.java:203)
    at gestion.habillts.Articles.jButton3ActionPerformed(Articles.java:673)
    at gestion.habillts.Articles.access$500(Articles.java:25)
    at gestion.habillts.Articles$7.actionPerformed(Articles.java:377)
    at javax.swing.AbstractButton.fireActionPerformed(AbstractButton.java:2022)
    at javax.swing.AbstractButton$Handler.actionPerformed(AbstractButton.java:2346)
    at javax.swing.DefaultButtonModel.fireActionPerformed(DefaultButtonModel.java:402)
    at javax.swing.DefaultButtonModel.setPressed(DefaultButtonModel.java:259)
    at javax.swing.plaf.basic.BasicButtonListener.mouseReleased(BasicButtonListener.java:252)
    at java.awt.Component.processMouseEvent(Component.java:6525)
    at javax.swing.JComponent.processMouseEvent(JComponent.java:3324)
    at java.awt.Component.processEvent(Component.java:6290)
    at java.awt.Container.processEvent(Container.java:2234)
    at java.awt.Component.dispatchEventImpl(Component.java:4881)
    at java.awt.Container.dispatchEventImpl(Container.java:2292)
    at java.awt.Component.dispatchEvent(Component.java:4703)
    at java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4898)
    at java.awt.LightweightDispatcher.processMouseEvent(Container.java:4533)
    at java.awt.LightweightDispatcher.dispatchEvent(Container.java:4462)
    at java.awt.Container.dispatchEventImpl(Container.java:2278)
    at java.awt.Window.dispatchEventImpl(Window.java:2750)
    at java.awt.Component.dispatchEvent(Component.java:4703)
    at java.awt.EventQueue.dispatchEventImpl(EventQueue.java:758)
    at java.awt.EventQueue.access$500(EventQueue.java:97)
    at java.awt.EventQueue$3.run(EventQueue.java:709)
    at java.awt.EventQueue$3.run(EventQueue.java:703)
    at java.security.AccessController.doPrivileged(Native Method)
    at java.security.ProtectionDomain$1.doIntersectionPrivilege(ProtectionDomain.java:75)
    at java.security.ProtectionDomain$1.doIntersectionPrivilege(ProtectionDomain.java:86)
    at java.awt.EventQueue$4.run(EventQueue.java:731)
    at java.awt.EventQueue$4.run(EventQueue.java:729)
    at java.security.AccessController.doPrivileged(Native Method)
    at java.security.ProtectionDomain$1.doIntersectionPrivilege(ProtectionDomain.java:75)
    at java.awt.EventQueue.dispatchEvent(EventQueue.java:728)
    at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:201)
    at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:116)
    at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:105)
    at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:101)
    at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:93)
    at java.awt.EventDispatchThread.run(EventDispatchThread.java:82)

**NoClassDefFound** · 24/08/2017, 15h34

ArrayIndexOutOfBounds signifie que tu cherches à récupérer un élément dans un tableau à une position qui n'existe pas dans ce tableau (Ex: le cinquième élément d'un tableau de deux)
La stacktrace te donnes le numéro de ligne où ça plante: gestion.habillts.Articles.jButton3ActionPerformed(Articles.java:673)

**gani_95** · 24/08/2017, 20h00

C'est en fait l'appel de la fonction MaBase.queryUpdate qui pose problème ...

**joel.drigo** · 24/08/2017, 20h54

at cnx.MaBase.queryUpdate(MaBase.java:203)

Donc tu regardes en ligne 203 de MaBase.java quel est le code. De toute manière, nous, on ne peut pas, tu ne montres que 80 lignes de cette classe, donc voir la ligne 203 est exclu.

**gani_95** · 26/08/2017, 02h58

Envoyé par joel.drigo

at cnx.MaBase.queryUpdate(MaBase.java:203)

Donc tu regardes en ligne 203 de MaBase.java quel est le code. De toute manière, nous, on ne peut pas, tu ne montres que 80 lignes de cette classe, donc voir la ligne 203 est exclu.

Dsl Joel pour le retard mais a la ligne 203 y'a le code suivant :

Code :

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

for (i = 0; i <= nomColonne.length - 1; i++) {

CODE COMPLETE de la classe MaBase :
Pour mes autres formulaire ça marche du tonnerre je ne sais ce qui cloches dans cette table ou dans cette interface. J'ai pourtant mit le nomColonne moin 1 ...

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
/*
 * To change this license header, choose License Headers in Project Properties.
 * To change this template file, choose Tools | Templates
 * and open the template in the editor.
 */
package cnx;
 
 
 
import java.net.Socket;
import java.sql.*;
 
/**
 *
 * @author Drizzy
 */
 
public class MaBase {
 
    Connection connection;
    Statement statement;
    String SQL;
 
    String url;
    String username;
    String password;
    Socket client;
    int Port;
    String Host;
 
 
  public MaBase(String url, String username, String password, String Host, int Port) {
 
 
  this.url = url;
        this.username = username;
        this.password = password;
        this.Host = Host;
 
   this.Port = Port;
    }
 
 
 public Connection connexionDatabase() {
 
 
  try {
            Class.forName("com.mysql.jdbc.Driver");
 
  connection = DriverManager.getConnection(url, username, password);
        } catch (Exception e) 
{System.err.println(e);//
        }
        return connection;
    }
 
 
   public Connection closeconnexion() {
 
        try {
            connection.close();
        } catch (Exception e) {System.err.println(e);//
        }
        return connection;
    }
 
    public ResultSet exécutionQuery(String sql) {
        connexionDatabase();
        ResultSet resultSet = null;
        try {
            statement = connection.createStatement();
            resultSet = statement.executeQuery(sql);
            System.out.println(sql);
        } catch (SQLException ex) {System.err.println(ex);//
        }
        return resultSet;
    }
 
    public String exécutionUpdate(String sql) {
        connexionDatabase();
        String result = "";
        try {
            statement = connection.createStatement();
            statement.executeUpdate(sql);
            result = sql;
 
        } catch (SQLException ex) {
            result = ex.toString();
        }
        return result;
 
    }
 
//Fonction d'exécution de la requête pour sélectionner toutes les colonnes
    public ResultSet querySelectAll(String nomTable) {
 
        connexionDatabase();
        SQL = "SELECT * FROM " + nomTable;
        System.out.println(SQL);
        return this.exécutionQuery(SQL);
 
    }
 
//Fonction d'exécution de la requête  pour sélectionner toutes les colonnes avec la condition
    public ResultSet querySelectAll(String nomTable, String état) {
 
        connexionDatabase();
        SQL = "SELECT * FROM " + nomTable + " WHERE " + état;
        return this.exécutionQuery(SQL);
 
    }
 
//Fonction d'exécution de la requête pour sélectionner les colonnes spécifique
    public ResultSet querySelect(String[] nomColonne, String nomTable) {
 
        connexionDatabase();
        int i;
        SQL = "SELECT ";
 
        for (i = 0; i <= nomColonne.length - 1; i++) {
            SQL += nomColonne[i];
            if (i < nomColonne.length - 1) {
                SQL += ",";
            }
        }
 
        SQL += " FROM " + nomTable;
        return this.exécutionQuery(SQL);
 
    }
 
//La fonction d'exécution des requêtes de surcharge pour sélectionner les colonnes spécifiques avec la close WHERE
    public ResultSet fcSelectCommand(String[] nomColonne, String nomTable, String état) {
 
        connexionDatabase();
        int i;
        SQL = "SELECT ";
 
        for (i = 0; i <= nomColonne.length - 1; i++) {
            SQL += nomColonne[i];
            if (i < nomColonne.length - 1) {
                SQL += ",";
            }
        }
 
        SQL += " FROM " + nomTable + " WHERE " + état;
        return this.exécutionQuery(SQL);
 
    }
 
    //Fonction d'insertion
    public String queryInsert(String nomTable, String[] contenuTableau) {
 
        connexionDatabase();
        int i;
        SQL = "INSERT INTO " + nomTable + " VALUES(";
 
        for (i = 0; i <= contenuTableau.length - 1; i++) {
            SQL += "'" + contenuTableau[i] + "'";
            if (i < contenuTableau.length - 1) {
                SQL += ",";
            }
        }
 
        SQL += ")";
        return this.exécutionUpdate(SQL);
 
    }
//Fonction d'execution de la requete INSERT
 
    public String queryInsert(String nomTable, String[] nomColonne, String[] contenuTableau) {
 
        connexionDatabase();
        int i;
        SQL = "INSERT INTO " + nomTable + "(";
        for (i = 0; i <= nomColonne.length - 1; i++) {
            SQL += nomColonne[i];
            if (i < nomColonne.length - 1) {
                SQL += ",";
            }
        }
        SQL += ") VALUES(";
        for (i = 0; i <= contenuTableau.length - 1; i++) {
            SQL += "'" + contenuTableau[i] + "'";
            if (i < contenuTableau.length - 1) {
                SQL += ",";
            }
        }
 
        SQL += ")";
        return this.exécutionUpdate(SQL);
 
    }
 
//Fonction d'execution de requete update
   public String queryUpdate(String nomTable, String[] nomColonne, String[] contenuTableau, String état) {
 
       try{
        connexionDatabase();
        int i;
        SQL = "UPDATE " + nomTable + " SET ";
 
        for (i = 0; i <= nomColonne.length - 1; i++) {
            SQL += nomColonne[i] + "='" + contenuTableau[i] + "'";
            if (i < nomColonne.length - 1) {
                SQL += ",";
            }
        }
 
        SQL += " WHERE " + état;
        return this.exécutionUpdate(SQL);
 
    }
       catch (Exception e){
           e.printStackTrace();
       }
    return null;
   }
 
 
 
//Fonction d'execution de requete delete
    public String queryDelete(String nomtable) {
 
        connexionDatabase();
        SQL = "DELETE FROM " + nomtable;
        return this.exécutionUpdate(SQL);
 
    }
 
//Fonction d'execution de la requete delete avec la clause where
    public String queryDelete(String nomTable, String état) {
 
        connexionDatabase();
        SQL = "DELETE FROM " + nomTable + " WHERE " + état;
        return this.exécutionUpdate(SQL);
 
    }
}

**adiGuba** · 26/08/2017, 13h17

Salut,

Ton problème vient du fait que les deux tableaux que tu passes à queryUpdate() n'ont pas la même taille :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
           // 7 éléments :
           String[] colon = {"code","libelle","taille","type","fourni","policier","quantite"};
           // 6 éléments
           String[] inf = {
               txtcode.getText(),
                txtlib.getText(),
                taille.getText(),
                combtype.getSelectedItem().toString(),
                combfour.getSelectedItem().toString(),
 
                quantite.getText()
            };

Pour éviter ce genre de désagrément à l'avenir, tu devrais vérifier la taille des tableaux avant d'effectuer le traitement :

Code :

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

1
2
3
4
5
public String queryUpdate(String nomTable, String[] nomColonne, String[] contenuTableau, String état) {
    if (nomColonne.length != contenuTableau.length) {
        throw new IllegalArgumentException("nomColonne.length != contenuTableau.length");
    }
    ...

Maintenant quelques remarques sur ton code :

Tu te contentes d'afficher un message sur la console en cas d'exception.
C'est pas top surtout sur une interface graphique car l'information ne remontera pas à l'utilisateur...

C'est quoi tout ces new connection() ???

Code :

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

1
2
3
4
5
6
           db = new MaBase(
                new connection().HOST_DB, 
                new connection().USERNAME_DB, 
                new connection().PASSWORD_DB, 
                new connection().IPHOST, 
                new connection().PORT);

Tu gères très mal tes ressources en les ouvrant n'importe quand et en ne les fermant jamais.
Par exemple lorsque tu appelles queryUpdate() tu ouvres deux connexions à la BD que tu ne libère jamais.
=> C'est une fuite de mémoire !

Il est préférable d'utiliser le try-with-ressources de Java 7 pour libérer proprement tes ressources (Connection, Statement, ResultSet).
Ca t'obligera à bien voir le scope d'utilisation de chaque variable.
Par exemple tes variables Connection et Statement sont des variables locales : cela n'a rien à faire en tant qu'attribut de classe !
Ta méthode queryUpdate() est sujette à une faille de sécurité via l'injection SQL.
=> Il faut éviter de concaténer des chaines dans une requête SQL, et préférer l'utilisation d'un PreparedStatement.
La concaténation avec + sur des String pour générer ton SQL c'est pas terrible non plus.
Ca passe car ton SQL est relativement petit mais si la chaine devient grande çà peut être très mauvais coté perf.
=> Préférer un StringBuilder pour concaténer des chaines...
Enfin les accès à la DB dans l'interface UI c'est pas top et cela peut te geler toute ton interface.
=> Préférer des SwingWorker pour exécuter cela en tâche de fond.

a++

**joel.drigo** · 26/08/2017, 16h05

Envoyé par gani_95

J'ai pourtant mit le nomColonne moin 1 ...

Oui, et en complément de la réponse de @adiGuba (+1) :

Tu pourrais faire plus simplement (et ça évite un calcul (n-1) à chaque itération)
Code : Sélectionner tout - Visualiser dans une fenêtre à part
```
for(int i=0; i<n; i++) {
```
au lieu de :
Code : Sélectionner tout - Visualiser dans une fenêtre à part
```
1
2
int i;
for(i=0; i<=n-1; i++) {
```

Au lieu de traiter le problème des tableaux de tailles différentes par une exception (en remplaçant l'ArrayIndexOutOfBoundException par une IllegalArgumentException), tu peux aussi faire :

Code :

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

1
2
int n = Math.min(nomColonne.length, contenuTableau.length); // on prend au maximum le nombre de cellules du plus petit des deux tableau
for (int i = 0; i < n; i++) {

Cela ne causera plus d'exception sur le parcours de tabeau. Avec le traitement de l'update, au pire, si l'un des tableaux est de taulle 0, tu auras une SQLException, donc tu peux traiter ce cas à part (en n'exécutant pas la requête par exemple).

Exception in thread "AWT-EventQueue-0" java.lang.ArrayIndexOutOfBoundsException:

AWT/Swing Java

Vue hybride

Discussions similaires

Partager

Partager