装饰图案杂物

Question

我有一个问题，如果我使用的装饰模式是正确的方式。让我们假设我正在处理一个控制台应用程序。在这个应用程序中，我定义了一个简单的IConfigPathProvider接口，它将提供某个类的配置文件路径。

public interface IConfigPathProvider
{
    string GetConfigPath();
}

public class AppSettingsConfigPathProvider : IConfigPathProvider
{
    public string GetConfigPath()
    {
        return System.Configuration.ConfigurationManager.AppSettings["configPath"];
    }
}

public class DecryptingConfigPathProvider : IConfigPathProvider
{
    private readonly IConfigPathProvider _provider;
    private readonly IStringDecrypter _decrypter;

    public DecryptingConfigPathProvider(IConfigPathProvider provider, 
        IStringDecrypter decrypter)
    {
        _provider = provider ?? throw new ArgumentNullException(nameof(provider));
        _decrypter = decrypter ?? throw new ArgumentNullException(nameof(decrypter));
    }

    public string GetConfigPath()
    {
        var path = _provider.GetConfigPath();
        //decrypting method of IStringDecrypter interface
        return _decrypter.DecryptString(path);
    }
}

public class AppendSectionConfigPathProvider : IConfigPathProvider
{
    private readonly IConfigPathProvider _provider;

    public AppendSectionConfigPathProvider(IConfigPathProvider provider)
    {
        _provider = provider ?? throw new ArgumentNullException(nameof(provider));
    }

    public string GetConfigPath()
    {
        var path = _provider.GetConfigPath();

        return System.IO.Path.Combine(
            System.IO.Path.GetDirectoryName(path),
            "section", 
            System.IO.Path.GetFileName(path));
    }
}

public class LoggingConfigPathProvider : IConfigPathProvider
{
    private readonly static ILog _log = 
        LogManager.GetLogger(typeof(LoggingConfigPathProvider));

    private readonly IConfigPathProvider _provider;

    public LoggingConfigPathProvider(IConfigPathProvider provider)
    {
        _provider = provider ?? throw new ArgumentNullException(nameof(provider));
    }

    public string GetConfigPath()
    {
        _log.Info("Getting config path...");
        var path = _provider.GetConfigPath();

        _log.Info("Config path retrieved successfully!");
        return path;
    }
}

当然，即时的结果是关注点的分离，但是实例化对象时增加的复杂性呢？您需要知道哪个decorator首先出现，以及它们应该按什么顺序链接，否则您最终将得到一个有缺陷的IconfigPathProvider。

new LoggingConfigPathProvider(
    new AppendSectionConfigPathProvider(
        new DecryptingConfigPathProvider(
            new AppSettingsConfigPathProvider(), 
            decrypter));

而这只是一个简单的提供者。在相当复杂的应用程序中，您可能会遇到具有多个引用的多个组件...这很容易导致维护噩梦。现在，这是一个众所周知的缺点还是我只是以错误的方式使用了这个模式？

Answer 1

这是一个众所周知的缺点。GoF提到Decorator模式的以下责任。

很多小东西。使用装饰器的设计通常会导致由许多看起来相似的小对象组成的系统。这些对象的不同之处在于它们相互连接的方式，而不在于它们的类或变量的值。虽然这些系统很容易被理解它们的人定制，但它们可能很难学习和调试。