Indique si const char * et strdup ont la même fonction lorsqu'ils sont utilisés avec getopt dans C?

Question

Dans l'extrait de code ci-dessous, puis-je remplacer char * par const char * et supprimer l'appel de la fonction strdup() et prendre directement la valeur optarg définie par getopt()? Je suis conseillé d'utiliser const char * pour ignorer l'utilisation de la fonction strdup. Appréciez l'aide à l'avance.

/* Code Snippet */ 
char *dir = NULL; 
char *bld = NULL; 
int chr; 

while ((chr = getopt(argc, argv, "d:a:b:f:")) != -1) { 

switch (chr) { 

    case 'd': //Directory parameter 
    dir = strdup(optarg); 
    if (dir == NULL) { /*Error*/ } 

    case 'b': //Build parameter 
    bld = strdup(optarg); 
    if (bld == NULL) { /*Error*/ } 
    ...other code... 
} 
} 

Je ne comprends vraiment pas la nécessité de le faire.

Modifier: Merci pour les réponses. C'était vraiment utile. J'ai modifié le code à const char * et j'ai ignoré l'appel de strdup.

Merci, Liju

Answer 1

Comme optarg est déclaré comme 'extern char *optarg;', vous n'avez pas besoin d'utiliser 'const char *', mais il est recommandé de ne pas modifier les arguments de votre programme. Cela dépendra plus, je suggère, si les fonctions que vous appelez avec vos variables dir et bld sont const-correctes. Si elles sont votre code, vous pouvez les faire ainsi; sinon, vous pouvez trouver plus simple de garder les variables non-const. Votre appel à 'strdup()' (qui est standard dans POSIX, mais pas dans C - comme dans ISO/CEI 9899: 1999) est inutile.

Answer 2

Si je comprends bien (http://linux.die.net/man/3/optarg) est que les points de OPTARG au contenu du tampon de argv de sorte que le strdup n'est pas nécessaire. La suggestion d'utiliser constchar * semble être une idée raisonnable.

Answer 3

Les deux sont de bons conseils. Vous ne voulez probablement pas modifier dir et bld, donc les déclarer comme const char * au lieu de char * est utile dans tous les cas parce que le compilateur détectera plus d'erreurs de codage. En se débarrassant de la strdup vous évitez juste une copie inutile (en fonction de ce que vous faites avec dir ensuite), mais cela crée un alias à votre optarg.