Question

假设我有一个Image类

class Image {
private:
  unsigned char* data;
  int sizex;
  int sizey;

public:
  Image(std::string filename);
};

和一个从磁盘读取图像的类：

class JpegReader;

在通过文件名构造函数读取图像时，如何设计这些类？我想减少耦合并避免循环依赖。

一个选项是让JpegReader成为Image类的朋友，让它写成员变量：

class JpegReader {
public:
  static void read(std::string filename, Image& image);
};

Image::Image(std::string filename) {
  JpegReader::read(filename, *this);
}

void JpegReader::read(std::string filename, Image& image) {
  image.data = readdata();
  image.sizex = readsizex();
  image.sizey = readsizey();
}

另一个选择是让一个构造函数包含所有成员变量作为参数：

Image(unsigned char* aData, int aSizex, int aSizey) : data(aData),
                                                      sizex(aSizex),
                                                      sizey(aSizey) {}

void JpegReader::read(std::string filename, Image& image) {
  Image tmp(readdata(), readsizex(), readsizey());
  std::swap(tmp, image);
}

或者让read（）返回一个Image，让Image构造函数执行swap：

class JpegReader {
public:
  static Image read(std::string filename);
};

Image::Image(std::string filename) {
  Image tmp = JpegReader::read(filename);
  std::swap(*this, tmp);
}

Image JpegReader::read(std::string filename) {
  Image tmp(readdata(), readsizex(), readsizey());
  return tmp;
}

你会如何设计它？

Answer 1

我可能会这样设计：

class Image {
public:
  Image(unsigned char* aData, int aSizex, int aSizey) { ... }
}

图像阅读器的抽象基类（允许您在不知道任何内容的情况下传递读者）：

class ImageReader {
public:
  virtual Image read(const std::string& filename) = 0;
};

具体读者（除了通过公共构造函数之外，不与Image耦合）：

class JpegReader: public ImageReader {
public:
  virtual Image read(const std::string& filename) {
    return Image(readdata(), readsizex(), readsizey());
  }
};

数据类型类和读取器/解析器之间的耦合

1 个答案: