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

[szpinc1102]

给大家观摩一下，给我看傻了

[levelworm]

这大概就是不停的粘贴复制造成的吧，懒得重构了？

[TsubasaHanekaw]

导入导出 excel 表格真是最屎的活

[szpinc1102]

@levelworm 这是新写的代码，新代码都能写成这样

[robinchina]

能用就行

[szpinc1102]

@TsubasaHanekaw 但是也不能写成这样吧

[zizon]

你看中间凸出的 应用服务器 等几个你就知道这种写法的某种形式的干净存粹了.

[jiabing520a]

就说是不是一目了然吧，(●'◡'●)

[jorneyr]

你就说这是不是执行效率最高的实现吧！！！

[moudy]

@szpinc1102 有可能是脚本生成的

[moudy]

@jorneyr 这堆 equal 走下来显然不是啊

[fruitmonster]

所以最合理的方式是使用 map? 还是怎么

[tool2dx]

90%的业务代码又没性能需求，我都是随便写的，怎么好维护，就怎么来。

[lervard358]

好奇 正确的写法是啥 弄个 table ，提前把映射关系弄好吗

[puzzle9]

能用 直观
虽然看起来不优雅
不过 你给对方发工资吗
不发的话 那管啥子

[greensea]

说不定是 copilot 写的呢，反正 tab 一下就都出来了，毫不费力

[bug123]

好奇有更好的写法吗

[AlexRoot]

想知道这个长截图是如何实现的？

[laoyur]

CA IP 表示看不到图

[smdbh]

其实也没啥不好

[laoyur]

额，发完上面的回复突然又能看到了 😅
如果确实需要通过字符串比较，那这样写好像也没毛病啊，至少很清晰
可能大部分执行的都是同样的一句代码，显得比较啰嗦，可以把条件合并起来

[yul36074]

你们都能看到图吗

[Track13]

还行，我手上一个项目一个 vue 页面 6000 多行，一个 echart 图表依赖 2 个 json ，文件合计 20 几 k😢

[woodwhales]

这段代码，从语法结构上看是 Java 。可以推测这段代码在导出 excel 文件并设置满足条件的指定列进行隐藏。如果是我重构，我能想到的是去除多重 if else ，用 map 集合先保存每个条件，然后一个循环遍历 map 中是否存在满足条件。伪代码如下：

```java

// 业务逻辑外部定义
static Map<String, Function<ExportColumn>> map = new HashMap<>();

static {
map.put("服务器名称", data -> !data.getAppServer().isServerName());
}


// 业务逻辑内

if(map.containsKey(cell.getStringValue())) {
boolean flag = map.get(cell.getStringValue()).apply(dto.getExportColumn());
sheet.setColumnHidden(index, flag);
}

```

[ooxx2123]

要我重构的话，也是 23L 的思路，好奇大家有什么其他方式吗？
或者保留原样，我感觉也还好，就是看起来不太简洁。

[potatowish]

excel 导入导出是初级 java 必备技能，怎么把很容易写成💩的代码写的优雅这就是中级 java 的技能了

[ma836323493]

用反射， 另外 model 上一般也有注解 ApiModelProtery

[yb2313]

丑陋的 equal

[ice]

挺好，我想不到重构的方法，而且也没想到最后生成的 Excel 是个多复杂的玩意

[loocao]

清晰明了，简单又方便维护

[adminii]

项目组代码量考核，第一了

[awalkingman]

@yul36074 第一次加载需要挺长时间

[chenyu8674]

虽然不优雅，但好处是随便换个开发都能懂

离真正的屎山差远了

[jackOff]

整个枚举类，初始化时把枚举类灌入一个 map 里面

[xiaoshengwpp]

这代码写的很优雅啊，谁上手看一眼都能维护的来。

[justfindu]

先不要吐槽, 先改一版你觉得好的. 一起放出来

[luohaiyang]

又不是不能用，狗头

[angenin]

@adminii 这个方法写完，当天代码行数的任务量直接完成🤣

[junwind]

这种千万千万不要想着重构，能跑就行。

[enihcam]

这不是代码问题，这是管理问题。

[enihcam]

如果老板是“能用就行”的心态，这个团队长不了。

[meeop]

我觉得写得挺好的，逻辑简洁清晰易于维护


如果觉得这都不好，你手动封装成其他逻辑，出了 bug 你背锅就行
如果不敢这么做，这也是上一个人不这么做的原因

[toan]

挺好，要逻辑有逻辑，要代码量有代码量，还能正常运行 👍

[5sheep]

我对这段代码的评价是：满意

[byc4i]

看到等保测评，不知道是不是某地网安部门的外包。你这个属于泄露源码了。

[yhxx]

曾经我也觉得看不下去
现在我也这么写
清晰易懂，更重要的是代码量拉满了

[lisxour]

最起码一目了然，你就该偷笑了

[barrysj]

全量 map 还是省不了的吧，顶多把 set true 那行抽出来

[wangritian]

不考虑反射吗，真的一行行硬写

[QXDM]

[fresco]

好奇有更好的写法吗

[hekunhotmail]

gpt 一键重构

[RobertWu]

真诚发问，反射要怎么写？

[janda]

代码和人有一个能跑就行

[opengps]

我问下更好的写法是啥？我有过类似的经历，因为具体字段完全是现场要求的，没法提前知道，所以做成了一个批量赋值的模式，别看内容多，其实只是几下纵向复制粘贴就能出来这么多代码

[NEPv5NA6R8R3Y11u]

[daybreakfangyang]

看着头大🤦‍♂️，空指针警告

[wyx119911]

其实还不错的，很清晰。如果是我的话会改成用 switch case 优化下分支判断的性能，但大体结构保持。

[ShaoLongFei]

@QXDM 我觉得这种代码明显更糟糕

[teenight]

你行你上

[panlatent]

改 if-else 到 switch :

title = cell.getStringCellValue()
switch (title) {
case "xxxx":
break;
}

这样可以确保每列处理方法集中在一个 case 。

对于最多的检查+隐藏操作，可以封装为一个函数来改进 if 结构：
hiddenColumnIfNot(dto.getExpartColumn().getAppProject().isProjectYea())

dto.getExpartColumn() 输入起来费劲可改成一个中间变量

[plasticman64]

说不定他的工资很低呢，收多少钱干什么活呗

[ZGame]

@Track13 解耦阿 1. widget/chart/echart-wrapper 2. data/json

[gollwang]

没人注意到这些关键词吗？
密码评测、等保、国产化

[Mandelo]

我们用的是自定义注解+导出工具类,类似 easy-excel

[CodeCodeStudy]

代码很直观啊，换个人都懂

[yb2313]

@QXDM py 里面字符串不转成枚举吗, 不然未来改字符串改得你头痛, 还不报错

[lucasdev]

很常见的场景，个人觉得比较“优雅”的做法是，在实体类字段上面加注解，然后通过反射来做

[x86]

别的不说，接盘的人应该庆幸了，你见过 IDE 右边红的跟牛市一样的代码吗

[NoOneNoBody]

10 个以内判断我还可以写，10 个以上我就想用状态机了

[qiaobeier]

不是挺工整的吗，有什么问题？

[nb85144]

能用就行了，反正最后都得变成屎山

[shiguiyou]

读起来很易懂

[Smilencer]

挺好的，应该是高手写的。
随时应对产品经理提出的改动需求，你看有些字段确实需要特殊处理，简单迅速明了。
你吭哧吭哧搞个封装啥的，改的时候有的你苦恼的！

[justdoit123]

需求太多变的场景下，73# 的说法挺适用的。

[Greendays]

看代码都能想象出来 Excel 有多复杂了。这种写法可能真不算差的。

[zhangdawei]

这样好理解

[asdfasasdf]

我觉得挺好的一目了然，后期功能应该也挺容易修改，要是把这个再封装在一个模块那就更完美了

[pkoukk]

每一行的检查都有略有区别，我觉得没多大问题

[QXDM]

@yb2313 #66 没有 enumerate 这都是小事，更恐怖的是所有 write_rows 既没抽象也没做任何映射，光逻辑就快到 7 层，拉了 800 多行

[xcsoft]

< 能跑就行了 > 真的

[oneisall8955]

又不是不能用.jpg

[pecsj]

分支不多就还好，优雅不优雅另说

[retrocode]

看起来是 excel 和 dto 之间缺失了对应关系, 一边可能是甲方提供的 excel 和 项目已有的对象, 这个明显是复制粘贴出来的, 我一眼看到 应用名称 的 case 出现了两次, 感觉用 map 的确会好很多, 这里绝大多数操作都是一样的

[alexsz]

整洁壮观，非常好懂，没毛病，真的

[shylockhg]

@moudy 一般来说编译器应该把这种优化成查表

[Track13]

@ZGame 解不动，2 个 json 快 30kb 了，一个文件 6000 多行没有几行注释。echart 的配置就靠这 2 个 json 各种循环遍历，还有一堆本地数据映射。json 里全是各种金融行业术语缩写。

[woniuppp]

按行数定 kpi

[nightlight9]

@opengps 如果是我，我选择用 excel 模板写，更直观一点

[Felldeadbird]

能跑就行了。
这业务代码基本是这样的：

老板：小明，我想要个 ABCDEFG 的导出。你去实现一下。
小明：好的老板。 老板我已经完成了。（老板应该不常用，直接硬编码导出）
若干天后。
老板：小明，GHJIT 列加一下。
小明：加完了老板（擦，半年后回来，告诉我调整这个导出，弱智老板）

如此反复。

[xuanbg]

我导入导出都是搞个实体类，字段加注解就完了，像这种需求也一样用注解搞定。反正注解这玩意可以很多，一个搞不定就再来几个。哈哈哈

[C0dEr]

如果只是某个特定业务，这么写也无可厚非，如果考虑复用的话，建议优化一下

[xloger]

就这还一堆人说好？这一排下来谁知道哪里有没有哪里有特殊逻辑，哪里直观了？
很明显，改法就是把这里的类型名和判断函数抽出来，弄个 Map 或者类似的合适的数据结构。然后特殊逻辑也拎一块，这才叫直观才叫一目了然好么？
一大堆 if else 里藏几个特殊逻辑，也好意思管这叫好代码？

[git00ll]

我觉得没啥问题，导出功能而已比较边缘的能力不需要过度设计。 而且他还写了注释，看起来一目了然，如果要改这里任何人都能马上上手

[onichandame]

挺直观的

[liquid207]

添加一个 cell name 和 dto column 的映射，可以解决大部分的重复代码。添加一个匿名函数处理 “应用服务器” 之类有特殊逻辑的 cell 就可以了。

[lasuar]

大道至简，楼主还得再看看，不要只从技术角度看现象。

[28Sv0ngQfIE7Yloe]

@xloger

[wusheng0]

说优雅的，说得我都怀疑自己了。

哪怕你按照类型分开来组织呢

唉

[kristofer]

@xloger #92 这论坛里大神有，水货更多，放松心态。有的人本来就不是真心干这一行的，指望他们能有什么代码情怀。

[wusheng0]

@wusheng0 一说 35 就是失业，一说代码就是能跑就行。我是不想遇到这种同事。写得菜真的无所谓，对代码一点追求没有才是最痛苦的。除非你拉的屎永远不会让别人擦屁股