Lecture de fichiers passés comme arguments de ligne de commande

Je suis en train d'analyser un fichier texte, mais jusqu'à présent, mon programme ne semble pas être lu correctement.

#include <stdio.h>


int main(int argc, char *argv[])
{
    FILE *fr; //file pointer
    int buildingFloors = 1;

    printf("sanity check\n");  

    fr = fopen (argv[0], "r");
    fscanf(fr, "%d", &buildingFloors );

    printf("%d\n", buildingFloors);

    fclose(fr);

    return 0;
}

J'ai compiler le programme et l'exécuter sur ma redhat linux de la machine avec la commande suivante:

./sjf file.text

fichier.le texte est un document texte avec un "4" comme premier caractère. Donc je m'attends à ma sortie de la

sanity check
4

Cependant, lorsque je lance mon programme, j'ai plutôt obtenir

sanity check
1

Qui implique que fscanf n'avait pas bien lu le premier caractère -- 4. J'ai un erreur de syntaxe qui empêche l'attend fonctionnalité de code? Suis-je censé scanf pour un personnage, et puis convertir un int en quelque sorte?

OriginalL'auteur Raven Dreamer | 2011-02-25

  1. 4

    Une chose qui vient immédiatement à l'esprit est que le programme args inclure le nom de l'exécutable que le premier élément

    argv[0] est "sjf"

    argv[1] est "fichier.texte"

    de sorte que vous devriez être en utilisant 

    fr = fopen (argv[1], "r");

    Rappelez-vous lors du débogage de toujours essayer et de limiter le problème, si vous connaissez l'emplacement de l'erreur de la cause devient souvent évidente ou au moins investigatable.

    Dans ce cas, vous devriez vérifier argc >= 2, imprimer argv[1] pour vous assurer que vous êtes en essayant d'ouvrir le bon fichier, puis vérifiez également que le fichier a été ouvert avec succès.

    Enfin vérifier le fscanf codes d'erreur de voir que fscanf était capable de lire le numéro.

    Il doit vérifier si argc == 2 ainsi.
    Buyukliev: très vrai, je vais l'ajouter à la réponse

    OriginalL'auteur Akusete

  2. 5

    argv[0] est le nom du programme (./sjf dans votre cas), de sorte que vous êtes en train de lire dans votre propre exécutable du programme. Utilisation argv[1] au lieu d'obtenir le premier véritable programme d'argument.

    OriginalL'auteur Adam Rosenfield

  3. 0

    Votre code est clair et simple, mais il y a une chose importante manquant: erreur de manipulation.

    Ce qui se passe si le fichier que vous voulez ouvrir n'existe pas? fopen renvoie la valeur NULL dans ce cas.

    Ce qui se passe si le fichier ne commence pas par un chiffre? fscanf retourne le nombre de champs qui ont été correctement lire, alors vous devez vérifier que la valeur de retour est au moins 1.

    Vous avez besoin pour une certaine façon de gérer ces cas, probablement par l'impression des messages d'erreur et quitter le programme. Lorsque vous faites cela, assurez-vous d'inclure les informations pertinentes dans les messages d'erreur. Ensuite, vous trouverez le bug que les autres réponses ont déjà mentionné.

    OriginalL'auteur Roland Illig