Discussion :

[tribaleur]

Bonjour,

J'aimerai savoir qu'elle indentation vous préférez pour votre code. Si possible je veux bien savoir pourquoi. Et si aucune de mes propositions ne vous convient je veux bien un exemple de ce que vous utilisez dans ces cas

1. Var declaration

Code :

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

1
2
3
4
5
6
7
8
9
10
11
var myVar1 = "v1";
var mySecondVar = randomFunction("v2");
var aVarWithLongName = "v3";
var myVar 4 = true;

or

var myVar1             = "v1";
var mySecondVar        = randomFunction("v2");
var aVarWithLongName   = "v3";
var myVar 4            = true;

2. Function parameters

Code :

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

1
2
3
4
5
6
7
8
9
myFunction1("p1", randomFunction("value1"), true);
myFunction1("parameter2", "value1", false);
myFunction1("param3", randomFunction("value3"), true);

or

myFunction("p1",         randomFunction("value1"), true);
myFunction("parameter2", "value1",                 false);
myFunction("param3",     randomFunction("value3"), true);

3. IfElse

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
if(myFunction("value1"){
    value = "value1";
}else{
    value = "";
}

or

if(myFunction("value1"){ value = "value1";}
else{ value = ""; }

or

if(myFunction("value1")
{
    value = "value1";
}
else
{
    value = "";
}

4. Function declaration

Code :

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

1
2
3
4
5
6
7
8
9
10
public void myFunction (String myString, Boolean myBool){
    ...
}

or

public void myFunction (String myString, Boolean myBool)
{
    ...
}

5. Generic object

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
myVar = {
    "property1" : "random string",
    "p2" : true
};

or

myVar = {
    "property1" : "random string",
    "p2"        : true
};

or

myVar = { "property1" : "random string",
          "p2" : true };

Merci pour vos avis

[tatayo]

Bonjour,
Pour ma part:
Déclaration:

Code :

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

1
2
3
4
5
var myVar1 = "v1";
var mySecondVar = randomFunction("v2");
var aVarWithLongName  = "v3";
var myVar 4 = true;

J'aimerai bien aligner les valeurs, mais IntelliSens de Visual Studio ne me laisse pas faire. A chaque mise ne page, tout est "collé" (mais avec des espaces quand même !).


Paramètres de fonction:

Code :

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

1
2
3
4
myFunction1("p1", randomFunction("value1"), true);
myFunction1("parameter2", "value1", false);
myFunction1("param3", randomFunction("value3"), true);

If et compagnie:
Aucune des proposition.

Code :

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

1
2
3
4
5
6
7
8
9
if(myFunction("value1")
{
    value = "value1";
}
else
{
    value = "";
}

J'aime bien avoir l'accolade ouvrante sur une nouvelle ligne.

Déclaration de fonction:
En gros la même mise en page que pour les blocs IF ELSE.

Code :

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

1
2
3
4
5
public void myFunction (String myString, Boolean myBool)
{
    ...
}

Objet générique:

Code :

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

1
2
3
4
5
6
myVar = 
{
    "property1" : "random string",
    "p2" : true
};

Comme pour la déclaration de variable, j'aimerai bien aligner les valeurs, mais Visual Studio n'est pas d'accord.

Tatayo.

[tribaleur]

Merci pour ta réponse !

Pour le ifElse c'était ma troisième proposition mais on la vois pas bien sans scroller

Pas contre je suis surpris car de mon côté je n'ai jamais de souci sur Visual Studio. Par contre je ne génère pas l'indentation automatiquement sinon en effet dans certains cas ça me supprime les tabulations en milieu de ligne.

D'autres volontaires pour répondre ?

[gangsoleil]

1. Var declaration

Code :

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

1
2
3
4
var myVar1             = "v1";
var mySecondVar        = randomFunction("v2");
var aVarWithLongName   = "v3";
var myVar 4            = true;

Dans la mesure du possible, je n'initialise pas mes variables à la déclaration, ça permet de vérifier qu'on les assigne dans le code -- mais parfois on veut une valeur par défaut. Dans ce cas, j'essaye de les aligner.

2. Function parameters

Code :

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

1
2
3
myFunction1("p1", randomFunction("value1"), true);
myFunction1("parameter2", "value1", false);
myFunction1("param3", randomFunction("value3"), true);

Un appel de fonction, c'est au milieu du code, je trouve ça plus logique de passer les paramètres sans alignement particulier.

3. IfElse

Code :

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

1
2
3
4
5
6
7
8
if(myFunction("value1")
{
    value = "value1";
}
else
{
    value = "";
}

Je trouve ça plus lisible comme ça, mais c'est vrai que pour juste une courte ligne c'est parfois un peu overkill.
En cas d'utilisation de l'opérateur ternaire, je mets tout sur une ligne.

4. Function declaration

Code :

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

1
2
3
4
public void myFunction (String myString, Boolean myBool)
{
    ...
}

5. Generic object

Code :

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

1
2
3
4
myVar = {
    "property1" : "random string",
    "p2" : true
};

Ce qui est le plus important :
- se mettre d'accord au sein de l'équipe
- avoir une unité dans le code
- régler correctement tous les éditeurs

[tribaleur]

Merci pour ce retour.

Un appel de fonction, c'est au milieu du code, je trouve ça plus logique de passer les paramètres sans alignement particulier.

Est-ce que le fait que ce soit au milieu du code à une importance ? La lisibilité du code ne passe-t'elle pas avant ?

Dans tous les cas je suis d'accord avec toi, l'important c'est d'être iso au sein de l'équipe