运算符重载-索引不打印

时间:2019-07-09 09:21:14

标签: c++

我有使用C ++生成“人生游戏”的课程,必须通过讲师的指导才能通过。板的状态存储在std :: vector中。需要一个在x和y位置上的辅助索引函数来获得向量中的一维位置。需要2个运算符重载类,其中1个(非const)用于覆盖向量中的值,另一个(const)用于打印向量中的值。这是到目前为止的课程:

下面的代码编译。

在主类中,设置网格值可以,例如grid(6,1)= 1。 在特定的网格上打印单元格的值不会打印该值。

我不确定问题的原因是运算符重载函数,还是ostream运算符<<重载。

在主类中,我在此行上放置了一个断点:

std::cout << "(6, 1) = " <<  grid(6,1) << std::endl;

在调试时评估grid(6,1)的值时,其内容为1,因此这使我怀疑我的重载ostream编程不正确。

World.h文件:

#include <vector>
#include <iostream>

class World {

private:
    int width, height, timestep;
    std::vector<char> currentCells, nextCells;
    size_t index(int x, int y) const;

public:
    World(int width, int height);
    friend std::ostream& operator<<(std::ostream &os, const World &rhs);
    friend std::ostream& headerLine(std::ostream &os, const World &rhs);

    char & operator()(int x, int y);
    const char& operator()(int x, int y) const;
};

World.cpp文件:

#include "World.h"

World::World(int _width, int _height) : width(_width), height(_height), timestep(0),
                currentCells(_width * _height, ' '), nextCells(_width * _height) {}

std::ostream& operator<<(std::ostream& os, const World &rhs) {

    os << "timestep " << rhs.timestep << "\n";

    headerLine(os, rhs);
    for (auto i = 0; i < rhs.height; i++) {
        os << "|";
        for (auto j = 0; j < rhs.width; j++) {
            if (rhs.currentCells[j + (i * rhs.width)] == ' ') {
                os << ' ';
            } else {
                os << '0';
            }
        }
        os << "|\n";
    }
    headerLine(os, rhs);
    return os;
}

std::ostream& headerLine(std::ostream& os, const World &rhs) {
    os << "+";
    for (auto i = 0; i < rhs.width; i++) {
        os << "-";
    }
    os << "+\n";
    return os;
}

size_t World::index(int x, int y) const{
    return y * width + x;
}

char& World::operator()(int x, int y) {
    return currentCells[index(x, y)];
}

const char& World::operator()(int x, int y) const {
    return currentCells[index(x,y)];
}

主类文件:

#include "World.h"

int main() {
    World grid(10,5);

    grid(6,1) = 1;
    grid(5,1) = 1;

    std::cout << grid << std::endl;

    std::cout << "(6, 1) = " <<  grid(6,1) << std::endl;

    return 0;
}

控制台上的预期输出:

(6, 1) = 1

控制台上的实际输出:

(6, 1) =

1 个答案:

答案 0 :(得分:1)

这两个重载运算符:

char& World::operator()(int x, int y) {
    return currentCells[index(x, y)];
}

const char& World::operator()(int x, int y) const {
    return currentCells[index(x,y)];
}

正在返回char&const char&,并在cout语句中使用它时,您已经完成了以下操作:

std::cout << "(6, 1) = " <<  grid(6,1) << std::endl;

它要么根据您的计算机的体系结构和操作系统来打印一些字符。它可能是ASCII可打印或不可打印的字符,您甚至可能听到来自内部蜂鸣器的提示音,或者可能是ASCII以外的其他编码字符格式。

正如其他人在评论中所述,您需要将其提升为int类型。最简单的方法是执行以下操作:

std::cout << "(6, 1) = " << +grid(6,1) << std::endl;

我在这里所做的只是在+调用之前添加一个operator。这是一元加法,会将char类型提升为int类型。