从不同的类访问对象 - 设计

Question

我有三个类，TImageProcessingEngine、TImage 和 TProcessing

TImageProcessingEngine 是我用来公开所有我的类的类。走向世界的方法。

TImage 是我计划使用通用图像读取和图像写入功能的一个。

TProcessing 包含执行成像操作的方法。

class TImageProcessingEngine
{
    public:
        TImage* mpImageProcessingEngine;

};

class TImage
{
    public:
        int ReadImage();
        int WriteImage();

    private:
            //a two dimensional array holding the pixel values
        tImageMatrix*  mpImageMatrix;
};

class TProcessing
{
    public:
        int ConvertToBinary();
        int ConvertToGrayScale();
};

我的问题是如何访问TProcessing类中的对象mpImageMatrix？以便我的调用应用程序可以使用以下内容

TImageProcessingEngine* vEngine = new TImageProcessingEngine;

//Converts an input gray scsale image to binary image
vEngine->ReadImage().ConvertToBinary();

//Write the converted image to disk
vEngine->WriteImage();

delete vEngine;
vEngine = NULL;

//During this whole processing internally, 
//the image is read in to `mpImageMatrix` 
//and will also be holding the binarised image data, 
//till writing the image to disk.

或者您会吗推荐我的班级设计的任何其他方法吗？

Answer 1

我当然会推荐一种不同的实现，但让我们先检查一下设计。

我不太明白 TImageProcessingEngine 的附加值，它没有带来任何功能。

事实上，我的建议非常简单：

• Image类，保存值
• Processing类（接口），应用操作
• Encoder和Decoder 类（接口），用于读取和写入不同的格式

只有当您能够从中获得效率时，Processing 类才能访问内部图像，这确实有意义（这很可能），在这种情况下，您可以简单地使 Processing 成为朋友，并让它解压其派生的 Encoder 和 Decoder 的值，

class Image
{
public:
  Image();

  void Accept(Processing& p);
  void Encode(Encoder& e) const; // Image is not modified by encoding

  void Decode(Decoder& d); // This actually resets the image content

private:
  friend class Processing;

  size_t mHeight;
  size_t mWidth;
  std::vector<Pixel> mPixels; // 2D array of Pixels
};

class Processing
{
public:
  void apply(Image& image)
  {
    this->applyImpl(image.mHeight, image.mWidth, image.mPixels);
  }

private:
  virtual void applyImpl(size_t h, size_t w, std::vector<Pixel>& pixels) = 0;
};

遵循相同的原则。

请注意，我从来不需要显式指针，以及由此保证的正确性。

Answer 2

首先，根据您提供的代码，没有 ReadImage() & WriteImage() 函数位于 TImageProcessingEngine 类中，因此后面使用此类功能的代码存在缺陷。

至于解决方案，您可以为 tImageMatrix 指针创建一个 getter 函数，如下所示：

tImageMatrix* GetImageMatrix() { return mpImageMatrix; }

然后将该指针（或指向整个 TImage 实例的指针）传递给您要调用的 TProcessing 函数。

Answer 3

为什么你想要一个单独的 TProcessing 进程，当它专门具有仅访问 mpImageMatrix; 的函数时

，在 OOP 中，你必须绑定 数据成员及其操作..

因此，IMO，删除您的 TProcessing 类并在 TImage 中拥有这两个函数..

您的 TImage就像，

class TImage
{
public:
    int ReadImage();
    int WriteImage();
    int ConvertToBinary();
    int ConvertToGrayScale();

private:
        //a two dimensional array holding the pixel values
    tImageMatrix*  mpImageMatrix;
};

Answer 4

您可以创建一个访问器 TImage 类：

byte * pixelAt(unsigned x, unsigned y);