如何优雅解决业务代码中必须要重复的嵌套检查返回结果是否为 None？

@SpicyCat @Abirdcfly
他们的代码的本意应该就是出了异常，认为可以忽略异常，然后就返回 None 。但是这样的做法假设有一个函数或者方法，它被**嵌套**调用了多次，所以在一系列嵌套调用的过程必须每一次都要检查 None ，要不然等出了错就不知道异常到底是哪一步骤抛出来的。

我现在有两种做法：
1. 原本返回 None 的地方我不兼容异常，全部抛出
2. 在返回 None 的地方做检查，并且记录日志，但是这样的话代码结构会很丑

请问上面做法是否合理？

linkdesu

2017-03-06 18:21:50 +08:00

这个问题，我觉得可以等同于：请问 V 站各位前辈不得不吃屎时是如何优雅处理的？

其中的煎熬难以言喻，其路漫漫，其修远兮~

huigeer

2017-03-06 19:05:30 +08:00

@linkdesu 把鼻子堵上，

lhw45202

2017-03-06 19:12:33 +08:00

先为现有的业务逻辑建立一些测试用例，然后再开始修改现有实现，保证前面的测试用例通过，逐步修改

guyskk

2017-03-06 21:07:45 +08:00 via Android

把需要用到的代码封装一下再用

lxy

2017-03-06 21:20:01 +08:00

装饰器不就是在这时候用的吗。

cabing

2017-03-06 21:24:16 +08:00

1 可以把一些逻辑拆出来。然后写单元测试~
2 抛出异常，捕获异常写日志~~

billgreen1

2017-03-06 21:30:32 +08:00

try catch 就是这个时候用的啊。我一般是喜欢在关键部分： try: except KeyError xxx except IndexError.....

yunshansimon

2017-03-06 22:20:53 +08:00

Python 不熟，但 script 类语言应该都可以写一个调用函数的函数解决：_call_func 有 3 个参数，函数名，参数个数，参数对象数组（或集合什么的能放下所有种类对象的）。用这个函数调用所有的其他函数，在这个函数里面检查返回值，如果是 none ，就抛出异常，或者打印函数名称。对象方法也可以，重载_call_func 第一个参数是对象，第二个参数是对象的方法，其他一样。然后把所有直接调用都改成调用函数调用。

lrh3321

2017-03-07 09:21:40 +08:00

用日志打印出来吧，出问题返回 None 之前，先把当前的函数名打印出来，再返回。