[debutant] NullPointerException java

**flora806** · 07/11/2007, 12h35

Envoyé par Tasumy

Qu'est-ce qui ne fonctionne pas?
Tu as toujours l'erreur "NullpointerException"?

Oui, j'ai toujours cette Exception. c'est au niveau de la ligne:

Code :

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

1
2
3
4
5
public class MaClasse1 {......
public void Calcul() throws Exception {....
if (mot != null)
{for (int i = 0; i < mot.getClientNbr(); i++)....} 
}

c'est aussi au niveau du Main de cette classe1:
MaClasse1 obj = new MaClasse1();
obj.Calcul();
et également au niveau de MaClasse2 : ligne de la méthode "getClientNbr" (je l'ai posté dans mon message précédent)..
Merci.

**Tasumy** · 07/11/2007, 13h24

également au niveau de MaClasse2 : ligne de la méthode "getClientNbr"

Ca viendrait de cette méthode donc, c'est à quelle ligne?
Sinon, tu peux poster l'erreur en entier s'il te plaît?

**flora806** · 07/11/2007, 15h25

Envoyé par Tasumy

Ca viendrait de cette méthode donc, c'est à quelle ligne?
Sinon, tu peux poster l'erreur en entier s'il te plaît?

Exception in thread "main" java.lang.NullPointerException:
at MaClasse1.main (ligne: obj.Calcul()

at MaClasse2.getClientNbr (ligne: for (int[]DataLine : data ))

**Benouze** · 07/11/2007, 15h30

data ne serait-il pas null ?

**jhaythem** · 07/11/2007, 15h31

pourquoi tu ne deboggue pas ton code step by step?

**Tasumy** · 07/11/2007, 15h34

Apparemment, ce serait ton data qui serait null.
Fais-nous voir comment il est déclaré et instancié.

**jhaythem** · 07/11/2007, 15h40

oui ton data il est instancie mais il est vide je crois.

Cheers,

**flora806** · 07/11/2007, 15h48

je pense que vous avez raison, c'est data qui est certainement null.

Dans MaClasse2 {
int [][]data = 0;
.....

et dans la méthode:

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
22
 
public  int getClientNbr()
	    {	int ClientId = 0;
	    	int TotalClients = 0;
 
	    	for (int[]DataLine : data )
	    	{
	    		if (DataLine.length > 2)
	    		{
	    		DataLine[0] = ClientId;
 
	    			{
					for (int i=0; i < DataLine.length; i++)
		   			{ TotalClients = TotalClients + DataLine[i];
 
		   				}				
	    										}
 
	    	}
	    	}
	    	return TotalClients;
	    }

je dois l'instancier au niveau de cette méthode?
je peux faire comment si je veux récupérer vers "data", les éléments de "list" qui est dans le return de la méthode ParseFile de MaClasse2?

Merci bcp pour votre aide.

**flora806** · 07/11/2007, 15h49

Pardon, je rectifie la déclaration de data dans MaClasse2:
public int [][] data;

**Tasumy** · 07/11/2007, 16h17

je dois l'instancier au niveau de cette méthode?

Oui tu peux.
Mais, étant donné que tu l'as déclaré en attribut, je suppose que tu dois l'utiliser autre part et donc le mieux est de l'instancier dès sa première utilisation une fois pour toute.

je peux faire comment si je veux récupérer vers "data", les éléments de "list" qui est dans le return de la méthode ParseFile de MaClasse2?

comme data vient de la classe MaClasse2 :

Code :

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

data = ParseFile();

Bien sûr, il faut que ta méthode ParseFile retourne un "int [][]".

[EDIT]

oui ton data, il est instancié mais il est vide je crois.

non justement, ça ne provoquerait pas de NullPointerException s'il était instancié et vide.

**flora806** · 07/11/2007, 17h56

Envoyé par Tasumy

Oui tu peux.
Mais, étant donné que tu l'as déclaré en attribut, je suppose que tu dois l'utiliser autre part et donc le mieux est de l'instancier dès sa première utilisation une fois pour toute.
comme data vient de la classe MaClasse2 :

Code :

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

data = ParseFile();

Bien sûr, il faut que ta méthode ParseFile retourne un "int [][]".

[EDIT]
non justement, ça ne provoquerait pas de NullPointerException s'il était instancié et vide.

j'ai instancié data comme tu m'as dis :data = ParseFile() au niveau de getClientNbr, mais comme ParseFile retournait un résultat de type ArrayList, je suis entrain de le modifier:

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
22
23
24
25
26
27
28
29
30
 
public int [][] ParseFile() throws IOException
 
	      {
 
		 BufferedReader reader = new BufferedReader(new FileReader("C:file.txt"));
 
		  String line = null;
		  int [][]items;
		  StringTokenizer splitter;
		  while ((line = reader.readLine()) != null) 
		  {
 
		  for (String item : items) 
		  {
			  splitter = new StringTokenizer(item, " ");
			  while (splitter.hasMoreTokens())
			  	{
 Client = Integer.parseInt(splitter.nextToken());;
		   Article = Integer.parseInt(splitter.nextToken());
		   Prix = Integer.parseInt(splitter.nextToken());
 
		   items.add(item);
 
			  	}
 
		  }
		 }
		  return items;	
  }

j'ai toujours une erreur bien sûr concernant cette méthode, il y a un conflit entre String et int... Comment je pourrais utiliser à la fois StringTokenizer et retourner le résultat ss forme de int [][]?
je sais que je pose trop de questions, mais c'est grace à vous que j'apprends des choses, merci

**Tasumy** · 07/11/2007, 18h20

Code :

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

1
2
3
int [][]items;
...
for (String item : items)

Je vois une erreur NullPointerException à ce niveau (items).

Il y a quelque chose que je ne comprends pas :
ParseFile retournait un résultat de type ArrayList<String>=> dimension : 1
Mais maintenant, ParseFile retourne un résultat de type int [][] => dimension : 2.

Qu'est-ce que tu veux retourner exactement?
Qu'est ce que tu veux mettre dans data exactement? Car j'ai l'impression que les types que tu utilises ne sont pas bons.

je sais que je pose trop de questions

non, non on est là pour y répondre.

**flora806** · 07/11/2007, 18h43

En fait, je saivais bien qu'il y avait cette erreur à ce niveau du code, je l'ai posté qd même, même si j'étais entrain de le modifier..
en fait dans ParseFile, je veux retourner un tableau int à 2 dimensions, une concernant ClientId et l'autre ArticleId. parceque j'ai besoin de réutiliser ce tableau par d'autres méthodes dans la classe.

**Tasumy** · 07/11/2007, 20h36

Dans ce cas, je te déconseille le int[][] puisque tu ne connais pas la taille du tableau et te conseille d'utiliser un ArrayList<Achat> pour ton data.
classe Achat :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
	public class Achat
	{
		private int clientId;
		private int articleId;
		public Achat(int clientId, int articleId){
			this.clientId = clientId;
			this.articleId = articleId;
		}
		public int getClientId(){return clientId;}
		public int getArticleId(){return articleId;}
	}

Dans MaClasse2:

Code :

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

ArrayList<Achat>  data;

puis faire (où tu en as besoin) :

Code :

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

data = parseFile();

ta méthode parseFile() :

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
22
23
24
25
26
27
public ArrayList<Achat> ParseFile() throws IOException
{
	BufferedReader reader = new BufferedReader(new FileReader("C:file.txt"));
         
	String line;
	ArrayList<Achat> items;
	StringTokenizer splitter;
	while ((line = reader.readLine()) != null) 
	{
		for (String item : items)
		  {
			 splitter = new StringTokenizer(item, " ");
			  while (splitter.hasMoreTokens())
			  	{
				  Client = Integer.parseInt(splitter.nextToken());;
				  Article = Integer.parseInt(splitter.nextToken());
				  Prix = Integer.parseInt(splitter.nextToken());
		      		  Achat unAchat = new Achat(Client,Article);

				  items.add(unAchat);
		   
			  	}
		  	  
		  }
		 }
		  return items;	
  }

Note: Ta boucle for(String item : items) ne te sert à rien.

Pour accéder au clientID/articleID de data tu fais:

Code :

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

1
2
3
Achat unAchat = data.get(index);
unAchat.getClientID();
unAchat.getArticleId();

Bien sûr, en faisant cela, il faut que tu modifies partout où tu as manipulé data.

**rebel64** · 07/11/2007, 22h58

J'ai lu en diagonal les messages et il y a des trucs qui me turlupine...

1) Dans ParseFile() items est déclaré mais jamais alloué.

Code :

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

ArrayList<Achat> items = new ArrayList<Achat>();

2) La boucle devrait alors s'écrire:

Code :

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

for(Achat achat:items){...}

3) Mais çà ne marcherais PAS parcequ'elle est vide !
Ce bloc for(Achat achat:items) ne sert à rien...
Si chaque ligne lue est un achat, alors ce sont les lignes lues qu'il faut splitter (et pas les items).
La liste des items c'est ton résultat.

4) Si le but est de remplir cette liste avec les éléments "achats de clients"
(ici les objets de type Achat) et si chaque achat est une ligne...
tu devrais avoir un truc du style:

Code :

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

1
2
3
4
5
6
7
8
splitter = new StringTokenizer(line, " ");
if (splitter.hasMoreTokens()) {
   client = Integer.parseInt(splitter.nextToken());;
   article = Integer.parseInt(splitter.nextToken());
   prix = Integer.parseInt(splitter.nextToken());	
   items.add(new Achat(client,article,prix));	      		 
}

5) Une autre remarque:
Conventionellement (mais c'est important en JAVA) le nom des propriétés et des méthodes commencent par une lettre minuscule. Le nom des classes commencent par une lettre majuscule.
On donc peu déclarer :

Article article; /* ma variable "article" de classe "Article" */
Article getArticle(){...}; /* getter de l'article */

Mais JAMAIS l'inverse (les programmes deviennent illisibles)

**Tasumy** · 08/11/2007, 09h40

Code :

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

1
2
3
4
5
ArrayList<Achat> items = new ArrayList<Achat>();
...
splitter = new StringTokenizer(line, " ");
if (splitter.hasMoreTokens())
...

Oui pardon, c'est du copier-coller un peu trop rapide.

Ce bloc for(Achat achat:items) ne sert à rien...

Note: Ta boucle for(String item : items) ne te sert à rien.

Conventionellement (mais c'est important en JAVA) le nom des propriétés et des méthodes commencent par une lettre minuscule. Le nom des classes commencent par une lettre majuscule.

Cette remarque lui a déjà été faite.

**flora806** · 08/11/2007, 10h34

Je viens de lire vos réponses, je vais tester ce que vous m'avez proposé, je vous dirai ce que ça donne

Merci beaucoup pour votre aide

**flora806** · 08/11/2007, 14h42

Salut,
j'ai modifié la classe Achat suivant vos conseils, pour la méthode ParseFile, c'est bon.
maintenant quand je fais appel à ParseFile() à partir de la méthode getArticlesCount, j'ai une erreur

. je veux récupérer le nbr total des articles achetés.
voilà mon code:

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
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
60
 
public class Achat
{
	private int clientId;
	private int articleId;
	private int prix;
	ArrayList<Achat>  data;
	public Achat(int clientId, int articleId, int prix)
	{
		this.clientId = clientId;
		this.articleId = articleId;
		this.prix = prix;
	}
 
	public Achat() {
 
	}
 
	public int getClientId(){return clientId;}
	public int getArticleId(){return articleId;}
	public int getPrix(){return prix;}
 
	public ArrayList<Achat> ParseFile() throws IOException
	{
		BufferedReader reader = new BufferedReader(new FileReader("C:\file.txt"));
 
		String line;
		ArrayList<Achat> items = new ArrayList<Achat>();
		StringTokenizer splitter;
		while ((line = reader.readLine()) != null) 
		{						  
	splitter = new StringTokenizer(line, " ");
	if (splitter.hasMoreTokens()) {
				clientId = Integer.parseInt(splitter.nextToken());;
				articleId = Integer.parseInt(splitter.nextToken());
				prix = Integer.parseInt(splitter.nextToken());	
				  items.add(new Achat(clientId,articleId,prix));     		 			  								}					
		 }
		return items;	  	  
	}
 
 
	public int getArticlesCount() {
		//Achat test = new Achat();
//Ici j'ai unhandled exception file IO Exception		
data = ParseFile();
 
		int TotalItems = 0;
	       ArrayList <Integer> v = new ArrayList <Integer> ();
 
	        for (int i = 0; i < data.size(); i++) 
	        {	        
	        		  v.add(i);
	        		  TotalItems = Math.max(TotalItems,v.size());
	       }
 
	       return TotalItems;	   
 
	  }	
}

**Benouze** · 08/11/2007, 14h54

Bonjour,

C'est cette ligne-là qui entraîne l'exception :

Code :

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

while ((line = reader.readLine()) != null)

Je le sais car dans ton code seule cette méthode peut générer une IOException comme précisé dans la Javadoc

Il faudrait donner la trace complète du message pour savoir plus précisemment ce qu'il en est.

**Duc Lebowski** · 08/11/2007, 14h58

Remplace :

Code :

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

"C:\file.txt"

Par :

Code :

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

"C:\\file.txt"

Ou encore mieux :

Code :

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

"C:"+System.getProperty("path.separator")+"file.txt"

[debutant] NullPointerException java

Langage Java

Discussions similaires

Partager

Partager