嗨我有一个用于我的映射的类,它正在绘制到屏幕,但它只在窗口的左侧垂直绘制。我可以弄清楚我哪里出错了。任何帮助将非常感激。非常确定它与绘制函数中的for循环有关。
void Map::Initialise(const char *filename)
{
std::ifstream openfile(filename);
std::string line;
std::vector <int> tempvector;
while(!openfile.eof())
{
std::getline(openfile, line);
for(int i =0; i < line.length(); i++)
{
if(line[i] != ' ') // if the value is not a space
{
char value[1] = {line[i]}; // store the character into the line variable
tempvector.push_back(atoi(value)); // then push back the value stored in value into the temp vector
}
mapVector.push_back(tempvector); // push back the value of the temp vector into the map vector
tempvector.clear(); // clear the temp vector readt for the next value
}
}
}
void Map::DrawMap(sf::RenderWindow &Window)
{
sf::Shape rect = sf::Shape::Rectangle(0, 0, BLOCKSIZE, BLOCKSIZE, sf::Color(255, 255, 255, 255));
sf::Color rectCol;
sf::Sprite sprite;
for(int i = 0; i < mapVector.size(); i++)
{
for(int j = 0; j < mapVector[i].size(); j++)
{
if(mapVector[i][j] == 0)
rectCol = sf::Color(44, 117, 255);
else if (mapVector[i][j] == 1)
rectCol = sf::Color(255, 100, 17);
rect.SetPosition(j * BLOCKSIZE, i * BLOCKSIZE);
rect.SetColor(rectCol);
Window.Draw(rect);
}
}
}
答案 0 :(得分:1)
首先,让你的提取成为while
循环的条件:
while(std::getline(openfile, line))
{
// ...
}
使用openfile.eof()
是一个非常糟糕的主意;只是因为你还没有到达文件的末尾,并不意味着下一次提取会成功。你只是不知道你是否真的有一个有效的line
。
其次,这不能正常工作:
char value[1] = {line[i]}; // store the character into the line variable
tempvector.push_back(atoi(value));
我可以看到您使用char[1]
的原因 - 您希望将其转换为指针,因为atoi
需要const char*
。但是,atoi
也期望它指向的数组为空终止。你的不是。它只是一个char
。
有一种更好的方法可以将char
(数字数字)转换为它所代表的整数。所有数字字符的值都保证是连续的:
char value = line[i];
tempvector.push_back(value - '0');
现在,这是真正的问题。在阅读完一个字符后,您将tempvector
推入mapVector
。您需要将其移到行的for
循环之外:
while(std::getline(openfile, line))
{
for(int i =0; i < line.length(); i++)
{
// ...
}
// Moved here
mapVector.push_back(tempvector); // push back the value of the temp vector into the map vector
tempvector.clear(); // clear the temp vector readt for the next value
}