首页 文章

我该怎么办才能不使用!openFile.eof()?

提问于
浏览
1

我一直在关注allegro 5平台游戏的教程,他的文件管理器使用!openFile.eof(),我听说它不好,我很确定它是什么给了我一个向量下标超出范围的错误 . 除此之外还有什么我可以使用的吗?你也可以检查我的图层类,这会给我矢量下标超出范围错误吗?我无法弄明白,我很确定它来自filemanager,但我不能告诉 .

它只输出 Map 的第一行 . 当我将它改为while(std :: getline(openFile,line))时,我甚至都没有得到std :: cout << contents [i] [k] <<“k:”<< k <<“内容:” << contents [i] .size()<< std :: endl << std :: endl;

当我将其更改为while(std :: getline(openFile,line))时,属性和内容每个只有8个 .

仍然需要帮助= \

FileManager.CPP

#include "FileManager.h"


FileManager::FileManager()
{
    identifierFound = false;
}


FileManager::~FileManager()
{
}

void FileManager::LoadContent(const char *filename, std::vector<std::vector<std::string>> &attributes, std::vector<std::vector<std::string>> &contents)
{
    std::ifstream openFile(filename);
    std::string line, newLine;

    if(openFile.is_open())
    {
        while(std::getline(openFile, line))
        {
            std::stringstream str;

            if(line.find("Load=") != std::string::npos)
            {
                type = LoadType::Attributes;
                line = line.erase(0, line.find("=") + 1);
                tempAttributes.clear();
            }
            else
            {
                type = LoadType::Contents;
                tempContents.clear();
            }

            str << line;

            while(std::getline(str, newLine, ']'))
            {
                newLine.erase(std::remove(newLine.begin(), newLine.end(), '['), newLine.end());

                std::string erase = " \t\n\r"; 

                newLine.erase(newLine.find_last_not_of(erase) + 1);

                if(type == LoadType::Attributes)
                    tempAttributes.push_back(newLine);
                else
                    tempContents.push_back(newLine);

                std::cout << newLine << std::endl;
            }

            if(type == LoadType::Contents && tempContents.size() > 0)
            {
                attributes.push_back(tempAttributes);
                contents.push_back(tempContents);
            }
        }
    }
    else
    {

    }
}

void FileManager::LoadContent(const char *filename, std::vector<std::vector<std::string>> &attributes, std::vector<std::vector<std::string>> &contents, std::string identifier)
{
    std::ifstream openFile(filename);
    std::string line, newLine;

    if(openFile.is_open())
    {
        while(!openFile.eof())
        {
            std::stringstream str;
            std::getline(openFile, line); 

            if(line.find("EndLoad=") != std::string::npos && line.find(identifier) != std::string::npos)
            {
                identifierFound = false;
                break;
            }
            else if(line.find("Load=") != std::string::npos && line.find(identifier) != std::string::npos)
            {
                identifierFound = true;
                continue;
            }

            if(identifierFound)
            {

                if(line.find("Load=") != std::string::npos)
                {
                    type = LoadType::Attributes;
                    line = line.erase(0, line.find("=") + 1);
                    tempAttributes.clear();
                }
                else
                {
                    type = LoadType::Contents;
                    tempContents.clear();
                }

                str << line;

                while(std::getline(str, newLine, ']'))
                {
                    newLine.erase(std::remove(newLine.begin(), newLine.end(), '['), newLine.end());

                    std::string erase = " \t\n\r"; 

                    newLine.erase(newLine.find_last_not_of(erase) + 1);

                    if(type == LoadType::Attributes)
                        tempAttributes.push_back(newLine);
                    else
                        tempContents.push_back(newLine);

                    std::cout << newLine << std::endl;
                }

                if(type == LoadType::Contents && tempContents.size() > 0)
                {
                    attributes.push_back(tempAttributes);
                    contents.push_back(tempContents);
                }
            }
        }
    }
    else
    {

    }
}

Layer.CPP

#include "Layer.h"


Layer::Layer(void)
{
}


Layer::~Layer(void)
{
}

std::pair<int, int> Layer::SetTiles(std::string tileString)
{
    std::pair<int, int> tile;
    tile.first = atoi(tileString.substr(0, tileString.find(',')).c_str());
    tile.second = atoi(tileString.substr(tileString.find(',') + 1).c_str());
    return tile;
}

void Layer::LoadContent(std::string layerID, std::string mapID)
{
    std::string fileName = "Maps/"+mapID+".txt";
    fileManager.LoadContent(fileName.c_str(), attributes, contents, layerID);
    int indexY = 0;

    for(int i = 0; i < attributes.size(); i++)
    {
        for(int j = 0; j < contents[i].size(); j++)
        {
            if(attributes[i][j] == "SolidTiles")
            {
                solidTiles.push_back(SetTiles(contents[i][j]));
                std::cout << contents[i][j] << std::endl << std::endl;
            }
            else if(attributes[i][j] == "TileSheet")
            {
                std::cout << contents[i][j] << std::endl << std::endl;
                tileSheet = al_load_bitmap(contents[i][j].c_str());
            }
            else if(attributes[i][j] == "StartLayer")
            {
                for(int k = 0; k < contents[i].size(); k++)
                {
                    std::cout << contents[i][k] << " k: " << k << " contents: " << contents[i].size() << std::endl << std::endl;
                    if(contents[i][k] != "---")
                    {
                        std::cout << contents[i][k] << std::endl << std::endl;
                        ALLEGRO_BITMAP *tileImage;
                        Tile::State tempState = Tile::State::Passive;
                        std::pair<int, int> tile = SetTiles(contents[i][k]);

                        if(std::find(solidTiles.begin(), solidTiles.end(), tile)  != solidTiles.end())
                        {
                            tempState = Tile::State::Solid;
                        }

                        tileImage = al_create_sub_bitmap(tileSheet, tile.first * 32, tile.second * 32, 32, 32);

                        std::pair<float, float> position(k * 32, indexY * 32);

                        Tile tileInstance;
                        tiles.push_back(tileInstance);
                        tiles[tiles.size()-1].SetContent(tileImage, tempState, position);
                        std::cout << tiles.size() << std::endl;
                    }
                }
                indexY++;
            }
        }
    }
}

void Layer::UnloadContent()
{
    for(int i = 0; i < tiles.size(); i++)
        tiles[i].UnloadContent();

    al_destroy_bitmap(tileSheet);
}

void Layer::Update()
{
}

void Layer::Draw(ALLEGRO_DISPLAY *display)
{
    for(int i = 0; i < tiles.size(); i++)
        tiles[i].Draw(display);
}

map1.txt

Load=[MapProperties]

Load=[TileDimensions]
[32,32]

EndLoad=[MapProperties]

Load=[Layer1]

Load=[SolidTiles]
[2,0]
[1,0]

Load=[NullTile]
[---]

Load=[Motion]
[2,0:Static]

Load=[TileSheet]
[TileSheets/tilesheet1.png]

Load=[StartLayer]

[2,0][---][---][---][---][---][---][---][---][---][---][---][---][---][---][---]
[---][---][---][---][---][---][---][---][---][---][---][---][---][---][---][---]
[---][---][---][---][---][---][---][---][---][---][---][---][---][---][---][---]
[---][---][---][---][---][---][---][---][2,0][2,0][2,0][---][---][---][---][---]
[---][---][---][---][---][---][---][2,0][---][---][---][---][---][---][---][---]
[---][---][---][---][---][---][2,0][---][---][---][---][---][---][---][---][---]
[---][---][---][---][---][---][---][---][---][---][---][---][---][---][---][---]
[---][---][2,0][2,0][2,0][---][---][---][---][---][---][---][---][---][---][---]
[---][---][---][---][---][---][---][---][---][---][---][---][---][---][---][---]
[---][---][---][---][---][---][---][---][---][---][---][---][---][---][---][---]
[1,0][1,0][1,0][1,0][1,0][1,0][1,0][1,0][1,0][1,0][1,0][1,0][1,0][1,0][1,0][1,0]

Load=[EndLayer]
[dummy]

EndLoad=[Layer1]

Load=[PlayerPosition]
[0,0]

1 回答

  • 1

    首先,真正的问题不是使用 file.eof() ;在某些情况下,它是你想要的(尽管我无法想到它在_2606200中的任何位置 - 在你已经检测到输入失败之后它是合适的情况) . 真正的问题是代码使用 std::getline 读取的值而不验证它是否成功 . 由于这只在代码中出现一次,因此只需使用与其他循环相同的解决方案: while ( std::getline( ... ) ) .

相关问题