problème avec les accesseurs (getters)

Version imprimable

Code:

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
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
public class Document {

    //attributs 
    private int numero;
    private String titre;
    
     //constructeur
    public Document(int numero,String titre)
    {
     this.numero=numero;
     this.titre=titre;
    }
    
    //ACCESSEURS
    //1ère methode 
         public int getNumero()
         {
          return this.numero;   
         }
	
	 public String getTitre()
         {
         return this.titre;   
         }

	
	//2ème methode

	public int getNumero()
	{
	  return numero;
	}
	
       public String getTitre()
      {
        return titre;   
      }

    
}

Quelle est la bonne méthode (première ou deuxième )?

20/09/2017, 12h01
joel.drigo

Salut,

C'est exactement la même chose dans les deux cas : ça va fonctionner et donner le même résultat.

La première manière est peut-être plus rigoureuse. L'emploi du this. explicite lorsqu'il n'est pas indispensable va être sujet à débat entre développeurs. Pour moi, les deux sont acceptables, et, préférant le code le plus concis, je m'abstiendrais de mettre un this. explicite dans ce cas. Je suis partisan du "l'implicite quand l'explicite n'est pas indispensable (soit techniquement, soit pour la compréhension du code)". Donc, j'écris plutôt mes méthodes avec la seconde manière (sauf bien sûr pour les accolades que je mets à la manière Java, pas C ;) ).
20/09/2017, 22h14
tchize_

La bonne méthode c'est celle qui est consistante dans tout ton programme. C'est une question de convention. La convention choisie est peu importante, ce qui est important c'est de la respecter.
21/09/2017, 12h13
gautier1er

Si je commence avec this.numero alors je continue toujours avec this.numero dans le reste de mon code ? En claire pas de numero mais plutôt this.numero

Toujours dans la même classe

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
   
//1ère méthode
    public String toString()
    {
        String ch=new String("n°:"+ numero + "titre:"+ titre );
     return ch ;   
    }


//2ème méthode
   public String toString()
    {
        String ch=new String("numero:"+ getNumero() + "titre:"+ getTitre() );
     return ch ;   
    }

25/09/2017, 00h23
tchize_

Citation:

Envoyé par gautier1er

Si je commence avec this.numero alors je continue toujours avec this.numero dans le reste de mon code ? En claire pas de numero mais plutôt this.numero

En clair tu te fixe une convention et tu la suis :)

soit tu applique systématiquement this
soit uniquement en cas d'ambiguité
soit uniquement en setter
etc.
tous les gouts sont dans la nature.

Par contre, pour le

Code:

String ch=new String("n°:"+ numero + "titre:"+ titre );

:scarymov:

tu construis un String (la concaténation de 4 éléments), puis tu en fais une nouvelle String avec new String. D'une manière générale, en débutant, oublie que le constructeur existe sur String, tu n'aura pas besoin de l'appeler directement.

Code:

String ch="n°:"+ numero + "titre:"+ titre;
25/09/2017, 02h59
thelvin

En fait, juste

Code:

return "n°:" + numero + "titre:" + titre;

et envisager si on préfère pas l'usage de String.format() :

Code:

return String.format("n°: %d titre: %s", numero, titre);
26/09/2017, 14h32
gautier1er

merci :D