pylint false positive для неиспользуемого аргумента

Question

Я занимаюсь очисткой кода с помощью pylint, чтобы иметь возможность использовать его для проверки перед фиксацией. У меня есть много предупреждений о неиспользованном аргументе, когда они используются. Вот пример, вызывающий ложное срабатывание.

def addSeven(foo): #Here I have a warning "Unused argument 'foo'" 
    foo += [7] 

example = [3, 4, 5, 6] 

addSeven(example) 
print example 

Я не хочу, чтобы подавить в глобальном масштабе это предупреждение, потому что я хотел бы видеть время, когда аргумент действительно не используется. Есть ли другой вариант, который вручную добавляет отключенный комментарий в каждом случае? Это известная проблема с pylint?

Answer 1

Это разумное поведение от pylint; если переданный объект неизменен, то данное утверждение по существу не является оператором. Только когда он изменен, он кажется неправильным.

К сожалению, если вы не хотите отключать предупреждение в глобальном масштабе, вам необходимо отключить его на один экземпляр.

Answer 2

Вы можете отключить его для любой сферы, добавив:

def myfunc(a): 
    # pylint: disable=W0612,W0613 

см https://pylint.readthedocs.io/en/latest/faq.html#is-it-possible-to-locally-disable-a-particular-message

Answer 3

pylint, как правило, является хорошим показателем плохого стиля. Даже когда он дает «ложный положительный результат», вероятно, это связано с тем, что он делает что-то против конвенции. Я не эксперт, но я бы сказал, что функция, которая имеет только побочный эффект, не является оптимальной. Некоторые люди (например, Роберт Мартин в «Чистом коде»), говорят, что все побочные эффекты - ложь.

Я рекомендую (опять же, я не являюсь специалистом):

def addSeven(foo): 
    return foo + [7] 

example = [3, 4, 5, 6] 

example = addSeven(example) 

Аргументы должны быть только для ввода и вывода должны быть с помощью возвращаемого значения. Насколько мне известно, аргументы вывода - это плохая практика.