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

你们都能看到图吗

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

这段代码，从语法结构上看是 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);
}

```

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

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

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

丑陋的 equal

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

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

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

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

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

离真正的屎山差远了

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

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

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

又不是不能用，狗头

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

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

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

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