主页
代码审核之 重新造轮子
标签:, , ,

代码审核之 重新造轮子

No Comments 2017年8月17日 I.T., 代码审核, 程序设计

今天在看代码修改记录的时候发现有这么一处改动, 虽然这个改动已经很久了,但是我觉得有必要拿出来大家讨论一下:

re-inventing-the-wheel-coding-review-linq 代码审核之 重新造轮子 I.T. 代码审核 程序设计

re-inventing-the-wheel-coding-review-linq

2007年 .NET 3.5 之后推出LINQ,其实整个函数只是在做一件事,就是返回类成员 layoutList 中是 LayoutDevice (后面改成LayoutAnt )的列表.但实际上这通过 C#的LINQ只需要用 OfType<LayOutDevice> 或者 OfType<LayOutAnt> 即可(暂且不说改动包括重构类型)

左边的版本实际上是OK的,这就是学校的标教科书版本,无可厚非,但右边的这个版本就大有问题,因为参数含有引用 ref, 也就是说每次都把外面传进的变量给清空了,这种函数拿来单元测试并不友好.

如果一定要重新造轮子,两个版本都有小问题,一个是 private 方法不好单元测试,另一个是都使用了成员变量 layoutList, 最好是改成 public static 公有静态方法,传入 layoutList, 然后像第一种方式返回新的List.这样的话,这个公有静态方法就是不会更改 (immutable), 较容易被单元测试.

英文: [Code Review] – Reinventing the Wheel

GD Star Rating
loading...
本文一共 287 个汉字, 你数一下对不对.
代码审核之 重新造轮子. (AMP 移动加速版本)
上一篇: 如何在 SteemIt MarkDown 编辑器里添加数学公式?
下一篇: SteemIt 通过回溯算法确定更好的点赞策略 (高级版)
赞赏我的几个理由.
¥ 打赏支持

扫描二维码，分享本文到微信朋友圈
ba1c05bd21ddf9903929dfd2d1e2060a 代码审核之 重新造轮子 I.T. 代码审核 程序设计

相关文章：

  1. GoPro运动相机可以拿来做什么? 自从新年打折入了GoPro Hero+运动相机以来, 也没怎么用上, 基本上是几天的热性就和其它电子产品一样被冷落到一旁. GoPro运动相机可以拿来做什么, 主要的几个用途是: GoPro 不太适合拍人像 拍孩子孩子容易动, 特别容易就糊了, GoPRO的摄像参数可以调的很少, 所以不要期望有大光圈能背景虚化, 唯一能拍的可能是大长腿, GoPro是广角镜头, 能拍出细长的身段和大长腿,...
  2. 梨视频(PearVideo)下载解析的方法和技巧 梨视频(PearVideo)下载的最好方法 以下几种都是下载解析梨视频(PearVideo)的靠谱方法: 网页在线梨视频下载解析工具 (PearVideo Downloader) Chrome 浏览器插件, 支持数十种视频网站下载, 又方便又准确. 当然还可以加我的 微信公众号 JustYYUK 通过机器人问答交互方式来 下载视频. IOS...
  3. 整理 谢菲尔得办公室 上周末我开车和同事又回了谢菲尔得.这次公差的任务就是整理办公室的东西.因为过不久,公司在英国的办公室都会关了, 所有人都会搬到 剑桥附近的 CAMBOURNE 新办公室. 这个办公室是在 2011年初建立的.有近四年之久了.我是2010年进公司的, 2011年10月份搬到了谢菲尔得,算起来也三年多了.这次很戏剧化, 本来想在谢菲尔得买房(并且已经进行到了一半),后来公司HR三次找我谈话,最终我决定往南搬,并很快的速度处理了搬家买房的事情. 现在基本已经安顿好(感谢公司提供住房),买房正在继续,相信不久就能搞定.剑桥的生活很不错,起码天气比较暖和(南边),晴天好像也比较多. 本文一共 228 个汉字, 你数一下对不对. 整理...
  4. 微软顶配 Surface Studio Laptop 开箱和测评 我家的联想笔记本十几年了, 换了SSD固态硬盘勉强一直够用, 虽然平时不玩游戏, 对显卡要求不是很高, 但是在视频编辑方面就很有点弱鸡了. 所有笔记本几乎都有散热的问题, 特别是用久了的, 灰尘多, 散热效率不高, 所以温度一高风扇就会狂转. 家里的服务器放车库, 一直用小PC盒子远程连接除了玩游戏不太行, 视频编辑用PC版的剪映在PC盒子上勉强能用. 但想着还是搞台笔记本合适一些. 有朋友说视频剪辑要用苹果电脑,...
  5. 公司实行新的在家办公制度 – 你允许员工在家办公么? 在家办公 – Work From Home, 公司的老外简称 WFH. 什么工作可以在家办公? 不是所有的工作性质都能在家办公. 程序员, 搞IT的有台电脑网络就能在家办公 甚至可以去公园去咔啡店. 但是主要是和人打交道的HR, 或者跑销售的, 还有一些需要到现场的就不可能选择在家办公....
  6. 为啥我不使用 AdsBlocker 等广告屏蔽软件? 我相信很多人都会使用 AdsBlocker 等类似的广告屏蔽插件来浏览网页, 特别是搞技术的几乎是必备的上网装备. 但是我却对它无爱, 原因有几个: AdsBlocker 把网页搞得太干净了 生活需要点噪音, 我需要点噪音. 就像我受不了半夜的寂静 特别是能很清楚的听到家里的表时针在走. AdsBlocker 并不完美 经常会有页面显示问题...
  7. 怎么样快速有效的清洁车的内部? 每几个月都会按心情(其实是车脏得不行了) 花个大约20英镑去 Tesco 洗车. 20几镑还能享受10%的Tesco员工价, 虽然不贵 但是其实并不能撑很久, 原因是, 两孩子上车吃零食经常弄的车内部很脏. 英国空气很好, 所以车的外部清洁只需要几个周甚至几个月做一次即可, 而车内部则得几天就清理一次, 总不能每周花个20英镑去做个全清理(车的外部和内部), 这也太土豪了. 还好,...
  8. 在英国多久到超市买一次菜合适? 时间就是金钱, 时间也是种成本. 至从搬了新家 路边有水果, 花还有一些蔬菜. 村里还有个酒巴挺不错的. 其它真的没有什么了 感觉几条街就能把这个小镇走完. 每周就去 BAR HILL 的 TESCO(乐购) 买一周的食物. 这样由于我中午在公司附近吃饭,...

评论