一个类,最好只做一件事,只有一个引起它变化的原因。
一个违背SRP原则的设计实例:对数据库的操作和用户权限判别封装在一个类中实现,如:
public void Add()
{
if(GetPermission(id.permission)=="CanAdd")
{
Console.WriteLine("CanAdd");
}
}
显然这样的实现是很僵化的,一旦权限设置的规则发生变化,则所有的方法都必须跟着改变,这是不可想象的事情。我们应该把权限判断的职责与数据库操作的职责分开。P可以通过实现一个Proxy模式来解决职责交叉的问题
public interface IDBAction
{
void Add();
}
public class DBManager: IDBAction
{
#region IDBAction 成员
public void Add()
{
Console.WriteLine("DBManager.Add()");
}
#endregion
public DBManager()
{
}
}
public class DBManagerProxy : IDBAction
{
#region 私有成员
private IDBAction dbManager;
#endregion
public DBManagerProxy(IDBAction dbAction)
{
dbManager = dbAction;
}
public string GetPermission(string id)
{
return id;
}
#region IDBAction 成员
public void Add()
{
if (GetPermission("CanAdd") == "CanAdd")
{
dbManager.Add();
}
}
#endregion
}
class DBClient
{
static void Main(string[] args)
{
IDBAction DBManager = new DBManagerProxy(new DBManager());
DBManager.Add();
}
}
关于单一职责原则的建议:
一个类只有一个引起它变化的原因,否则就应当考虑重构
SRP由引起变化的原因决定,而不是由功能职责决定。虽然职责常常是引起变化的轴线,但是有时却未必,应该审时度势。
测试驱动开发,有助于实现合理分离功能的设计。
可以通过Facade模式和Proxy模式进行职责分离。
- 黑暗天堂 - CSDNBlog
最新评论