今天老大让我们底下几个开发相互 review 代码，然后我的代码很不幸的成了重点 review 对象，因为我经常偷懒懒得动脑筋，就会在 for 循环里套数据查询，尤其是一年十二个月统计数据这种，当时开发过程中的确也想了很多办法去想一次性查询出来，奈何开发时间有限，于是就用了 for 循环十二个月，自认为对效率的影响不是特别大。然而今天同事就揪着这个问题非让我改代码，想听听你们有什么意见。
本来打算一开始就发这个帖子，但是感觉自己当时有点生气，有点失去理智，所以等到现在睡了一个午觉之后再来发帖，可以比较客观的来叙述一下事实，各位看官觉得我做的不对的，也尽管可以指出，我虚心接受
前几天我们也搞了 code review。
是只 review 我的代码····
因为我们组就 3 个人，另两个人当时很多东西没有搞完。
我其实是希望他们提一些建设性的意见，可以提高效率啦，或者更优雅的写法了。
但是提的问题，基本还是比较基础的，命名不好了（那个命名已经是我想了好多，最后根据我的方法确定出来的），然后还有代码写的递进性太强了（就是说我的一个方法里面做的东西太多了，很多其实查询出来是什么我直接给处理了，他们意思是应该查询出来什么类型就是什么类型，方法返回以后处理，我是在那个方法里面都处理完了在返回。。。反正这个说不太清楚），还有就是枚举用的不彻底（这个其实我用的还好吧，我很喜欢用枚举的，因为这个枚举的问题，还跟领导吵了一下）。
我有一个方法我是直接返回了 StringBuffer，因为我要的就是 a,b,a,d 这样的。但是要我先返回一个 list 然后在处理成我要的那个样子，后来我就用了 StringUtils.join 这个方法，但是其实里面就是用的 StringBuilder，我当时就想直接用 StringBuilder，不用 list。。。领导的意见是什么方法该干什么事儿就只让干什么事，这就上面说的我的代码递进性太强了。。。
当时因为这个 list 问题，我又跟他争论了，他说对性能没有影响的，因为我的 list 里面就是我们定义的枚举，不多的。但是先 list，然后有 join 里遍历，在成为 StringBuilder，性能就算影响不大，应该也会有的呀。。。
让我做个测试，当时忙，也没做。。。
另外还有一个我的 try-catch 的问题，我喜欢用大 try-catch，但是给我建议是 catch 跟在那句 try 后面，然后我就改了，用了好几个布尔判断·····，自己又改出几个 bug。。。
我写代码，在性能方面我觉得考虑的真的挺多的了。另外，我会用我认为那是最优雅的方式来写。
我挺喜欢 review 的，因为有时候我认为优雅的方式，别人有更优雅的方式，然后我学会了，就会很开心。
最后就是，我的那些名字不好的方法，最后我也没改。因为我就是想不到更好的了，那已经是绞尽脑汁想到的符合我方法含义的名字···
我就是没找到更好的办法，无奈之下才用的这个 for 循环套数据查询，可气的是我跟他讲我这个一次性查询不好做的时候，他说这个他不管，他只负责提出问题。好歹大家都是开发，提点有用的意见也行啊
最近写的验证代码里有好多 for、if 嵌套来嵌套去，深度达到 3 层，虽然不是特别深，但是这样的太多了，看着也不是很舒服
然而不知道怎么写出更简洁而又符合逻辑的 orz
前几天我们也搞了 code review。
是只 review 我的代码····
因为我们组就 3 个人，另两个人当时很多东西没有搞完。
我其实是希望他们提一些建设性的意见，可以提高效率啦，或者更优雅的写法了。
但是提的问题，基本还是比较基础的，命名不好了（那个命名已经是我想了好多，最后根据我的方法确定出来的），然后还有代码写的递进性太强了（就是说我的一个方法里面做的东西太多了，很多其实查询出来是什么我直接给处理了，他们意思是应该查询出来什么类型就是什么类型，方法返回以后处理，我是在那个方法里面都处理完了在返回。。。反正这个说不太清楚），还有就是枚举用的不彻底（这个其实我用的还好吧，我很喜欢用枚举的，因为这个枚举的问题，还跟领导吵了一下）。
我有一个方法我是直接返回了 StringBuffer，因为我要的就是 a,b,a,d 这样的。但是要我先返回一个 list 然后在处理成我要的那个样子，后来我就用了 StringUtils.join 这个方法，但是其实里面就是用的 StringBuilder，我当时就想直接用 StringBuilder，不用 list。。。领导的意见是什么方法该干什么事儿就只让干什么事，这就上面说的我的代码递进性太强了。。。
当时因为这个 list 问题，我又跟他争论了，他说对性能没有影响的，因为我的 list 里面就是我们定义的枚举，不多的。但是先 list，然后有 join 里遍历，在成为 StringBuilder，性能就算影响不大，应该也会有的呀。。。
让我做个测试，当时忙，也没做。。。
另外还有一个我的 try-catch 的问题，我喜欢用大 try-catch，但是给我建议是 catch 跟在那句 try 后面，然后我就改了，用了好几个布尔判断·····，自己又改出几个 bug。。。
我写代码，在性能方面我觉得考虑的真的挺多的了。另外，我会用我认为那是最优雅的方式来写。
我挺喜欢 review 的，因为有时候我认为优雅的方式，别人有更优雅的方式，然后我学会了，就会很开心。
最后就是，我的那些名字不好的方法，最后我也没改。因为我就是想不到更好的了，那已经是绞尽脑汁想到的符合我方法含义的名字···
一般很忌讳在 for 里面去直接访问数据库, 代码评审的时候架构师是一定会提出来的~~ 不过你应该是被态度气到了, 如果自己暂时么有解决方案,可以想架构师帮忙想方案,你实施就好
我入职这个新公司，我是第一个被 review 的，因为别的组压根就没有这回事，我看过别的组写的代码。。。至少不格式化的代码，我就懒得看。。。
我们现在那个领导，技术还是很 nb 的，跟着他学了不少。不过他也挺扭的，我也扭，我们就吵吵。。。
他想要表达的就是你这两个意思的，我觉得挺对的，就改了。
我写东西很喜欢抽共通的，我特别讨厌代码重复，idea，重复代码会有波浪线的，我无法忍受我的代码有一个波浪线。。。
如果好的方法，我也喜欢写成别人也可以调用的 util。
只有我们坚持，别的组都不坚持，最后我们到了别的组就被同化了。。。
好不容易弄好了 Jenkins ，结果因为测试一句不习惯用，，， 就不用了。。。
听的我都懵逼了···。。。好东西因为不习惯就放弃了，，，我也是无语了。。。
表面笑嘻嘻，心里 MMP，你就答应改就行了，反正没想好就别改.
找到自己的位置，萌新烦恼少。
MyBatis 本是 Apache 软件基金会 的一个开源项目 iBatis，2010 年这个项目由 Apache 软件基金会迁移到了 google code，并且改名为 MyBatis ，2013 年 11 月再次迁移到了 GitHub。
InfluxDB 是一个开源的没有外部依赖的时间序列数据库。适用于记录度量，事件及实时分析。
C 语言是一门通用计算机编程语言，应用广泛。C 语言的设计目标是提供一种能以简易的方式编译、处理低级存储器、产生少量的机器码以及不需要任何运行环境支持便能运行的编程语言。
ReactiveX 是一个专注于异步编程与控制可观察数据（或者事件）流的 API。它组合了观察者模式，迭代器模式和函数式编程的优秀思想。
Microsoft Windows 是美国微软公司研发的一套操作系统，它问世于 1985 年，起初仅仅是 Microsoft-DOS 模拟环境，后续的系统版本由于微软不断的更新升级，不但易用，也慢慢的成为家家户户人们最喜爱的操作系统。
Notion - The all-in-one workspace for your notes, tasks, wikis, and databases.
你若安好，便是晴天。
Dubbo 是一个分布式服务框架，致力于提供高性能和透明化的 RPC 远程服务调用方案，是 [阿里巴巴] SOA 服务化治理方案的核心框架，每天为 2,000+ 个服务提供 3,000,000,000+ 次访问量支持，并被广泛应用于阿里巴巴集团的各成员站点。
frp 是一个可用于内网穿透的高性能的反向代理应用，支持 TCP、UDP、 HTTP 和 HTTPS 协议。
思源笔记是一款隐私优先的个人知识管理系统，支持完全离线使用，同时也支持端到端加密同步。
融合块、大纲和双向链接，重构你的思维。
Webswing 是一个能将任何 Swing 应用通过纯 HTML5 运行在浏览器中的 Web 服务器，详细介绍请看 将 Java Swing 应用变成 Web 应用 。
Swift 是苹果于 2014 年 WWDC（苹果开发者大会）发布的开发语言，可与 Objective-C 共同运行于 Mac OS 和 iOS 平台，用于搭建基于苹果平台的应用程序。
提问之前请先看《提问的智慧》，好的问题比好的答案更有价值。
LeetCode（力扣）是一个全球极客挚爱的高质量技术成长平台，想要学习和提升专业能力从这里开始，充足技术干货等你来啃，轻松拿下 Dream Offer！
好记性不如烂笔头。
生活是指人类生存过程中的各项活动的总和，范畴较广，一般指为幸福的意义而存在。生活实际上是对人生的一种诠释。生活包括人类在社会中与自己息息相关的日常活动和心理影射。
Tomcat 最早是由 Sun Microsystems 开发的一个 Servlet 容器，在 1999 年被捐献给 ASF（Apache Software Foundation），隶属于 Jakarta 项目，现在已经独立为一个顶级项目。Tomcat 主要实现了 JavaEE 中的 Servlet、JSP 规范，同时也提供 HTTP 服务，是市场上非常流行的 Java Web 容器。
禅道是一款国产的开源项目管理软件，她的核心管理思想基于敏捷方法 scrum，内置了产品管理和项目管理，同时又根据国内研发现状补充了测试管理、计划管理、发布管理、文档管理、事务管理等功能，在一个软件中就可以将软件研发中的需求、任务、bug、用例、计划、发布等要素有序的跟踪管理起来，完整地覆盖了项目管理的核心流程。
Android 是一种以 Linux 为基础的开放源码操作系统，主要使用于便携设备。2005 年由 Google 收购注资，并拉拢多家制造商组成开放手机联盟开发改良，逐渐扩展到到平板电脑及其他领域上。
面试造航母，上班拧螺丝。多面试，少加班。
Vditor 是一款浏览器端的 Markdown 编辑器，支持所见即所得、即时渲染（类似 Typora）和分屏预览模式。它使用 TypeScript 实现，支持原生 JavaScript、Vue、React 和 Angular。
又拍云是国内领先的 CDN 服务提供商,国家工信部认证通过的“可信云”，乌云众测平台认证的“安全云”，为移动时代的创业者提供新一代的 CDN 加速服务。
ngrok 是一个反向代理，通过在公共的端点和本地运行的 Web 服务器之间建立一个安全的通道。
京东是中国最大的自营式电商企业，2015 年第一季度在中国自营式 B2C 电商市场的占有率为 56.3%。2014 年 5 月，京东在美国纳斯达克证券交易所正式挂牌上市（股票代码：JD），是中国第一个成功赴美上市的大型综合型电商平台，与腾讯、百度等中国互联网巨头共同跻身全球前十大互联网公司排行榜。