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

开局一张图？

@lxk11153 故事全在图里了啊

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

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

没人觉得 unused variable 很难受吗

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

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

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

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

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

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

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

@john6lq 我看了 怎么了？

@john6lq 尴尬不尴尬不尴尬不

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