今日review一个有关文本编辑框最大长度限制的一个问题。 开发人员修改了两个文件,这两个都有关于字符串最大长度的限制。 开发人员还算细心,本来测试人员提出的问题只是关于一处的,他把相关的另一处地方也想到了,顺便进行了修改。
这里我目前想到了两个问题:
- 开发人员虽然进行了相关地方的修改,但是这两处的修改几乎一模一样,他只是把一处代码又搬到另一处,修改了一下其中的局部变量。
- 这个问题我想本来应该是不需要修改的。此处的修改只是在超过长度是发出了一个toast提示。大家知道,android中的toast提示是可以叠加的,如果用户不停的输入,那么toast提示是会弹出很多,很长时间不会消失。 这就有增加了用户体验的负担。
首先来说下问题1:这里有经验的程序员一般都会做一下提取。将两处的代码公共部分提取出来,用输入参数来控制下行为结果就可以了,这里就是常听到的“重构代码”,具体的重构方法和原则很多,比较经典的推荐是:Martin Fowler的“重构 改善既有代码的设计“。 上面开发人员的问题就是所谓的”鼠标程序员“。
问题2就是风格和需求输入的问题了。这里一般的刚入行不久的程序员(或者反馈渠道不畅 或其他)只是解决分配过来的问题。我把目前的问题解决了就行了。没有站在用户的角度反思一下问题。只是我把任务完成了就行了。
由此,想到了一些东西,还不完全,有空继续思考。 发博以记之。
20130415 周一 晴