代码重构：如何优雅处理for循环中的多个continue语句

我来分享几个能让这段代码更整洁优雅的思路——毕竟这种满是continue的条件判断堆在一起，确实容易让代码显得杂乱难读，还不好维护：

思路1：把验证逻辑抽成独立函数 #

把所有需要判断的条件和对应的日志打印逻辑，统一封装到一个专门的验证函数里，主函数只需要关注“验证不通过就跳过”的核心逻辑：

def validate_current_item():
    # 第一个条件：只判断不打日志
    if somevar1 and somevar2 is None:
        return True
    # 检查somevar3
    if somevar3 is None:
        logger.info('对应的somevar3缺失日志消息')
        return True
    # 检查somevar4
    if somevar4 is None:
        logger.info('对应的somevar4缺失日志消息')
        return True
    # 处理需要计算中间变量的检查
    somevar5 = x.split('\n')
    somevar6 = y.split('\n')
    somevar7 = do_something_2(somevar5, somevar6)
    if somevar7 is None:
        logger.info('对应的somevar7生成失败日志消息')
        return True
    # 所有检查通过，返回False表示不跳过
    return False

def somefunc(*args, **kwargs):
    # 注意Python3里用items()替代iteritems()
    for p_key, p_det in somedict.items():
        get_all_somevars(p_det)
        if validate_current_item():
            continue
        do_something_3()

这种方式的好处是把分散的验证逻辑集中管理，主函数的逻辑瞬间清爽，后续要加新的检查或者修改现有条件，直接去验证函数里操作就行，不用动主循环的核心代码。

思路2：用检查项列表统一处理 #

如果不想单独拆函数，也可以把不需要中间变量的检查项做成一个列表，循环遍历判断，减少重复的if...continue结构：

def somefunc(*args, **kwargs):
    for p_key, p_det in somedict.items():
        get_all_somevars(p_det)
        
        # 定义检查项：(判断逻辑, 日志消息)，日志为None则只判断不打印
        check_items = [
            (lambda: somevar1 and somevar2 is None, None),
            (lambda: somevar3 is None, 'somevar3缺失的日志内容'),
            (lambda: somevar4 is None, 'somevar4缺失的日志内容'),
        ]
        
        # 遍历检查项，判断是否需要跳过
        skip = False
        for check_func, log_msg in check_items:
            if check_func():
                if log_msg:
                    logger.info(log_msg)
                skip = True
                break
        if skip:
            continue
        
        # 处理需要计算中间变量的检查（前面都通过才执行）
        somevar5 = x.split('\n')
        somevar6 = y.split('\n')
        somevar7 = do_something_2(somevar5, somevar6)
        if somevar7 is None:
            logger.info('somevar7生成失败的日志内容')
            continue
            
        do_something_3()

这种方式能把重复的判断逻辑收敛起来，避免一遍又一遍写if...continue，同时也能清晰区分开“简单条件检查”和“需要计算中间变量的检查”，还能减少不必要的计算开销（前面检查不通过的话，后面的变量就不用算了）。

思路3：拆分循环体为子函数，用return替代continue #

把每个循环项的处理逻辑单独拆成子函数，用return代替continue，让每个逻辑块的职责更单一：

def process_single_item(p_det):
    get_all_somevars(p_det)
    if somevar1 and somevar2 is None:
        return
    if somevar3 is None:
        logger.info('somevar3对应的日志消息')
        return
    if somevar4 is None:
        logger.info('somevar4对应的日志消息')
        return
    somevar5 = x.split('\n')
    somevar6 = y.split('\n')
    somevar7 = do_something_2(somevar5, somevar6)
    if somevar7 is None:
        logger.info('somevar7对应的日志消息')
        return
    do_something_3()

def somefunc(*args, **kwargs):
    for p_key, p_det in somedict.items():
        process_single_item(p_det)

这种写法的可读性非常强，主函数只负责遍历字典并调用处理函数，子函数里的每一步判断都清晰明了，一旦某个条件不满足就直接退出子函数，自然就跳过了后续的逻辑，完全替代了continue的作用。

以上几种思路都能在保留不同条件对应不同日志的前提下，让代码更精简整洁，你可以根据项目的现有代码风格选择最适合的方式。

内容的提问来源于stack exchange，提问作者Pro