工作 8 年，这样 while 循环调别人接口是什么水平？

[Mannnnning]

同事写的代码，循环调了别人接口 50+次

var callRpc(var id) {
	此处省略(P99 = 2S)
}
    
var getId() {
	var id = 0;
	while(true) {
    	tmp = callRpc(id);
        if (tmp <=0) {
        	break;
        }
        id = tmp;
    }
    return id;
}

[Levox]

所以楼主有什么高见可以看看吗

[sir283]

抛开业务背景直接谈代码，没有意义，就像有人会问：会 printf("hello");是什么水平一样。

[wanei]

大部分代码都是处理边界条件和异常的。功能实现就行。此外 id 返回的值不符合预期，return 应该在 break 那。

[wanei]

@wanei #23 缩进看错了...

[lingxipaofan]

这个缩进看的好难受

[Mannnnning]

@lesismal 想要吐槽一下同事写的代码，昨天我刚把这个循环去掉了～

[zuixinwenyue]

你真闲的！没事给自己找优越感，在代码上贬低一下同事

[GoogleQi]

@defphilip Ai 更容易写垃圾代码

[lesismal]

@Mannnnning 但是你描述的上下文不全，也不知道上下游具体的代码实现，并不能说明去掉循环更好，这也是我说看不懂的原因。。。