C - Wie der Benutzer aufgefordert, Dateiname eingeben

Sehr neu in der C-Programmierung. Ich Schreibe ein Programm, wo ich will, fordert den Benutzer zur Eingabe der Namen der Datei, die zum Lesen geöffnet. Auf meinem code (siehe unten) ich will keinen Fehler werfen, wenn Sie nicht offen ist oder wenn die Datei nicht existiert, aber wenn ich es starten, mein code explodiert, und ich muss das Programm schließen (DOS)

/*ask user for the name of the file*/
  printf("enter file name: ");
  gets(fname);


//Opens the file from where the text will be read. 
fp=fopen(fname, "r");

//Checks if the file us unable to be opened, then it shows the error message 
if (fp == NULL)
    {
     printf("\nError, Unable to open the file for reading\n");
    }

//Sie können dies testen, indem Sie die Schaffung eines name.txt Datei. Bitte lassen Sie mich wissen, wenn Sie weitere Informationen benötigen.

  • Wie haben Sie fname? Oh, und bitte, bitte, bitte nicht verwenden gets - niemals! Betrachten fgets als eine mögliche alternative.
  • Sie sollten sich auch darüber im klaren sein, was Sie meinen, wenn Sie sagen, "wenn ich es laufen", vor allem, wenn Sie neu sind C. meinst du es nicht kompilieren, stürzt es ab sofort auf laufen, oder es stürzt ab, nachdem Sie versuchen, geben Sie einen Dateinamen ein und drücken Sie enter,?
InformationsquelleAutor NewLearner | 2012-04-11
  1. 3

    Gut, Sie sollten stellen Sie sicher, es gibt genug Platz in fname zum speichern Ihrer Datei-Namen, sonst werden Sie fast sicher die Ursache der Korruption und der "Sprengung" 🙂

    Beispielsweise den folgenden Codeausschnitt:

    char fname[10];
gets (fname);

    wird dann problematisch, wenn Sie weitere Informationen eingeben als fname halten kann. An diesem Punkt, Sie sind in ein Undefiniertes Verhalten Hoheitsgebiet und alles passieren kann.

    Aber, weil gets keine Möglichkeit bietet, zu beschränken, was der Benutzer eingibt, sollten Sie nie verwenden!

    Ordnungsgemäße, geschützt, Methode der Benutzereingaben kann gefunden werden in diese Antwort.

    Es nutzt fgets da das kann zu begrenzen, was der Benutzer eingibt. Es ermöglicht auch eine Aufforderung, liefert ein Fehlerbild, wenn etwas schief geht, behandelt das Ende der Datei korrekt ist, und entfernt den Rest von zu großen Linie, so dass es keinen Einfluss auf die nächste Eingabe.

    In der Tat, ich werde es kopieren, hier zu machen, diese Antwort selbst zu enthalten:

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

#define OK       0
#define NO_INPUT 1
#define TOO_LONG 2
static int getLine (char *prmpt, char *buff, size_t sz) {
    int ch, extra;

    //Get line with buffer overrun protection.
    if (prmpt != NULL) {
        printf ("%s", prmpt);
        fflush (stdout);
    }
    if (fgets (buff, sz, stdin) == NULL)
        return NO_INPUT;

    //If it was too long, there'll be no newline. In that case, we flush
    //to end of line so that excess doesn't affect the next call.
    if (buff[strlen(buff)-1] != '\n') {
        extra = 0;
        while (((ch = getchar()) != '\n') && (ch != EOF))
            extra = 1;
        return (extra == 1) ? TOO_LONG : OK;
    }

    //Otherwise remove newline and give string back to caller.
    buff[strlen(buff)-1] = '\0';
    return OK;
}

    Sie können es nennen, wie folgt, mit Angabe der Puffer und die Größe, und erhalten eine Fehleranzeige bei der Rückkehr:

    //Test program for getLine().

int main (void) {
    int rc;
    char buff[10];

    rc = getLine ("Enter string> ", buff, sizeof(buff));
    if (rc == NO_INPUT) {
        //Extra NL since my system doesn't output that on EOF.
        printf ("\nNo input\n");
        return 1;
    }

    if (rc == TOO_LONG) {
        printf ("Input too long [%s]\n", buff);
        return 1;
    }

    printf ("OK [%s]\n", buff);

    return 0;
}
    • Kann ich nur verwenden scanf?
    • es gibt andere Probleme mit scanf - besonders, wenn Sie verwenden die 'unbounded string' format-string %s. Es ist auch Thema, um buffer-overflow. scanf bedeutet "scan formatted" und da ist nicht viel mehr unformatiert als user input 🙂 versteh mich nicht falsch, es ist gut, in den meisten Fällen, vor allem für das lernen C, oder Unterricht, oder auch quick-und-dirty-Programme, aber es ist nicht gut für die Produktion von hochwertigem code.
    InformationsquelleAutor paxdiablo
  2. 0

    Nicht wissen, was Sie fehlen, aber das Programm kompiliert und läuft bei mir mit cl.exe in der Befehlszeile.

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

int main(){
    char fname[100];
    FILE* fp;
    memset(fname, 0, 100);
    /*ask user for the name of the file*/
    printf("enter file name: ");
    gets(fname);


    //Opens the file from where the text will be read. 
    fp = fopen(fname, "r");

    //Checks if the file us unable to be opened, then it shows the error message 
    if (fp == NULL)
    {
        printf("\nError, Unable to open the file for reading\n");
    }

}

    Beachten Sie auch, dass bestimmte Compiler (einschließlich Microsoft) benötigen Sie, um alle Deklarationen an den Anfang der Funktion beim Aufruf mit default-Optionen.

    InformationsquelleAutor merlin2011
  3. 0

    gut,Das ist, was passiert mit mir, indem eine "else" Bedingung "wenn" für mich gearbeitet.Unten erwähnt ist der code.

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

int  main() 
{

char fname[100];
FILE* fp;
memset(fname, 0, 100);
/*ask user for the name of the file*/
printf("enter file name: ");
gets(fname);

fp = fopen(fname, "r");

//Checks if the file us unable to be opened, then it shows the error message 
if (fp == NULL)
{
    printf("\nError, Unable to open the file for reading\n");
}

else
{
    printf("hello");
}


getch();
}

    Stellen Sie außerdem sicher, dass Sie Hinzugefügt haben, wird "#include" als header-Datei.

    InformationsquelleAutor rishanth chavali
Schreibe einen Kommentar