190 条回复  ·  1717 次点击
yippees 小成 2024-9-5 15:00:42

网上看到的段子照进了现实,这种代码出现在我的项目中!

excel 表头不是固定顺序吗。。我可能就是最后 getAllIs ()返回一个 bool list 。就是里面要把 isXXX 排序下
然后循环 setColumnHidden
cslive 小成 2024-9-5 15:07:42

网上看到的段子照进了现实,这种代码出现在我的项目中!

唯一有风险的就算空指针问题,其它没啥问题,还方便后续更改
IdJoel 小成 2024-9-5 15:17:02

网上看到的段子照进了现实,这种代码出现在我的项目中!

对大部分人的工作水平产生了怀疑,这东西不优化还有人说好?  这玩意一旦有要求改字段加字段有多麻烦?
这个要是我用 PHP 实现,我先吧每个字段 和 取值的方法搞到一个数组里面,然后单独写个 tansformer 处理下要处理的数据
以后加字段,改字段直接修改 array 就行了。

就这代码还有人说好,我真服了
chenqh 小成 2024-9-5 15:19:32

网上看到的段子照进了现实,这种代码出现在我的项目中!

@IdJoel 你这个样子每个字段,都有对应的方法,那所有的方法不就满天飞了?和这个有区别吗?
tmplinshi 小成 2024-9-5 15:32:28

网上看到的段子照进了现实,这种代码出现在我的项目中!

cell.getStringCellValue() 需要执行一大段代码才返回结果,明明调用一次就够了,截图中几十上百次的重复调用,为什么很多人不觉得有问题?

https://github.com/trlogic-developer/birlikteal/blob/2064c2c270907d9589d0052af9e4d28fff8ad3c1/crowdfunding-web/src/main/java/org/apache/poi/xssf/usermodel/XSSFCell.java#L350-L403
suofeiya 小成 2024-9-5 15:55:28

网上看到的段子照进了现实,这种代码出现在我的项目中!

这段代码折叠起来,眼不见心不烦.https://i.imgur.com/L62ZP7V.png
dbing 小成 2024-9-5 16:05:02

网上看到的段子照进了现实,这种代码出现在我的项目中!

if 不 if 的先不说,主要问题不该是有空概率的变量.equals(字符串常量)吗?不担心空指针?
jiangzm 小成 2024-9-5 16:15:49

网上看到的段子照进了现实,这种代码出现在我的项目中!

好的问题来了, 该功能有个改动安排你来负责, 那你是继续按原来的方式改动呢,还是花时间重构呢?

如果你随波逐流那你何尝不是屎山代码的贡献者呢?

如果花时间重构,那你的问题在团队中提出来才有意思。

屎山代码肯定不是从第一行代码就是屎, 就是因为有太多不断的添屎的人才有了高高的屎山。
esile 小成 2024-9-5 16:59:23

网上看到的段子照进了现实,这种代码出现在我的项目中!

写的很好 一目了然
jayneyang 小成 2024-9-5 17:05:21

网上看到的段子照进了现实,这种代码出现在我的项目中!

如果是我,直接扔给前端做 excel 导出的功能
返回顶部