Funkcja wczytuje z pliku dane i dodaje je do tablicy

Question

Witam mam problem. Program nie chce się uruchomić. 

 

#include <iostream>

using namespace std;

int counter = 0;
int liczba;

void readfile(const char *file, int *t, int size)
{
    std::fstream plik;
    plik.open(file, ios::in);

    if (plik.good() != true)
    {
        cout<<"plik sie nie otworzył";
    }
    while (!plik.eof ())
    {
        plik>>liczba;
        t[counter++] = liczba;
    }

}

 

Comments

Gdzie funkcja main?

---

@Kraczek, tak, nie widząc wywołania

• jeśli plik się nie otworzy to warto byłoby coś zrobić poza wypisaniem komunikatu. Np. nie czytać go dalej tylko od razy return albo exit.
• po co te zmienne są globalne? Nie lepiej zwrócić counter jako wynik funkcji, w przypadku błedu otwarcia możesz zwrócić -1 (tak linuxowo)
• metoda eof daje true dopiero jak walniesz w koniec pliku - jak spróbujesz czytać poza tym miejscem. Więc ostatni przebieg pętli już nie zapisze prawidłowej wartości.
• Dlaczego używasz fstream zamiast ifstream? Lepiej jak kompilator sprawdzi czy używasz tylko operacji dopuszczalnych w trybie odczytu. Inaczej będziesz miał bład wykonania zamiast kompilacji, a te drugie się znacznie prościej poprawia.

I jak poprzednik, sugeruję być pokazał wywołanie tej funkcji.

No i sprecyzuj co rozumiesz przez "Program nie chce się uruchomić.".