不发帖子根本不够活跃啊

本贴最后更新于 2408 天前,其中的信息可能已经水流花落

今天老大让我们底下几个开发相互 review 代码,然后我的代码很不幸的成了重点 review 对象,因为我经常偷懒懒得动脑筋,就会在 for 循环里套数据查询,尤其是一年十二个月统计数据这种,当时开发过程中的确也想了很多办法去想一次性查询出来,奈何开发时间有限,于是就用了 for 循环十二个月,自认为对效率的影响不是特别大。然而今天同事就揪着这个问题非让我改代码,想听听你们有什么意见。

本来打算一开始就发这个帖子,但是感觉自己当时有点生气,有点失去理智,所以等到现在睡了一个午觉之后再来发帖,可以比较客观的来叙述一下事实,各位看官觉得我做的不对的,也尽管可以指出,我虚心接受

  • 职场

    找到自己的位置,萌新烦恼少。

    126 引用 • 1699 回帖
  • Review
    2 引用 • 52 回帖

相关帖子

优质回帖
  • wuhongxu 1 赞同

    那这种就没办法了啊,喵的,我脾气一上来,for 改成 while

  • yangyujiao 1 1 赞同

    前几天我们也搞了 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、if 嵌套来嵌套去,深度达到 3 层,虽然不是特别深,但是这样的太多了,看着也不是很舒服
    然而不知道怎么写出更简洁而又符合逻辑的 orz

    2 回复
  • 其他回帖
  • yangyujiao

    他想要表达的就是你这两个意思的,我觉得挺对的,就改了。

    我写东西很喜欢抽共通的,我特别讨厌代码重复,idea,重复代码会有波浪线的,我无法忍受我的代码有一个波浪线。。。
    如果好的方法,我也喜欢写成别人也可以调用的 util。

  • zhunengfei

    表面笑嘻嘻,心里 MMP,你就答应改就行了,反正没想好就别改.

    1. 看看能不能拆开来做。如果能拆开做,可以试试,不管实现如何复杂
    2. 他说性能有问题就有问题?业务和表结构就是这样设计的,你给他做他不也是这样做。你去网上找找有关性能的工具,你自己测试看看,然后对他问你,你就让他测试,拿不出证据艹死他
    3. 去京东订购一把菜刀 -。-
  • yangyujiao

    只有我们坚持,别的组都不坚持,最后我们到了别的组就被同化了。。。
    好不容易弄好了 Jenkins ,结果因为测试一句不习惯用,,, 就不用了。。。
    听的我都懵逼了···。。。好东西因为不习惯就放弃了,,,我也是无语了。。。

    1 回复
  • 查看全部回帖

推荐标签 标签

  • 开源

    Open Source, Open Mind, Open Sight, Open Future!

    393 引用 • 3380 回帖 • 1 关注
  • Ubuntu

    Ubuntu(友帮拓、优般图、乌班图)是一个以桌面应用为主的 Linux 操作系统,其名称来自非洲南部祖鲁语或豪萨语的“ubuntu”一词,意思是“人性”、“我的存在是因为大家的存在”,是非洲传统的一种价值观,类似华人社会的“仁爱”思想。Ubuntu 的目标在于为一般用户提供一个最新的、同时又相当稳定的主要由自由软件构建而成的操作系统。

    123 引用 • 168 回帖
  • 设计模式

    设计模式(Design pattern)代表了最佳的实践,通常被有经验的面向对象的软件开发人员所采用。设计模式是软件开发人员在软件开发过程中面临的一般问题的解决方案。这些解决方案是众多软件开发人员经过相当长的一段时间的试验和错误总结出来的。

    198 引用 • 120 回帖
  • BND

    BND(Baidu Netdisk Downloader)是一款图形界面的百度网盘不限速下载器,支持 Windows、Linux 和 Mac,详细介绍请看这里

    107 引用 • 1281 回帖 • 22 关注
  • 电影

    这是一个不能说的秘密。

    120 引用 • 597 回帖
  • Dubbo

    Dubbo 是一个分布式服务框架,致力于提供高性能和透明化的 RPC 远程服务调用方案,是 [阿里巴巴] SOA 服务化治理方案的核心框架,每天为 2,000+ 个服务提供 3,000,000,000+ 次访问量支持,并被广泛应用于阿里巴巴集团的各成员站点。

    60 引用 • 82 回帖 • 590 关注
  • Swagger

    Swagger 是一款非常流行的 API 开发工具,它遵循 OpenAPI Specification(这是一种通用的、和编程语言无关的 API 描述规范)。Swagger 贯穿整个 API 生命周期,如 API 的设计、编写文档、测试和部署。

    26 引用 • 35 回帖 • 7 关注
  • 博客

    记录并分享人生的经历。

    270 引用 • 2386 回帖
  • Maven

    Maven 是基于项目对象模型(POM)、通过一小段描述信息来管理项目的构建、报告和文档的软件项目管理工具。

    185 引用 • 318 回帖 • 352 关注
  • SQLServer

    SQL Server 是由 [微软] 开发和推广的关系数据库管理系统(DBMS),它最初是由 微软、Sybase 和 Ashton-Tate 三家公司共同开发的,并于 1988 年推出了第一个 OS/2 版本。

    19 引用 • 31 回帖
  • 心情

    心是产生任何想法的源泉,心本体会陷入到对自己本体不能理解的状态中,因为心能产生任何想法,不能分出对错,不能分出自己。

    59 引用 • 369 回帖
  • 钉钉

    钉钉,专为中国企业打造的免费沟通协同多端平台, 阿里巴巴出品。

    15 引用 • 67 回帖 • 380 关注
  • WordPress

    WordPress 是一个使用 PHP 语言开发的博客平台,用户可以在支持 PHP 和 MySQL 数据库的服务器上架设自己的博客。也可以把 WordPress 当作一个内容管理系统(CMS)来使用。WordPress 是一个免费的开源项目,在 GNU 通用公共许可证(GPLv2)下授权发布。

    45 引用 • 113 回帖 • 321 关注
  • golang

    Go 语言是 Google 推出的一种全新的编程语言,可以在不损失应用程序性能的情况下降低代码的复杂性。谷歌首席软件工程师罗布派克(Rob Pike)说:我们之所以开发 Go,是因为过去 10 多年间软件开发的难度令人沮丧。Go 是谷歌 2009 发布的第二款编程语言。

    491 引用 • 1383 回帖 • 370 关注
  • 以太坊

    以太坊(Ethereum)并不是一个机构,而是一款能够在区块链上实现智能合约、开源的底层系统。以太坊是一个平台和一种编程语言 Solidity,使开发人员能够建立和发布下一代去中心化应用。 以太坊可以用来编程、分散、担保和交易任何事物:投票、域名、金融交易所、众筹、公司管理、合同和知识产权等等。

    34 引用 • 367 回帖 • 2 关注
  • iOS

    iOS 是由苹果公司开发的移动操作系统,最早于 2007 年 1 月 9 日的 Macworld 大会上公布这个系统,最初是设计给 iPhone 使用的,后来陆续套用到 iPod touch、iPad 以及 Apple TV 等产品上。iOS 与苹果的 Mac OS X 操作系统一样,属于类 Unix 的商业操作系统。

    84 引用 • 139 回帖
  • JWT

    JWT(JSON Web Token)是一种用于双方之间传递信息的简洁的、安全的表述性声明规范。JWT 作为一个开放的标准(RFC 7519),定义了一种简洁的,自包含的方法用于通信双方之间以 JSON 的形式安全的传递信息。

    20 引用 • 15 回帖 • 18 关注
  • Sillot

    Sillot (汐洛)孵化自思源笔记,致力于服务智慧新彖乄,具有彖乄驱动、极致优雅、开发者友好的特点
    Github 地址:https://github.com/Hi-Windom/Sillot

    12 引用 • 26 关注
  • Log4j

    Log4j 是 Apache 开源的一款使用广泛的 Java 日志组件。

    20 引用 • 18 回帖 • 36 关注
  • 大数据

    大数据(big data)是指无法在一定时间范围内用常规软件工具进行捕捉、管理和处理的数据集合,是需要新处理模式才能具有更强的决策力、洞察发现力和流程优化能力的海量、高增长率和多样化的信息资产。

    89 引用 • 113 回帖
  • Scala

    Scala 是一门多范式的编程语言,集成面向对象编程和函数式编程的各种特性。

    13 引用 • 11 回帖 • 101 关注
  • 导航

    各种网址链接、内容导航。

    37 引用 • 168 回帖
  • React

    React 是 Facebook 开源的一个用于构建 UI 的 JavaScript 库。

    192 引用 • 291 回帖 • 444 关注
  • 尊园地产

    昆明尊园房地产经纪有限公司,即:Kunming Zunyuan Property Agency Company Limited(简称“尊园地产”)于 2007 年 6 月开始筹备,2007 年 8 月 18 日正式成立,注册资本 200 万元,公司性质为股份经纪有限公司,主营业务为:代租、代售、代办产权过户、办理银行按揭、担保、抵押、评估等。

    1 引用 • 22 回帖 • 674 关注
  • Gzip

    gzip (GNU zip)是 GNU 自由软件的文件压缩程序。我们在 Linux 中经常会用到后缀为 .gz 的文件,它们就是 Gzip 格式的。现今已经成为互联网上使用非常普遍的一种数据压缩格式,或者说一种文件格式。

    9 引用 • 12 回帖 • 106 关注
  • OAuth

    OAuth 协议为用户资源的授权提供了一个安全的、开放而又简易的标准。与以往的授权方式不同之处是 oAuth 的授权不会使第三方触及到用户的帐号信息(如用户名与密码),即第三方无需使用用户的用户名与密码就可以申请获得该用户资源的授权,因此 oAuth 是安全的。oAuth 是 Open Authorization 的简写。

    36 引用 • 103 回帖 • 6 关注
  • InfluxDB

    InfluxDB 是一个开源的没有外部依赖的时间序列数据库。适用于记录度量,事件及实时分析。

    2 引用 • 47 关注