Question

以下是一些我试图提高可读性的代码。它可以工作，但是嵌套的for循环和try / if使得初看起来有点难以理解。

有人可以就如何加入嵌套for循环或压缩此代码给我一些建议吗？

matcher = None
if re.match(_RE_OBJECT, nodes.replace(LQMN, '')):
  matcher = alias
else:    
  for x in lister[0].conditions:
    for y in x.codes:
      try:
        if y.id.split(',')[1] == condition:
          matcher = x.codenames
      except IndexError:
        pass

Answer 1

您可以使用生成器表达式来嵌套循环并添加一个使IndexError处理程序过时的过滤器：

candidates = ((x, y) for x in lister[0].conditions for y in x.codes if ',' in y.id)
for x, y in candidates:
    if y.id.split(',')[1] == condition:
        matcher = x.codenames

除了x和y之外，使用更有意义的名称，可以提高可读性：

candidates = ((cond, code) for cond in lister[0].conditions for code in cond.codes
              if ',' in code.id)
for cond, code in candidates:
    if code.id.split(',')[1] == condition:
        matcher = cond.codenames

Answer 2

您可以使用单行if和continue语句而不是try语句：

some_var = y.id.split(',')
if len(some_var) < 2: continue
if some_var[1] == condition:
    matcher = x.codenames

并将some_var替换为有意义的名称。

在python中嵌套for循环的单行

2 个答案: