Question

我正在为课堂编写Conway的环形人生游戏的实现。函数cargarToroide（loadToroid）应从文件中将每个单元格（celda）的适当状态（有效或无效-正确或错误-1或0）加载到向量中，其签名如下：

toroide cargarToroide(string nombreArchivo, bool &status);

nombreArchivo是文件的名称，如果加载文件或文件格式有任何问题，status应该为false。

数据结构的定义如下（我无法更改）：

typedef vector< vector<bool> > toroide;

文件的结构如下：

numberOfLines numberOfColums
list of the values of the cells
number of live cells

例如：

问题是，我找不到使它起作用的方法。我已经在线阅读了vector<bool>的问题，当您尝试以常规方式加载它时，这是我尝试的第一件事。

toroide cargarToroide(string nombreArchivo, bool &status)
{
    toroide t;
    ifstream fi (nombreArchivo);
    int cantidadFilas, cantidadColumnas;
    int celda;

    if(!fi){
        status = false;
    }

    fi >> cantidadFilas;
    fi >> cantidadColumnas;

    for(int i = 0; i < cantidadFilas; i++) {
        for (int j = 0; j < cantidadColumnas; j++) {
            fi >> celda;
            if(celda == 1) {
                t[i].push_back(true);
            }
            else if(celda == 0){
                t[i].push_back(false);
            }
            else{
                status = false;
                return t;
            }
        }
    }
    return t;
}

我还尝试过将celda定义为布尔值，并且仅使用

t[i].push_back(celda);

使用C ++ 11解决此问题的最佳方法是什么？

Answer 1

您需要先调整外部向量的大小，然后才能对其使用operator []。在读取数据时，还应使用正确的类型（布尔型），并检查输入文件中是否有错误。我在代码中评论了：

#include <iostream>
#include <fstream>
#include <vector>

typedef std::vector< std::vector<bool> > toroide;

// Both cargarToroide and cargarToroide_improved can be used
bool cargarToroide_improved(const std::string& nombreArchivo, toroide& in_toroide)
{
    std::ifstream fi(nombreArchivo);
    if(!fi) return false;

    int cantidadFilas, cantidadColumnas, liveCells=0;
    // use a bool to read the bool data
    bool celda;

    fi >> cantidadFilas;
    fi >> cantidadColumnas;
    // check if the stream is in a failed state
    if(fi.fail()) return false;

    // Temporary used to not mess with in_toroide until we're finished.
    // Create it with cantidadFilas default inserted rows
    toroide t(cantidadFilas);

    for(auto& row : t) {
        // default insert columns into the row
        row.resize(cantidadColumnas);
        for (int col = 0; col < cantidadColumnas; ++col) {
            fi >> celda;
            // check if the stream is in a failed state
            // (non-bool read or the file reached eof())
            if(fi.fail()) return false;
            // set column value in the row
            row[col] = celda;
            // count live cells
            liveCells += celda;
        }
    }

    // compare live cells in matrix with checksum
    int cmpLive;
    fi >> cmpLive;
    if(fi.fail() || cmpLive!=liveCells) return false;

    // a successful toroide was read, swap your temporary
    // toroide with the user supplied one
    std::swap(t, in_toroide);
    return true;
}

// if the signature of this function really can't be changed (which it should),
// make it a proxy for the function with a slightly nicer interface
// Like this:
toroide cargarToroide(std::string nombreArchivo, bool &status)
{
    toroide rv;
    status = cargarToroide_improved(nombreArchivo, rv);
    return rv;
}

使用改进的签名：

int main(int argc, char* argv[]) {
    std::vector<std::string> args(argv+1, argv+argc);

    for(auto& file : args) {
        toroide my_toroide;
        if(cargarToroide_improved(file, my_toroide)) {
            for(auto& r : my_toroide) {
                for(auto c : r) {
                    std::cout << c << " ";
                }
                std::cout << "\n";
            }
        } else {
            std::clog << "failed loading " << file << "\n";
        }
    }
}

使用您必须使用的签名：

int main(int argc, char* argv[]) {
    std::vector<std::string> args(argv+1, argv+argc);

    for(auto& file : args) {
        bool status;
        toroide my_toroide = cargarToroide(file, status);
        if(status) {
            for(auto& r : my_toroide) {
                for(auto c : r) {
                    std::cout << c << " ";
                }
                std::cout << "\n";
            }
        } else {
            std::clog << "failed loading " << file << "\n";
        }
    }
}

Answer 2

如果您知道编译时的行数（如本例所示），则可以使用resize。在这种情况下，您必须在两个for循环之前，先进行以下操作。

t.resize(cantidadFilas);

实际上，您也可以对列执行相同的操作。这样一来，您就不再需要在内部push_back循环中使用for。

如果不知道行数，则只需使用push_back并将行添加到toroide。然后，您可以在第二个for循环之前添加以下行。

vector<bool> row; 
t.push_back(row)

写入vector <vector <bool>>

2 个答案: