c++ - 涉及类别的变异困难

我有一个类characterData和此类型的列表:

class characterData {
    private:
        vector<vector<int> > pixelData;
        int area;
    public:
        void setPix(vector<vector<int> > in) {
            pixelData = in;
            area = in.size() * in[0].size();
        }
        vector<vector<int> > getPix() {
            return pixelData;
        }
        int getArea() {
            return area;
        }
};

list<characterData> listChars;

我正在尝试通过其他方法来改变pixelData，但是仍然遇到问题。我有一个对pixelData起作用的方法:

void processHandler(list<characterData> &inList) {
    while (!inList.empty()) {
        normalize(inList.front().getPix());
        inList.pop_front();
    }
}

void normalize(vector<vector<int> > &input) {
    if (input.size() > input[0].size()) {
        affixRows(input, input.size() - input[0].size());
    } else if (input.size() > input[0].size()) {
        affixCols(input, input[0].size() - input.size());
    }
}

void affixRows(vector<vector<int>> &input, int dimDiff) {
    vector<vector<int>> temp(input.size(), vector<int>(input[0].size() + dimDiff));
    for (unsigned int i = 0; i < temp[0].size(); i++) {
        for (unsigned int j = 0; j < temp.size(); j++) {
            if (i < (unsigned int)dimDiff / 2) {
                temp[j][i] = 255;
            }
            else if (i < input[0].size() + (int)dimDiff / 2) {
                temp[j][i] = input[j][i - (int)dimDiff / 2];
            }
            else {
                temp[j][i] = 255;
            }
        }
    }
    input = temp; // point of mutation
}

void affixCols(vector<vector<int>> &input, int dimDiff) {
    vector<vector<int>> temp(input.size() + dimDiff, vector<int>(input[0].size()));
    for (unsigned int i = 0; i < temp[0].size(); i++) {
        for (unsigned int j = 0; j < temp.size(); j++) {
            if (j < (unsigned int)dimDiff / 2) {
                temp[j][i] = 255;
            }
            else if (j < input.size() + (int)dimDiff / 2) {
                temp[j][i] = input[j - (int)dimDiff / 2][i];
            }
            else {
                temp[j][i] = 255;
            }
        }
    }
    input = temp; // point of mutation
}

我得到这些错误:

error: invalid initialization of non-const reference of type 'std::vector<std::vector<int> >&' from an rvalue of type 'std::vector<std::vector<int> >'
normalize(inList.front().getPix());
-------------------------------^--

如果不是:

normalize(inList.front().getPix());

我制作了一个临时 vector 并使用它，它没有错误。

vector<vector<int> > temp = inList.front().getPix();
normalize(temp);

但这并不能实现我想要的，即使pixelData的characterData突变。为什么我不能直接出于突变目的访问pixelData？

最佳答案

您正在混淆使用指针和引用。

在void processHandler(list<characterData> *inList)函数中，inList是一个指针，但是函数内的代码将其用作引用。

while (!inList.empty())
{
    normalize(inList.front().getPix());
    inList.pop_front();
}

需要将inList视为指针。因此，将inList.替换为inList->。

while (!inList->empty())
{
    normalize(inList->front().getPix());
    inList->pop_front();
}

由于characterData::getPix()按值返回，这将带来第二个问题。但是normalize()需要一个引用(因此，编译器会抱怨类似于将引用传递给临时文件-临时文件包含getPix()返回的值，该值将是pixelData的副本)。相反，要以这种方式使用它，getPix()需要返回一个引用。

vector<vector<int> > &getPix() {return pixelData;}    // note the ampersand

这可以回答您的问题。更一般而言，您的设计存在致命缺陷-characterData的成员函数提供对任何private成员的直接访问(通过指针或引用)是一个坏主意，因为它允许任何代码更改该成员。如果要执行此操作，则最好将成员设为public而不是private。

您需要提供characterData的成员函数，以允许调用者请求private成员函数的更改(例如，添加元素)。这些功能-如果您选择-可以忽略此类请求。更重要的是，它们允许您的类及其成员函数共同确保数据始终处于有效状态。如果 call 者可以进行任意更改，则不可能。