【发布时间】:2012-12-02 13:11:17
【问题描述】:
我一直致力于重构应用程序,以通过依赖注入和所有其他好东西使其更易于管理。这样做时,我不止一次遇到循环依赖。
所以这里是一个循环依赖的典型例子:
interface IA
{
int Data { get; }
}
interface IBefore
{
void DoStuffBefore();
}
class A: IA
{
public int Data { get; private set; }
IBefore before;
public A(IBefore before)
{
this.before = before;
}
public void Increment()
{
before.DoStuffBefore();
Data++;
}
}
class B: IBefore
{
IA a;
public B(IA a)
{
this.a = a;
}
public void WriteADataToConsole()
{
Console.Write(a.Data);
}
public void DoStuffBefore() //From IBefore
{
WriteADataToConsole();
}
}
我不能创建这两个类,因为它们需要彼此。现在,在这种情况下要做的标准(?)事情是将 A 的数据与 A 分开:
public interface IA
{
int Data { get; set; }
}
public interface IBefore
{
void DoStuffBefore();
}
class AData : IA
{
public int Data { get; set; }
}
class A
{
public IA Data { get; private set; }
IBefore before;
public A(IA data, IBefore before)
{
this.Data = data;
this.before = before;
}
public void Increment()
{
before.DoStuffBefore();
Data.Data++;
}
}
class B : IBefore
{
IA a;
public B(IA a)
{
this.a = a;
}
public void WriteADataToConsole()
{
Console.Write(a.Data);
}
public void DoStuffBefore() //From IBefore
{
WriteADataToConsole();
}
}
以上解决了循环依赖,因为我现在可以先创建 AData,然后将其注入 B 并将 B 注入 A。但我也可以将事件 i IA 放入 B 可以听的:
public interface IA
{
int Data { get; }
event Action BeforeEvent;
}
class A: IA
{
public int Data { get; private set; }
public event Action BeforeEvent;
public void Increment()
{
BeforeEvent();
Data++;
}
}
class B
{
IA a;
public B(IA a)
{
this.a = a;
a.BeforeEvent += new Action(WriteADataToConsole);
}
void WriteADataToConsole() //Event listener
{
Console.Write(a.Data);
}
}
这是我偶然发现的,因为我试图将事件方法转换为依赖注入,并意识到这样做我得到了一个循环依赖。
困扰我大脑的一些问题是:
- 这两种解决方案都解决了循环依赖问题(对吗?),据我所知,它们都可以将 A 扩展到同等程度,但哪种设计会被认为是最佳设计?
- 对于何时使用事件以及何时使用 DI 来解决循环依赖以及一般情况,有哪些指导方针?
- 如果 A 需要 B 的返回值,显然事件是不好的。这是否意味着当返回 void 时总是首选事件?
- 每种解决方案的优缺点是什么?
【问题讨论】:
-
我已经把你所有的类声明一个一个[第一个,第二个和最后一个]每个都编译了
-
@Kamran 是的,它们确实可以编译。第一个问题是您无法将它们连接在一起。 IA a = new A(这里你需要注入 B)但你不能,因为要创建 B,你需要先创建 A。我也不知道代码块发生了什么。缩进在预览中看起来不错。
-
任何可以帮助您设计和实施周期的东西都只是隐藏了您的设计非常非常糟糕的事实。如果两件事相互依赖,你就没有两件事,你只有一件事。
-
@PeterRitchie 是不是过于简单化了?您可能对糟糕的设计是正确的,但您并没有说服我解决方案。我发现解决循环依赖的常见答案是:如果两件事相互依赖,你就没有两件事,你有三件事。我以前从未见过合并提议作为对此的答案。另外,你是说如果我有一个课,例如一个DataGridView,那么那个类就不能监听DataGridView的任何事件,因为那时我已经设计并实现了一个循环?
-
关于相互之间存在循环依赖的项目有各种各样的可用信息,解决方案是制作一个项目而不是两个。上课也是如此。当然,您可以重新设计以添加第三类;但是您只是从二类循环依赖项更改为三类循环依赖项。许多人会争辩说这并没有更好。
标签: c# oop events dependency-injection