8 层嵌套， 400 多行代码的函数。。。

[asanelder]

遇到一个老的项目，有一个函数是 8 层嵌套，if 套 while 再套 if 再套 while 再套 if ....,而且每个条件判断中，各种&& 套 || 再套 &&，这个 8 层嵌套俺数了一下，一共 400 多行代码。。。

让俺感觉到奇怪的是，之事从事这个项目的老员工对这个函数竟然没有丝毫的意见，他们似乎很平静。

那么问题来了，俺之前受的教育是，函数应该尽量短小，嵌套应该尽量浅。所以遇到这个函数俺自然感觉浑身不适，但那些老员工似乎都很平静。是俺太笨了，理解不了这种 8 层嵌套，还是俺太矫情，太理想主义了，太教条了，还是老员工已经习惯了？

求解惑

[GopherTT]

@noobsheldon hhah 特地数了下 八个：）

[KasonPasser]

好想见识一下，我一般都是复制一个新的函数出来改，原来的能不动尽量不动。
改完了测试没问题再迁移，不过一般出问题了都是没有好结果。
确实想改送你六个字：慎之慎之慎之。

[deplives]

我见过一个实习生写的 Python request handle 多达 1500 多行，仔细一看，他一个 handle 处理了 get post put delete 不同请求

[w516322644]

@justin2018 你是不是在黑 php，我遇见过 java 这么类似写的。

[asanelder]

@codergrowing #58 哈哈，笑尿了

[zuokanyunqishi]

你是没见过 800/1500 行的函数🐶

[cwek]

没坏，别动

[akira]

老员工都知道 这种代码不能动，一动就必然会死人.

[sampeng]

去年这类帖子出来还是尽量自己慢慢重构了。除非是算法类，普通逻辑类的大函数就没有改不掉的。是会有风险，现在程序员都居安思危到这个地步了？太可怕了…

[codingbody]

spring jdbctemplate 中 setvalue 方法 200 多行全是 if else else if

[soulmt]

不要慌，常见操作， 老代码别动就对了

[Daniel17]

一个 switch800 行的见过没。。还全部写在 controller 层

[royan]

补测试，动手改。大不了跑路，怕什么[狗头 /摊手]

[wisetc]

你如果能改并且保证不出问题也要看看做了这件事对谁有利，是否在为前人背锅，成为搏傻游戏里的傻人呢？如果要扩展，并且之前开闭原则都不遵守，这家公司的管理是不是有问题呢？这个问题是你的吗？

[EscYezi]

还真拆过一个 400 行的，按业务逻辑拆成了三个。后来有需求要在这块加东西，还好之前拆了，不然真的要吐

[ruzztok]

逻辑没有问题、不影响性能、没有复用价值的 1000 层又怎样？？现实一点，不要看太多阿里巴巴的编程手册和一些编码规范之类的书