Code Fragment-删掉那些认为有用而注释掉的code。
2013-08-28 23:50
162 查看
本文思想参考自《Clean Code》
在android的源码里,有这样一段code。
之前我也有过类似的行为,注释而不是删掉,常常有下面的原因:
这些code将来可能会用到。
而实际上:
这些code将来也不会用到。
这些code将来不能直接用,因为在注释掉的一段时间里,它本来的场景已经不合适。
这些code需要用到的时候,别人也不敢用,别人不知道你为什么注释掉,不清楚这些code现在有没有问题。
这些code常常没有用,但是非作者一般不会去删掉,这就会使注释掉的code越来越不适用。
即便这些code有用,完全可以通过代码控制工具恢复。
所以:对于一些代码直接删掉,而不是注释掉!
在android的源码里,有这样一段code。
// This is to replace p.setStyle(Style.STROKE); canvas.drawRect() since it // doesn't work well with hardware acceleration // private void drawEmptyRect(Canvas canvas, Rect r, int color) { // int linesIndex = 0; // mLines[linesIndex++] = r.left; // mLines[linesIndex++] = r.top; // mLines[linesIndex++] = r.right; // mLines[linesIndex++] = r.top; // // mLines[linesIndex++] = r.left; // mLines[linesIndex++] = r.bottom; // mLines[linesIndex++] = r.right; // mLines[linesIndex++] = r.bottom; // // mLines[linesIndex++] = r.left; // mLines[linesIndex++] = r.top; // mLines[linesIndex++] = r.left; // mLines[linesIndex++] = r.bottom; // // mLines[linesIndex++] = r.right; // mLines[linesIndex++] = r.top; // mLines[linesIndex++] = r.right; // mLines[linesIndex++] = r.bottom; // mPaint.setColor(color); // canvas.drawLines(mLines, 0, linesIndex, mPaint); // }
之前我也有过类似的行为,注释而不是删掉,常常有下面的原因:
这些code将来可能会用到。
而实际上:
这些code将来也不会用到。
这些code将来不能直接用,因为在注释掉的一段时间里,它本来的场景已经不合适。
这些code需要用到的时候,别人也不敢用,别人不知道你为什么注释掉,不清楚这些code现在有没有问题。
这些code常常没有用,但是非作者一般不会去删掉,这就会使注释掉的code越来越不适用。
即便这些code有用,完全可以通过代码控制工具恢复。
所以:对于一些代码直接删掉,而不是注释掉!
相关文章推荐
- Code Fragment-提供相对安全和相对粗暴的两种接口胜过单一的接口
- myeclipse注释模板配置,codetemplates.xml
- EF Codefirst(二)数据注释
- 【转】那些令人喷饭的注释
- 有用的Code
- Code Fragment-尤大表达式在Java中没有必要
- Code Fragment-使用继承封装技术层次
- Android开发 Fragment中调用startActivityForResult返回错误的requestCode
- 删掉C程序中的注释
- [转]有用的SAP System Administration T-CODE
- 注释你的代码 Commenting your code
- TCExam文件代码注释分析(shard/code/tce_authorization.php)
- Python Show-Me-the-Code 第 0007 题 统计代码行数(注释,空行,总行数)小程序
- Code Fragment-设置自己的Logger工具类
- java code fragment colletion 070913
- 前几天参加了阿里巴巴的电话面试,把他们问的问题都记录了下来,希望对那些正准备面试的人有用。
- Code Fragment-对于if语句中的boolean值分开判断的处理。
- 整理自认为有用的Visual Studio 快捷键
- Fragment 踩过的那些坑
- 一些我认为有用的博客文章链接