Discussion :

[viou25]

Bonjour, je suis étudiante et je dois programmer en java mais il se trouve que je suis face à une erreur que je n'arrive pas à résoudre pourriez-vous m'aider ?
merci d'avance

L'erreur est Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: 2

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
package tp2;
/**
 *
 * @author 4lepoelv
 */
public class Gestion {
    public static void main (String[]args){
 
        int n,i,an, p;
        String is, au, ti, isl;
 
        System.out.println("Nombre de Livre à enregistrer ?");
        n = Lire.i();
        Livre t[] = new Livre [n];
 
        // création du tableau de livre
        for(i=0;i<n;i++){
        System.out.println("Enter l'ISBN du livre :\t" + "n°" +i);
        is = Lire.S();
        System.out.println("Quel est l'auteur de ce livre ?\t" + "n°"+i);
        au = Lire.S();
        System.out.println("Quel est le titre de ce livre ? \t" +"n°" +i);
        ti = Lire.S();
        System.out.println("Quelle est l'année d'édition de ce livre ?\t" +"n°"+ i);
        an = Lire.i();
 
 
        t[i] = new Livre(is,au,ti,an);
 
        }     
 
        // ajout d'un nouveau livre
        int m,ann;
        String isn,aun,tin;
        m = n+1;
        Livre g[] = new Livre [m];
        for(i=0;i<m-1;i++){
            g[i] = t[i];
        }
        System.out.println("Enter l'ISBN du nouveau livre :");
        isn = Lire.S();
        System.out.println("Quel est l'auteur de ce nouveau livre ?");
        aun = Lire.S();
        System.out.println("Quel est le titre de ce nouveau livre ? ");
        tin = Lire.S();
        System.out.println("Quelle est l'année d'édition de ce nouveau livre ?");
        ann = Lire.i();
 
        g[n+1] = new Livre(isn,aun,tin,ann);

[guigui5931]

Cela signifie que tu essaye d'accéder à une case d'un tableau qui n'existe pas. Je pense que c'est sur cette ligne

Code :

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

1
2
 
g[n+1] = new Livre(isn,aun,tin,ann);

puisque tu créé le tableau comme ça

Code :

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

1
2
3
 
m = n+1;
Livre g[] = new Livre [m];

donc g va de 0 à n il n'y a pas de case n+1.

[viou25]

j'avais aussi repéré ce problème. Mais je ne sais pas comment crée ce nouveau tableau g en lui attribuant une case de plus que l'autre pour ensuite remplir cette case. Si je dispose de ce tableau de taille m si je veux accèder à la dernière case je dois mettre m-1 ?
merci pour ta réponse.

[viou25]

Le problème était bien celui la je vous remerci. J'ai continué mon programme mais j'ai de nouveau une erreur. Pouvez-vous m'aider ?
Je n'arrive pas bien à saisir comment fonctionne un compareto.

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
// modification des données d'un livre
        System.out.println("Enter l'ISBN du livre à* modifier :");
        String ism;
        ism = Lire.S();
        int c,p,rep;
        p=0;
        c=1;
        while(c!=0){
            c = g[p].ISBN.compareTo(ism);
            p = p+1;     
        }
        System.out.println("Changer d'auteur : taper 1"+"\n"+
                "Changer de titre : taper 2"+ "\n"+
                "Changer année d'édition : taper 3");
        rep = Lire.i();
        switch (rep){
            case 1 :
            System.out.println("Donner le nouvel auteur : ");
            au = Lire.S();
            g[p-1].auteur = au ;
            break;
            case 2:
            System.out.println("Donner le nouveau titre : ");
            ti = Lire.S();
            g[p-1].titre = ti ; 
            break;
            case 3 :
            System.out.println("Donner la nouvelle année d'édition ");
            an = Lire.i();
            g[p-1].anedition = an ;    
            break;
        }

[maisonneuve01]

la fonction compareTo est utilisée le plus souvent pour trier les objets. Dans ton cas la fonction equals est préférable.

Code :

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

1
2
3
4
5
6
p=0;
c=1;
while(c!=0){
c = g[p].ISBN.compareTo(ism);
p = p+1;
}

Attention à cette boucle while, car si aucun ISBN correspond à l'ism saisie, tu vas sortir du tableau (ArrayIndexOutOfBoundsException).

Si tu code en java 1.5 ou >, il semblerais plus facile de faire :

Code :

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

1
2
3
4
5
6
7
 
for(Livre livre : g) {
    if(l.ISBN.equals(ism) {
        ...
        break;
    }
}

Si je peux me permettre, le code est un peu brouillon, il ne faut pas être radin sur le nommage des variables. Exemple Lire.S() <=> Lire.String() ...
g ne renseigne en rien que c'est un tableau de Livre ...

Je te conseille de lire attentivement le tutorial de Sun pour découvrir tous les atouts de Java. Plutôt que de redimensionner un tableau, il est préférable d'utiliser une collection de type ArrayList ...
Une fois le tutorial compris tu pourras attaquer la javadoc des API.

Bonne chance pour la suite.