Discussion :

[gregoire.stein]

Bonjour, bonsoir, j'aimerais avoir vos avis concernant la position des opérateurs * et & en C++.

J'utilise toujours : int& integer;

J'ai remarqué aussi que dans "The C++ Programming language Fourth Edition" de Bjarne Stroustrup, Stroustrup écrit aussi int& integer.

Est-ce une question de style ?

Qu'en pensez-vous ?

[Neckara]

Bonjour,

Ce n'est pas la même utilisation.

• un pointeur peut être nullptr, pas une référence ;
• un pointeur peut changer de valeur, pas une référence qui référencera toujours le même objet ;

On préférera souvent la référence au pointeur, il se comporte comme un "alias" et n'est pas toujours implémenté sous forme de pointeurs (optimisations du compilateur).
Il évite aussi de devoir tester si le pointeur est nullptr ou non.

Ensuite, pour les pointeurs, on préfère très souvent utiliser des containers (std::vector, std::map, etc.), des pointeurs intelligents (std::shared_ptr, std::unique_ptr, std::weak_ptr), voir même ne pas utiliser de pointeurs du tout.

[Neckara]

Arf, je me suis fais avoir en lisant trop vite, j'ai loupé deux petits mots "la position" (je n'ai pas vu le sondage) .

Donc moi j'utilise Type & variable.
Je trouve que c'est un plus lisible que Type& variable.

Par contre je déconseillerais Type &variable car cela n'a pas trop de sens à mes yeux de coller '&' au nom de la variable alors qu'il fait parti du type de la variable.

C'est donc surtout une question de style.

[ternel]

Justement, & et * sont des marqueurs de la variable.

c'est visible dans des codes comme:

Code :

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

1
2
3
4
 
int source = 2;
int* a, b;
int& one=source, two =source;

Ici, a est un int*, b un int, one un int& (sur source), et two un int.

Par contre, effectivement, pour déclarer une seule variable, je place la référence sur le type.

Cela dit, la majorité des références sont des arguments de fonctions, et j'adopte le deux espaces:

Code :

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

1
2
void swap( int & a, int & b);
void export(int const& a, int const& b);

Je préfère toujours l'étoile sur la variable à cause de cette grosse subtilité, qui veut que int* a, b; se lise "sont des int le pointé de la variable a, et la variable b"

Ca m'évite des erreurs.

[Neckara]

Justement, & et * sont des marqueurs de la variable.

Ce que je veux dire c'est qu'on dit que ptr est un int *, on ne dit pas que c'est un int. De plus, '*' ne fait pas parti du nom de la variable, on parle de 'ptr', pas de '*ptr'.
C'est pour cela que je sépare le nom de la variable du "type" par un espace.

c'est visible dans des codes comme:

Code :

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

1
2
3
4
 
int source = 2;
int* a, b;
int& one=source, two =source;

Ici, a est un int*, b un int, one un int& (sur source), et two un int.

Donc en effet, il vaudrait mieux éviter de coller '&' et '*' au "type" pour éviter la confusion.
Même si on essaye généralement d'éviter de déclarer plusieurs variables sur la même ligne.

Par contre, effectivement, pour déclarer une seule variable, je place la référence sur le type.

Comme on ne change pas la méthode d'indentation au milieu d'un code, je ne pense pas qu'il soit judicieux d'utiliser les "trois méthodes" dans le code en fonction des circonstances.
Ne vaut-il mieux pas en choisir une et s'y tenir ?

Code :

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

1
2
void swap( int & a, int & b);
void export(int const& a, int const& b);

Par contre, vous collez le & au const. Pourquoi?

[Ehonn]

int const& a

J'aime pas cette notation car ce qui est const est int, pas la référence (une référence "est toujours constante").

Personnelement, j'utilise cette notation :

Code :

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

1
2
3
T &
T const &
char const * const

Et m'interdit les déclarations sur la même ligne.

[ternel]

Je vais observer un peu mes codes, pour voir ce que je fais effectivement.
Je mets toujours const& pour les arguments, probablement parce que dans ma 'tite tête, c'est une "référence-constante-en-un-seul-mot".

Pour les déclarations, je n'ai quasiment jamais deux variables locales qui soient des pointeurs, et je ne déclare jamais un pointeur avec un pointé.
(pas de int *a,b;)

[Bousk]

Code :

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

int const& a

J'aime pas cette notation [...]

Personnelement, j'utilise cette notation :

Code :

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

T const &

Faudrait te mettre d'accord avec toi-même