涉及 类 的变异困难
Difficulty with mutation involving classes
我有一个 class、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<int> > temp = inList.front().getPix();
normalize(temp);
但这不会完成我想要的,即改变 characterData
的 pixelData
。为什么不能直接访问pixelData
进行变异?
您的 getPix
方法 return 是一个临时对象。
vector<vector<int> > getPix() {
~~~~~~~~~~~~~~~~~~~~
return pixelData;
}
您可能想要return它作为参考。
您混淆了指针和引用的使用。
在函数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()
returns。但是 normalize()
需要一个引用(因此编译器会抱怨类似于将引用传递给临时对象的东西 - 临时对象保存由 getPix()
编辑的值 return,它将是pixelData
)。相反,要以这种方式使用它,getPix()
需要 return 一个参考。
vector<vector<int> > &getPix() {return pixelData;} // note the ampersand
这解决了您提出的问题。更一般地说,您的设计存在致命缺陷 - characterData
的成员函数提供对任何 private
成员的直接访问(通过指针或引用)是一个坏主意,因为它允许任何代码更改该成员.如果你打算这样做,你还不如让成员 public
而不是 private
.
您需要为 characterData
提供成员函数,允许调用者请求更改 private
成员函数(例如添加元素)。如果您选择,这些功能可以忽略此类请求。更重要的是,它们允许您的 class 及其成员函数共同确保数据始终处于有效状态。如果调用者可以进行任意更改,这是不可能的。
我有一个 class、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<int> > temp = inList.front().getPix();
normalize(temp);
但这不会完成我想要的,即改变 characterData
的 pixelData
。为什么不能直接访问pixelData
进行变异?
您的 getPix
方法 return 是一个临时对象。
vector<vector<int> > getPix() {
~~~~~~~~~~~~~~~~~~~~
return pixelData;
}
您可能想要return它作为参考。
您混淆了指针和引用的使用。
在函数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()
returns。但是 normalize()
需要一个引用(因此编译器会抱怨类似于将引用传递给临时对象的东西 - 临时对象保存由 getPix()
编辑的值 return,它将是pixelData
)。相反,要以这种方式使用它,getPix()
需要 return 一个参考。
vector<vector<int> > &getPix() {return pixelData;} // note the ampersand
这解决了您提出的问题。更一般地说,您的设计存在致命缺陷 - characterData
的成员函数提供对任何 private
成员的直接访问(通过指针或引用)是一个坏主意,因为它允许任何代码更改该成员.如果你打算这样做,你还不如让成员 public
而不是 private
.
您需要为 characterData
提供成员函数,允许调用者请求更改 private
成员函数(例如添加元素)。如果您选择,这些功能可以忽略此类请求。更重要的是,它们允许您的 class 及其成员函数共同确保数据始终处于有效状态。如果调用者可以进行任意更改,这是不可能的。