Блог

почему pylint жалуется на этот блок кода?

R1705: Unnecessary "elif" after "return" (no-else-return)

 def f(a):
    if a == 1:
        return 1
    elif a == 2:
        return 2
    return 3
  

Чтобы предотвратить ошибку, мне пришлось создать временную переменную, которая кажется менее приятной.

 def f(a):
    if a == 1:
        b = 1
    elif a == 2:
        b = 2
    else:
        b = 3

    return b
  

Решение:

 def f(a):
    if a == 1:
        return 1
    if a == 2:
        return 2
    return 3
  

Цель else блока — определить код, который не будет выполняться, если условие истинно, поэтому выполнение не будет продолжаться до следующего блока.

Однако в вашем коде основной условный блок имеет оператор return , что означает, что выполнение завершит функцию, поэтому нет необходимости в блоке else: весь последующий код после возврата, по определению, не будет выполнен, если условие истинно. Это избыточно. Его можно заменить простым if .