实践中的重构07_能不能通过名字猜用途

代码中如果有一行code是这样的，

String email=userInfo.getEmail();

那么我们会对这行代码做出什么样的判断呢？
我对这行代码的判断是从一个用户信息的模型中取出了用户的email。我相信大部分的人和我的猜测应该一致的。我把这个叫做合理猜测。

但是很遗憾，这个猜测是错误的。同事给我指出在数据库中email的字段的存储值并不是该用户的email，该字段的说明是这样写的。如果该用户是用email注册的话，则该字段存储email值，否则存储该用户的手机号（系统目前只支持用email和手机号注册）。

有没有抓狂的感觉，我有。

拜托，能不能让我的合理猜测是正确的。

更遗憾的是，这个东西如果要想改成符合合理猜测的样子，要花费的代价远远超出了我们的想象，各个系统中都有对方法的调用。

评论

4 楼 imacback 2010-11-29  

自己写代码的时候一定在注意，要不然以后重构的工作基本上没人做。。。。于是代码就是越写越烂，如此循环。。。

3 楼 zhang_xzhi_xjtu 2010-11-29  

抛出异常的爱 写道

zhang_xzhi_xjtu 写道

代码中如果有一行code是这样的，

String email=userInfo.getEmail();

那么我们会对这行代码做出什么样的判断呢？
我对这行代码的判断是从一个用户信息的模型中取出了用户的email。我相信大部分的人和我的猜测应该一致的。我把这个叫做合理猜测。

但是很遗憾，这个猜测是错误的。同事给我指出在数据库中email的字段的存储值并不是该用户的email，该字段的说明是这样写的。如果该用户是用email注册的话，则该字段存储email值，否则存储该用户的手机号（系统目前只支持用email和手机号注册）。

有没有抓狂的感觉，我有。

拜托，能不能让我的合理猜测是正确的。

更遗憾的是，这个东西如果要想改成符合合理猜测的样子，要花费的代价远远超出了我们的想象，各个系统中都有对方法的调用。

伪敏捷......
一般来说烂代码没人改
都是出于
恐惧
懒惰
无知

无知的人写了数据库表结构
懒惰的人写了java变态的代码
楼主恐惧的不敢改动.

作孽是要还的
你不还后面的人还是要吃苦的.

我从不觉得我没有勇气去改动自己看着不顺眼的东西。
问题是没有权限，涉及的系统和人太多，不是我想改就能改的。

2 楼 抛出异常的爱 2010-11-29  

zhang_xzhi_xjtu 写道

代码中如果有一行code是这样的，

String email=userInfo.getEmail();

那么我们会对这行代码做出什么样的判断呢？
我对这行代码的判断是从一个用户信息的模型中取出了用户的email。我相信大部分的人和我的猜测应该一致的。我把这个叫做合理猜测。

但是很遗憾，这个猜测是错误的。同事给我指出在数据库中email的字段的存储值并不是该用户的email，该字段的说明是这样写的。如果该用户是用email注册的话，则该字段存储email值，否则存储该用户的手机号（系统目前只支持用email和手机号注册）。

有没有抓狂的感觉，我有。

拜托，能不能让我的合理猜测是正确的。

更遗憾的是，这个东西如果要想改成符合合理猜测的样子，要花费的代价远远超出了我们的想象，各个系统中都有对方法的调用。

伪敏捷......
一般来说烂代码没人改
都是出于
恐惧
懒惰
无知

无知的人写了数据库表结构
懒惰的人写了java变态的代码
楼主恐惧的不敢改动.

作孽是要还的
你不还后面的人还是要吃苦的.

1 楼 sambean 2010-11-29  

直接把 Email和手机号统一为 用户名 userName 呗