0

嗨，我有一个用于映射的类，它正在绘制到屏幕上，但它只在窗口左侧垂直向下绘制。我可以弄清楚我哪里出错了。任何帮助将非常感激。很确定这与我在绘图函数中的 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);

        }
    }
}
4

1 回答 1

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));

我明白你为什么使用 a char[1]- 你希望它被转换为指针，因为atoi需要一个const char*. 但是，atoi还期望它指向的数组以空值终止。你的不是。它只是一个char.

有一种更好的方法可以将 a 转换为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
}
于 2013-04-03T20:03:42.687 回答