我有一个类，你在一个文件夹中传递它，然后它会关闭并处理指定文件夹中的大量数据。
例如：
MyClass myClass = new MyClass(@"C:\temp");
现在它会启动并读取几千个文件并用数据填充类。
我是否应该将这些数据从构造函数中移出并将其作为单独的方法，例如以下？
MyClass myClass = new MyClass();
myClass.LoadFromDirectory(@"C:\temp");
【问题讨论】：
标签： c# constructor
也许您应该使用返回对象实例的静态方法尝试这种方式。
var myClass = MyClass.LoadFromDirectory(@"C:\temp");
这会将初始化代码保留在您的构造函数之外，并为您提供您正在寻找的“单行”声明。
从海报下方继续评论，通过添加 State 实现可能是这样的：
public class MyClass
{
#region Constructors
public MyClass(string directory)
{
this.Directory = directory;
}
#endregion
#region Properties
public MyClassState State {get;private set;}
private string _directory;
public string Directory
{
get { return _directory;}
private set
{
_directory = value;
if (string.IsNullOrEmpty(value))
this.State = MyClassState.Unknown;
else
this.State = MyClassState.Initialized;
}
}
#endregion
public void LoadFromDirectory()
{
if (this.State != MyClassState.Initialized || this.State != MyClassState.Loaded)
throw new InvalidStateException();
// Do loading
this.State = MyClassState.Loaded;
}
}
public class InvalidStateException : Exception {}
public enum MyClassState
{
Unknown,
Initialized,
Loaded
}
【讨论】：
这取决于。您应该评估课程的基本目的。它执行什么功能？
我通常更喜欢让类构造函数执行类运行所需的初始化。然后我调用类上的方法，这些方法可以安全地假设必要的初始化已经完成。
通常，初始化阶段不应过于密集。执行上述操作的另一种方法可能是：
// Instantiate the class and get ready to load data from files.
MyClass myClass = new MyClass(@"C:\temp");
// Parse the file collection and load necessary data.
myClass.PopulateData();
【讨论】：
我同意 Ari 和其他人的观点 - 将他们分开。
构造函数确实应该做最少的工作（只需初始化准备好使用的对象并保留它）。通过使用单独的方法来完成工作：
【讨论】：
这就是你的班级所做的一切吗？如果是这样，那么我会说这并不重要。但很可能你的班级实际上比你所展示的要多。例如，它是否有任何错误处理？
构造函数的目的是构造一个对象。方法的目的是执行一个动作。所以我的投票是这种形式：
MyClass myClass = new MyClass();
myClass.LoadFromDirectory(@"C:\temp");
【讨论】：
我认为您应该根据您是否计划重用同一个对象以对不同的输入执行相同的操作来决定上述两种方法（“首先初始化，然后执行”与“空初始化，使用参数执行”） .
如果该类仅用于在固定参数上运行任务，我将在构造函数中对其进行初始化（使其成为只读），然后以不同的方法执行任务。
如果你想继续在不同的参数上执行任务，我会把它放在任务方法本身中。
如果该类所做的只是这个任务，我也会考虑将其全部更改为静态类/方法 - 它不需要保持其内部状态。
无论如何，我永远不会将任务本身放在构造函数中。正如 Cerebrus 所说，初始化应该很快。
【讨论】：
除非您的类的主要目的是执行 I/O，否则您可能不应该在构造函数中执行 I/O（可能引发 IOException）。
考虑将班级一分为二：
interface IMyDataSource
{
// ...
}
class FileDataSource: IMyDataSource
{
public FileDataSource(string path)
{
// ...
}
}
class MyClass
{
public MyClass(IMyDataSource source)
{
// ...
}
}
IMyDataSource myDS = new FileDataSource(@"C:\temp");
MyClass myClass = new MyClass(myDS);
这样主类可以专注于管理自己的状态，而数据源则根据文件内容构建初始状态。
【讨论】：
如果这是该类使用的唯一资源，最好将路径传递给构造函数。否则，它将成为您班级成员的参数。
【讨论】：
我个人的偏好是使用 C# 3.0 初始化程序。
class MyClass {
public string directory;
public void Load() {
// Load files from the current directory
}
}
MyClass myClass = new MyClass{ directory = @"C:\temp" };
myClass.Load();
这有几个优点：
【讨论】：
我将在这里回应“将他们分开”的人们。如果有帮助，试试这个：
在这里应用它，你会得到这个：
在我看来，这比“构造函数应该创建对象并从文件系统加载其数据。
【讨论】：