在构造函数中创建目录是个坏主意吗?

如何解决在构造函数中创建目录是个坏主意吗?

| 乍一看,这似乎很自然-如果不存在一组目录,则在其上工作的对象将无法履行其合同。因此,在构造函数中,我有一些逻辑来检查是否存在某些目录,如果不存在则创建它们。尽管实际上还不是单例对象,但该对象的使用就像一个对象。 构造函数是否对这种设置逻辑不利? 背景 该类称为FileGetter。它抽象了从远程服务器获取特定文件,提取它们,准备文件并将它们放置在另一个目录中的另一个目录,该目录中的第二类将是文件系统监视/处理数据。     

解决方法

从控制反转或依赖反转的角度来看,是的,这是不正确的。 您声明,如果目录中不存在的对象无法工作,那么它们将无法工作。我将目录的提供和检查/创建抽象为另一个抽象,然后将该抽象的实现传递给您的对象。 然后,您的对象只需从该抽象中获取目录,然后从那里继续。 举例来说,这就是我的意思。首先,存在目录提供程序的抽象,如下所示:
public interface IDirectoryProvider
{
    // Gets the full paths to the directories being worked on.
    IEnumerable<string> GetPaths();
}
然后是实现。
public sealed class DirectoryProvider
{
    public DirectoryProvider(IEnumerable<string> directories)
    {
        // The validated directories.
        IList<string> validatedDirectories = new List<string>();

        // Validate the directories.
        foreach (string directory in directories)
        {
            // Reconcile full path here.
            string path = ...;

            // If the directory doesn\'t exist,create it.
            Directory.CreateDirectory(path);

            // Add to the list.
            validatedDirectories.Add(path);
        }
    }

    private readonly IEnumerable<string> _directories;

    public IEnumerable<string> GetPaths()
    {
         // Just return the directories.
         return _directories;
    }
}
最后,您的类将处理目录,如下所示:
public sealed DirectoryProcessor
{
    public DirectoryProcessor(IDirectoryProvider directoryProvider)
    {
        // Store the provider.
        _directoryProvider = directoryProvider;
    }

    private readonly IDirectoryProvider _directoryProvider;

    public void DoWork()
    {
        // Cycle through the directories from the provider and
        // process.
        foreach (string path in _directoryProvider.GetPaths())
        {
            // Process the path
            ...
        }
    }
}
    ,我会说这取决于。一般来说,使对象构造尽可能便宜是一个好主意。也就是说,使构造函数包含尽可能少的逻辑量。这不利于在构造函数中创建目录。另一方面,如果没有目录该对象实际上根本无法运行,那么最好尽早执行失败(例如,由于某种原因而无法创建目录)。这可能代表在构造函数中创建它们。 就我个人而言,我可能倾向于不在构造函数中创建它们,而是让每个需要它们的方法调用某个创建目录的方法(如果尚未创建)。     ,您可以在类本身或将占用该类的每段代码中放入处理缺少文件夹的逻辑。 选择实际上取决于您。如果选择将其放在类本身中,则构造函数非常适合它,因为构造函数可以“设置”该类及其所需的所有功能。     ,我喜欢casperOne的回答。但是我会考虑以下内容: 此独特对象多久创建一次。 正在创建多少个目录。 构造函数多久用于创建一个目录 您是否需要有关目录状态的反馈 您是否需要有关目录不存在或任何其他错误的原因的反馈。 最后但并非最不重要的一点:值得尝试另一种方式吗? 根据这些问题,您可能会意识到,这只是对象生命周期中一次被调用的情况。它不应失败,并且如果执行其他操作,则将完全出错。因此,我决定保持逻辑不变。 如果您有时间并且打算扩展程序,请在其他地方使用该类,或者创建更多目录,如casperOne所述,可能值得创建目录提供程序。如果要创建一个以上的目录,例如在程序的每个开始处创建多个目录,并且/或者您可以重用代码并尽可能灵活……那么,我强烈建议您使用目录提供者,从而使对象更加灵活,并减少了在构造函数内部创建SPOF的机会。     ,我认为有一种构造器可用于这种事情?     

版权声明:本文内容由互联网用户自发贡献,该文观点与技术仅代表作者本人。本站仅提供信息存储空间服务,不拥有所有权,不承担相关法律责任。如发现本站有涉嫌侵权/违法违规的内容, 请发送邮件至 dio@foxmail.com 举报,一经查实,本站将立刻删除。

相关推荐


依赖报错 idea导入项目后依赖报错,解决方案:https://blog.csdn.net/weixin_42420249/article/details/81191861 依赖版本报错:更换其他版本 无法下载依赖可参考:https://blog.csdn.net/weixin_42628809/a
错误1:代码生成器依赖和mybatis依赖冲突 启动项目时报错如下 2021-12-03 13:33:33.927 ERROR 7228 [ main] o.s.b.d.LoggingFailureAnalysisReporter : *************************** APPL
错误1:gradle项目控制台输出为乱码 # 解决方案:https://blog.csdn.net/weixin_43501566/article/details/112482302 # 在gradle-wrapper.properties 添加以下内容 org.gradle.jvmargs=-Df
错误还原:在查询的过程中,传入的workType为0时,该条件不起作用 &lt;select id=&quot;xxx&quot;&gt; SELECT di.id, di.name, di.work_type, di.updated... &lt;where&gt; &lt;if test=&qu
报错如下,gcc版本太低 ^ server.c:5346:31: 错误:‘struct redisServer’没有名为‘server_cpulist’的成员 redisSetCpuAffinity(server.server_cpulist); ^ server.c: 在函数‘hasActiveC
解决方案1 1、改项目中.idea/workspace.xml配置文件,增加dynamic.classpath参数 2、搜索PropertiesComponent,添加如下 &lt;property name=&quot;dynamic.classpath&quot; value=&quot;tru
删除根组件app.vue中的默认代码后报错:Module Error (from ./node_modules/eslint-loader/index.js): 解决方案:关闭ESlint代码检测,在项目根目录创建vue.config.js,在文件中添加 module.exports = { lin
查看spark默认的python版本 [root@master day27]# pyspark /home/software/spark-2.3.4-bin-hadoop2.7/conf/spark-env.sh: line 2: /usr/local/hadoop/bin/hadoop: No s
使用本地python环境可以成功执行 import pandas as pd import matplotlib.pyplot as plt # 设置字体 plt.rcParams[&#39;font.sans-serif&#39;] = [&#39;SimHei&#39;] # 能正确显示负号 p
错误1:Request method ‘DELETE‘ not supported 错误还原:controller层有一个接口,访问该接口时报错:Request method ‘DELETE‘ not supported 错误原因:没有接收到前端传入的参数,修改为如下 参考 错误2:cannot r
错误1:启动docker镜像时报错:Error response from daemon: driver failed programming external connectivity on endpoint quirky_allen 解决方法:重启docker -&gt; systemctl r
错误1:private field ‘xxx‘ is never assigned 按Altʾnter快捷键,选择第2项 参考:https://blog.csdn.net/shi_hong_fei_hei/article/details/88814070 错误2:启动时报错,不能找到主启动类 #
报错如下,通过源不能下载,最后警告pip需升级版本 Requirement already satisfied: pip in c:\users\ychen\appdata\local\programs\python\python310\lib\site-packages (22.0.4) Coll
错误1:maven打包报错 错误还原:使用maven打包项目时报错如下 [ERROR] Failed to execute goal org.apache.maven.plugins:maven-resources-plugin:3.2.0:resources (default-resources)
错误1:服务调用时报错 服务消费者模块assess通过openFeign调用服务提供者模块hires 如下为服务提供者模块hires的控制层接口 @RestController @RequestMapping(&quot;/hires&quot;) public class FeignControl
错误1:运行项目后报如下错误 解决方案 报错2:Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.8.1:compile (default-compile) on project sb 解决方案:在pom.
参考 错误原因 过滤器或拦截器在生效时,redisTemplate还没有注入 解决方案:在注入容器时就生效 @Component //项目运行时就注入Spring容器 public class RedisBean { @Resource private RedisTemplate&lt;String
使用vite构建项目报错 C:\Users\ychen\work&gt;npm init @vitejs/app @vitejs/create-app is deprecated, use npm init vite instead C:\Users\ychen\AppData\Local\npm-