不发帖子根本不够活跃啊

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

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

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

  • 职场

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

    127 引用 • 1708 回帖
  • Review
    2 引用 • 52 回帖

相关帖子

优质回帖
  • wuhongxu 1 赞同 via Linux

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

  • yangyujiao 1 1 赞同 via macOS

    前几天我们也搞了 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 的,因为有时候我认为优雅的方式,别人有更优雅的方式,然后我学会了,就会很开心。

    最后就是,我的那些名字不好的方法,最后我也没改。因为我就是想不到更好的了,那已经是绞尽脑汁想到的符合我方法含义的名字···

欢迎来到这里!

我们正在构建一个小众社区,大家在这里相互信任,以平等 • 自由 • 奔放的价值观进行分享交流。最终,希望大家能够找到与自己志同道合的伙伴,共同成长。

注册 关于
请输入回帖内容 ...
  • zhunengfei

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

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

    最近写的验证代码里有好多 for、if 嵌套来嵌套去,深度达到 3 层,虽然不是特别深,但是这样的太多了,看着也不是很舒服
    然而不知道怎么写出更简洁而又符合逻辑的 orz

    2 回复
  • yangyujiao via macOS

    我入职这个新公司,我是第一个被 review 的,因为别的组压根就没有这回事,我看过别的组写的代码。。。至少不格式化的代码,我就懒得看。。。
    我们现在那个领导,技术还是很 nb 的,跟着他学了不少。不过他也挺扭的,我也扭,我们就吵吵。。。

    1 回复
  • scorpion via macOS

    我比较偏向结合实际场景来判断,纯理论瞎 BB 怎么都有优化的空间。我们都是在做工程,又不是搞科研。进取的态度要有,可是不能偏执。遇到这种领导,我同你一起蛋疼一会儿。

    1 回复
  • 查看全部回帖

推荐标签 标签

  • Oracle

    Oracle(甲骨文)公司,全称甲骨文股份有限公司(甲骨文软件系统有限公司),是全球最大的企业级软件公司,总部位于美国加利福尼亚州的红木滩。1989 年正式进入中国市场。2013 年,甲骨文已超越 IBM,成为继 Microsoft 后全球第二大软件公司。

    107 引用 • 127 回帖 • 354 关注
  • Hadoop

    Hadoop 是由 Apache 基金会所开发的一个分布式系统基础架构。用户可以在不了解分布式底层细节的情况下,开发分布式程序。充分利用集群的威力进行高速运算和存储。

    88 引用 • 122 回帖 • 620 关注
  • Ubuntu

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

    127 引用 • 169 回帖 • 1 关注
  • Hibernate

    Hibernate 是一个开放源代码的对象关系映射框架,它对 JDBC 进行了非常轻量级的对象封装,使得 Java 程序员可以随心所欲的使用对象编程思维来操纵数据库。

    39 引用 • 103 回帖 • 719 关注
  • 面试

    面试造航母,上班拧螺丝。多面试,少加班。

    325 引用 • 1395 回帖 • 1 关注
  • Telegram

    Telegram 是一个非盈利性、基于云端的即时消息服务。它提供了支持各大操作系统平台的开源的客户端,也提供了很多强大的 APIs 给开发者创建自己的客户端和机器人。

    5 引用 • 35 回帖
  • Access
    1 引用 • 3 回帖 • 3 关注
  • Wide

    Wide 是一款基于 Web 的 Go 语言 IDE。通过浏览器就可以进行 Go 开发,并有代码自动完成、查看表达式、编译反馈、Lint、实时结果输出等功能。

    欢迎访问我们运维的实例: https://wide.b3log.org

    30 引用 • 218 回帖 • 644 关注
  • Flutter

    Flutter 是谷歌的移动 UI 框架,可以快速在 iOS 和 Android 上构建高质量的原生用户界面。 Flutter 可以与现有的代码一起工作,它正在被越来越多的开发者和组织使用,并且 Flutter 是完全免费、开源的。

    39 引用 • 92 回帖 • 1 关注
  • OnlyOffice
    4 引用 • 21 关注
  • jQuery

    jQuery 是一套跨浏览器的 JavaScript 库,强化 HTML 与 JavaScript 之间的操作。由 John Resig 在 2006 年 1 月的 BarCamp NYC 上释出第一个版本。全球约有 28% 的网站使用 jQuery,是非常受欢迎的 JavaScript 库。

    63 引用 • 134 回帖 • 736 关注
  • MySQL

    MySQL 是一个关系型数据库管理系统,由瑞典 MySQL AB 公司开发,目前属于 Oracle 公司。MySQL 是最流行的关系型数据库管理系统之一。

    693 引用 • 537 回帖
  • 开源

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

    410 引用 • 3588 回帖
  • Caddy

    Caddy 是一款默认自动启用 HTTPS 的 HTTP/2 Web 服务器。

    12 引用 • 54 回帖 • 162 关注
  • PHP

    PHP(Hypertext Preprocessor)是一种开源脚本语言。语法吸收了 C 语言、 Java 和 Perl 的特点,主要适用于 Web 开发领域,据说是世界上最好的编程语言。

    179 引用 • 408 回帖 • 485 关注
  • Firefox

    Mozilla Firefox 中文俗称“火狐”(正式缩写为 Fx 或 fx,非正式缩写为 FF),是一个开源的网页浏览器,使用 Gecko 排版引擎,支持多种操作系统,如 Windows、OSX 及 Linux 等。

    7 引用 • 30 回帖 • 390 关注
  • V2EX

    V2EX 是创意工作者们的社区。这里目前汇聚了超过 400,000 名主要来自互联网行业、游戏行业和媒体行业的创意工作者。V2EX 希望能够成为创意工作者们的生活和事业的一部分。

    16 引用 • 236 回帖 • 270 关注
  • Ngui

    Ngui 是一个 GUI 的排版显示引擎和跨平台的 GUI 应用程序开发框架,基于
    Node.js / OpenGL。目标是在此基础上开发 GUI 应用程序可拥有开发 WEB 应用般简单与速度同时兼顾 Native 应用程序的性能与体验。

    7 引用 • 9 回帖 • 400 关注
  • 创业

    你比 99% 的人都优秀么?

    82 引用 • 1395 回帖
  • BND

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

    107 引用 • 1281 回帖 • 34 关注
  • frp

    frp 是一个可用于内网穿透的高性能的反向代理应用,支持 TCP、UDP、 HTTP 和 HTTPS 协议。

    20 引用 • 7 回帖 • 3 关注
  • 印象笔记
    3 引用 • 16 回帖
  • 安装

    你若安好,便是晴天。

    132 引用 • 1184 回帖 • 1 关注
  • 人工智能

    人工智能(Artificial Intelligence)是研究、开发用于模拟、延伸和扩展人的智能的理论、方法、技术及应用系统的一门技术科学。

    159 引用 • 305 回帖
  • Openfire

    Openfire 是开源的、基于可拓展通讯和表示协议 (XMPP)、采用 Java 编程语言开发的实时协作服务器。Openfire 的效率很高,单台服务器可支持上万并发用户。

    6 引用 • 7 回帖 • 102 关注
  • 禅道

    禅道是一款国产的开源项目管理软件,她的核心管理思想基于敏捷方法 scrum,内置了产品管理和项目管理,同时又根据国内研发现状补充了测试管理、计划管理、发布管理、文档管理、事务管理等功能,在一个软件中就可以将软件研发中的需求、任务、bug、用例、计划、发布等要素有序的跟踪管理起来,完整地覆盖了项目管理的核心流程。

    6 引用 • 15 回帖 • 31 关注
  • CAP

    CAP 指的是在一个分布式系统中, Consistency(一致性)、 Availability(可用性)、Partition tolerance(分区容错性),三者不可兼得。

    12 引用 • 5 回帖 • 640 关注