我可以有一个 VegetableManager（或
东西) 加载方法所在的类
是，但这实际上是否意味着更少
重复代码？然后在每个屏幕中我
必须创建对象
蔬菜管理器并调用它的负载
反正方法。
这样做的重点不是效率（即性能）。关键是将加载该数据的细节封装到单个隔离对象中。例如，假设您的站点变得非常大，并且您决定将数据存储移动到数据库以实现可伸缩性和性能。在您描述的现有代码中，您必须遍历每个用户控件或页面并更改加载方法的逻辑。最好的情况是这很痛苦，最坏的情况是你错过了一些或复制粘贴不正确。如果逻辑被封装到一个专用对象中，其唯一职责就是知道如何从某个地方加载数据，那么您只需进行一次更改。
用户控件的代码隐藏：
protected void Page_Load(object sender, EventArgs e) {
var veggieManager = new VegetableManager();
VeggieListControl.DataSource = veggieManager.GetAll();
VeggieListControl.DataBind();
}
蔬菜管理器.cs：
public class VegetableManager {
private static Collection<Vegetable> _veggies;
private static object _veggieLock;
public ReadOnlyCollection<Vegetable> GetAll() {
if (_veggies == null) {
lock(_veggieLock) { //synchronize access to shared data
if (_veggies == null) { // double-checked lock
// logic to load the data into _veggies
}
}
}
return new ReadOnlyCollection(_veggies);
}
public void Add(Vegetable veggie) {
GetAll(); // call this to ensure that the data is loaded into _veggies
lock(_veggieLock) { //synchronize access to shared data
_veggies.Add(veggie);
// logic to write out the updated list of _veggies to the file
}
}
}
因为_veggies 是static，所以内存中只有一个蔬菜集合，尽管多个调用者会实例化VegetableManager。但是因为它是静态的，所以如果您有一个多线程应用程序（例如网站），您必须在所有线程中同步对该字段的访问（因此是 locks）。
就良好的面向对象而言，这只是冰山一角。我建议仔细阅读 UncleBob's SOLID principles 和 Domain-Driven Design (free e-book)。
所以，是的，您正在重复某些事情，但您所重复的只是一个方法调用，并且可以重复。 DRY 意味着减少“逻辑”代码的重复，即决策和算法；简单的方法调用不属于此范围。但是，如果您愿意，您可以将逻辑合并到一个基类中来执行此操作，从而有效地将用户控件与必须了解蔬菜管理器的情况隔离开来，尽管我认为这是面向对象的过度杀伤力，或者 OOO :-)
public abstract class FoodUserControl : UserControl {
protected List<Vegetable> GetVeggies() {
return new VegetableManager().GetAll();
}
}
那么您的实际控件将来自此而不是来自 UserControl。
更新
急切加载的 VegetableManager.cs：
public class VegetableManager {
private static Collection<Vegetable> _veggies;
private static object _veggieLock;
static VegetableManager() {
// logic to load veggies from file
}
public ReadOnlyCollection<Vegetable> GetAll() {
return new ReadOnlyCollection(_veggies);
}
public void Add(Vegetable veggie) {
lock(_veggieLock) { //synchronize access to shared data
_veggies.Add(veggie);
// logic to write out the updated list of _veggies to the file
}
}
}
请注意，这个预加载版本不必在构造函数中围绕加载代码进行双重检查锁定。另请注意，加载代码位于static 构造函数中，因为此代码初始化static 字段（否则，您将在每次构造时将文件中的数据重新加载到相同的共享static 字段中）。因为蔬菜是预先加载的，所以您不需要在 GetAll 或 Add 中加载。