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

[InkAndBanner]

https://imgchr.com/i/tf1XdO

[lxk11153]

开局一张图？

[InkAndBanner]

@lxk11153 故事全在图里了啊

[l1nyanm1ng]

我感觉楼主是想表达通过 fileName 去建立 InputStream 如果这个文件在磁盘不存在的话，这个函数就要返回 null 了，感觉这个函数有待加强，应该在 catch 部分去创建文件，再返回 XSSFWorkbook

[InkAndBanner]

不仅是代码鲁棒性的问题 首先：不关流，其次用捕捉异常直接用 Exception， 这和不 catch 有什么区别 ，即使出问题了也看不出来因为什么抛异常。

[Vegetable]

没人觉得 unused variable 很难受吗

[GM]

@InkAndBanner 说不定 XSSWorkBook 里自己维护了流呢？
出错直接 printStackTrace 、没用日志框架倒是个槽点。

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

所以，知足吧骚年，你根本没见过那种真正的“这种代码”！

[wangkun025]

哈哈，我承认是猪队友了。

[pinkSlime]

起个吓人的标题就想搞个大新闻
凭良心想 谁都有犯迷糊的时候 要思考的不该是这代码怎么能顺利提交的吗
应该是某种静态检查工具把第一道关 codereview 把第二道关吗
如果你正在做 review 的事 这不正好体现了你的价值吗

[john6lq]

刚毕业呢？
你先看看现在的 InputStream 实现了哪些接口，再看看 JDK1.7 关于 try 中进行 closable 对象操作的相关知识？

[msg7086]

这质量比我司国外招来打工的写出来的代码强太多了。

[suqingzhou2020]

@john6lq 他也没用 try-with-resource 啊

[InkAndBanner]

@john6lq 我看了 怎么了？

[InkAndBanner]

@john6lq 尴尬不尴尬不尴尬不

[InkAndBanner]

@pinkSlime 只是感慨一下，同时也为自己的垃圾感到无奈罢了