Erreur execution Java : java.lang.NullPointerException

**milym** · 01/12/2008, 21h31

bonjour,
comme l'indique le titre, j'ai une erreur d'execution : java.lang.NullPointerException

Voiçi les morceaux de codes d'où viendrait le pb :

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
public class TesteurSansTransformation {
 
    public IFigureFactory factory;
 
    public TesteurSansTransformation(IFigureFactory factory) {
        this.factory = factory;
    }
 
    public IFigure pleinDeIPoints(int nbr) {
        IFigureComposee res = this.factory.nouvelleFigureComposeeVide();
        for (int i = 0; i < nbr; i++) {
            IPoint p = this.factory.nouveauPoint(Math.random(), Math.random());
            res.ajouteSousFigure(p);
        }
        return res;
 
    }
 
    public IFigure pleinDeIPoints(int nbr) {
        IFigureComposee res = this.factory.nouvelleFigureComposeeVide();
        for (int i = 0; i < nbr; i++) {
            IPoint p = this.factory.nouveauPoint(Math.random(), Math.random());
            res.ajouteSousFigure(p);
        }
        return res;
 
    }
}

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
import java.util.ArrayList;
 
 
public class FigureComposee implements IFigureComposee {
 
    public ArrayList <IFigure> figs;
 
    public FigureComposee() {
        this.figs = new ArrayList <IFigure>();
    }
 
    public void ajouteSousFigure(IFigure sousFig) {
        this.figs.add(sousFig);
    }
// ...etc...}

java renvoie cette erreur :
init:
deps-jar:
compile-single:
run-single:
Exception in thread "main" java.lang.NullPointerException
at fr.****.geo2d.FigureComposee.ajouteSousFigure(FigureComposee.java:23)
at fr.****.tps.m2.dessin2DV1.test.TesteurSansTransformation.pleinDeIPoints(TesteurSansTransformation.java:34)
at fr.****.tps.m2.dessin2DV1.test.TesteurSansTransformation.testPrincipal(TesteurSansTransformation.java:105)
at fr.****.geo2d.TestFdB.main(TestFdB.java:19)

Donc l'erreur viendrait de ajouteSousFigure()
où est le pb ? Comment le résoudre ? je ne vous pas d'où vient cette erreur.
merci de votre aide

**maisonneuve01** · 01/12/2008, 22h01

A première vue, l'erreur ne vient pas de la classe FigureComposee car la list est bien initialisée dans le constructeur.

L'erreur viendrait plutôt de la classe IFigureFactory.

Car c'est elle qui créer

Code :

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

IFigureComposee res = this.factory.nouvelleFigureComposeeVide();

et

Code :

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

IPoint p = this.factory.nouveauPoint(Math.random(), Math.random());

Pour avoir un NullPointerException il faut que res ou p soit à null.

**milym** · 01/12/2008, 22h21

Je te remercie de ton aide.

Voiçi le morceau du FigureFactory qui poserait donc pb :

Code :

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

1
2
3
4
5
6
7
8
9
public class FigureFactory implements IFigureFactory {
 
    public IPoint nouveauPoint(double x, double y) {
        return new Point(x, y);
    }
    public IFigureComposee nouvelleFigureComposeeVide() { 
        return new FigureComposee();
    }
....etc...}

Tu trouves une erreur ou tout est ok ?
Je comprend vraiment rien de ces erreurs null et tout, ça me dépasse pour le moment. En tout cas merci de ton aide.

**g0up1l** · 02/12/2008, 08h51

si l'erreur est à cette ligne, je ne vois que la variable 'figs' qui pourrait être null.
Essaye d'ajouter un texte, genre :

Code :

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

if( this.figs != null ){...}

**CheryBen** · 02/12/2008, 09h50

Bonjour, une NullPointerException n'est pas une erreur de compilation mais une erreur d'exécution.

Es-tu sûr que la trace que tu donnes dans le 1er message est complète? car si on s'y réfère, l'erreur survient à la ligne 23 de FigureComposee.java. Or sur cette ligne, il n'y a que figs qui pourrait être null, sauf que cet attribut est initialisé dans ton constructeur.
Il y donc quelque chose qui cloche. Pourrais-tu poster la classe FigureComposee en entier?

**milym** · 02/12/2008, 13h26

oui la trace du 1er message est complete
voiçi la classe FigureComposee :

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
package fr.****.geo2d;
 
import fr.****.tps.m2.dessin2DV1.api.IFigure;
import fr.****.tps.m2.dessin2DV1.api.IFigureComposee;
import java.awt.Graphics2D;
import java.awt.geom.AffineTransform;
import java.util.ArrayList;
 
/**
 *
 * @author ilym
 */
public class FigureComposee implements IFigureComposee {
 
    public ArrayList <IFigure> figs;
 
    public FigureComposee() {
        this.figs = new ArrayList <IFigure>();
    }
 
    public void ajouteSousFigure(IFigure sousFig) {
        this.figs.add(sousFig);
    }
 
    public void supprimeSousFigure(IFigure sousFig) {
        this.figs.remove(sousFig);
    }
 
    public double maxX() {
        double max = 0;
        if (this.figs.size() > 0) max = this.figs.get(0).maxX();
        for(int i = 0; i < this.figs.size(); i++) max = Math.max(this.figs.get(i).maxX(), max) ;
        return max;
    }
 
    public double minX() {
        double min = 0;
        if (this.figs.size() > 0) min = this.figs.get(0).minX();
        for(int i = 0; i < this.figs.size(); i++) min = Math.min(this.figs.get(i).minX(), min) ;
        return min;
    }
 
    public double maxY() {
        double max = 0;
        if (this.figs.size() > 0) max = this.figs.get(0).maxY();
        for(int i = 0; i < this.figs.size(); i++) max = Math.max(this.figs.get(i).maxY(), max) ;
        return max;
    }
 
    public double minY() {
        double min = 0;
        if (this.figs.size() > 0) min = this.figs.get(0).minY();
        for(int i = 0; i < this.figs.size(); i++) min = Math.min(this.figs.get(i).minY(), min) ;
        return min;
    }
 
    public void dessine(Graphics2D g, AffineTransform trans) {
        for(int i = 0; i < this.figs.size(); i++) this.figs.get(i).dessine(g, trans); 
    }
 
}

**g0up1l** · 02/12/2008, 14h13

As-tu essayé de tester ta variable 'figs' ?

**milym** · 02/12/2008, 18h17

comment ça tester ?

**g0up1l** · 02/12/2008, 20h57

Comme ça :

Code :

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

if( this.figs != null ){...}

tu testes et tu mets le résultat sur la console ou dans une boîte de dialogue, ça t'aidera à cerner le problème !

**milym** · 02/12/2008, 21h16

il faudrait le mettre où ?

**g0up1l** · 03/12/2008, 08h37

Par exemple ici :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 public void ajouteSousFigure(IFigure sousFig) 
{
 if( this.figs != null )
 { 
  this.figs.add(sousFig)
 }
else
{ 
  System.out.println("valeur nulle");
}
}