我看到的最多被违反的原则是“命令,不要去询问(Tell, Don’t Ask)”原则。这个原则讲的是,一个对象应该命令其它对象该做什么,而不是去查询其它对象的状态来决定做什么(查询其它对象的状态来决定做什么也被称作 ‘功能嫉妒(Feature Envy)’)。
这篇文章里有个很生动的例子,我至今记忆犹新:
if (person.getAddress().getCountry() == “Australia”) {
这违反了得墨忒耳定律,因为这个调用者跟Person过于亲密。它知道Person里有一个Address,而Address里还有一个country。它实际上应该写成这样:
if (person.livesIn(“Australia”)) {
非常的明了。今天我又看到一个关于“Tell, Don’t Ask”原则的文章,里面提供了4个关于这个原则的例子,都很有价值。
例一
不好:
1 |
<% if current_user.admin? %> |
2 |
<%= current_user.admin_welcome_message %> |
4 |
<%= current_user.user_welcome_message %> |
好:
1 |
<%= current_user.welcome_message %> |
例二
不好:
1 |
def check_for_overheating(system_monitor) |
3 |
if system_monitor.temperature > 100 |
4 |
system_monitor.sound_alarms |
好:
01 |
system_monitor.check_for_overheating |
04 |
def check_for_overheating |
例三
不好:
4 |
if user.is_a?(TwitterUser) |
5 |
user.send_to_feed(contents) |
好:
04 |
user.send_to_feed(contents) |
09 |
def send_to_feed(contents) |
11 |
twitter_client.post_to_feed(contents) |
16 |
def send_to_feed(contents) |
例四
不好:
4 |
user.address.street_name |
7 |
'No street name on file' |
好:
03 |
user.address.street_name |
09 |
@address || NullAddress. new |
16 |
'No street name on file' |
好的面向对象编程是告诉对象你要做什么,而不是询问对象的状态后根据状态做行动。数据和依赖这些数据的操作都应该属于同一个对象。
命令,不要去询问!