V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
V2EX 提问指南
InkAndBanner
V2EX  ›  问与答

不好好学习就只能和写出这种代码的人做同事了

  •  
  •   InkAndBanner · 2020-06-08 16:54:48 +08:00 · 2622 次点击
    这是一个创建于 1389 天前的主题,其中的信息可能已经有所发展或是发生改变。
    lxk11153
        1
    lxk11153  
       2020-06-08 16:59:04 +08:00
    开局一张图?
    InkAndBanner
        2
    InkAndBanner  
    OP
       2020-06-08 17:28:08 +08:00
    @lxk11153 故事全在图里了啊
    l1nyanm1ng
        3
    l1nyanm1ng  
       2020-06-08 17:29:17 +08:00
    我感觉楼主是想表达通过 fileName 去建立 InputStream 如果这个文件在磁盘不存在的话,这个函数就要返回 null 了,感觉这个函数有待加强,应该在 catch 部分去创建文件,再返回 XSSFWorkbook
    InkAndBanner
        4
    InkAndBanner  
    OP
       2020-06-08 18:09:18 +08:00
    不仅是代码鲁棒性的问题 首先:不关流,其次用捕捉异常直接用 Exception, 这和不 catch 有什么区别 ,即使出问题了也看不出来因为什么抛异常。
    Vegetable
        5
    Vegetable  
       2020-06-08 18:12:16 +08:00
    没人觉得 unused variable 很难受吗
    GM
        6
    GM  
       2020-06-08 18:17:11 +08:00
    @InkAndBanner 说不定 XSSWorkBook 里自己维护了流呢?
    出错直接 printStackTrace 、没用日志框架倒是个槽点。

    其实吧,这代码质量已经算可以的了,满分 100 的话,能打 70 分吧。

    所以,知足吧骚年,你根本没见过那种真正的“这种代码”!
    wangkun025
        7
    wangkun025  
       2020-06-08 18:25:42 +08:00
    哈哈,我承认是猪队友了。
    pinkSlime
        8
    pinkSlime  
       2020-06-08 19:12:43 +08:00
    起个吓人的标题就想搞个大新闻
    凭良心想 谁都有犯迷糊的时候 要思考的不该是这代码怎么能顺利提交的吗
    应该是某种静态检查工具把第一道关 codereview 把第二道关吗
    如果你正在做 review 的事 这不正好体现了你的价值吗
    john6lq
        9
    john6lq  
       2020-06-09 01:12:54 +08:00 via iPhone
    刚毕业呢?
    你先看看现在的 InputStream 实现了哪些接口,再看看 JDK1.7 关于 try 中进行 closable 对象操作的相关知识?
    msg7086
        10
    msg7086  
       2020-06-09 06:45:23 +08:00
    这质量比我司国外招来打工的写出来的代码强太多了。
    suqingzhou2020
        11
    suqingzhou2020  
       2020-06-09 09:06:31 +08:00
    @john6lq 他也没用 try-with-resource 啊
    InkAndBanner
        12
    InkAndBanner  
    OP
       2020-06-09 10:16:48 +08:00
    @john6lq 我看了 怎么了?
    InkAndBanner
        13
    InkAndBanner  
    OP
       2020-06-09 10:17:01 +08:00
    @john6lq 尴尬不尴尬不尴尬不
    InkAndBanner
        14
    InkAndBanner  
    OP
       2020-06-09 10:18:01 +08:00
    @pinkSlime 只是感慨一下,同时也为自己的垃圾感到无奈罢了
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   我们的愿景   ·   实用小工具   ·   1135 人在线   最高记录 6543   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 27ms · UTC 18:42 · PVG 02:42 · LAX 11:42 · JFK 14:42
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.