Erreur NullPointerException incompréhensible

Version imprimable

Bonjour,
Code:

1 2 3 4 5 6 7 File entryFile = new File("C:\\www.xml"); cl1 essai = new cl1(); String logPass =""; String userCherch = "ladjm"; logPass = essai.recherche(userCherch, entryFile); logPass = "lulu"; textLog.setText("Identifiant : "+logPass);
Voici mon code, la fonction recherche ne fonctionne pas et ne renvoi rien.
Dans ce code le JTextField affiche bien "Identifiant : lulu". Pourtant quand je commente

Code:

//logPass = essai.recherche(userCherch, entryFile);

(la ligne qui ne fonctionne pas) le compilateur me dit :

Citation:

Exception in thread "main" java.lang.NullPointerException
at essai.main(essai.java:39)

Si quelqu'un a une idée de la provenance de mon problème...

Merci

24/10/2012, 16h30
Robin56

Et c'est quoi le code de essai.java ? (spécialement la ligne 39)
24/10/2012, 16h34
liouan

Je pense que ce n'est pas logPass qui pose problème mais textLog. (Qu'il soit null ou pas, la chaîne résultante de "Identifiant : "+logPass vaudra au pire "Identifiant : null")

Peux-tu nous montrer où tu l'initialises?

Par intuition, je dirais que c'est ta fonction recherche qui initialise textLog donc en commentant la ligne, il ne serait plus initialisé -> NPE

Citation:

Envoyé par Robin56

Et c'est quoi le code de essai.java ? (spécialement la ligne 39)

La ligne 39 de essai.java est :

Code:

textLog.setText("Identifiant : "+logPass);

@liouan
Code:

1 2 3 4 5 6 7 8 9 private void initialize() { frame = new JFrame(); frame.setBounds(100, 100, 450, 300); frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); textLog = new JTextField(); frame.getContentPane().add(textLog, BorderLayout.NORTH); textLog.setColumns(10); }
Que veux dire NPE ?

Merci pour vos réponse.

24/10/2012, 16h45
Robin56

Citation:

Envoyé par Carlito08

La ligne 39 de essai.java est :

Code:

//logPass = essai.recherche(userCherch, entryFile);

[...]

Que veux dire NPE ?

Donc tu veux dire qu'il te mentionne une NullPointerException sur ta ligne commentée. :aie: Je crois que ton IDE perd les pédales.

NPE = NullPointerException
24/10/2012, 16h46
liouan

(NPE : NullPointerException)
24/10/2012, 16h47
TheZopo

Un pointeur qui pointe sur rien, c'est une erreur d'inattention très fréquente sur l'initialisation.
Tu n'a pas initialiser un tableau par exemple, donc la fonction suivante cherche dans le vide...

Pourrais-tu nous montrer tout ton code source, parce que là on peut pas vraiment t'aider...

@Robin56 C'est moi perd les pédales :?, je viens d'éditer mon post.

@liouan A ok merci.

Voici tout mon code essai.java :

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
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
import java.awt.EventQueue;
import javax.swing.JFrame;
import java.io.File;
import javax.swing.JTextField;
import java.awt.BorderLayout;
 
public class essai {
 
	private JFrame frame;
	private static JTextField textLog;
	private static JTextField textPass;
 
	/**
         * Launch the application.
         */
	public static void main(String[] args) {
		EventQueue.invokeLater(new Runnable() {
			public void run() {
				try {
					essai window = new essai();
					window.frame.setVisible(true);
				} catch (Exception e) {
					e.printStackTrace();
				}
			}
		});
 
File entryFile = new File("C:\\www.xml");
cl1 essai = new cl1();
String logPass ="";
String userCherch = "ladjm";
logPass = essai.recherche(userCherch, entryFile);
logPass = "lulu";
textLog.setText("Identifiant : "+logPass);
	}
 
	/**
         * Create the application.
         */
	public essai() {
		initialize();
	}
	/**
         * Initialize the contents of the frame.
         */
	private void initialize() {
		frame = new JFrame();
		frame.setBounds(100, 100, 450, 300);
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
 
		textLog = new JTextField();
		frame.getContentPane().add(textLog, BorderLayout.NORTH);
		textLog.setColumns(10);
 
		textPass = new JTextField();
		frame.getContentPane().add(textPass, BorderLayout.SOUTH);
		textPass.setColumns(10);
	}
}

24/10/2012, 16h54
TheZopo

Est-ce que tu exécute bien ta fonction initialize();, avant ton :

Code:

textLog.setText("Identifiant : "+logPass);

?
24/10/2012, 16h56
Carlito08
Oui je pense vu qu'elle se trouve dans le constructeur :
Code:

1 2 3 public essai() { initialize(); }
Non ?
24/10/2012, 17h00
liouan

Tu as apparemment une classe qui s'appelle cl1. Peux-tu nous donner son code également ?

Note : Les noms de classes en Java doivent traditionellement commencer par une majuscule.
24/10/2012, 17h01
TheZopo

Effectivement, tu éditais ton message pendant que j'écrivais le mien...
Mais pourquoi avoir fait cela ? Tu a soif de te compliquer la vie ?

Questions idiotes :
A quoi sert cette parenthèse fermante, ligne 26 ?
Et pourquoi tu a fait une fonction dans une fonction ? (run dans le main...)

@liouan Merci je ne savais pas, je débute.

Voici mon code cl1 :

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
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
import java.io.File;
import java.io.IOException;
import java.util.Iterator;
import java.util.List;
import org.jdom2.Element;
import org.jdom2.JDOMException;
import org.jdom2.input.SAXBuilder;
import org.jdom2.xpath.XPath;
 
 
public class cl1
{
    public cl1() {
    }  
    String recherche(String userACherche, File _FilePath)
    {
    	String logPass="";
        org.jdom2.Document document = null ;
        try {
            SAXBuilder sxb = new SAXBuilder();
            document = sxb.build(_FilePath);
        } catch (IOException e) {
            System.out.println("Erreur lors de la lecture du fichier " 
				+ e.getMessage() );
            e.printStackTrace();
        } catch (JDOMException e){
            System.out.println("Erreur lors de la construction du fichier JDOM " 
				+ e.getMessage() );
            e.printStackTrace();
        }
        try {
            Element racine = document.getRootElement();
            XPath xpa = XPath.newInstance("//User");
            List results = xpa.selectNodes(racine) ;
            Iterator iter = results.iterator() ;
            Element noeudCourant = null;
            while (iter.hasNext())
            {
                noeudCourant = (Element) iter.next();                  
                xpa = XPath.newInstance("./@uid");
                if(xpa.valueOf(noeudCourant) == userACherche)
                {
                	logPass = xpa.valueOf(noeudCourant);
 
                	/*xpa = XPath.newInstance("./@password");
                	logPass[1] = xpa.valueOf(noeudCourant);*/
                	break;
                }
             }
        } catch (JDOMException e) {
            System.out.println("Erreur JDOM " + e.getMessage() );
            e.printStackTrace();            
        }
    	return logPass;
    }
}

@TheZopo Ceci n'est pas de moi c'est mon IDE (eclipse) qui l'a écris à la création de ma classe.

24/10/2012, 17h12
TheZopo
Ha ? N'empêche que ça sert a rien, mais bon :D

Je trouve ça bizarre :
Code:

1 2 3 public cl1() { }
Tu a fermer la méthode juste après l'avoir ouverte, elle ne contiens rien du coup... ;)

Ha et enlève la parenthèse (L. 26), j'ai beau chercher je ne vois pas à quoi elle sert...
24/10/2012, 17h16
Carlito08

@TheZopo Un oubli :oops: La parenthèse ligne 26 referme la parenthèse ouverte ligne 17 juste avant "new".

Peux-tu retirer ces lignes du main :

Code:

1
2
3
4
5
6
7
8
 
File entryFile = new File("C:\\www.xml");
cl1 essai = new cl1();
String logPass ="";
String userCherch = "ladjm";
logPass = essai.recherche(userCherch, entryFile);
logPass = "lulu";
textLog.setText("Identifiant : "+logPass);

et les mettre à la fin de la méthode initialize comme ceci :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
 
	private void initialize() {
		frame = new JFrame();
		frame.setBounds(100, 100, 450, 300);
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
 
		textLog = new JTextField();
		frame.getContentPane().add(textLog, BorderLayout.NORTH);
		textLog.setColumns(10);
 
		textPass = new JTextField();
		frame.getContentPane().add(textPass, BorderLayout.SOUTH);
		textPass.setColumns(10);
 
		File entryFile = new File("C:\\www.xml");
		cl1 essai = new cl1();
		String logPass ="";
		String userCherch = "ladjm";
		logPass = essai.recherche(userCherch, entryFile);
		logPass = "lulu";
		textLog.setText("Identifiant : "+logPass);
	}

puis retirer les mots-clé static sur ces 2 lignes :

Code:

1
2
3
 
	private static JTextField textLog;
	private static JTextField textPass;

et me dire si tu as toujours le problème?

24/10/2012, 17h20
Carlito08

@liouan Il n'y a plus le problème, merci, mais sa venait de quoi ?
24/10/2012, 17h21
TheZopo

Pas grave, met bien l'accolade a la fin.
Tu devrais te renseigner sur les conventions d'écriture, ton code sera ainsi beaucoup plus lisible, par exemple :
- MaClass
- MonObjet
- maVariable
- MADEFINE

Bonne continuation :)
24/10/2012, 17h23
Carlito08

@TheZopo Comme je l'ai dit je débute, merci pour le conseil et merci pour ton aide :ccool:
24/10/2012, 17h32
andry.aime
@Carlito08

1- Vire la méthode initialize() et met son contenu dans le constructeur essai.
2- La variable window que tu instancie dans public void run() ne sera visible que cette bloc donc tu auras toujours un NullPointerException à la ligne **. Déplace ces lignes après le bloc EventQueue.invokeLater(new Runnable() ... dans le bloc de run().
Citation:
Envoyé par TheZopo

Ha ? N'empêche que ça sert a rien, mais bon :D

Je trouve ça bizarre :

Code:

1 2 3 public cl1() { }

Tu a fermer la méthode juste après l'avoir ouverte, elle ne contiens rien du coup... ;)

Ha et enlève la parenthèse (L. 26), j'ai beau chercher je ne vois pas à quoi elle sert...
C'est un constructeur, y a rien de bizarre :aie:.

A+.

Pour tout te dire, je ne comprends meme pas pourquoi ça marchait en commentant la ligne.

Au début de ton main, tu invoques :
Code:

1 2 3 4 5 6 7 8 9 10 11 EventQueue.invokeLater(new Runnable() { public void run() { try { essai window = new essai(); window.frame.setVisible(true); } catch (Exception e) { e.printStackTrace(); } } });
A cause de ça, tu ne sais pas exactement quand sera exécutée la méthode run(). Elle sera exécutée "dès que possible".

En attendant, ton programme java continue à éxecuter les lignes suivantes:
Code:

1 2 3 4 5 6 7 8 File entryFile = new File("C:\\www.xml"); cl1 essai = new cl1(); String logPass =""; String userCherch = "ladjm"; logPass = essai.recherche(userCherch, entryFile); logPass = "lulu"; textLog.setText("Identifiant : "+logPass);
Si tu en arrives à la dernière ligne alors que ton run() n'a pas été exécuté, textLog est toujours null...

Un petit copier/coller vite fait

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
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
public class essai {
 
	private JFrame frame;
	private static JTextField textLog;
	private static JTextField textPass;
 
	/**
         * Launch the application.
         */
	public static void main(String[] args) {
		EventQueue.invokeLater(new Runnable() {
			public void run() {
				try {
					essai window = new essai();
					window.frame.setVisible(true);
					File entryFile = new File("C:\\www.xml");
					// cl1 essai = new cl1();
					String logPass ="";
					String userCherch = "ladjm";
					// logPass = essai.recherche(userCherch, entryFile);
					logPass = "lulu";
					textLog.setText("Identifiant : "+logPass);
				} catch (Exception e) {
					e.printStackTrace();
				}
			}
		});
	}
 
 
 
 
 
	/**
         * Create the application.
         */
	public essai() {
		frame = new JFrame();
		frame.setBounds(100, 100, 450, 300);
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
 
		textLog = new JTextField();
		frame.getContentPane().add(textLog, BorderLayout.NORTH);
		textLog.setColumns(10);
 
		textPass = new JTextField();
		frame.getContentPane().add(textPass, BorderLayout.SOUTH);
		textPass.setColumns(10);
	}
 
}

24/10/2012, 17h36
Carlito08

Citation:

Envoyé par andry.aime

@Carlito08

1- Vire la méthode initialize() et met son contenu dans le constructeur essai.
2- La variable window que tu instancie dans public void run() ne sera visible que cette bloc donc tu auras toujours un NullPointerException à la ligne **. Déplace ces lignes après le bloc EventQueue.invokeLater(new Runnable() ... dans le bloc de run().

D'ailleurs pourquoi tu utilises EventQueue.invokeLater ?

Le point numéro 1 c'est fais, le 2e je n'est pas compris ce que tu voulais dire.
Et pour "EventQueue.invokeLater" c'est l'IDE qui me l'a créé avec la fonction run().
24/10/2012, 17h42
TheZopo

Citation:

C'est un constructeur, y a rien de bizarre :aie:

TheZopo prend une grande inspiration, puis dit -en faisant exprès d'exagérer l'articulation- : LOL
24/10/2012, 17h43
andry.aime

Pour l'EventQueue, c'est expliqué sur le javadoc.
Mais essaie de lire un tutoriel sur Swing et AWT avant de générer des codes avec un IDE.

A+.
24/10/2012, 17h45
Carlito08

@andry.aime Ton code marche nickel merci. Merci pour la doc et merci a tous pour votre aide.