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