对此处使用的适当设计模式感到困惑

Question

我正在实现一个 class，它可以从 ftp、http 等各种来源下载文件。我从以下界面开始

class IInternetFileDownloader
{
public:
    IInternetFileDownloader() = default;

    virtual void Download(const std::string& urlToDownloadFrom) = 0;

};

然后我实施了 classes 将从适当的端点执行实际下载。所以我有一个 HttpFileDownloader.h 如下

#include "IInternetFileDownloader.h"
class HttpFileDownloader : public IInternetFileDownloader
{
public:
    HttpFileDownloader() = default;

    virtual void Download(const std::string& urlToDownloadFrom)
    {
        // download implementation
    }

};

所以我有一个 FtpFileDownloader.h 如下

#include "IInternetFileDownloader.h"
class FtpFileDownloader : public IInternetFileDownloader
{
public:
    FtpFileDownloader() = default;

    virtual void Download(const std::string& urlToDownloadFrom)
    {
        // download implementation
    }

};

我可以如下调用适当的 class

#include "IInternetFileDownloader.h"
#include "HttpFileDownloader.h"

int main()
{
    std::string downloadFromUrl = "http://www.example.org/xyz.zip";
    IInternetFileDownloader* download = new HttpFileDownloader();
    download->Download(downloadFromUrl);
}

但是，我不想在这里实例化特定的 HttpFileDownloader 或 FtpFileDownloader。在我看来，应该有另一个 class 可以只接受 url 并且根据协议，它可以构造适当的 class。这样客户端代码 (main.cpp) 就不需要担心适当 class 的实例化。我阅读了有关工厂和构建器设计模式的内容，但对在这种情况下最适合使用哪一个感到有点困惑？

Answer 1

最简单的方法是在 IInternetFileDownloader 上有一个静态函数来实例化正确的子class。

此外，我认为您不需要基 class 中的默认构造函数，但您可能需要基 class 中虚拟的默认析构函数。这样，我建议的工厂函数 CreateDownloader 可以 return 一个指向您稍后删除的 IInternetFileDownloader 实例的指针（或 shared_ptr）。

class IInternetFileDownloader
{
public:
    virtual ~IInternetFileDownloader() = default;
    virtual void Download(const std::string& urlToDownloadFrom) = 0;

    // parses the url to infer the protocol and construct an instance of a derived class
    static IInternetFileDownloader* CreateDownloader(const std:string& url);
};

class HttpFileDownloader : public HttpFileDownloader 
{
public:
    virtual void Download(const std::string& urlToDownloadFrom) override;
};

class FtpFileDownloader : public IInternetFileDownloader
{
public:
    virtual void Download(const std::string& urlToDownloadFrom) override;
};

这可能就是我要采用的方法。

另一个变体是将 "factory" 作为一个单独的 class 可能有意义。一个优点是它可以更好地在单元测试中模拟下载程序的实例。

class IInternetFileDownloader
{
public:
    virtual ~IInternetFileDownloader() = default;
    virtual void Download(const std::string& urlToDownloadFrom) = 0;
};

class InternetFileDownloaderFactory
{
public:
    // parses the url to infer the protocol and construct an instance of a derived class
    virtual IInternetFileDownloader* CreateDownloader(const std:string& url);
};

class InternetFileDownloaderFactoryMock : public InternetFileDownloaderFactory
{
public:
    IInternetFileDownloader* CreateDownloader(const std:string& url) override
    {
         return new MockFileDownloaderFactoryMock();
    }
};

class MockFileDownloaderFactoryMock : public IInternetFileDownloader
{
public:
    virtual void Download(const std::string& urlToDownloadFrom)
    {
       // do nothing or simulate a download
    }
};

对此处使用的适当设计模式感到困惑

Confused about the appropriate design pattern to be used here

c++

design-patterns

factory

builder