Erreur NullPointerException incompréhensible

Version imprimable

Voir 40 message(s) de cette discussion en une page

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, 15h30
Robin56

Et c'est quoi le code de essai.java ? (spécialement la ligne 39)
24/10/2012, 15h34
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, 15h45
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, 15h46
liouan

(NPE : NullPointerException)
24/10/2012, 15h47
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, 15h54
TheZopo

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

Code:

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

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

1 2 3 public essai() { initialize(); }
Non ?
24/10/2012, 16h00
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, 16h01
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, 16h12
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, 16h16
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, 16h20
Carlito08

@liouan Il n'y a plus le problème, merci, mais sa venait de quoi ?
24/10/2012, 16h21
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, 16h23
Carlito08

@TheZopo Comme je l'ai dit je débute, merci pour le conseil et merci pour ton aide :ccool:
24/10/2012, 16h32
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+.

Voir 40 message(s) de cette discussion en une page