Discussion :

[ToTo13]

Bonjour,

je dispose d'une classe image :

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
 
public class Image
{
	private int width ; // Largeur de l'image
	private int height ; // Hauteur de l'image
	private int size ; // Taille totale de l'image = hauteur * largeur
	short data[][]  ; // Matrice des pixels
 
 
public Image()
	{
	}
 
public Image(int largeur, int hauteur)
	{
	setData(largeur, hauteur) ;
	}
 
public void setData(int largeur, int hauteur)
	{
	width = largeur ;
	height = hauteur ;
	size = largeur * hauteur ;
	data = new short[hauteur][largeur] ;
	}
}

Je déclare donc une varable Image dans mon main, puis j'appelle une classe contenant la méthode "LireImage" :

Code :

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

1
2
3
 
Image image = null ;
OutilPGM.LireImage(nom, image) ;

Dans LireImage je récupère les dimensions de mon image et donc j'alloue :

Code :

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

1
2
3
4
5
6
7
 
width = Integer.valueOf(ligne.substring(0, n)).intValue() ;
height = Integer.valueOf(ligne.substring(n+1)).intValue() ;
 
System.out.println("Dimension image : " + width + " & " + height) ;
 
image.setData(width, height) ;

Mais lorsque je veux utiliser le tableau data, j'ai une erreur de pointeur null !!!

Est ce que quelqu'un sait pourquoi ???

[ToTo13]

Pour être exacte, j'ai un "java.lang.NullPointerException" lorsque j'appelle : image.setData(width, height) ;

[herve91]

Bonjour,

Code :

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

1
2
3
Image image = null ;
OutilPGM.LireImage(nom, image) ;

Code :

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

1
2
image.setData(width, height) ;

C'est normal ! tu passes une référence null à la méthode LireImage...
Fais plutôt :

Code :

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

image = OutilPGM.LireImage(nom);

et dans LireImage :

Code :

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

1
2
Image image = new Image(width, height);
return image;

[ToTo13]

Bonjour,

j'avais pensé à cette éventualité.
Disons que je préférais avoir une solution qui ressemble à ce que je faisais en C :

Code :

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

1
2
 
int LireImage(char nom[64], Image *image)...

Et ainsi pouvoir renvoyer un code d'erreur ou de réussite.

[herve91]

Disons que je préférais avoir une solution qui ressemble à ce que je faisais en C :
Et ainsi pouvoir renvoyer un code d'erreur ou de réussite.

Tu ne peux pas renvoyer de valeur via les paramètres en Java : ces derniers sont passés par valeur et non par référence (= "par adresse" en C).
Pour le code d'erreur, soit :

• une valeur null en retour de méthode signale un problème
• ta méthode lève une exception en cas d'erreur

[ToTo13]

Bonjour,

je viens de regarder en détail ce qui se passe et c'est dans Image que l'allocation ne se fait pas :

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
 
public class Image
{
	private int width ; // Largeur de l'image
	private int height ; // Hauteur de l'image
	private int size ; // Taille totale de l'image = hauteur * largeur
	Pixel data[][]  ;
 
public Image()
	{
	}
 
public Image(int largeur, int hauteur)
	{
	setData(largeur, hauteur) ;
	}
 
public void setData(int largeur, int hauteur)
	{
	width = largeur ;
	height = hauteur ;
	size = largeur * hauteur ;
	data = new Pixel[hauteur][largeur] ;
	}
}

Quand je tape :

Code :

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

1
2
3
 
Image image = new Image(width, height) ;
System.out.println(image.data[0][0]) ;

J'obtient un pointeur null !!!

Pourquoi ?

[herve91]

Code :

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

1
2
3
 
Image image = new Image(width, height) ;
System.out.println(image.data[0][0]) ;

Le tableau data est bien alloué mais ses éléments ne sont pas alloués c-à-d que :

Code :

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

data[i][j] vaut null pour 0 <= i < width et 0 <= j < height

Il faut allouer chaque Pixel en i, j.

[benratti]

Code :

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

1
2
 
	data = new Pixel[hauteur][largeur] ;

Cette ligne t'alloue la place memoire pour le tableau ... mais ne le remplit pas. C'est à toi de le remplir manuellement en faisant :

Code :

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

1
2
3
for ( int i = 0 ; i < hauteur ; i++ )  
    for ( int j = 0 ; j < largeur ; j++ )
        data[i][j] = new Pixel() ;