Wczytywanie pliku w grze i error

Question

Witam wszystkich.

Mam problem dotyczy on wczytywania pliku do gry. Wydaję mi się że wszystko jest okej lecz perror zwraca błąd 

plik istnieje w folderze lecz go nie wczytuje.

Proszę o pomoc ponieważ nie mam pojęcia jak temu zaradzić poniżej wysyłam kod funkcji która ładuje plik i jej wywołanie w mainie.
 

void Level::load(string fileName, Player &player)
{
	//WCZYTUJE LEVEL
	ifstream plik;

	plik.open(fileName);
	if (plik.fail());
	{
		perror(fileName.c_str());
		system("PAUSE");
		exit(1);
	}

	string mapa;

	while (getline(plik, mapa))
	{
		_levelData.push_back(mapa);
	}

	plik.close();
	// POZYCJA GRACZA NA PLANSZY
	char pozycja;
	// WCZYTUJE CALE LINIE
	for (int i = 0; i < _levelData.size(); i++)
	{
		// WCZYTUJE POJEDYNCZE ZNAKI W LINIACH
		for (int j = 0; j < _levelData[i].size(); j++)
		{
			pozycja = _levelData[i][j];

			switch (pozycja)
			{
			case '@':
				player.getposition(j, i);
				break;
			}
		}
	}
}

___________________________________________________________________

#include <iostream>
#include <string>
#include <cstdlib>
#include <conio.h>
#include "GameCore.h"
#include "Level.h"


using namespace std;

int main()
{
	
	GameCore gameCore("poziom1.txt");
	gameCore.playGame();
	

 

Answer 1

if (plik.fail()); //tu masz średnik, którego nie powinno tu być

 

Comments

Zastanawia mnie tylko to dlaczego Visual Studio nie pokazało mi takiego błędu. Dzięki za odpowiedź taka głupota a zabrała mi tyle czasu

---

Jako błąd na pewno nie (bo taka konstrukcja jest dopuszczalna - co wcale nie oznacza, że jest sensowana), ale mój VS w takim przypadku pokazałby ostrzeżenie:

warning C4390: ";": znaleziono kontrolowaną pustą instrukcję; czy to było zamierzone?

Answer 2

        perror(fileName.c_str());
        system("PAUSE");
        exit(1);

Kiedy std::ifstream sfailuje, nie ustawia zmiennej std::errno. To mechanizm z C i nie jest on najlepszy do stosowania w C++, który posiada system wyjątków. Tak samo - używanie funkcji exit jest niewskazane - to funkcja projektowana do kończenia programów C - w C++ nie pozwala na wywołanie destruktorów obiektów, które kończą życie razem z programem. Staraj się tak sterować przepływem sterowania w programie, żeby kończyć działanie z funkcji main przez zwykły return.

Wracając do Twojego problemu - jego rozwiązaniem może być ustawienie katalogu roboczego na katalog, w którym trzymasz plik "poziom.txt" lub skopiowanie tego pliku do lokalizacji, w której buduje się plik wykonywalny, czyli u Ciebie gdzieś w katalogu "Debug".

 

Comments

Dzięki za odpowiedź i rady lecz katalog roboczy jest ustawiony tam gdzie trzymam wszystkie pliki projektu więc to raczej jest ok. Jak wrzucam plik do folderu debug lub gdziekolwiek indziej program nie znajduję drogi do niego.