装饰员模式混乱

Question

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

public interface IConfigPathProvider
{
    string GetConfigPath();
}

该路径存储在控制台应用程序的app.config文件的appSettings部分中。

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

划分et impera

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

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

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

Answer 1

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

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

Answer 2

你不一定正确。 而不是立即装饰对象，保持某种装饰shema ，validatable，lazy，可以通过调用，比如说.Build()将其转换为所需的（最终的，即用的）对象。 只是代码草图： obj.DecorateWith<Decorator1>().DecorateWith<Decorator2>().DecorateWith(() => new Decorator3(IContainer.Resolve<SomeWhatArgument> ...).Build() 。这让事情变得更加困难但是，只要装饰是正确的方法，你的项目确实足够大，可以从如此高的抽象中受益，它将解决你的问题。