Discussion :

[Invité]

Bonjour,

J'ai un objet (disons conteneur), qui en contient un autre (disons contenu). J'aimerais que lors de la construction de contenu, si certaines conditions ne sont pas réalisées, l'objet s'initialise à null.

Le plus simple est d'illustrer immédiatement mon problème par du 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
public class Test2
{
	private Test1 t1;
 
	public Test2()
	{
		this.t1 = new Test1(this);
	}
 
	public void set(Test1 t1)
	{
		this.t1 = t1;
	}
 
	public Test1 get()
	{
		return t1;
	}
 
	public static void main(String[] args)
	{
		Test2 t2 = new Test2();
		System.out.println(t2.get() == null);
	}
}

Code :

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

1
2
3
4
5
6
7
public class Test1
{
	public Test1(Test2 t2)
	{
		t2.set(null);
	}
}

Pourquoi est-ce que l'exécution de ce programme m'affiche false, et comment résoudre ce problème ?

Merci d'avance,

Maxime

[Baptiste Wicht]

Ce genre de code est vraiment à éviter...

Mais le résultat est logique. Si on prend le constructeur de t2 :

this.t1 = new Test1(this);

Voici ce qui va être fait

exécuter constructeur de Test1
t2.set(null); ==> donc t1 dans Test2 = null
retourner du constructeur
this.t1 = new Test1(this); ==> donc t1 dans Test2 = new Test1(this);

On ne peut pas stopper un constructeur et lui faire retourner null...

La solution c'est de tester les conditions dans Test2 et si elle se réalisent instancier Test1, sinon ne rien faire et laisser t1 à null.

[Invité]

Je ne peux pas tester les conditions dans conteneur, parce qu'en fait contenu appartient à une classe abstraite dont chacune des classes filles a ses propres conditions d'existence.

[Baptiste Wicht]

Je ne peux pas tester les conditions dans conteneur, parce qu'en fait contenu appartient à une classe abstraite dont chacune des classes filles a ses propres conditions d'existences.

Alors, il faut que tu utilises une factory qui va contrôler les conditions d'existences. Et qui, si les conditions ne sont pas vérifiées, va te retourner un objet null.

Tu devrais regarder du côté du design pattern Abstract Factory qui devrait satisfaire tes exigences.

Sinon, un workaround simple mais très moche serait :

Code :

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

1
2
3
4
5
this.t1 = new Test1(this);
 
if(t1.isNotValid()){
    this.ti = null;
}

mais une solution avec design pattern sera beaucoup plus propre et souple.

[Invité]

Merci de la réponse.

[adiGuba]

Salut,


Le constructeur de Test1 qui référence Test2 pour modifier la référence de Test1 dans le constructeur de ce dernier qui recevra le résultat du constructeur de Test1 ! Tout ceci est bien alambiqué


Tu ne peux pas retourner null depuis un constructeur. Tout ce que tu peux faire c'est remonter une exception et la traiter.

Tu peux éventuellement utiliser une méthode factory pour cela :

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
class Test {
 
	public Test() {
		if (condition) {
			throw new IllegalStateException("info");
		}
	}
 
	public static Test create() {
		try {
			return new Test();
		} catch (IllegalStateException e) {
			return null;
		}
	}
}

a++