以前一直很自大,觉得自己C/C++还是很牛的,实际上C方面硬件资源方面不了解,C++方面更是只知道better C。代码怎么写都觉得不漂亮,自己关都过不了。
首 当其冲资源管理问题(也是看了Effective C++才知道还有这方面的问题),拿mvr_image来说ViewImage是基本的数据对象的接口类,如何来管理ViewImage的构造是个问题, 原则是要在使用时采用Factory方法而不是需要用户直接调用new。重构一晚上(一晚上就这点成果......)后代码如下:
namespace
mvr
{
![](https://i-blog.csdnimg.cn/blog_migrate/6a9c071a08f1dae2d3e1c512000eef41.gif)
class ViewImage
{
public:
// Virtual Destructor
virtual ~ViewImage() = 0;
![](https://i-blog.csdnimg.cn/blog_migrate/6a9c071a08f1dae2d3e1c512000eef41.gif)
virtual void ShowImage() const = 0;
virtual void DrawFeatures() = 0;
virtual int GetHeight() const = 0;
virtual int GetWidth() const = 0;
virtual const CvMat * GetCVMat() const = 0;
virtual vector& GetFeatures() = 0;
virtual string GetFilename() const = 0;
};
![](https://i-blog.csdnimg.cn/blog_migrate/6a9c071a08f1dae2d3e1c512000eef41.gif)
// View Image Resource Manager
class ViewManager
{
public:
// Virtual Destructor
virtual ~ViewManager() = 0;
![](https://i-blog.csdnimg.cn/blog_migrate/6a9c071a08f1dae2d3e1c512000eef41.gif)
// Create ViewImage from Image File
virtual shared_ptr CreateViewImage(string filename) = 0;
};
![](https://i-blog.csdnimg.cn/blog_migrate/6a9c071a08f1dae2d3e1c512000eef41.gif)
// Factory Method For ViewManager
shared_ptr CreateViewManager();
![](https://i-blog.csdnimg.cn/blog_migrate/6a9c071a08f1dae2d3e1c512000eef41.gif)
}
//
namespace mvr
使用ViewManager来统一管理ViewImage的构造,这样还保留了扩展性,以后还可以通过ViewManager来构造图像序列对象。ViewManager也是接口类,使用独立的工厂方法来构造。
工厂方法返回的都是智能指针std::tr1::shared_ptr,析构就不用咱管了。
这代码也还看得过去了吧
补充一下,上面的代码编译不过,纯虚的析构函数也需要定义,改成
virtual
~
ViewManager()
{}
就可以了
首 当其冲资源管理问题(也是看了Effective C++才知道还有这方面的问题),拿mvr_image来说ViewImage是基本的数据对象的接口类,如何来管理ViewImage的构造是个问题, 原则是要在使用时采用Factory方法而不是需要用户直接调用new。重构一晚上(一晚上就这点成果......)后代码如下:
![](https://i-blog.csdnimg.cn/blog_migrate/a41954a27d6ad96fa2c2cf816e677448.gif)
![](https://i-blog.csdnimg.cn/blog_migrate/6a9c071a08f1dae2d3e1c512000eef41.gif)
![](https://i-blog.csdnimg.cn/blog_migrate/6a9c071a08f1dae2d3e1c512000eef41.gif)
![](https://i-blog.csdnimg.cn/blog_migrate/37c8bf68cdc3cc81759c34160776bc53.gif)
![](https://i-blog.csdnimg.cn/blog_migrate/6a9c071a08f1dae2d3e1c512000eef41.gif)
![](https://i-blog.csdnimg.cn/blog_migrate/6a9c071a08f1dae2d3e1c512000eef41.gif)
![](https://i-blog.csdnimg.cn/blog_migrate/6a9c071a08f1dae2d3e1c512000eef41.gif)
![](https://i-blog.csdnimg.cn/blog_migrate/6a9c071a08f1dae2d3e1c512000eef41.gif)
![](https://i-blog.csdnimg.cn/blog_migrate/6a9c071a08f1dae2d3e1c512000eef41.gif)
![](https://i-blog.csdnimg.cn/blog_migrate/6a9c071a08f1dae2d3e1c512000eef41.gif)
![](https://i-blog.csdnimg.cn/blog_migrate/6a9c071a08f1dae2d3e1c512000eef41.gif)
![](https://i-blog.csdnimg.cn/blog_migrate/6a9c071a08f1dae2d3e1c512000eef41.gif)
![](https://i-blog.csdnimg.cn/blog_migrate/6a9c071a08f1dae2d3e1c512000eef41.gif)
![](https://i-blog.csdnimg.cn/blog_migrate/6a9c071a08f1dae2d3e1c512000eef41.gif)
![](https://i-blog.csdnimg.cn/blog_migrate/6a9c071a08f1dae2d3e1c512000eef41.gif)
![](https://i-blog.csdnimg.cn/blog_migrate/717446ca04a6125dc5b6b54e0fa14ab4.gif)
![](https://i-blog.csdnimg.cn/blog_migrate/6a9c071a08f1dae2d3e1c512000eef41.gif)
![](https://i-blog.csdnimg.cn/blog_migrate/6a9c071a08f1dae2d3e1c512000eef41.gif)
![](https://i-blog.csdnimg.cn/blog_migrate/6a9c071a08f1dae2d3e1c512000eef41.gif)
![](https://i-blog.csdnimg.cn/blog_migrate/37c8bf68cdc3cc81759c34160776bc53.gif)
![](https://i-blog.csdnimg.cn/blog_migrate/6a9c071a08f1dae2d3e1c512000eef41.gif)
![](https://i-blog.csdnimg.cn/blog_migrate/6a9c071a08f1dae2d3e1c512000eef41.gif)
![](https://i-blog.csdnimg.cn/blog_migrate/6a9c071a08f1dae2d3e1c512000eef41.gif)
![](https://i-blog.csdnimg.cn/blog_migrate/6a9c071a08f1dae2d3e1c512000eef41.gif)
![](https://i-blog.csdnimg.cn/blog_migrate/6a9c071a08f1dae2d3e1c512000eef41.gif)
![](https://i-blog.csdnimg.cn/blog_migrate/6a9c071a08f1dae2d3e1c512000eef41.gif)
![](https://i-blog.csdnimg.cn/blog_migrate/717446ca04a6125dc5b6b54e0fa14ab4.gif)
![](https://i-blog.csdnimg.cn/blog_migrate/6a9c071a08f1dae2d3e1c512000eef41.gif)
![](https://i-blog.csdnimg.cn/blog_migrate/6a9c071a08f1dae2d3e1c512000eef41.gif)
![](https://i-blog.csdnimg.cn/blog_migrate/6a9c071a08f1dae2d3e1c512000eef41.gif)
![](https://i-blog.csdnimg.cn/blog_migrate/6a9c071a08f1dae2d3e1c512000eef41.gif)
![](https://i-blog.csdnimg.cn/blog_migrate/0196c3df5ea9e936f21e9932cca91014.gif)
使用ViewManager来统一管理ViewImage的构造,这样还保留了扩展性,以后还可以通过ViewManager来构造图像序列对象。ViewManager也是接口类,使用独立的工厂方法来构造。
工厂方法返回的都是智能指针std::tr1::shared_ptr,析构就不用咱管了。
这代码也还看得过去了吧
补充一下,上面的代码编译不过,纯虚的析构函数也需要定义,改成
![](https://i-blog.csdnimg.cn/blog_migrate/a41954a27d6ad96fa2c2cf816e677448.gif)
就可以了