Discussion :

[NFHnv]

Bonjour,

j'aurais besoin d'une petite confirmation concernant les accesseurs et modificateurs.

Si j'ai une classe LouePar qui a deux champs : un de type Vehicule veh et l'autre de type Client cl
est-ce que je peux écrire :

Code :

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

1
2
3
4
 
public void setVC(Vehicule v, Client c)
{ veh = v;
   cl = c;}

???

Merci pour vos réponses.

[Jidefix]

Bien sur, c'est comme ça que ça marche

Si tu veux que ce soit plus clair tu peux aussi écrire

Code :

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

1
2
3
4
 
public void setVC(Vehicule v, Client c)
{ this.veh = v;
   this.cl = c;}

Ca revient au même mais le sens est plus évident quand on ne connait pas bien la classe.

[Antoine_935]

Il est conseillé de séparer les accesseurs et mutateurs pour les différents attributs.
Ainsi, tu auras deux méthodes de chaque:

• une pour la voiture
• une pour le client

Ca permet de pouvoir facilement changer une des deux propriétés sans obligatoirement devoir conserver ou récupérer l'autre.

[likoudA]

Pour aller dans le même sillage de Antoine_935, il te faut les 4 méthodes suivantes :

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 Client getClient() {
        return client;
    }
 
    public void setClient(Client cli) {
        this.client = cli;
    }
 
    public Vehicule getVehicule() {
        return vehicule;
    }
 
    public void setVehicule(Vehicule veh) {
        this.vehicule = veh;
    }

ainsi tout est séparé et plus visible.

[professeur shadoko]

IL me semble que la définition des accesseurs/mutateurs dépend plutot de tes contraintes fonctionelles:
- mettre des accesseurs/mutateurs pour chaque champ n'est pas une bonne pratique si elle est systématique: quelle est la logique? l'objet garde t'il une paire non-modifiable Client/voiture (alors ça passe par le constructeur et il n'y a pas de méthode de modification); cette paire est elle modifiable mais les valeurs changent ensemble (alors une méthode pour changer la paire comme dans l'exemple initial), etc... le risque d'avoir des mutateurs séparés est d'avoir des modifications incohérentes et un objet dans un état que la morale réprouve (par exemple avec des champs null : bonjour les bugs).
Donc il n'y a pas de recette mais une réflexion sur ce que doit faire l'objet.

[virulent]

Bonjour à tous,

Juste pour info, le 'this' est souvent utilisé pour permettre aux nouveaux venus de se familiariser avec la syntaxe Java. Cela permet de bien clarifier le code lors d'un relecture.

A la base le 'this' ne devient obligatoire que lorsque l'on a le même nom de handlers afin de pouvoir différencier les références :

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 LouePar {
 
Vehicule vehicule;
Client client;
 
public void setVC(Vehicule vehicule, Client client)
{ 
   this.vehicule=vehicule;
   this.client=client;
}
 
}

Je préfère d'ailleurs coder de cette façon, car je trouve cela plus lisible...mais bon c'est personnel.

====
JAVA bien merci

[Alain Defrance]

Le this est indispensable lors d'une affectation lorsque le paramètre provoque un shadowing (possède le même nom qu'un des membres).

Les accesseurs/mutateurs ne doivent pas toujours être présents. Il se peut que la mutation d'un état soit dangereux pour l'intégrité de l'objet, il faut donc tout simplement ne pas le créer. Si les accesseurs/mutateurs sont systématiques, l'encapsulation perd tout son sens. Il en va de même pour les setters : certains membres doivent obligatoirement exister (présents dans le constructeur), mais ne doivent jamais être modifier (pas de setter).

[Ivelios]

Récapitulatif :
Si tu as juste besoin d'initialiser : tu mets des paramètres dans le constructeur.
Si tu as besoin de consulter : tu mets des "gets" (accesseurs)
Si tu as besoin de modifier : tu met des "sets" (modificateurs)

Après, comme le signalait professeur shadoko c'est toi qui décide de quel manière tu les écries (un accesseurs par attributs ; un seul accesseurs avec tous les attributs...).

ps : Si tu n'as pas besoin d'accesseurs ni de modificateurs alors ne les écrits pas... moins tu en fais mieux s'es