假设您有以下两种方法:
1号:
void AddPerson(Person person) { // Validate person if(person.Name != null && IsValidDate(person.BirthDate) DB.AddPersonToDatabase(person); }
2号:
void AddPerson(string name, DateTime birthDate) { Person p = new Person(name, birthDate); DB.AddPersonToDatabase(person); }
这两种方法中哪一种最好?我知道第一个更正确OO-wise,但我觉得第二个更可读,并且你不必确保对象是有效的,因为参数确保这一点.我只是不喜欢在传递它们作为参数的任何地方验证对象.还有其他方法吗?
编辑:Thx的所有答案.为了澄清,在构造函数和IsValid方法中进行验证当然是一种很好的方法,但在我的代码中,人的有效状态通常取决于上下文,并且可能因方法而异.这当然可能是糟糕设计的标志.
代码只是描述问题的一个例子.
第一个不应该验证person.Name和person.BirthDate - 它们应该由Person
构造函数自动验证.换句话说,如果你传递了一个人,你应该知道它是有效的.
另一方面,您必须检查它person
不是空引用.
它有时值得拥有像第二个版本这样的便捷方法,以避免必须经常显式地调用构造函数.它通常应该只调用Person
构造函数,然后将工作委托给第一个表单.