对这里使用的适当设计模式感到困惑

Question

我正在实现一个类，可以从各种来源下载文件，如ftp，http等。我开始使用以下界面

class IInternetFileDownloader
{
public:
    IInternetFileDownloader() = default;

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

};

然后，我实现了将从适当的端点执行实际下载的类。所以我有一个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
    }

};

我可以调用适当的类，如下所示

#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。在我看来，应该有另一个类可以只接受url并根据协议，它可以构造适当的类。这样客户端代码（main.cpp）不需要担心适当类的实例化。我阅读了有关工厂和构建器设计模式的内容，并对在这种情况下最适合使用哪种模式感到困惑？

Answer 1

最简单的方法是在IInternetFileDownloader上使用静态函数来实例化正确的子类。

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

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;
};

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

另一个变体是将“工厂”作为一个单独的类可能是有意义的。一个优点是它可以在单元测试中更好地模拟下载器的实例。

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
    }
};