Discussion :

[jioupy]

Bonjour,

J'ai un petite problème, lorsque j'appelle un constructeur en passant par un "if".

Voici le code de mon programme :

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
public class Point {
	int abscisse;
	int ordonnee;
 
	public Point(int x, int y) {
		abscisse = x;
		ordonnee = y;
	}
 
	public Point(int z) {
		if (z == 0) {
			this(0,0);
		}
		else this(1,1);
	}
}

Quand j'appelle "this", j'ai une erreur : "Constructor call must be the first statement in a constructor".

En fait j'ai l'impression que je dois impérativement appeler "this" à un "niveau 0", si je le met à l'extérieur des "if", cela marche très bien, mais ce n'est pas ce que je souhaite.

Savez vous comment je pourrais régler ce problème?

Merci d'avance.

[nnovic]

Ce que tu veux faire n'est pas possible. Comme indiqué dans le message d'erreur, l'appel au constructeur est obligatoirement la première instruction.

De la même manière, il est interdit de faire ça:

Code :

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

1
2
3
4
5
 
	public Point(int z) {
		int a = 9;
		this(1,1);
	}

Du coup la solution est de faire quelque chose dans ce style:

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
 
public class Point {
	int abscisse;
	int ordonnee;
 
	public Point(int x, int y) {
		abscisse = x;
		ordonnee = y;
	}
 
	public Point(int z) {
		this(1,1);
		if (z == 0) {
			abscisse=0;
			ordonnee=0;
		}
	}
}

[tchize_]

Ceci devrait peut etre faire l'affaire?

Code :

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

1
2
3
	public Point(int z) {
			this((z==0)?0:1,(z==0)?0:1);
}

sinon y a cette possibilité:

Code :

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

1
2
3
4
public Point(int z) {
			this(getInit(z),getInit(z));
}
private static int getInit(int z) {return z==0?0:1}

Ou alors, comme déjà suggérer, appeler le constructeur normale (0,0) puis redéfinir les abscices

[herch]

salut,

tu pourrais faire le test à l'intérieur de l'appel de this

Code :

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

1
2
3
public Point(int z) {
	this((z==0?0:1), (z==0?0:1));
}

mais c'est pas très lisible comme code

[EDIT] Raté

[jioupy]

Merci, de vos réponses.

Toutes ces méthodes fonctionnent très bien.