这是一个创建于 1658 天前的主题，其中的信息可能已经有所发展或是发生改变。
 |
1
lxk11153 2020-06-08 16:59:04 +08:00
开局一张图？
|
 |
2
InkAndBanner OP @lxk11153 故事全在图里了啊
|
 |
3
l1nyanm1ng 2020-06-08 17:29:17 +08:00
我感觉楼主是想表达通过 fileName 去建立 InputStream 如果这个文件在磁盘不存在的话，这个函数就要返回 null 了，感觉这个函数有待加强，应该在 catch 部分去创建文件，再返回 XSSFWorkbook
|
|
4
InkAndBanner OP 不仅是代码鲁棒性的问题 首先：不关流，其次用捕捉异常直接用 Exception， 这和不 catch 有什么区别 ，即使出问题了也看不出来因为什么抛异常。
|
 |
5
Vegetable 2020-06-08 18:12:16 +08:00
没人觉得 unused variable 很难受吗
|
 |
6
GM 2020-06-08 18:17:11 +08:00
@InkAndBanner 说不定 XSSWorkBook 里自己维护了流呢？
出错直接 printStackTrace 、没用日志框架倒是个槽点。 其实吧，这代码质量已经算可以的了，满分 100 的话，能打 70 分吧。 所以，知足吧骚年，你根本没见过那种真正的“这种代码”！ |
 |
7
wangkun025 2020-06-08 18:25:42 +08:00
哈哈，我承认是猪队友了。
|
 |
8
pinkSlime 2020-06-08 19:12:43 +08:00
起个吓人的标题就想搞个大新闻
凭良心想 谁都有犯迷糊的时候 要思考的不该是这代码怎么能顺利提交的吗 应该是某种静态检查工具把第一道关 codereview 把第二道关吗 如果你正在做 review 的事 这不正好体现了你的价值吗 |
 |
9
john6lq 2020-06-09 01:12:54 +08:00 via iPhone
刚毕业呢？
你先看看现在的 InputStream 实现了哪些接口，再看看 JDK1.7 关于 try 中进行 closable 对象操作的相关知识？ |
 |
10
msg7086 2020-06-09 06:45:23 +08:00
这质量比我司国外招来打工的写出来的代码强太多了。
|
 |
11
suqingzhou2020 2020-06-09 09:06:31 +08:00
@john6lq 他也没用 try-with-resource 啊
|
|
12
InkAndBanner OP @john6lq 我看了 怎么了？
|
|
13
InkAndBanner OP @john6lq 尴尬不尴尬不尴尬不
|
|
14
InkAndBanner OP @pinkSlime 只是感慨一下，同时也为自己的垃圾感到无奈罢了
|
Select Language