NullPointerException

NullPointerException - Java - Programmation

Marsh Posté le 02-11-2013 à 11:57:17    

Bonjour à tous !
 
 
Et bienvenue dans l'enfer de ma nouvelle meilleure amie, la NullPointerException.
 
 
Pour information, j'ai bien lu le topic du débutant, mais je n'ai toujours pas trouvé de solution pour me débarrasser de mon encombrante nouvelle amie. Je viens donc chercher de l'aide ici.
 
 
Il s'agit donc d'un programme de Java. En gros, j'ai créé plusieurs classes me permettant de créer des graphes. J'ai donc à ma disposition une classe Graph, une classe Edge (arête) et une classe Vertex (sommet).  
 
 
Quelques morceaux de code :
 

Code :
  1. public class Graph {
  3. private TreeSet<Edge> edges;
  4. private TreeSet<Vertex> vertices;
  7. /** Constructeur de la classe Graph
  8.  *  
  9.  */
  10. public Graph() {
  12.  TreeSet<Edge> edges = new TreeSet<Edge>();
  13.  TreeSet<Vertex> vertices = new TreeSet<Vertex>();
  14. }


 
 

Code :
  1. public class Vertex implements Comparator, Comparable {
  3. private int num ;
  6. /** Constructeur de la classe Sommet
  7.  *  
  8.  * @param a qui représente le numéro du sommet
  9.  */
  10. public Vertex(int a){
  12.  this.num = a;
  13. }


 

Code :
  1. public class Edge<E,F> implements Couples<E, F>, Comparator, Comparable {
  3. private Vertex pere;
  4. private Vertex fils;
  7. /** Constructeur de la classe Edge
  8.  *  
  9.  * @param a qui représente le sommet père
  10.  * @param b qui représente le sommet fils
  11.  */
  12. public Edge(Vertex a, Vertex b) {
  14.  this.pere = a;
  15.  this.fils = b;
  16. }


 
Edit : Je précise que l'interface couple est une interface qui m'a été imposée dans l'exercice et qui ne contient qu'une fonction "afficher()".
 
 
Dans mon application, j'essaie de créer un graphe, deux sommets, une arête. Jusque là, tout fonctionne correctement (toString() à l'appui).
 
L'exception survient lorsque j'essaie d'ajouter une arête ou un sommet au graphe.  
 
 
Code de l'application :
 

Code :
  1. public static void main(String[] args) throws UnknownVertexException {
  4.  Graph g = new Graph();
  5.  g.toString(); //fonctionne
  7.         Vertex a = new Vertex(1);
  8.  Vertex b = new Vertex(2);
  9.  a.toString(); //fonctionne
  10.  b.toString(); //fonctionne
  12.  Edge e = new Edge(a,b);
  13.  e.toString();//fonctionne
  15.         g.addEdge(e);//NullPointerException ici


 
 
J'ai également essayé de simplement ajouter un sommet au graphe, mais l'exception est également levée.  
 
 
Code de l'ajout d'une arête :
 

Code :
  1. public void addEdge(Edge a) {
  4.  this.edges.add(a);
  5.  vertices.add(a.getFils());
  6.  vertices.add(a.getPere());
  7. }


 
 
J'ai également essayé de commenter les vertices.add, mais j'ai toujours ma NullPointerException. J'ai pensé que le problème pouvait venir de mes TreeSet, qui sont les seuls éléments null de tout mon code, mais je ne vois pas en quoi l'ajout d'un nouvel élément poserait problème. J'ai testé en créant un TreeSet<Vertex> directement dans mon application, puis en ajoutant un sommet, et cela fonctionne parfaitement.
 
 
J'attends vos idées  :whistle:


Message édité par arhyaa le 02-11-2013 à 12:03:30
Reply

Marsh Posté le 02-11-2013 à 11:57:17   

Reply

Marsh Posté le 02-11-2013 à 22:39:54    

Code :
  1. public class Graph {
  3. private TreeSet<Edge> edges;
  4. private TreeSet<Vertex> vertices;
  7. /** Constructeur de la classe Graph
  8.  *  
  9.  */
  10. public Graph() {
  12.  TreeSet<Edge> edges = new TreeSet<Edge>();
  13.  TreeSet<Vertex> vertices = new TreeSet<Vertex>();
  14. }

Ton probleme vient de la. Dans ton constructeur, tu n'initialises pas tes variables privees, mais plutot deux variables locales et temporaires, qui disparaissent lors de la sortie du constructeur. Du coup tes variables "edges" et "vertices" de ton objet sont nulles et ca coince quand tu essaies d'y ajouter quelque chose. Pour corriger, il suffit d'enlever la re-declaration du constructeur:

Code :
  1. public class Graph {
  3. private TreeSet<Edge> edges;
  4. private TreeSet<Vertex> vertices;
  7. /** Constructeur de la classe Graph
  8.  *  
  9.  */
  10. public Graph() {
  12.  this.edges = new TreeSet<Edge>();
  13.  this.vertices = new TreeSet<Vertex>();
  14. }


Un petit truc: dans le futur, essaie de systematiquement "qualifier" tes variables quand tu t'en sers (je veux dire, utilise "this.tavariable" ). Ici ton erreur t'aurais alors saute aux yeux je pense.


---------------
C'était vraiment très intéressant.
Reply

Marsh Posté le 03-11-2013 à 03:30:33    

Rolala, je n'avais pas du tout pensé à ça ! Merci beaucoup de ton aide !

Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed