我需要一些关于为我的基于Web的业务层功能选择签名的意见:
function void CreateUser(Company UserCompany,string name ...)
或
function void CreateUser(int CompanyID,string name ...)
如果这是一个基于窗口的我会选择函数来取对象而不是int,因为既然公司已经加载到窗口形式,为什么不只是利用它。另外,我认为业务层应该采用业务对象,并且它的类型安全(防止意外传入0或-1)。
但另一方面,如果这是一个基于Web的应用程序,则只会将id加载到客户端站点。因此加载公司对象是多余的,因为最终只有companyID将保存在dabasebase中。
希望我的描述不会太混乱= P. 谢谢你的阅读。
答案 0 :(得分:0)
我认为您回答了自己的问题,这取决于您使用的表示层。但是,如果您希望保持此业务层的通用性,则必须使用ID。
这是另一个想法。我个人喜欢将所有突变包装到类中,我称之为命令。例如,您可以从基本命令继承CreateUserCommand
。这将允许你像这样使用它(我在这里假设C#):
var command = new CreateUserCommand();
command.UserCompanyId = companyId;
command.UserName = name;
command.Execute();
在命令中包装此逻辑是非常好的模式。它允许您在单个命令中放置一个用例。特别是当命令中的逻辑数量增加时,你会欣赏这种模式,但我发现它对CRUD操作也非常有效。
此模式还允许您在基类中抽象事务模型。基类可以包装调用以在数据库事务中执行。这是一个简单的实现:
public abstract class CommandBase
{
public void Execute()
{
this.Validate();
using (var conn = ContextFactory.CreateConnection())
{
conn.Open();
using (var transaction = conn.BeginTransaction())
{
using (var db = ContextFactory.CreateContext(conn))
{
this.ExecuteInternal(db);
db.SubmitChanges();
}
transaction.Commit();
}
}
}
protected virtual void Validate() { }
protected abstract void ExecuteInternal(YourDataContext context);
}
这就是CreateUserCommand
的样子:
public class CreateUserCommand : CommandBase
{
public int UserCompanyId { get; set; }
public string UserName { get; set; }
protected override void ExecuteInternal(YourDataContext context)
{
this.InsertNewUserInDatabase(context);
this.ReportCreationOfNewUser();
}
protected override void Validate()
{
if (this.UserCompanyId <= 0)
throw new InvalidOperationException("Invalid CompanyId");
if (string.IsNullOrEmpty(this.UserName))
throw new InvalidOperationException("Invalid UserName");
}
private void InsertNewUserInDatabase(YourDataContext context)
{
db.Users.InsertOnSubmit(new User()
{
Name = this.UserName,
CompanyId = this.CompanyId
});
}
private void ReportCreationOfNewUser()
{
var message = new MailMessage();
message.To = Configuration.AdministratorMailAddress;
message.Body = "User " + this.Name + " was created.";
new SmtpClient().Send(message);
}
}
我希望这会有所帮助。