Discussion :

[ptitdje]

Bonjours à tous,

Voilà mon souci ,

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
  public void dessinerLiaison(){
 
		  Iterator<Station> it = stations.iterator();
 
		  while(it.hasNext()){
			  Station temp=it.next();
			  int [] coorddep = r.find(temp.getId()).getCoord();
			  System.out.println(coorddep[0]+ " " + coorddep[1]);
 
			  Iterator<Station> it2 = stations2.iterator();
			  while(it2.hasNext()){
			  Station temp2=it2.next();
 
			  int[] coordarr = r.find(temp.ligneCommune(temp2)).getCoord();
 
 
		  System.out.println("Depart"+coorddep[0] +" "+coorddep[1]+" Arrivée"+coordarr[0]+" "+coordarr[1]);
		  ligne(null, coorddep[0],coorddep[1],coordarr[0],coordarr[1]);
			  }
		 	}
 	}

Le nulExceptionPointer se situe au niveau de la ligne suivante :

Code :

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

r.find(temp.ligneCommune(temp2)).getCoord();

J'ai pourtant utiliser 2 listes différentes ( qui contienne exactement la meme chose ) car j'ai cru que c'etait le fait de parcourir la meme liste avec 2 iterator mais non ...

Que faire ?

Merci d'avance !!!

[thelvin]

Que faire ?

On va pas deviner... Il faut le code !

Bon, sinon, c'est bien gentil de trouver la ligne qui fait le NullPointerException, mais est-ce que ce ne serait pas plus malin de juste regarder quel pointer est null, provoquant ainsi cette NullPointerException ?

[Mopmop]

Je suis d'accord. Ta ligne donne trois possibilités :

Code :

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

1
2
 
r.find(temp.ligneCommune(temp2)).getCoord();

Soit r est null, soit temp est null soit r.find(temp.ligneCommune(temp2))... soit plusieurs ^^.
Trouve déjà lequel est null et ensuite voit ce que tu peux y faire

[ptitdje]

En fait ce que j'aurai aimé savoir c'est surtout si ma syntaxe au niveau des 2 iterator était correct =) Car je vois pas pourquoi la ligne indiquée par le compilateur serrait mauvaise ... =/

[Mako 5013]

Bonjour,

Pour l'exception, je rejoins les avis précédent : en l'état, difficile de savoir ce qui ne va pas. Il faudrait que tu débugues un peu de ton côté (pour savoir ce qui est vraiment null).

Par contre, qu'est-ce que tu cherches à réaliser exactement ? Parce que là, en parcourant deux fois toutes tes stations, tu vas "dessiner des liaisons" (les relier je présume) toutes entre elles.

Par exemple, si tu as S1, S2 et S3, tu vas créer les liaisons suivantes :
(première itération de la première boucle)
- S1/S1 ;
- S1/S2 ;
- S1/S3 ;
(deuxième itération de la première boucle)
- S2/S1 ;
- S2/S2 ;
- S2/S3 ;
(troisième itération de la première boucle)
- S3/S1 ;
- S3/S2 ;
- S3/S3 ;

Non seulement je ne suis pas sûr que cela soit ce que tu recherches (chaque station reliée à toutes les autres), mais en plus, comme on peut le constater sur l'exemple ci-dessus, il y a des doublons.

Mako.

[ptitdje]

Bonjour bonjour !!

Alors Mako, merci pour tes conseils, j'ai en effet bien fait une belle bétise !!!
Du coup j'ai refait sa proprement et je me retrouve à la fin de ma fonction avec les coordonnées de la Station de départ et celle d'arrivée.
Seulement, comme je suis dans une fonction, j'ai tenter ( on sait jamais ) un DrawLine(dep[0],dep[1],arr[0],arr[1]), mais sa fonctionne pas du tout =/

Alors pour déssiner les stations j'ai fait une class spécialement pour( avec un implement MouseInputListener ) . Mais pour les lignes j'aurai aimé éviter de refaire une class juste pour les dessiner.

Je fait tout dans le même JPanel ( pan ) .

Une idée ?

merci d'avance !!

ps : j'ai aussi essayer de mettre en paramètre de ma fonction

Code :

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

public void dessinerLiaison(Graphics g){

puis appelé g.DrawLine(dep[0],dep[1],arr[0],arr[1])
Idem , fonctionne pas =/