Mit fopen mit Eingabedateinamen in C

Question

An einem gewissen Punkt in meinem Code, ich möchte für eine Datei einen Namen lesen, die ich (und/oder Bearbeitung) wird die Erstellung und ich habe mit den folgenden kommen:

FILE *fp; 
char filename[15]; 
fgets(filename, 15, stdin); 
fp = fopen(filename, "a"); 
fprintf(fp, "some stuff goes here"); 
fclose(fp); 

Auch wenn das kompiliert und ausgeführt wird, erstellt es (oder öffnet, wenn ich es manuell erstelle) nicht die Datei, die von filename angegeben wird.
Was würden Sie vorschlagen?

Answer 1

fgets() speichert das Newline-Zeichen, das nach dem Lesen einer Eingabezeile aus stdin gelesen wird. Sie müssen es manuell entfernen, z.

size_t len = strlen(filename); 
if (len > 0 && filename[len - 1] == '\n') 
    filename[len - 1] = '\0'; 

Sie auch, dass fopen() nicht zurück NULL überprüfen sollten, die es tun, wenn es nicht in der Lage war, die Datei zu öffnen. Ich denke, mit fprintf mit einem NULL Dateizeiger ist undefiniertes Verhalten.

Answer 2

Sie müssen filename[16] deklarieren, um 15 Zeichen zu verwenden, Sie benötigen Platz für die abschließende Null.

Answer 3

der Regel (aber nicht immer), fgets() geben Ihnen einen zusätzlichen '\n' der eingegebenen Zeichenfolge anhängt, weil

Ein Newline-Zeichen fgets macht aufhören zu lesen, aber es gilt als gültiges Zeichen von der Funktion und enthalten in der Zeichenfolge kopiert auf str.

_Referenz:

fgets(filename, 15, stdin); 
filename[strcspn(filename, "\n")] = '\0'; 

Answer 4

Seit fgets werden Zeichenfolge util ein EOF oder eine neue Zeile, lesen und wenn ein Newline gelesen wird: http://www.cplusplus.com/reference/cstdio/fgets/

zu dem '\n' mit minimalem Code loszuwerden , wird es in den Puffer gespeichert. Das bedeutet, dass Sie die abschließende Zeilenumbrüche gegebenenfalls manuell beschneiden müssen. Ich würde vorschlagen, verwenden Sie stattdessen fscanf, es wird nicht die neue Zeile zum Puffer hinzufügen:

char filename[15]; 
int ret = fscanf(stdin, "%s", filename); 
if (ret != 1) { // handle input error } 
fp = fopen(filename, "a");