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

227 条回复 • 2024-09-06 14:59:53 +08:00

1 2 3

❮

❯

201

dddd1919

77 天前

真正诠释了什么叫“代码即注释”

202

wysnxzm

77 天前

@juzisang #107 一看就是被产品的需求狠狠拷打过😂

203

dwlovelife

77 天前

新增一个注解然后标在属性上，注解里是文案，然后用反射我估摸不超过 20 行，而且后面只需要加属性字段和注解就行

204

yippees

77 天前

excel 表头不是固定顺序吗。。我可能就是最后 getAllIs （）返回一个 bool list 。就是里面要把 isXXX 排序下
然后循环 setColumnHidden

205

cslive

77 天前

唯一有风险的就算空指针问题，其它没啥问题，还方便后续更改

206

IdJoel

77 天前

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

就这代码还有人说好，我真服了

207

chenqh

77 天前

@IdJoel 你这个样子每个字段,都有对应的方法,那所有的方法不就满天飞了?和这个有区别吗?

208

tmplinshi

77 天前

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

https://github.com/trlogic-developer/birlikteal/blob/2064c2c270907d9589d0052af9e4d28fff8ad3c1/crowdfunding-web/src/main/java/org/apache/poi/xssf/usermodel/XSSFCell.java#L350-L403

209

suofeiya

77 天前

这段代码折叠起来，眼不见心不烦.

210

dbing

77 天前

if 不 if 的先不说，主要问题不该是有空概率的变量.equals(字符串常量)吗？不担心空指针？

211

jiangzm

77 天前

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

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

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

屎山代码肯定不是从第一行代码就是屎，就是因为有太多不断的添屎的人才有了高高的屎山。

212

jiangzm

77 天前

#211 错别字: 有意思 => 有意义

213

esile

77 天前 via Android

写的很好一目了然

214

jayneyang

77 天前

如果是我，直接扔给前端做 excel 导出的功能

215

NCE

77 天前

you can you up, no can no BB.

这代码可读性，可维护性都是最好的。面对不可知的需求变化，要做到简单可维护，而不是优雅，这里不需要复杂的算法。

216

tool2dx

77 天前

@IdJoel 你别光看前面的字符串比较，看后面的 dto 映射函数，基本上都不带重复的。

217

iceheart

77 天前 via Android

直观的代码就是好代码。不直观，封装的越多就越容易藏污纳垢，隐藏错误。

218

realpg

77 天前

@szpinc1102 #3
我最近也写了这种代码而且不少
主要是这种代码写的很快也不需要复制粘贴
因为是 copilot 替我从列表补全的提供范式 ai tab 一秒写完
除了罗嗦点没啥缺点而且看起来清晰

219

cndenis

77 天前

这代码非常好啊, 清晰明了, 可读性高, 容易维护.

业务的复杂性是不可能消除的, 能消除的只有实现的额外的复杂性.

这段代码没有引入额外复杂性和理解成本, 可以说是最佳实践了.

220

kiddyu

76 天前

仅就图中的代码来说，这样写如何？

switch (true) {
// 大部分
case equals("应用 ID") && !dto.xxxxx:
case equals("应用名称") && !dto.xxxxx:
case equals("应用编码") && !dto.xxxxx:
case ...:
sheet.setXXXXXX();
break;
// 需要特殊处理的
case equals("应用服务器"):
xxxxxxx;
break;
case equals("安全评测信息"):
xxxxxxx;
break;
}