passe par adresse, scanf, de-référencement

Question

j'ai écrit un programme utilisant le concept suivant:

Je crée un entier x passe par adresse à une fonction, avec un nom de fichier, ladite fonction ouvre le fichier si elle est disponible, balaye la première ligne et définit la valeur que pX points égal à la ligne analysée.

Seulement ça ne marche pas, et je ne vois pas ce que je fais de mal. Pour autant que je peux dire le code ci-dessous est généralement comment on pourrait l'accomplir, mais je ne suis pas sûr si je n'utilise pas scanf() correctement avec le pointeur ou quoi.

void foo() { 
    char input[20] = "test.txt"; 
    int x = 1; 
    bar(input, &x); 
} 

void bar(char *fileName, int *pX) { 
    FILE *fp = fopen(fileName, "r"); 
    char *buffer = malloc(15 * sizeof(int)); 
    fgets(buffer, 15, fp); 
    scanf(buffer, "%d", *pX); 
    free(buffer); 
    fclose(fp); 
} 

Answer 1

ligne de changement:

scanf(buffer, "%d", *pX); 

à:

sscanf(buffer, "%d", pX); 

Vous devez fonction sscanf pour ce que vous essayez de faire. Les deux scanf et sscanf prennent un pointeur comme argument. pX est de type int *, donc un pointeur sur un int et devrait fonctionner pour vous. Ce que vous passez, *pX, est le contenu de ce pointeur, en d'autres termes un int.

En outre, la ligne de changement:

char *buffer = malloc(15 * sizeof(int)); 

à:

char *buffer = malloc(15 * sizeof(char)); 

ou simplement:

char *buffer = malloc(15); 

et toujours check the result of malloc:

if (buffer == NULL){ 
    ... 
} 

Answer 2

Tout d'abord, il n'y a pas de passage par référence en C, les paramètres de fonction sont tous passés par valeur. En passant un pointeur sur des données, nous faisons la simulation de l'obtention du même effet que le passage par référence, mais cela ne signifie pas que C a un concept de passage par référence.

Cela dit, le problème semble être

scanf(buffer, "%d", *pX); 
        ^^^^ 

où

• la syntaxe actuelle est invalide et invoque undefined behavior. Vous avez probablement besoin de sscanf().

• px est déjà un pointeur vers int. Passer px sera correct et suffisant.

Morale de l'histoire: activer les avertissements du compilateur et y tenir compte. Ils sont là pour une raison.Avec les avertissements appropriés sont activés, vous devriez voir quelque chose comme

avertissement: le format %d attend un argument de type int *, mais l'argument 3 a le type int [-Wformat=]

Enfin,

• Toujours vérifiez la valeur de retour de fopen() pour réussir avant d'utiliser le pointeur de fichier.
• Vérifiez la valeur de retour de scanf() pour garantir une numérisation réussie.
• Vérifier la valeur de retour de fgets() pour assurer le succès

... en gros, vérifier la valeur de retour de tous les appels de bibliothèque pour vous assurer qu'ils ont travaillé comme prévu.

Answer 3

Vous utilisez scanf() correctement: soit utiliser directement scanf pour analyser l'entrée standard ou utiliser sscanf() pour analyser la chaîne lue par fgets(). En outre, pX est déjà un pointeur vers int, qui est ce que sscanf() attend pour stocker la valeur int il convertit, passez directement: sscanf(buffer, "%d", pX);

Voici une version modifiée:

int bar(const char *fileName, int *pX) { 
    char buffer[15]; 
    FILE *fp = fopen(fileName, "r"); 
    int success = 0; 

    if (fp != NULL) { 
     fgets(buffer, sizeof buffer, fp); 
     if (sscanf(buffer, "%d", pX) == 1) 
      success = 1; 
     fclose(fp); 
    } 
    return success; 
} 

void foo(void) { 
    int x = 1; 
    bar("test.txt", &x); 
    /* do something with x */ 
} 

Notes:

• allocation buf est inutile, juste en faire un tableau local avec stockage automatique.

• char *buffer = malloc(15 * sizeof(int)); est incorrect: vous allouez de l'espace pour 15 int au lieu de 15 caractères, qui ont une taille de 1 par définition. Utilisez la taille du type de destination afin d'éviter toute incohérence:

  char *buffer = malloc(15 * sizeof(*buffer)); 
  

• toujours vérifier la valeur de retour de malloc() pour éviter un comportement non défini potentiel.

• lecture de fp sans vérifier si fopen réussi a un comportement potentiel indéfini.
• le contenu du tableau pointé par filename n'est pas modifié, en fait un const char *.
• il peut être utile pour bar de renvoyer un indicateur de réussite.
• activer plus d'avertissements au moment de la compilation: gcc -Wall -Wextra -Werror ou clang -Weverything -Werror aurait pu attraper les erreurs dans scanf.