История изменений
Исправление CrX, (текущая версия) :
Основное, что следует осознать: линтер — не догмат и не священное писание. Это инструмент. Он показывает, на что обратить внимание, подсвечивает, так сказать, потенциальные точки для рефакторинга. Но это не значит, что на каждое его замечание стоит реагировать. Если бы было именно куча if’ов с return’ами, где по сути просто от какого-то проверяемого значения зависит возвращаемое — тогда да, идеоматичнее было бы переписать в виде словаря. В примере не оно — значит просто игнорируем мнение линтера по этому поводу и идём дальше.
Исходная версия CrX, :
Основное, что следует осознать: линтерн — не догмат и не священное писание. Это инструмент. Он показывает, на что обратить внимание, подсвечивает, так сказать, потенциальные точки для рефакторинга. Но это не значит, что на каждое его замечание стоит реагировать. Если бы было именно куча if’ов с return’ами, где по сути просто от какого-то проверяемого значения зависит возвращаемое — тогда да, идеоматичнее было бы переписать в виде словаря. В примере не оно — значит просто игнорируем мнение линтера по этому поводу и идём дальше.