constructeurs d'une class

**sofiene D** · 29/12/2009, 14h47

bonsoir
j'ai une classe avec deux constructeurs

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
 
public class maclass
{
private int v3;
public maclass(int v1,int v2)
{
//traitement
}
public maclass(int v1,int v2,String v3)
{
this.v3=v3;
 
this(v1,v2);
}
}

mais le compilateur génère une erreur au niveau de l'instruction this(v1,v2)
je comprant pas pourquoi

et est-ce que je doit redéfinir le code du premier constructeur dans le deuxième
merci

**Ivelios** · 29/12/2009, 14h59

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
public class maclass{
      private String v3;
      private int v1,v2;
 
      public maclass(int v1,int v2){
            this.v1 = v1;
            this.v2 = v2;
            this.v3 = null;
            this.traitement();
 
      }
 
      public maclass(int v1,int v2,String v3){
            this.v1 = v1;
            this.v2 = v2;
            this.v3=v3;
            this.traitement();
 
      }
 
      private void traitement(){
            //traitement
      }
}

**tchize_** · 29/12/2009, 16h37

Envoyé par sofiene D

mais le compilateur génère une erreur au niveau de l'instruction this(v1,v2)
je comprant pas pourquoi

et est-ce que je doit redéfinir le code du premier constructeur dans le deuxième
merci

La délégation à un autre constructeur doit obligatoirement etre la premier instruction de ton constructeur. Ton code devrais donc etre:

Code :

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

1
2
3
4
5
6
7
8
9
public maclass(int v1,int v2)
{
//traitement
}
public maclass(int v1,int v2,String v3)
{
this(v1,v2);
this.v3=v3;
}

PS: essaie de respecter les convention de nommage java, Les classes commencent toujours par une majuscule.

**sofiene D** · 29/12/2009, 21h44

bonsoir

PS: essaie de respecter les convention de nommage java, Les classes commencent toujours par une majuscule.

maclass est juste un exemple réalment ma classe est R_dgs

**tchize_** · 29/12/2009, 22h59

un nommage correct dans ce cas serait RDgs

**adiGuba** · 30/12/2009, 10h06

Salut,

Ivelios : Justement l'objectif est d'éviter la duplication du code...

Sinon pour l'utilisation de multiple constructeur il y a deux approches :

La première, utilisé ici, consiste à ne traiter les nouveaux paramètres QUE dans le constructeur où on les déclare. Par exemple :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
	public Maclass(int v1) {
		this.v1 = v1;
	}
	public Maclass(int v1, int v2) {
		this(v1);
		this.v2 = v2;
	}
	public Maclass(int v1, int v2, String v3) {
		this(v1, v2);
		this.v3 = v3;
	}

Cette approche la plus simple et la plus logique de prime abord pose quand même des soucis :

Selon le constructeur utilisé certains attributs ne seront pas initialisés. Ils prendront donc forcément la valeur 0 ou null selon leurs types...
A cause de cela, on ne peut pas définir d'attribut final mis à part dans le constructeur initial.

La seconde approche consiste tout simplement à faire l'inverse. C'est à dire qu'au lieu de partir du constructeur le plus simple, on part du plus complet.

En clair on utilise le constructeur avec le plus de paramètres et on y initialise TOUS les champs, éventuellement en gérant les valeurs nulles si besoin. Les autres constructeur font alors appel à ce constructeur plus ou moins directement, en rajoutant les valeurs par défaut pour les champs absents :

Cela donne quelque chose comme cela :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
	public Maclass(int v1) {
		this(v1, 10); // 10 = valeur par défaut pour v2
	}
	public Maclass(int v1, int v2) {
		this(v1, v2, null);
 
	}
	public Maclass(int v1, int v2, String v3) {
		this.v1 = v1;
		this.v2 = v2;
		this.v3 = (v3==null ? "" : v3);
	}