Discussion :

[pcdwarf]

Bonjour,

Je trouve assez souvent des morceaux de code dans ce genre

Code :

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

fd = open(std::string("/pathto/file").c_str(), O_RDWR);

considérant le prototype de open

Code :

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

int open(const char *pathname, int flags);

Je ne comprends pas l’intérêt de cette formulation par rapport à écrire simplement

Code :

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

fd = open("/pathto/file", O_RDWR);

A priori c'est plus lourd pour la machine (sauf optimisation faite par le compilateur) et surtout c'est plus complexe à relire.
Cependant, comme j'ai trouvé cette formulation dans de nombreux codes "officiels" différents, je suppose qu'il y a une raison.

quelqu'un peut-il m'éclairer.
Est-ce une bonne pratique ? (j'en doute)
et si oui, pourquoi ?

[ternel]

En effet, il n'y a pas d'avantage, puisqu'on passe par une string temporaire.

[Iradrille]

Hello,

Dans ce cas précis je vois vraiment pas l’intérêt.

Code :

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

1
2
3
4
5
6
std::string path = "/path/to/file";
fd = open(path.c_str(), O_RDWR); // ok, bien que std::ifstream serait probablement préférable,
// et qu'un char const* fait aussi bien le boulot qu'un std::string, mais pourquoi pas
 
// ou
fd = open("path/to/file", O_RDWR); // dans ce cas, un std::string n'a aucune utilité

[dalfab]

Bonjour,

Je pense que souvent il y besoin d'ajouter un path à un nom de fichier d'où l'intérêt de passer par un std::string,
et puis il ne reste qu'une chaine complète otant tout l'intêrêt du std::string.

[koala01]

Salut,

Dans le cas présent, je pense que l'on peut considérer que l'auteur du code a quelques peu tiré les ailes des mouches. Sauf si, pour une raison ou une autre, il s'attend à ce que le code finisse tôt ou tard sous une forme proche de

Code :

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

1
2
3
std::string filename("/path/to");
/* quoi qu'il se passe ici, on complète le nom du fichier */
fd=open(filename.c_str(),O_RDWR);

Mais, si le nom du fichier est écrit en dur, il n'y a absolument aucun intérêt