Discussion :

[Targan]

Bonjour,

ne développant pas en java/objet depuis longtemps j'ai une question existentielle sur la création d'une classe métier, avec ou non l'initialisation de ses attributs.

Je m'explique : imaginons une class ayant 3 atttributs de type String, un attribut de type List<String>, un d'un type "personnalisé" (d'une autre classe) et un de type int.

En gros ceci :

Code :

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

1
2
3
4
5
6
7
8
9
 
public class My_New_Class {
    private int id;
    private String myString1;
    private String myString2;
    private String myString3;
    private List<String> myList1;
    private MyObject myObject1;
}

Imaginons 2 constructeurs différents :

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 My_New_Class {
    ...
 
    public My_New_Class(String myString1, String myString2){
        this.myString1 = myString1;
        this.myString2 = myString2; 
    }
 
    public My_New_Class(String myString1, String myString2, String myString3){
        this.myString1 = myString1;
        this.myString2 = myString2; 
        this.myString3 = myString3; 
    }
}

Quand faut il initialiser directement les attributs dans leur déclaration ?

Je pars du principe (enfin ma logique) qu'il faut initialiser les objets "complexes" si il ne le sont pas dans les constructeurs.
Ici :

Code :

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

1
2
3
 
    private List<String> myList1 = null;
    private MyObject myObject1 = null;

Et j'aurais tendance à initialiser aussi les types "primitifs" qui ne sont pas commun à tous les constructeurs.
Ici, j'aurais fait :

Code :

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

1
2
3
4
 
    private int id = -1;
    ...
    private String myString3 = null;

Donc en gros, cela donnerait ceci au final :

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
 
public class My_New_Class {
    private int id = -1;
    private String myString1;
    private String myString2;
    private String myString3 = null;
    private List<String> myList1 = null;
    private MyObject myObject1 = null;
 
    public My_New_Class(String myString1, String myString2){
        this.myString1 = myString1;
        this.myString2 = myString2; 
    }
 
    public My_New_Class(String myString1, String myString2, String myString3){
        this.myString1 = myString1;
        this.myString2 = myString2; 
        this.myString3 = myString3; 
    }
}

Selon vous, est ce bon ? ou généralement on évite de le faire dans la déclaration, et on initialise tout dans chaque constructeur ?

Par avance merci.

[Rei Ichido]

Par défaut, tous les Objets sont initialisés à null, donc il est inutile de faire une telle affectation.
Quand on fait de l'affectation, c'est souvent pour initialiser une nouvelle liste, par exemple.

[Targan]

donc en gros, je ne devrais laisser que l'initialisation de ID = -1

Le reste s'initialisant à null quelque soit le type.

[Rei Ichido]

Les types primitifs s'initialisent à 0 pour les numériques.

Ensuite, savoir s'il vaut mieux initialiser par défaut ou initialiser dans le constructeur, j'ai tendance à trouver que si le constructeur peut changer la valeur (là, le cas de ton id), je préfère le mettre dans le constructeur.

[Alkhan]

bonjour,

pour ma part, je dirais que cela dépend de ton programme et de comment sont utiliser les variables, mais aussi de quel genre d'objet il s'agit.

prenons l'exemple de la liste que tu as, bien souvent lorsque tu as une liste privée, c'est que tu va l'alimenter avec des objets, pour ensuite la parcourir et / ou utiliser les objets qu'elle contient ou pas.

dans ce cas je fais plutôt :

Code :

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

1
2
 
private List<String> myList1 = new MonTypeList<String>();

La raison est qu'a aucun moment de ton code tu ne te demandera si myList1 est null !

je ferais une petite remarque sur tes constructeurs, il serais plus logique de faire :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
public class My_New_Class {
    ...
 
    public My_New_Class(String myString1, String myString2){
        this.myString1 = myString1;
        this.myString2 = myString2; 
    }
 
    public My_New_Class(String myString1, String myString2, String myString3){
        this(myString1, myString2);
        this.myString3 = myString3; 
    }
}

[Targan]

je ne connaissais pas ce type d'écriture pour le constructeur appelant un autre..

Merci

Pour ce qui est de mes attributs, la plupart sont setter par des valeurs externes. Donc je cherchais l'écriture la plus propre pour bien tout initialiser et ne pas avoir de surprise.