如何才能减少代码的嵌套层级？

[kuoruan]

Linus Torvalds:

If you need more than 3 levels of indentation, you ’ re screwed anyway, and should fix your program.

如果你的代码里需要有超过三层的缩进，那么你已经搞砸了，应该修改你的代码。

这里贴一段我写的代码

function do_kcptun_update(btn) {
	btn.disabled = true;
	btn.value = '<%:Downloading...%>';
	add_remove_page_notice(true);
	var kcptun_update_url = '<%=dsp.build_url("admin/services/kcptun/update/kcptun")%>';
	(new XHR()).get(kcptun_update_url, {
		token: token_str,
		url: kcptun_info ? kcptun_info.url.download : ''
	}, function (x, json) {
		if (x && x.status == 200) {
			if (json.code == 0) {
				btn.value = '<%:Extracting...%>';
				(new XHR()).get(kcptun_update_url, {
					token: token_str,
					task: "extract",
					file: json.file,
					subfix: kcptun_info ? kcptun_info.type : ''
				}, function (x, json) {
					if (x && x.status == 200) {
						if (json.code == 0) {
							btn.value = '<%:Moving...%>';
							(new XHR()).get(kcptun_update_url, {
								token: token_str,
								task: "move",
								file: json.file
							}, function (x, json) {
								if (x && x.status == 200) {
									if (json.code == 0) {
										on_update_success(btn);
									} else {
										on_request_error(btn, json);
									}
								} else {
									on_request_error(btn);
								}
							});
						} else {
							on_request_error(btn, json);
						}
					} else {
						on_request_error(btn);
					}
				});
			} else {
				on_request_error(btn, json);
			}
		} else {
			on_request_error(btn);
		}
	});
}

这是一段用来更新程序的 JavaScript，但是代码的继续执行依赖于上一次请求的成功执行。 可苦于没有好的优化方案，只能写出这种垃圾代码了。

[zn]

短路疗法了解一下。



其实就是楼上说的反向判断啦，先判断不符合条件的，尽快返回，后面剩下合法条件再做处理。

[GoLand]

early return.

[qiumaoyuan]

@kuoruan 封装是唯一正确的办法。不仅仅是封装成方法，还可以封装成类。

要嵌套多少层是业务逻辑决定的，而不是代码编写方式决定的。

其实你不知道你调用的代码库背后已经有了多少层的逻辑判断。

[evitceted]

rx

[yongjing]

callback hell

[sampeng]

封装成函数+promise 可以解决问题。。但 promise 也可能陷入调用链太长的问题。总比这样强不是。

楼上有说封装成函数没减少层级。有些逻辑是没办法减少的。但封装函数最少一眼能看明白逻辑，让代码清晰可见的最终目的不就是可读性尚可。

early return 也是一个很好的手段。稍微减少一些不必要判断层级。

唔。。少用匿名函数。。简直是噩梦

[pluschen]

啥？封装成函数没减少层数？
别逗，哪有什么层数？都是 JMP。

[sampeng]

仔细看了代码。。只是各种 else 处理错误而已。。。这个。。很难吗？

[xiaojunjor]

https://coolshell.cn/articles/17757.html

[kuoruan]

@sampeng #28 不好意思，我能力不行，让您见笑了

[zhlssg]

async await ,提早 return

[saulshao]

If you need more than 3 levels of indentation, you ’ re screwed anyway, and should fix your program.
这句话本来就是说的是一个函数里面的事情。

[cs923]

RxJs(没用过 应该能解决)

[Justin13]

就嵌套而言，可以考虑 early return，抽函数，就业务而言，可以考虑使用更优雅的库，promise,async

[we2ex]

@xiaojunjor #29 正解，第一时间想到的也是这篇文章

[banricho]

这个代码的根本原因在于直接原生 XHR …

不用任何库的情况下，封装一个 fetch
剩下是处理异步的问题，楼上都说了
最后才是 if 的问题

[wlwood]

其实，感觉，用 react.js, vue 这些框框，这种地狱应该挺好解决啊

[noNOno]

歪个楼,动感光波?

[Exia]

楼主的地狱回调，能自己看懂就还不错了，应该用 promise 吧，我也要向这个方面提升一下。

[LevineChen]

多个 return 也不好 可以用 do while （ false ） break 实现类似功能