首页 文章详情

学弟啊,你代码能不能不要写得这么烂?!

码农突围 | 342 2021-01-16 11:58 0 0 0
UniSMS (合一短信)
点击上方“码农突围”,马上关注
这里是码农充电第一站,回复“666”,获取一份专属大礼包
真爱,请设置“星标”或点个“在看”
作者:沙漠之鹰
www.cnblogs.com/buptzym/archive/2013/05/25/3099389.html

我是一个研二的即将实习和毕业的普通屌丝研究生,带着几个研一的同学一起做数据挖掘项目。为了在毕业的时候防止青黄不接导致老师不让我毕业,于是从他们刚进实验室起,就“开始了设计模式和代码风格的培养”。

具体的做法,就是扔给他们“大话设计模式”和金旭亮老师的“.NET 4.0面向对象编程漫谈”。于是项目接着做,我也很忙没怎么看过他们写的代码,直到昨天闲的没事,翻开SVN看各种他们写的代码,然后就很崩溃… 列举如下:

1. 以前命名变量用 a1、a2、a3,现在好多了,但能不能别用汉语拼音?聚类的函数就算不写 ClusterMethod,但也不要叫 JuleiMethod 啊!有了继承,那个新类怎么也得反应改进特点吧,最少也不能叫 XXXFinal、XXXTest 或者XXX2吧?


我在想,这个GuoDu又是什么?过度?国度?我实在想不到数据挖掘领域还有这个拼音的东东。

2. 超大类!大到什么程度呢?

五百行?不,一千行?不,是五千行!一个MainWindow的类里面,洋洋洒洒五千行代码,100个变量,50多个函数,功能倒是实现了,可是代码滚动条都细的看不见了,敢不敢把逻辑分开啊?“单一职责”真是发挥到淋漓尽致:一个类把一个老师的任务搞定!

3. 你是懒还是勤快?
几个类的功能差不多,只有一些细节不同,就算不做继承吧,也别把一样的函数拷贝四五遍啊。下面是两个不同类的截图:


4. 截图同上。一个对象可能是继承树里的叶子节点,在编程的时候这个基类没有你想要的属性,但不需要每次都as 吧?如果是单个变量,在代码前as并判断非空不就好了?

若是个集合,在代码开始时通过LINQ语句提取集合中相应类型不就好了?看到那么多as,你不觉得很烦么?

5. 一个类里面有很多个参数,但能不能不要把所有的参数设置都写到构造函数里面啊,你有10个参数,就要写十个参量的构造函数?能不能做成默认值?其他做成属性访问器不就好了?


6. 能别这么酷爱单例模式和静态变量和方法啊?我知道你用单例和静态很爽,到处都能调用,可是,内存回收呢?代码扩展性呢?


大哥,怎么说这个词典也能占个200M内存,你为了方便一个单例就搞定了,你让它啥时候被GC回收啊?

7. 我要你把一个代码从JAVA移植到C#上,还真是够辛苦,JAVA没有属性访问器,但是C#里面有啊,干嘛还在C#上把所有的属性都加上个 getXXX() 和setXXX()方法。你真勤快!而且还加上了javadoc风格的注释,真是辛苦你了


8. 代码注释是用来说明代码的,稍微有些个注释掉的东西还是能理解的,可是,你一个长长的代码文件里,注释比源代码多好几倍,都是以前随便写的测试代码,废弃的代码,舍不得删,可是,后来你再看过吗?你都没看过,这些大段的注释除了碍眼有什么用处?


9. 蛋疼的在集合中删除元素!

你学会了foreach, 可爱的迭代器模式,于是和我一样,不喜欢写for循环了。哪天你问我,foreach中remove会报错!我说,当然会报错,于是你在外面new一个集合,把要删的全拷贝进去,然后再来个for循环拷出来,然后。还是不对,最后只能再用for循环了。看得我都要心碎了,咱先不说性能,用LINQ写一个扩展方法RemoveElement不就好了?

///
       /// 对集合实现删除操作
   ///
       ///元素类型
       ///要删除的元素列表
       ///过滤器
       ///删除时执行的委托
       public static void RemoveElements(
           this IList source, Func filter, Action method)
       {
           var indexs = (from d in source where filter(d) select source.IndexOf(d)).ToList();
           indexs.Sort();
           for (var i = indexs.Count - 1; i >= 0; i--)
           {
               if (method != null)
               {
                   method(source[indexs[i]]);
               }
               source.RemoveAt(indexs[i]);
           }
       }

然后,你在实现IList接口上的集合都能删除元素了,并且还能通过一个委托做一些操作,多舒服啊…

10.  你会在一个类的成员里很高兴的定义一个变量,比如下面:


一个随机数产生器ran, 然后私有成员还很勤快的写了get,set访问器,下面诡异的arrh,这都不说啥了,关键是下面:


这个变量被你在函数中轻易的覆盖了,覆盖了也不管,反正好像程序结果也是正确的,可是以后没这么走运怎么办?

11.  你不懂什么叫写“库”,只知道写代码,代码重用就是代码搬运工,从这边拷贝到那边。真正的库是什么?稳定,高效,好用,扩展性强。 到现在都没写过一个库,老师布置什么任务,新开一个工程,名字命名成XXXTest, 然后一顿狂敲,毕业时候,工作就是那么20多个不同的文件夹,所有代码都是给自己写给自己看,到最后连自己都看不懂了… 那些能够重用的功能,为什么不好好重构一下,以后肯定还能用得着啊!

12. Console.WriteLine(“xxxxxxxx”)
Console.WriteLine(“xxxxxxxx”)
Console.WriteLine(“xxxxxxxx”)

你给写的类库,里面全是这些控制台输出,做Console程序还好,一个大黑框哗哗哗的输出出来,可是,做WPF界面程序和WCF呢?你总不至于输出到VS里那个小小的调试窗口吧!我哪天想让你输出到txt文件,你是不是就傻眼了?  敢不敢用用专门的日志组件,比如log4net!

13.  C风格的C#, 里面全是数组,数组,数组太怨念了。.NET疯了会搞那么多集合和接口进去?全用数组不就得了?

两个集合的对应元素需要做加法,你是这么做的:


可是,如果这个代码被执行1000遍,你就要new一千次数组,且不管GC效率如何,这种需求你完全可以返回一个IEnumerable过去,通过类似LINQ的延迟计算进行,也不用生成这么多数组啊啊啊。

13. for循环啊!


能不能写个for循环?敢不敢?不仅如此,你都愿意为一块仅仅有细微差别的代码复制十遍,却不敢写一个for循环?

14. 做界面时候的事件引用器…

在界面上,你会在每个按钮,每个菜单双击一下,然后在C#代码中,敲事件处理程序,无非就是一堆变量赋值,一堆初始化等等,若你是做Winform,那我也就勉强不说什么了,可是你做的是WPF,这么紧密的耦合,让以后我改界面或者改逻辑的时候怎么办?敢不敢去学一下绑定和MVVM?

15. 还有无数的槽点,比如你会在程序位置输出几百兆的txt文件方便你调试,然后就都被你无良的上传到SVN上面去了,SVN也从来不正确使用,也不管冲突不冲突就直接点确定,最最让我不能忍受的是:

// 学长真是一头猪  
(主页君注:@水华子 童鞋的建议是 : //XueZhang.isPig()==true;)

哼!明天开会收拾你!

学弟们如果看到这篇文章,千万别记恨我,或者到导师那边告我状,我还是很想正常毕业的啊…

主页君插播:这篇是旧文,原作者已经毕业了啊 

最近有有不少老铁在后台留言说,想进大厂,但是算法不好。最近我整理了一份刷题实录,这份刷题实录,也让我进了心仪的大厂。现在开放分享给大家。希望对大家有所帮助。

任何的算法题,如同写作文一样,都有一些模板可以套用的。比如面试常考的DP(动态规划),难的是一些关键点是否能想清楚。比如你能写出动态转移方程,这题基本上就可以AC了。


整个刷题实录内容,包括 双子针、动态规划、二分查找、贪心算法、深度优先搜索、字符串、递归、字典树、排序、链表等相关专题内容。图文并茂,附有刷题答案源码。


刷题任务的题目,是根据题目的类型来汇总的,总结了八个类别,每个类别下面也总结了5个左右的题型,帮助大家分门别类的突破,所以刷起来相对会更有重点和针对性。如果从头到尾的刷,每周按顺序刷42题,很容易让自己坚持不下来,也会觉得很枯燥。所以在制定计划的时候可以让这个计划变得更“有趣"和针对性,让它看起来更容易实现一点,才会更容易坚持。

目前上述内容已打包成完整电子书,具体获取方式如下:
  1. 扫描关注 程序猿进阶 公众号;

  2. 程序猿进阶 公众号后台回复关键词「9999」获取下载地址。

扫描关注,回复"9999"即可下载

最近热文

为什么国内流行的 MyBatis ,国外 Java 工程师却不愿意使用?
再见,胡阿姨!再见,共享单车!
他是阿里P11,靠写代码写成合伙人,身家几十亿,没有他,我们可能刷不了淘宝!
判了!40岁程序员被判7年,曾提出系统安全问题被无视,怒删9TB财务数据及系统数据!
•  别瞎学了,这几门语言要被淘汰了!

在这里,我为大家准备了一份2020年最新最全的《Java面试题及答案V3.0》,这套电子书涵盖了诸多后端技术栈的面试题和答案,相信可以帮助大家在最短的时间内复习Java后端的大多数面试题,从而拿到自己心仪的offer。
截了张图,大家可以仔细查看左边的菜单栏,覆盖的知识面真的很广,而且质量都很不错。


资料获取方法

  1. 扫描下方二维码

  2. 后台回复关键词:Java核心整理

明天见(。・ω・。)

good-icon 0
favorite-icon 0
收藏
回复数量: 0
    暂无评论~~
    Ctrl+Enter