java lang NullPointerException

Version imprimable

Bonjour,

Lorsque je run mon programme java, j'ai cette exception qui m'embête :

Code:

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
 
run:
Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
        at cleanfiles.RetagMp3.ReadMp3(RetagMp3.java:209)
        at cleanfiles.RetagMp3.ReadInfoAlbum(RetagMp3.java:98)
        at cleanfiles.RetagMp3.RetagAlbum(RetagMp3.java:80)
        at cleanfiles.RetagMp3.RetagAllAlbums(RetagMp3.java:65)
        at cleanfiles.RenameMp3.jbtRetagMp3MouseClicked(RenameMp3.java:163)
        at cleanfiles.RenameMp3.access$100(RenameMp3.java:18)
        at cleanfiles.RenameMp3$2.mouseClicked(RenameMp3.java:82)
        at java.awt.AWTEventMulticaster.mouseClicked(AWTEventMulticaster.java:253)
        at java.awt.Component.processMouseEvent(Component.java:6266)
        at javax.swing.JComponent.processMouseEvent(JComponent.java:3267)
        at java.awt.Component.processEvent(Component.java:6028)
        at java.awt.Container.processEvent(Container.java:2041)
        at java.awt.Component.dispatchEventImpl(Component.java:4630)
        at java.awt.Container.dispatchEventImpl(Container.java:2099)
        at java.awt.Component.dispatchEvent(Component.java:4460)
        at java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4574)
        at java.awt.LightweightDispatcher.processMouseEvent(Container.java:4247)
        at java.awt.LightweightDispatcher.dispatchEvent(Container.java:4168)
        at java.awt.Container.dispatchEventImpl(Container.java:2085)
        at java.awt.Window.dispatchEventImpl(Window.java:2475)
        at java.awt.Component.dispatchEvent(Component.java:4460)
        at java.awt.EventQueue.dispatchEvent(EventQueue.java:599)
        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)

Je ne sais pas du tout par ou commencer tellement il y a de lignes...

Si quelque peut éclairé ma lanterne ou directement peut me dire de quoi vient le problème...

Je vous remercie d'avance!!!

++

03/11/2009, 16h30
Jidefix

Le problème est arrivé ligne 209 du fichier RetagMP3 dans la méthode ReadeMP3
C'est une NullPointerException, c'est à dire que tu appelles une méthode sur un objet null.

Sans le code source de cette fonction, c'est tout ce qu'on peut te dire :)

Salut Jidefix !

Il me semble que tu participes souvent à mes problèmes :)
Je t'en remercie...
Pour ton information, je te met le code de ma fonction ReadMp3 :
(La ligne 209 correspond à la ligne Album.Track[i].NoTrack = Champs[0];)

Code:

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
    public static tAlbum ReadMp3 (tInfoAlbums InfoAlbums, tAlbum Album)
    {
        File PathDirMp3 = new File(InfoAlbums.PathAlbum+"\\"+InfoAlbums.CurrentAlbum);
        //Liste les fichiers de l'album
        File [] TrackList = PathDirMp3.listFiles();
 
        //Champs
        String [] Champs;
 
        //Pour chaque Mp3
        for(int i=0 ; i<TrackList.length ; i++)
        {
            if(IsMp3(TrackList[i].getName()))
            {
                //Sépare les champs des mp3 (NoTrack, Artiste, Album)
                Champs = TrackList[i].getName().split(" - ");
 
                //Récupère le titre et le No de la piste
                Album.Track[i].NoTrack = Champs[0];
                Album.Track[i].Title = Champs[2];
            }
        }
 
        return Album;
    }

Je te donne également mes "types" :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
    //Type d'information sur les albums (Genre, Chemin des albums, et type)
    static class tInfoAlbums{
        public String Genre;
        public String PathAlbum;
        public boolean TypeAlbum;
        public String CurrentAlbum;
    }
 
    //Type d'information sur un MP3
    static class tTrack {
        String NoTrack;
        String Title;
    }
 
    //Type d'information sur l'album (Artiste, Album, Année...)
    static class tAlbum {
        int Year;
        String Artist;
        String Album;
        tTrack [] Track;
        File Picture;
    }

Voilà, voilà !

Merci

03/11/2009, 16h59
Invité

dans ton "for" : "PathDirMp3.length()" , ça ne va pas je pense
03/11/2009, 17h04
Jidefix
Ah oui c'est vrai bah rebonjour! :D

Allez je propose quatre solutions:

A) Album est null, il faut regarder dans la méthode appelante pour voir ce qui est passé en paramètre
B) Album.Track est null. Il faut initialiser le tableau avant d'accéder à l'une de ses cases
C) Album.Track[i] est null. Il faut initialiser chaque case avant de la modifier.
D) La réponse D.

J'ajouterai un truc:
Code:

1 2 3 4 Album.Track[i].NoTrack = Champs[0];// Inutile car écrasé par la ligne juste en dessous Album.Track[i].NoTrack = Champs[1]; Album.Track[i].Title = Champs[2];
03/11/2009, 17h08
Invité

nan mais il boucle sur la taille d'un "File" : ça va pas !
il faut boucler sur la taille du tableau que tu récupères !

PS : revoir aussi les conventions de nommage
03/11/2009, 17h17
saket2114

Oui merci Pierre, j'ai modifié cette boucle dans mon code et dans mon second post (je sais pas ce que j'ai fais :p). Et merci pour les conventions de nommage :( si tu as un site pour cela ca m'aiderait vraiment...

Mais l'erreur persiste!

Je suis donc d'accord, après avoir vérifié, mon tableau n'est pas initialisé.

Mais heu... Je comptais l'initialisé quand je mettais des valeurs à l'intérieur...

Comment puis-je faire dans cette méthode pour l'initialisé avec le nombre de fichier trouvés ?

Merci de votre aide mes amis!
03/11/2009, 17h21
Jidefix

Code:

Album.Track = new tTrack[TrackList.length];

A placer juste avant ta boucle for.

De plus au début de chaque itération de la boucle, tu dois aussi initialiser chaque case avec

Code:

Album.Track[i] = new tTrack();

Pour les conventions, en bref:
- les variables et noms de méthodes commencent par une minuscule
- les classes par une majuscules (un peu l'inverse de ce que tu as fait en fait :D)

Il y en a d'autres mais déjà ces deux là clarifieraient grandement ton code!
03/11/2009, 17h28
saket2114

Je vous remercie, plus d'erreur et je note cela dans ma petite bible :)

Ok pour les conventions, je vais reprendre mon code et remettre tout ca en ordre!

Je pense avoir encore des problèmes avant la fin de mon développement et si pour mes interlocuteurs mon texte peut être lisible plus rapidement je peux bien faire l'effort!

Encore merci!

Et à bientot je suppose...

Bonne soirée !