Discussion :

[Deallyra]

Bonsoir,

Une erreur toute bête et pourtant...

J'ai deux classes que l'on retrouve dans des ArrayList.

La classe Avion et la classe Vol.

A savoir qu'un avion peut réaliser plusieurs vols, en ce sens, j'ai une collection de Vol dans avion.

Aussi, je souhaite pouvoir ajouter des vols à cette collection... Mais mon raisonnement me conduit à un NullPointerException...

Voici des extraits de code :

Code :

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

1
2
3
4
5
6
7
 
        /*0*/desAvions.add(new Avion("Airbus A340-300", 295, 129300, 55600, 100, 13000, 140640,140640));
        /*0*/desVols.add(new VolMilitaire(100,desAvions.get(0)));
        Vol unVol = desVols.get(0);
        Avion unAvion = desAvions.get(0);
 
        unAvion.addVol(unVol);

Dans la classe Avion

Code :

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

1
2
3
4
5
6
7
8
 
 
    private ArrayList<Vol> mesVols;
 
    public void addVol(Vol unVol)
    {
        mesVols.add(unVol);
    };

Pourquoi ce dump d'erreur ?

Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
at trafficaerien.Avion.addVol(Avion.java:141)
at trafficaerien.i_trafficAerien.initialiser(i_trafficAerien.java:165)
at trafficaerien.i_trafficAerien.initComponents(i_trafficAerien.java:36)
at trafficaerien.i_trafficAerien.<init>(i_trafficAerien.java:29)
at trafficaerien.i_trafficAerien$2.run(i_trafficAerien.java:224)
at java.awt.event.InvocationEvent.dispatch(InvocationEvent.java:209)
at java.awt.EventQueue.dispatchEvent(EventQueue.java:597)
at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:269)
at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:184)
at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:174)
at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:169)
at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:161)
at java.awt.EventDispatchThread.run(EventDispatchThread.java:122)

[Ricky81]

As tu instancié tes listes ?

Avec l'ensemble du code on y verrait plus clair

[Deallyra]

Les instancier ?

Hum... A quel endroit ?

Dans le code du main oui mais pas dans les classes...

Voici une archive

et s'il faut un modèle conceptuel pour la représentation des données j'en ai un à dispo

(merci)

[Ricky81]

Dans la classe Avion, tu n'as ni setter te permettant d'attribuer une valeur à "mesVols", ni initialisation.

Tu peux, au choix, compléter ton code ainsi :

Code :

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

1
2
 
private ArrayList<Vol> mesVols = new ArrayList();

ou

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
    /**
     * Constructeur de la classe
     * @param String marque Marque de l'avion
     * @param int capaciteDePersonnes Nombre de places passagers
     * @param int consommationEnVol Carburant consommé lors du trajet de l'avion
     * @param int altitudeMaximale Altitude maximale de couloir aérien pour l'avion
     * @param int volumeDuReservoir Volume de carburant du réservoir.
     */
  	public Avion(String marque, int capaciteDePersonnes, int masseAVide,
            int capaciteDeChargement, int consommationEnVol,
            int altitudeMaximale, int capaciteReservoir, int volumeDuReservoir)
	{
        this.identifiant = ++compteurIdentifiant;
        this.marque = marque;
        this.capaciteDePersonnes = capaciteDePersonnes;
        this.masseAVide = masseAVide;
        this.capaciteDeChargement = capaciteDeChargement;
        this.consommationEnVol = consommationEnVol;
        this.altitudeMaximale = altitudeMaximale;
        this.capaciteReservoir = capaciteReservoir;
        this.volumeDuReservoir = volumeDuReservoir;
        this.mesVols = new ArrayList();
	}

au passage, utilises plutôt l'interface List pour type tes listes

ex.

Code :

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

1
2
3
4
5
6
7
 
    private List<Vol> mesVols = new ArrayList();
 
    public List<Vol> getMesVols()
    {
        return this.mesVols;
    };

[Deallyra]

Si ce n'est pas abuser, que m'apporte de plus l'interface List par rapport à la déclaration en ArrayList ?

(je fais les modifications >.< )

[Ricky81]

Si ce n'est pas abuser, que m'apporte de plus l'interface List par rapport à la déclaration en ArrayList ?

(je fais les modifications >.< )

Ca te permet de manipuler la liste en dehors de ta classe sans être lié à une quelconque implémentation, et donc de changer sans impact l'implémentation le jours ou tu te dis qu'une liste chaînée fais mieux l'affaire. Bonne lecture