Question

在我的图像类中，我希望在将像素传递给图像后更改像素，并且它们仍然应该更改图像。

int main(int argc, char* argv[]){

  Image theImage(4, 8);//width/height
  Pixel aPixel(2,1);
  Pixel* p = &aPixel;
  theImage.setPixel(p);

  aPixel.setBlue(100);//change the blue (RGB) value of the pixel, but Pixel color doesnt change

  theImage.saveAsPixelMap("/Users/dan/Desktop/test.ppm");
  return 0;
}

我认为Pixel的颜色会发生变化，因为Imageclass保持指针，当指针仍然指向相同的Pixel时，Color会改变，图像中的Pixel的颜色是不是会改变？

这是Pixel构造函数：

Pixel::Pixel(int tx, int ty){
  red = 255;
  green = 0;
  blue = 0;
  x = tx;
  y = ty;
  hasBeenChanged = false;
}

和setPixel方法

void Image::setPixel(Pixel *aPixel){
  int tX = aPixel->getX();
  int tY = aPixel->getY();
  imageData.at(tY).at(tX)->setRed(aPixel->getRed());//value 0 - 255
  imageData.at(tY).at(tX)->setGreen(aPixel->getGreen());
  imageData.at(tY).at(tX)->setBlue(aPixel->getBlue());  
}

imageData看起来像这样

std::vector< std::vector<Pixel*> > imageData;

和saveAsPixelmap方法。

void Image::saveAsPixelMap(char aPath[]){

  std::ofstream myfile;
  myfile.open(aPath);

  myfile << "P3\n" << this->getWidth() <<" "<< this->getHeight() <<"\n255\n";
  std::vector < Pixel* > row;
  for (int y = 0; y < this->getHeight(); y++){
    row = imageData.at(y);
    for (int x = 0; x < this->getWidth(); x++){

        myfile << row.at(x)->getRed() << " ";
        myfile << row.at(x)->getGreen() << " ";
        myfile << row.at(x)->getBlue() << " ";
        std::cout <<"rot: "<< imageData.at(y).at(x)->getRed();

    }
  }
  std::cout << "\n Writing File to " << aPath << "\n \n";
  myfile.close();
}

好吧，这是很多代码，请问我是否需要更多关于某些事情的信息或者我的问题不够明确。任何提示如何解决这个问题

Answer 1

setPixel方法应该引用一个指针：

void Image::setPixel(Pixel *& aPixel) { .. }

Answer 2

您实施的概念与您描述的概念不同：

描述的是，Image
已实现纯复制语义（从给定像素读取并将值放到其他像素中）

你需要Image类的某种方法来返回一个像素 - 这个方法可以改变。

示例：

class Image { 
// ..
Pixel & get_pixel(int x, int y) { /* */ }
}

然后你可以用：

改变像素（之后）

image.get_pixel(2,1).setBlue(100)

更改指针指向的对象，但对象不会更改

2 个答案: