Discussion :

[Beginner.]

Bonsoir,

Voilà j'aimerais comprendre où est l'erreur dans ma tentative d'utilisation d'un tableau d'objets :

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
 
 
public class TestObject {
 
    /**
     * @param args
     */
    public static void main(String[] args) {
        // TODO Auto-generated method stub
 
        int nbrDeValeurs = 5 ;
 
        SommeEtCoordonnees [] SommeEtCoordonneesA ;
        SommeEtCoordonneesA  = new SommeEtCoordonnees[nbrDeValeurs]; 
 
        for (int i = 0 ; i < nbrDeValeurs ; i++)                       
               SommeEtCoordonneesA[i].initialise(2, 3, 3, 5, 5);
 
       // SommeEtCoordonneesA[1].affiche(); 
 
    }
 
}
 
 
class SommeEtCoordonnees 
{
    private int Somme ;                    // Somme et coordonnées des deux termes additionnés
    private int indiceColTerme1 ;      
    private int indiceColTerme2 ;
    private int indiceLigneTerme1 ; 
    private int indiceLigneTerme2 ;
 
 
    public void initialise (int Somme, int indiceLigneTerme1, int indiceColTerme1, int indiceLigneTerme2,  int indiceColTerm2)
      { 
 
        this.Somme = Somme ;
 
        this.indiceColTerme1 = indiceColTerme1 ;      
        this.indiceColTerme2 = indiceColTerm2 ;
        this.indiceLigneTerme1 = indiceLigneTerme1 ; 
        this.indiceLigneTerme2 = indiceLigneTerme2 ;
      }
 
    public void affiche ()
      { System.out.println ("L" + indiceLigneTerme1 + "C" + indiceColTerme1 + " + L" + indiceLigneTerme2 + "C" + indiceColTerme2 + " ---> Somme = " + Somme) ;
      }
 
}

Une erreur est signalée à la ligne 17 : SommeEtCoordonneesA[i].initialise(2, 3, 3, 5, 5);

Exception in thread "main" java.lang.NullPointerException
at TestObject.main(TestObject.java:16)

Sinon à part ça faut-il obligatoirement remplir le tableau avant son utilisation ?

Merci.

[tchize_]

Avec ce code:

Code :

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

SommeEtCoordonneesA  = new SommeEtCoordonnees[nbrDeValeurs];

tu crée un tableau de 5 entrées, mais il n'y a rien dedans. Que des null. Il faut donc y stocker des valeur avant d'essayer d'appeler des méthodes dessus:

Code :

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

1
2
3
4
        for (int i = 0 ; i < nbrDeValeurs ; i++) {  
               SommeEtCoordonneesA[i]=new SommeEtCoordonnees();
               SommeEtCoordonneesA[i].initialise(2, 3, 3, 5, 5);
        }

Enfin, ta méthode initialise ferait mieux d'être transformée en constructeur, puisque finalement c'est son but, ce qui donnerais ceci

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
public class TestObject {
 
    /**
     * @param args
     */
    public static void main(String[] args) {
        // TODO Auto-generated method stub
 
        int nbrDeValeurs = 5 ;
 
        SommeEtCoordonnees [] SommeEtCoordonneesA ;
        SommeEtCoordonneesA  = new SommeEtCoordonnees[nbrDeValeurs]; 
 
        for (int i = 0 ; i < nbrDeValeurs ; i++)                       
               SommeEtCoordonneesA[i] = new SommeEtCoordonnees(2, 3, 3, 5, 5);
 
       // SommeEtCoordonneesA[1].affiche(); 
 
    }
 
}
 
 
class SommeEtCoordonnees 
{
    private int Somme ;                    // Somme et coordonnées des deux termes additionnés
    private int indiceColTerme1 ;      
    private int indiceColTerme2 ;
    private int indiceLigneTerme1 ; 
    private int indiceLigneTerme2 ;
 
 
    public SommeEtCoordonnees(int Somme, int indiceLigneTerme1, int indiceColTerme1, int indiceLigneTerme2,  int indiceColTerm2)
      { 
 
        this.Somme = Somme ;
 
        this.indiceColTerme1 = indiceColTerme1 ;      
        this.indiceColTerme2 = indiceColTerm2 ;
        this.indiceLigneTerme1 = indiceLigneTerme1 ; 
        this.indiceLigneTerme2 = indiceLigneTerme2 ;
      }
 
    public void affiche ()
      { System.out.println ("L" + indiceLigneTerme1 + "C" + indiceColTerme1 + " + L" + indiceLigneTerme2 + "C" + indiceColTerme2 + " ---> Somme = " + Somme) ;
      }
 
}

[Beginner.]

Avec ce code:

Code :

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

SommeEtCoordonneesA  = new SommeEtCoordonnees[nbrDeValeurs];

tu crée un tableau de 5 entrées, mais il n'y a rien dedans. Que des null. Il faut donc y stocker des valeur avant d'essayer d'appeler des méthodes dessus:

Code :

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

1
2
3
4
        for (int i = 0 ; i < nbrDeValeurs ; i++) {  
               SommeEtCoordonneesA[i]=new SommeEtCoordonnees();
               SommeEtCoordonneesA[i].initialise(2, 3, 3, 5, 5);
        }

Enfin, ta méthode initialise ferait mieux d'être transformée en constructeur, puisque finalement c'est son but, ce qui donnerais ceci

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
public class TestObject {
 
    /**
     * @param args
     */
    public static void main(String[] args) {
        // TODO Auto-generated method stub
 
        int nbrDeValeurs = 5 ;
 
        SommeEtCoordonnees [] SommeEtCoordonneesA ;
        SommeEtCoordonneesA  = new SommeEtCoordonnees[nbrDeValeurs]; 
 
        for (int i = 0 ; i < nbrDeValeurs ; i++)                       
               SommeEtCoordonneesA[i] = new SommeEtCoordonnees(2, 3, 3, 5, 5);
 
       // SommeEtCoordonneesA[1].affiche(); 
 
    }
 
}
 
 
class SommeEtCoordonnees 
{
    private int Somme ;                    // Somme et coordonnées des deux termes additionnés
    private int indiceColTerme1 ;      
    private int indiceColTerme2 ;
    private int indiceLigneTerme1 ; 
    private int indiceLigneTerme2 ;
 
 
    public SommeEtCoordonnees(int Somme, int indiceLigneTerme1, int indiceColTerme1, int indiceLigneTerme2,  int indiceColTerm2)
      { 
 
        this.Somme = Somme ;
 
        this.indiceColTerme1 = indiceColTerme1 ;      
        this.indiceColTerme2 = indiceColTerm2 ;
        this.indiceLigneTerme1 = indiceLigneTerme1 ; 
        this.indiceLigneTerme2 = indiceLigneTerme2 ;
      }
 
    public void affiche ()
      { System.out.println ("L" + indiceLigneTerme1 + "C" + indiceColTerme1 + " + L" + indiceLigneTerme2 + "C" + indiceColTerme2 + " ---> Somme = " + Somme) ;
      }
 
}

+1 et problème résolu.

Merci je comprends maintenant qu'il fallait allouer un emplacement pour chaque objet du tableau avec l'opérateur new.

Sinon merci pour le conseil concernant le constructeur mais ici c'était volontaire car je voulais pouvoir affecter les champs de l'objet à tout moment.

EDIT : Je viens de voir que la réponse est présente dans la FAQ ici : Comment instancier un tableau?