C Anfänge: User input string, fgets(), max. Arraygröße

allgemein erstmal scanf_s verwenden - schon mal sicherer

z.B.

1

char string[4];

2

scanf("%3s", string);

oder

1

char string[4];

2

scanf("%*s", sizeof(string)-1, string);

fgets ist nichts Magisches, die benutzt auch nur fgetc/getchar um eine 
Zeile einzulesen.  Mach dir halt eine eigene Routine, z.B.:

1

// Get one line and store first size-1 chars in buf.

2

// Returns buf or NULL on EOF.

3

char *ngets(char *buf, int size)

4

{

5

    int i, c;

6

7

    i = 0;

8

    while ((c = getchar()) != EOF && c != '\n')

9

        if (i+1 < size)  // +1 for the '\0' below

10

            buf[i++] = c;

11

12

    if (i < size)

13

        buf[i] = '\0';

14

15

    return (c == EOF && i == 0) ? NULL : buf;

16

}

> Warum i==0? Was passiert bei einem I/O-Error oder wenn die letzte Zeile
> nicht mit '\n' aufhört?

Die Routine verhält sich wie ANSI gets: Wenn schon Zeichen gelesen 
wurden, werden die auf jeden Fall zurückgegeben; NULL gibt's nur bei EOF 
als erstem Zeichen.

>> Die Routine verhält sich wie ANSI gets
>
> Nichts ist so unbrauchbar, dass es nicht als schlechtes Beispiel dienen
> könnte...

Nicht das ich die stdio-Routine gerne verteidige, aber dieses Verhalten 
bzgl EOF ist in Ordnung.  Es stellt sicher, dass keine Zeichen verloren 
gehen (z.B. die von dir genannte "letzte Zeile ohne \n" wird geliefert). 
Erkauft wird das damit, dass ein EOF oder I/O-Error evtl erst beim 
nächsten Aufruf gemeldet wird.

...

Oh, gerade mal im avr-libc-1.8.0 nachgeschaut: da wird es falsch 
behandelt, ein EOF macht augenblicklich ein "return NULL;" - die letzte 
Zeile ohne \n geht verloren.

Bauform B. schrieb:
> Nichts ist so unbrauchbar, dass es nicht als schlechtes Beispiel dienen
> könnte... Die Steigerung davon sind Funktionen, die mal und mal nicht
> die '\0' mitliefern.

Unter welchen Umständen wird denn kein '\0' angehängt?

LG, Sebastian

foobar schrieb:
> if (i < size)
>         buf[i] = '\0';
>
>     return (c == EOF && i == 0) ? NULL : buf;

reicht da nicht einfach:

1

      buf[i] = '\0';

2

3

     return (i == 0) ? NULL : buf;

> reicht da nicht einfach: [...]

Das "if" schützt vor size<=0 und ohne das c==EOF würde auch eine 
Leerzeile ein NULL generieren.

foobar schrieb:
> Das "if" schützt vor size<=0

ach so, klar ;-)

foobar schrieb:
> ohne das c==EOF würde auch eine
> Leerzeile ein NULL generieren.

Ja, kompiziert.

Dieses "EOF" hat ja was mit Files zu tun. Und die Eingabe kann ja per 
Tastatur aber auch per File erfolgen.

1

./my_program < input.txt

Und falls "input.txt" leer ist, dann gibt es Probleme. Dann werden die 
Ausgaben

1

   ngets(name, NAME_LEN);                                        

2

   printf("%s\n", name);

und

1

    printf("%s\n", ngets(name, NAME_LEN));

untershiedlich sein. Wegen der letzten drei Zeilen.

> Dieses "EOF" hat ja was mit Files zu tun.

Kann evtl aber auch über die Tastatur simuliert werden, bei Linux z.B. 
default-mäßig per CTRL-D.

> Dann werden die Ausgaben [...] untershiedlich sein.

Korrekt.  Und zwar gewollt, um ein Ende der Datei überhaupt erkennen zu 
können.  Was sollte er sonst liefern?  Unendlich viele Leerzeilen?

Die typische Nutzung sähe z.B. so aus:

1

    char buf[32];  // short lines ...

2

    int i = 0;

3

4

    printf("Anfang\n");

5

6

    while (ngets(buf, sizeof(buf)))

7

        printf("Zeile %d: '%s'\n", ++i, buf);

8

9

    printf("Ende\n");

> Wegen der letzten drei Zeilen.

Nur die letzte.  Die beiden davor kümmern sich darum, dass der Buffer 
immer schön 0-terminiert ist.