Allocation à distance

Version imprimable

Bonjour,

je dispose d'une classe image :

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
 
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:

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:

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 ???

16/09/2006, 12h47
ToTo13

Pour être exacte, j'ai un "java.lang.NullPointerException" lorsque j'appelle : image.setData(width, height) ;
16/09/2006, 13h15
herve91
Bonjour,
Code:

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

1 2 image.setData(width, height) ;
C'est normal ! tu passes une référence null à la méthode LireImage...
Fais plutôt :

Code:

image = OutilPGM.LireImage(nom);

et dans LireImage :
Code:

1 2 Image image = new Image(width, height); return image;
16/09/2006, 14h40
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:

1 2 int LireImage(char nom[64], Image *image)...
Et ainsi pouvoir renvoyer un code d'erreur ou de réussite.
16/09/2006, 14h50
herve91
Citation:

Envoyé par ToTo13

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

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:

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:

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

J'obtient un pointeur null !!!

Pourquoi ?

16/09/2006, 15h50
herve91
Code:

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:

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

Il faut allouer chaque Pixel en i, j.

Voilà la classe pixel !

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
 
public class Pixel
{
	private short Value = 0 ;
 
public Pixel()
	{
	}
 
public Pixel(short value)
	{
	setValue(value) ;
	}
 
public short getValue()
	{
	return Value ;
	}
 
public int setValue(short value)
	{
	if ( (value >= 0) && (value < 256) ) Value = value;
    else return 0 ;
    return 1 ;
	}
}

Je ne vois pas comment faire...

16/09/2006, 17h30
benratti
Citation:
Envoyé par ToTo13

Code:

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:

1 2 3 for ( int i = 0 ; i < hauteur ; i++ ) for ( int j = 0 ; j < largeur ; j++ ) data[i][j] = new Pixel() ;
16/09/2006, 20h40
ToTo13

Bonsoir,

merci infiniment pour ces réponses !!!
ça fontionne maintenant !!!