Ajouter des numéros de ligne au début de chaque ligne dans un fichier en utilisant C

vous avez un programme qui compte le nombre de lignes dans le fichier, vous devez le changer pour qu'au lieu d'afficher le nombre à la fin, vous deviez faire écho au contenu du fichier

(par exemple en utilisant putchar (c); après chaque getc ()) et

à chaque fois que vous voyez un \ n (et aussi au début du fichier) vous devez imprimer un nombre avec des zéros non significatifs, puis un espace. printf ("% 06d", ln); est probablement ce que vous voulez pour cela.

Vous avez un grand nombre de petits problèmes à résoudre. Premièrement, sauf si vous utilisez un compilateur non conforme, les déclarations conformes pour main sont int main (void) et int main (int argc, char ** argv) (que vous verrez écrit avec l'équivalent char * argv [] ). Voir: C11 Standard §5.1.2.2.1 Démarrage du programme p1 (brouillon n1570) . Voir aussi: Que doit renvoyer main () en C et C ++?

Ensuite, c doit être de type int pas de type char pour correspondre à EOF , par exemple

$ ./bin/linenos ../dat/captnjack_noeol.txt
000001 This is a tale
000002 Of Captain Jack Sparrow
000003 A Pirate So Brave
000004 On the Seven Seas.

Don ' t coder en dur les noms de fichiers ou utiliser des nombres magiques . Soit passez le nom de fichier comme argument à main () , soit demandez-le dans votre programme. Vous pouvez facilement prendre un nom de fichier à ouvrir ou à lire à partir de stdin par default avec l'opérateur ternaire comme suit:

$ cat ../dat/captnjack.txt
This is a tale
Of Captain Jack Sparrow
A Pirate So Brave
On the Seven Seas.

Enfin pour votre code, puisque vous voulez Préfixer chaque ligne avec le numéro de ligne, vous devez sortir le numéro de ligne de la première ligne Avant de sortir les caractères de cette ligne ( et la même chose pour chaque ligne suivante). Vous pouvez le faire simplement en produisant d'abord le nombre (en utilisant le spécificateur de conversion "% 06zu" avec les modificateurs '0' pour afficher les zéros non significatifs et le champ largeur 6 pour correspondre à votre format affiché). Notez également que le type de votre compteur ln est passé de int à size_t , le type recommandé pour les compteurs en C (vous ne pouvez pas nombre de lignes négatif).

En associant ceci à l'utilisation du dernier caractère pour permettre de vérifier le '\ n' avant de sortir vos caractères , vous pouvez faire:

#include <stdio.h>

int main (int argc, char **argv) {

    int c, last = 0; /* c must be type int, not char to match EOF */
    size_t ln = 1;      /* use size_t for counters */
    /* use filename provided as 1st argument (stdin by default) */
    FILE *fp = argc > 1 ? fopen (argv[1], "r") : stdin;

    if (!fp) {  /* validate file open for reading */
        perror ("file open failed");
        return 1;
    }
    printf ("%06zu ", ln++);            /* output line 1 number */
    while ((c = getc(fp)) != EOF) {     /* read each character */
        if (last)                       /* test if last set */
            putchar (last);             /* output all characters */
        if (last == '\n')               /* test if last is newline */
            printf ("%06zu ", ln++);    /* output next line number */
        last = c;                       /* set last to c */
    }
    putchar (last);                     /* output final character */
    if (last != '\n')                   /* check POSIX eof */
        putchar('\n');                  /* tidy up with newline */
    if (fp != stdin)                    /* close file if not stdin */
        fclose (fp);

    return 0;
}

( remarque: la vérification de if (last! = '\ n') après avoir quitté la boucle vérifie la présence d'une ligne POSIX se terminant sur la dernière ligne, et sinon, vous devez afficher manuellement un '\ n' afin que votre programme soit compatible POSIX)

Exemple de fichier d'entrée

    /* use filename provided as 1st argument (stdin by default) */
    FILE *fp = argc > 1 ? fopen (argv[1], "r") : stdin;

Exemple d'utilisation / de sortie

    int c, last = 0; /* c must be type int, not char to match EOF */

Pour numéroter les lignes, ou pour modifier n'importe quelle partie du fichier d'ailleurs, il est préférable de charger le fichier en mémoire sous forme de chaîne, de manipuler cette chaîne (dans ce cas, d'ajouter des numéros de ligne) puis de créer un espace nouveau fichier. Copiez la chaîne modifiée dans le nouveau fichier, puis renommez-la avec le chemin de l'ancien, afin de l'écraser.

/* fputs(numbered, fpNumbered); */ printf("%s", numbered);

Si vous voulez seulement imprimer les lignes numérotées sans même toucher le fichier, puis vous remplacez l'instruction fputs dans le code ci-dessus par un appel à printf:

/* fp: File whose lines are to be numbered 
   pathOld: Its path */
void giveLineNos(FILE *fp, const char *pathOld)
{
    char line[MAXLINE]; /* line that is read in */
    char numbered[MAXLINE + 8]; /* line that will include its number */
    int i;

    char tempName[L_tmpnam];
    tmpnam(tempName);  /* generate filename guaranteed to be unique */

    FILE *fpNumbered = fopen(tempName, "w"); /* create temporary file onto 
                              which we will write; error checking omitted */

    for (i = 0; fgets(line, MAXLINE, fp) != NULL; ++i) {
        /* write out the line number followed by its contents */
        snprintf(numbered, MAXLINE + 8, "%d: ", i + 1);
        strcat(numbered, line);

        fputs(numbered, fpNumbered); /* save line to file */
    }

    rename(tempName, pathOld);  /* rename the temporary file to that of 
                                    the old one, thereby overwriting it */
    fclose(fpNumbered);
    fclose(fp);
}

(mettant une nouvelle ligne après % s est erronée car fgets met déjà une nouvelle ligne dans le tampon ligne , qui est ensuite transférée dans numérotée .)

Notez également que votre main n'est pas standard. Utilisez int comme type de retour, et non void.

Je recommande de ne pas incrémenter le nombre de lignes avec chaque '\ n' et d'incrémenter à la place le nombre de lignes chaque fois qu'un caractère est lu après un ' \ n '. C'est aussi à ce moment que le code doit imprimer le numéro de ligne

unsigned long ln = 0;
int prev = '\n';
int c;  // Use int here, not char
while((c=getc(fp))!=EOF) {
  if (prev == '\n'){
    printf("%06lu ", ++ln);
  }
  putchar(c);
  prev = c;
}
if (prev != '\n') {
  putchar('\n');  // print a \n for input that lacks a final \n
}
printf("lines num: %lu\n", ln);

Remarquez que ce qui précède fonctionne bien lorsqu'il y a 0 ligne (pas de données) et lorsque l'entrée ne se termine pas par un '\ n '.

Une faiblesse courante de l'utilisation de fgets () est 1) lorsque la longueur de la ligne dépasse le tampon d'entrée, un traitement spécial est nécessaire 2) lorsque l'entrée contient de manière inattendue un caractère nul , ce qui nuit à la gestion des chaînes tropicales. getchar () n'a pas ces problèmes.