Problem z funkcją w c++

Question

Witam, proszę o pomoc. Napisałem funkcję w c++ i nie działa. Problem jest głównie w lini "string line[lines];"

void fileRead(string filename, int lines) {
	fstream file;

	file.open(filename, ios::in);
	if (file.good() == false) {
		cout << "File is broken or missing!";
	}
	else {
		string line[lines];
		int lineNum = 1;

		for (int i = 1; i <= lines; i++) {
			while (getline(file, line)) {
				lines[i] = line;
			}
			lineNum++;
		}
	}
}

 

Answer 1

Problem jest taki że C++ nie obsługuje VLA.

 https://en.m.wikipedia.org/wiki/Variable-length_array

Comments

Mógłbyś wyłumaczyć co to jest i czy można coś zrobić aby ta funkcja działała?

---

W trybie małych poprawek:

void fileRead(const string& filename, size_t lines) {
    ifstream file(filename);

    if (!file) {
        cerr << "File is broken or missing!";
    } else {
        vector<string> line;
        line.reserve(lines);

        for (int i = 0; i < lines; ++i) {
            while (getline(file, line)) {
                line.emplace_back(line);
            }
        }
    }
}

 

Answer 2

Musisz zadeklarować tą tablicę line w sposób dynamiczny, a nie statyczny.

http://cpp0x.pl/kursy/Kurs-C++/Dynamiczne-zarzadzanie-pamiecia-new-i-delete/307

https://www.p-programowanie.pl/cpp/tablice-dynamiczne/

Poza tym zmienna lineNum u Ciebie nic nie robi.

while (getline(file, line)) {
   lines[i] = line;
 }

Ta pętla też nie ma sensu, ponieważ wczytujesz cały czas do tej samej komórki nadpisując poprzednie linie.

Comments

Oczywiście, że nie w ten sposób. Od tego jest vector, albo array, albo dynamiczne wskaźniki.

---

Dokładnie, nawet lepiej skorzystać z vectora.

http://www.cplusplus.com/reference/vector/vector/

http://cpp0x.pl/dokumentacja/standard-C++/vector/819

---

@gagyn, Akurat ta pętla (mam na myśli pętle for) ma sens, bo od niej zależy do jakiej "szufladki" w tablicy, zostanie zapisana zawartość zmiennej line. Ale dziękuję za odpowiedź

---

Ta pętla for jak najbardziej ma sens, ale chodzi mi o while, który sensu nie ma.

---

Poza tym, jak skorzystasz z vectora, to nie będziesz musiał się przejmować do której komórki wstawić nową linię, tylko użyjesz .push_back(linia) i nowa linia zostanie wstawiona na koniec.