Discussion :

[Baptiste Wicht]

Bonjour,

Je suis en train de me poser une question sur les conventions de codage. J'ai déja regardé dans le tutoriel sur developpez.com et rien n'en parle.

Ma question est la suivante, quand on appelle une méthode avec plusieurs paramètres (dison x(a,b,c,d,e) ), comme organise t'on les paramètres, faut-il les mettre sur plusieurs lignes ou alors on les mets seulement sur plusieurs lignes si on a pas la place en largeur pour tous les mettre les uns à cotés des autres.

Donc faut-il écrire :

Code :

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

z = x(a,b,c,d,e);

Ou alors plutôt écrire :

Code :

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

1
2
3
4
5
6
z = x(
  a,
  b,
  c,
  d,
  e);

Ou encore le premier paramètre sur la première ligne :

Code :

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

1
2
3
4
5
z = x(a,
  b,
  c,
  d,
  e);

Et pareil pour la déclaration de la méthode en question, faut il les écrire tous sur une ligne ou un par ligne ?

donc :

Code :

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

1
2
3
public Type x(Type a, Type b, Type c, Type d, Type e){
 
}

ou :

Code :

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

1
2
3
4
5
6
7
8
public Type x(
   Type a, 
   Type b, 
   Type c, 
   Type d, 
   Type e){
 
}

P.S. S'il n'y a pas de conventions sur ce sujet, qu'est ce que vous vous employez comme façon de faire ?

[le y@m's]

Je ne sais pas s'il y a une convention sur le sujet, mais perso cela dépend du nombre de paramètres ou de leur type.
J'essaye de choisir la disposition que me paraît la plus clair (la moins bord****** ) au premier coup d'oeil.

Code :

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

1
2
3
4
5
6
7
8
9
public void method(int a, int b) {
    // ...
}
 
public void method( int i1, String s1, 
                    int i2, String s2,
                    int i3, String s3) {
     // ...
 }

Pour résumer je fait plutôt du cas par cas.

[mrkzed]

Idem, je ne suis pas expert mais je ne crois pas qu'il existe une convention la dessus.
Personnellement j'utilise les 2 manieres (aussi bien pour les appels que pour les declarations), en fonction de la taille des parametres, histoire que le code reste lisible.

Code :

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

1
2
3
4
 
String result = method(parameter.substring(0, parameter.length() - 4),
                       parameter.substring(parameter.length() - 4));
String result = method("java", ".exe");

[sironimo]

Salut

on les mets seulement sur plusieurs lignes si on a pas la place en largeur pour tous les mettre les uns à cotés des autres.

Pour ma part, j'utilise plutôt cette méthode. Je rentre les paramètres sur la même ligne et s'il y en a trop, je vais à la ligne. Pourtant, dans la javadoc, je déclare chaque paramètre sur une ligne disctincte donc pas très cohérent

Ensuite je préfèrerais ces formes

Code :

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

1
2
3
4
5
6
7
z = x(
  a,
  b,
  c,
  d,
  e
  );

Histoire d'encore mieux séparer. Ca me rappelle aussi le codage PHP de cette façon. Cela permet aussi de mieux commenter les paramètres pour les gens n'utilisant pas la javadoc.

Code :

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

1
2
3
public Type x(Type a, Type b, Type c, Type d, Type e){
 
}

Dans ce cas-là, j'essaie principalement de regrouper les paramètres par type. Par exemple, si j'attends 3 String et 2 Integer en paramètre, je vais ordonner de façon à pas mélanger les types, question d'esthétisme

Voilà mon opinion et mes "mauvaises?" habitudes

[sinok]

Mon opinion c'est que j'en ais pas et que je laisse Jalopy tout faire à ma place

Sinok, assisté primaire

[sironimo]

sinok terrible. M'enfin je rigole mais je vais me le prendre je pense

[g_rare]

Sinon il y a les conventions de code de Sun (voir chapitre 4 "Indentation") suivies par défaut par Checkstyle.

[Baptiste Wicht]

Dans les conventions de Sun, il ne disent rien là-dessus. Par contre, dans les exemples avec Jalopy, il met chaque paramètre sur une ligne.

J'hésite un peu, mais je pense que je vais rester comme actuellement, avec tout sur une ligne et un retour à ligne s'il faut.