C funkcja otwierająca pliki

Question

W jaki sposób mogę stworzyć funkcję, która będzie służyła do otwierania pliku/plików tekstowych tak, aby przekazany oryginalny argument trzymał ten plik?

Próbowałem w ten sposób, ale nie działa. Program powinien wypisać zawartość pliku na ekran.

#include <stdio.h>

int openFile(FILE** inputFile)
{
    *inputFile = fopen("input.txt", "r");

    if (*inputFile != NULL)
        return 0;
    else
        return -1;
}
void readWholeFile(FILE* inputFile)
{
    char str[20];
    while (feof(inputFile)) {
        fscanf(inputFile, str);
        printf("%s\n", str);
    }
}
int main() {

    FILE* inputFile;

    if (openFile(&inputFile) == 0) {
        readWholeFile(&inputFile);
        fclose(inputFile);
    }
    else
        printf("File didn't open");

    return 0;
}

 

Answer 1

Witam,

spróbuj tak

...
int openFile(FILE* inputFile)
...
void readWholeFile(FILE* inputFile)
...

a w funkcji main()

    if (openFile(inputFile) == 0) {
        readWholeFile(inputFile);
        fclose(inputFile);
    }

Wtedy program Ci pójdzie. Co do Twojego głównego problemu czy dereferencja czy kopiowania FILE* jest w ogóle dozwolone. Powinieneś używać FILE* lokalnie tzn wszystkie operacje powinny być w jednym zakresie blokowym. Tutaj masz więcej info

http://en.cppreference.com/w/c/io

Comments

Nie wiem czy dobrze zrozumiałem, ale dla tego kodu, program wywala brak odpowiedzi.

#include <stdio.h>

int openFile(FILE* inputFile)
{
    inputFile = fopen("input.txt", "r");

    if (inputFile != NULL)
        return 0;
    else
        return -1;
}
void readWholeFile(FILE* inputFile)
{
    char str[20];
    while (feof(inputFile)) {
        fscanf(inputFile, str);
        printf("%s\n", str);
    }
}
int main() {

    FILE* inputFile;

    if (openFile(inputFile) == 0) {
        readWholeFile(inputFile);
        fclose(inputFile);
    }
    else
        printf("File didn't open");

    return 0;
}

 

---

Właśnie tak,

u mnie kompilator zwraca coś takiego:

Invalid parameter passed to C runtime function.

Podałem jeszcze pod wątpliwość czy jest możliwość wykonania Twojego zadania. Nie wiem czy jest możliwość jakiejś wartości FILE* przypisania jej wartości, wyjście z jednego zakresu i dalej jej prawidłowe działanie.

---

Nie do końca rozumiem mimo wszystko dlaczego początkowy kod nie chce działać.

Przecież przesyłam wskaźnikowi na wskaźnik adres tego lokalnego dla maina wskaźnika na plik, którym chce otworzyć ten plik input.txt.

Program nie wypisuje "File didn't open", więc plik powinien zostać otwarty w oryginalnym wskaźniku, tym w mainie. Mimo to potem funkcja do wyczytywania pliku jakby dostaje pusty plik, bo wcale nie wchodzi do while'a który ma czytać kolejne słowa, tak, jakby od razu łapał koniec pliku. Nie bardzo rozumiem czemu, skoro plik nie jest pusty i zostaje otwarty dla odpowiedniej zmiennej,

---

http://www.cplusplus.com/reference/cstdio/FILE/

i.e., the pointer to a copy of a FILE object could be interpreted differently than a pointer to the original).

Może tutaj jest problem ale do końca nie jestem pewien

Answer 2

readWholeFile(&inputFile);

readWholeFile(inputFile);

 

fscanf(inputFile, str);

O format-stringu zapomniałeś. Do czytania całych linii użyj gets()/gets_s().

 

Comments

Przecież do readWholeFilemusze wrzucić właśnie sam wskaźnik, a nie jego adres.

Inna sprawa - ja nie chce czytać całych linii tylko właśnie zbierać pojedyncze słowa.

---

Przecież do readWholeFile musze ...

A co ja pokazałem? Poprawiłem Twój kod.

ja nie chce czytać całych linii ...

fscanf(inputFile, "%s", str); 

// tak bezpieczniej
fscanf_s(inputFile, "%s", str, (rsize_t)sizeof(str)) ;

str powinien mieć więcej niż 20 znaków.

 

---

Czy wobec tego deklaracja str powinna wyglądać w ten sposób?

char * str = NULL;

 

---

Skąd taki wniosek? str niech będzie tablicą, tylko większą niż 20 znaków. Tak małą tablicę łatwo przekroczyć (chyba że użyjesz bezpiecznej wersji fscanf).