Почему доступ к этой локальной переменной-члену вызывает исключение?

Я пытаюсь написать программу, которая будет считывать содержимое файла .csv в массив, а затем выводить результаты на экран (часть более крупного проекта), когда функция getNumberOfRooms() вызывается исключение выдается, когда он пытается вернуть значение numberOfRooms переменная, закрытый член в классе. У кого-нибудь была такая проблема раньше или кто-то помогал с такой проблемой? Если да, то как ты решил это?

Заранее спасибо,

Полный исходный код доступен здесь: https://bitbucket.org/skutov/micropuzzle/

Исключение, которое выбрасывается при вызове getNumberOfRooms ():

Unhandled exception at 0x01354aa6 in MICROPUZZLE.exe: 0xC0000005: Access violation
reading location 0xccccccd0.

Это рассматриваемые функции (все время на переменную ссылаются в классе)

ClassMap::ClassMap ()
{
numberOfRooms = 0;

// Get number of rooms in map.csv

/* Find number of entries in map.csv file */

numberOfRooms = number_of_lines;

// allocate memory for rooms array

/* loading data from file into array */
}
}

// self explanitory
int ClassMap::getNumberOfRooms()
{
// Exception occurs on this line when accessing the variable
return numberOfRooms;
}

int ClassMap::printRoomDescriptions ()
{
for(int j = this->getNumberOfRooms(); j > 0; j--)
{
cout << roomArray[j].getDescription();
}
return 0;
}

Вот заголовок класса:

class ClassMap
{
private:
int currentLocation;
int numberOfRooms;
// pointer to array initialised in constructor
ClassRoom *roomArray;

public:
// Constructors and Destructors
ClassMap();
~ClassMap();

// Print description, events and directions for current room
std::string getCurrentRoom();

// Change currentLocation to neighbour of current room if possible
int moveRoom(char direction);// self explanitory
int getNumberOfRooms();

// dump room descriptions to command line (debugging)
int printRoomDescriptions();

};

Вот конструктор для ClassMap, который также инициализирует roomArray:

ClassMap::ClassMap ()
{
numberOfRooms = 0;

// Get number of rooms in map.csv

unsigned int number_of_lines = 0;
FILE *infile = fopen("map.csv", "r");
int ch;

while (EOF != (ch=getc(infile)))
if ('\n' == ch)
++number_of_lines;
fclose(infile);
numberOfRooms = number_of_lines;// allocate memory for rooms array
roomArray = new ClassRoom[numberOfRooms+1];

// set starting room
int currentLocation = 1;

// load that shit up
{
// Holders for values read from file
int newRoomID = 0;
char newRoomDescription[79] = "";
int newRoomNorthNeighbour = 0;
int newRoomEastNeighbour = 0;
int newRoomSouthNeighbour = 0;
int newRoomWestNeighbour = 0;

// used for iterations
int i = 0;

// File stream for map.csv
std::ifstream mapFile;

// Crack that shit open
mapFile.open ("map.csv");

// Line buffer for parsing
std::string line;// For each line in the map.csv file read in the values into variables declared above then run initialise function for each room to store values into array
while (std::getline(mapFile, line))
{
// re-init parameters

newRoomID = 0;
newRoomNorthNeighbour = 0;
newRoomEastNeighbour = 0;
newRoomSouthNeighbour = 0;
newRoomWestNeighbour = 0;
for(i = 0;i<79;i++)
{
newRoomDescription[i] = ' ';
}int parameter = 0;

int paraStart = 0;
int paraEnd = 0;

std::string buffer;
std::istringstream iss(line);

for(parameter = 0; parameter <= 5; parameter++)
{
// Empty buffer from last iteration
buffer.clear();

// Find end of current parameter
paraEnd = line.find(',',paraStart+1);

switch (parameter)
{
case 0:
buffer = line.substr((paraStart),(paraEnd-paraStart));
newRoomID = atoi(buffer.c_str());
break;
case 1:
buffer = line.substr((paraStart+2),(line.find("\"",paraStart+2)-(paraStart+2)));
for(i = 0;i<(buffer.length());i++)
{
newRoomDescription[i] = buffer.c_str()[i];
}
//newRoomDescription
break;
case 2:
buffer = line.substr((paraStart+1),(paraEnd-paraStart));
newRoomNorthNeighbour = atoi(buffer.c_str());
break;
case 3:
buffer = line.substr((paraStart+1),(paraEnd-paraStart));
newRoomEastNeighbour = atoi(buffer.c_str());
break;
case 4:
buffer = line.substr((paraStart+1),(paraEnd-paraStart));
newRoomSouthNeighbour = atoi(buffer.c_str());
break;
case 5:
buffer = line.substr((paraStart+1),(paraEnd-paraStart));
newRoomWestNeighbour = atoi(buffer.c_str());
break;
} // switch

// Cycle paraEnd to paraStart
paraStart = paraEnd;

} // for parameters loop

// Init next room with data
new (&roomArray[newRoomID]) ClassRoom(  newRoomNorthNeighbour,
newRoomEastNeighbour,
newRoomSouthNeighbour,
newRoomWestNeighbour,
newRoomDescription);

} // while !EOF
// Close the file because we're a good little program and we don't need that shit no more
mapFile.close();
}
}

0

Решение

Ключ к этой проблеме:

Access violation reading location 0xccccccd0

0xcccccccc это специальное значение, используемое в режиме отладки для обозначения унифицированного указателя. (Увидеть Как в итоге получить указатель на 0xCCCCCCCC ) Он установлен в режиме отладки на причина этот вид сбоя — это означает, что используемый вами указатель еще не настроен. Как только вы установите указатель правильно, ошибка исчезнет. (Небольшое отличие от 0xcccccccc это смещение члена, к которому вы пытаетесь получить доступ внутри этого объекта.)

ДОБАВЛЕНО:

Это ваша ошибка:

ClassRoom* roomArray = static_cast<ClassRoom*>( ::operator new ( sizeof ClassRoom * numberOfRooms ) );

Это создает местный roomArray переменная, и скрывает переменную-член. То, что вы действительно хотите, это:

roomArray = static_cast<ClassRoom*>( ::operator new ( sizeof ClassRoom * numberOfRooms ) );

Или еще лучше:

roomArray = new ClassRoom[numberOfRooms];
2

Другие решения

Я думаю, проблема в том, что вы за цикл for(int j = this->getNumberOfRooms(); j > 0; j--), Это должно выглядеть так: for(int j = this->getNumberOfRooms()-1; j >= 0; j--),
Последний доступный индекс в массиве с N записями — N-1. С другой стороны, первый индекс равен 0.

1

По вопросам рекламы [email protected]