C - liest eine Zeile von der Standardeingabe mit fgets() in der Funktion

Ich versuche, liest eine Zeile von der Standardeingabe mit fgets(), will ich mit fgets() in meiner Funktion, die ich denke, dass ist das problem. Der string kann max 1024 chars lang. Wenn ich diesen code ausführen, ich erhalte die Meldung "Segmentation fault (core dumped)"

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

#define MAX_SIZE 1025

void print_fgets();

int main()
{
    print_select();
    return 0;
}

void print_select()
{
    char *str;
    int length;

    while (fgets( str, MAX_SIZE, stdin)!=NULL)
    {
        length=strlen(str);

        if (length==MAX_SIZE-1 && str[length-1]!='\n')
        {
             printf("Error, line overeached buffer!\n");
             return 1;
        }

        if (str[length-1]=='\n')
             str[length-1]='\0';
        printf("%s\n", str);
    }
}
  • wo werden Sie gehört?
  • Sie wusste nicht, reservieren Sie Speicher für str. das ist, warum.
  • Wie reservieren Sie Speicher? Ich bin neu in C.
  • siehe malloc () - Funktion (und deriviatives)
  • Für ein fixed-size-buffer, die man in der Regel weist s mit automatischer oder globalen Speicher: char s[MAX_SIZE]; oder static char s[MAX_SIZE]; wenn mehrere threads sind nicht ein Problem. Ihre Länge überprüft, sind auch völlig unnötig. fgets garantiert, dass Sie nicht den überlauf des Puffers.
  • Hinweis: In seltenen Fällen (Lesen von einer eingebetteten '\0'), str[0] == 0, dann str[length-1] außerhalb str. Empfehlen size_t len = strlen(str); if (len > 0 && str[len-1] == '\n') str[--len] = '\0';

InformationsquelleAutor Krop | 2014-11-03
  1. 0

    Das problem ist, dass Sie versuchen zu schreiben, an der Stelle, die die str - Zeiger.
    Anfangs wird es ein Müll-Adresse (wegen char *str nicht initialisiert werden).

    Können Sie versuchen, einen stack-basierten Lösung anstelle von ändern:

    /* This is just a pointer */
char *str;

    zu:

    /* This is a character array (on the stack) of size MAX_SIZE, initialized to zero */
char str[MAX_SIZE] = {0};

    Oder wenn Sie möchten, reservieren Sie Speicher dynamisch für das array, tun Sie dies:

    char *str;
str = calloc(1, MAX_SIZE);
if (str == NULL) {
    /* Failed to allocate memory */
}

... Your code

free(str);  /* You should always free dynamically allocated memory when you are done with it */
str = NULL; /* It's a good practice to point your pointer to NULL after it's free'd */

    Vergessen Sie nicht Dinge wie array-Indizes beginnen mit 0 und geht zu MAX_SIZE - 1 (in deinem Fall) und NUL-termination (strings müssen mit ihm enden).

    • Die Initialisierung ist unncessary in diesem Fall. fgets auch garantiert NULL-Terminierung.
    • Sicher, aber ich denke, es ist eine gute übung für Anfänger, um immer initialisieren auf null, um Probleme zu vermeiden. Nächsten minute kann es nicht fgets genannt, das ist aber etwas anderes, dass keine Garantie für die null-Terminierung zum Beispiel.
    InformationsquelleAutor Jite
  2. 0
    #include <stdio.h>
#include <stdlib.h>
#include <string.h>

#define MAX_SIZE 1025

int print_select();  /* Use correct name (instead of print_fgets()) */

int main()
{
    print_select();
    return 0;
}

int print_select()  /* Fix. Dhould return int if you have a return <int> statement. */
{
    char str[MAX_SIZE];  /* Initialize static memory. */
    int length;

    while (fgets( str, MAX_SIZE, stdin)!=NULL)
    {
        length=strlen(str);
        if (length==MAX_SIZE-1 && str[length-1]!='\n')
        {
             printf("Error, line overeached buffer!\n");
             return 1;
        }
        if (str[length-1]=='\n')
        {
             str[length-1]='\0';
        }
        printf("%s\n", str);
    }
    return 0;  /* Function may not be returning an int. Return it in those cases. */
}
    • Es gibt keine Notwendigkeit, fügen Sie den gesamten code wieder. Verwenden Sie die verfügbaren makro MAX_SIZE statt hardcoded 1025.
    • Ja, tut mir Leid. Ich bin irgendwie neu.
    InformationsquelleAutor Lucas Godoy
Schreibe einen Kommentar