今天老大让我们底下几个开发相互 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 里面去直接访问数据库, 代码评审的时候架构师是一定会提出来的~~ 不过你应该是被态度气到了, 如果自己暂时么有解决方案,可以想架构师帮忙想方案,你实施就好
只有我们坚持，别的组都不坚持，最后我们到了别的组就被同化了。。。
好不容易弄好了 Jenkins ，结果因为测试一句不习惯用，，， 就不用了。。。
听的我都懵逼了···。。。好东西因为不习惯就放弃了，，，我也是无语了。。。
最近写的验证代码里有好多 for、if 嵌套来嵌套去，深度达到 3 层，虽然不是特别深，但是这样的太多了，看着也不是很舒服
然而不知道怎么写出更简洁而又符合逻辑的 orz
Ant Design 是服务于企业级产品的设计体系，基于确定和自然的设计价值观上的模块化解决方案，让设计者和开发者专注于更好的用户体验。
据说 99% 的性能瓶颈都在数据库。
GitBook 使您的团队可以轻松编写和维护高质量的文档。 分享知识，提高团队的工作效率，让用户满意。
SVN 是 Subversion 的简称，是一个开放源代码的版本控制系统，相较于 RCS、CVS，它采用了分支管理系统，它的设计目标就是取代 CVS。
Openfire 是开源的、基于可拓展通讯和表示协议 (XMPP)、采用 Java 编程语言开发的实时协作服务器。Openfire 的效率很高，单台服务器可支持上万并发用户。
SQL Server 是由 [微软] 开发和推广的关系数据库管理系统（DBMS），它最初是由 微软、Sybase 和 Ashton-Tate 三家公司共同开发的，并于 1988 年推出了第一个 OS/2 版本。
SQLite 是一个进程内的库，实现了自给自足的、无服务器的、零配置的、事务性的 SQL 数据库引擎。SQLite 是全世界使用最为广泛的数据库引擎。
Quicker 您的指尖工具箱！操作更少，收获更多！
InfluxDB 是一个开源的没有外部依赖的时间序列数据库。适用于记录度量，事件及实时分析。
FreeMarker 是一款好用且功能强大的 Java 模版引擎。
Postman 是一款简单好用的 HTTP API 调试工具。
Redis 是一个开源的使用 ANSI C 语言编写、支持网络、可基于内存亦可持久化的日志型、Key-Value 数据库，并提供多种语言的 API。从 2010 年 3 月 15 日起，Redis 的开发工作由 VMware 主持。从 2013 年 5 月开始，Redis 的开发由 Pivotal 赞助。
Communication channel for makers and users.
Chrome 又称 Google 浏览器，是一个由谷歌公司开发的网页浏览器。该浏览器是基于其他开源软件所编写，包括 WebKit，目标是提升稳定性、速度和安全性，并创造出简单且有效率的使用者界面。
TextBundle 文件格式旨在应用程序之间交换 Markdown 或 Fountain 之类的纯文本文件时，提供更无缝的用户体验。
Log4j 是 Apache 开源的一款使用广泛的 Java 日志组件。
Google App Engine（GAE）是 Google 管理的数据中心中用于 WEB 应用程序的开发和托管的平台。2008 年 4 月 发布第一个测试版本。目前支持 Python、Java 和 Go 开发部署。全球已有数十万的开发者在其上开发了众多的应用。
Thymeleaf 是一款用于渲染 XML/XHTML/HTML5 内容的模板引擎。类似 Velocity、 FreeMarker 等，它也可以轻易的与 Spring 等 Web 框架进行集成作为 Web 应用的模板引擎。与其它模板引擎相比，Thymeleaf 最大的特点是能够直接在浏览器中打开并正确显示模板页面，而不需要启动整个 Web 应用。
GraphQL 是一个用于 API 的查询语言，是一个使用基于类型系统来执行查询的服务端运行时（类型系统由你的数据定义）。GraphQL 并没有和任何特定数据库或者存储引擎绑定，而是依靠你现有的代码和数据支撑。
ZeroNet 是一个基于比特币加密技术和 BT 网络技术的去中心化的、开放开源的网络和交流系统。
持续集成（Continuous Integration）是一种软件开发实践，即团队开发成员经常集成他们的工作，通过每个成员每天至少集成一次，也就意味着每天可能会发生多次集成。每次集成都通过自动化的构建（包括编译，发布，自动化测试）来验证，从而尽早地发现集成错误。
前端技术一般分为前端设计和前端开发，前端设计可以理解为网站的视觉设计，前端开发则是网站的前台代码实现，包括 HTML、CSS 以及 JavaScript 等。
星云链是一个开源公链，业内简单的将其称为区块链上的谷歌。其实它不仅仅是区块链搜索引擎，一个公链的所有功能，它基本都有，比如你可以用它来开发部署你的去中心化的 APP，你可以在上面编写智能合约，发送交易等等。3 分钟快速接入星云链 (NAS) 测试网
Kafka 是一种高吞吐量的分布式发布订阅消息系统，它可以处理消费者规模的网站中的所有动作流数据。 这种动作（网页浏览，搜索和其他用户的行动）是现代系统中许多功能的基础。 这些数据通常是由于吞吐量的要求而通过处理日志和日志聚合来解决。
V2EX 是创意工作者们的社区。这里目前汇聚了超过 400,000 名主要来自互联网行业、游戏行业和媒体行业的创意工作者。V2EX 希望能够成为创意工作者们的生活和事业的一部分。