История изменений
Исправление Aber, (текущая версия) :
Это не работает. Такие штуки всегда дают до жопы ложных срабатываний, в итоге через полгода проект или обрастает костылями для удовлетворения этого линтера, или его просто выкидывают.
Это потому что статические анализаторы кто-то внедрил для галочки, чтоб записать строчку в резюме, в «достижения на прошлой работе».
Новые правила статического анализа вносятся аккуратно. На еженедельном митинге предлагается внедрить новое правило/конвенцию в коде, которое должно форсироваться статическим анализатором. Выслушивается конструктивная критика команды. Люди обычно начинают сопротивляться, надо немного надавить. Новое правило внедряется на тестовый период. Через неделю две все недовольные должны кинуть примеры кода когда срабатывание было ложным. И это самое интересное…
Часто оказывается что «ложные» срабатывания (которые приводятся примером неработающей проверки) половина команды посчитает чистейшим, калиброванным говнокодом. И внезапно недовольные оказывается в клинче, ведь другие девелоперы такой код начинают критиковать прямо на этой разборке. И часто по итогам разборки такой код решают переписать а проверку оставить.
Иногда действительно есть ложное срабатывание в одном конкретном месте и тогда используется магическое слово которое выключает статическую проверку на конкретном фрагменте кода. Ну и иногда правило действительно оказывается непригодным и его отбрасывают.
Вот так работают хорошие команды со статическими анализаторами.
Исправление Aber, :
Это не работает. Такие штуки всегда дают до жопы ложных срабатываний, в итоге через полгода проект или обрастает костылями для удовлетворения этого линтера, или его просто выкидывают.
Это потому что статические анализаторы кто-то внедрил для галочки, чтоб записать строчку в резюме, в «достижения на прошлой работе».
Новые правила статического анализа вносятся аккуратно. На еженедельном митинге предлагается внедрить новое правило/конвенцию в коде, которое должно форсироваться статическим анализатором. Выслушивается конструктивная критика команды. Люди обычно начинают сопротивляться, надо немного надавить. Новое правило внедряется на тестовый период. Через неделю две все недовольные должны кинуть примеры кода когда срабатывание было ложным. И это самое интересное…
Часто оказывается что «ложные» срабатывания (которые приводятся примером неработающей проверки) половина команды посчитает чистейшим, калиброванным говнокодом. И внезапно недовольные оказывается в клинче, ведь другие девелоперы такой код начинают критиковать прямо на этой разборке. И часто по итогам разборки такой код решают переписать а проверку оставить.
Иногда действительно есть ложное срабатывание в одном место и тогда используется магическое слово которое выключает статическую проверку на конкретном фрагменте кода. Ну и иногда правило действительно оказывается непригодным и его отбрасывают.
Вот так работают хорошие команды со статическими анализаторами.
Исходная версия Aber, :
Это не работает. Такие штуки всегда дают до жопы ложных срабатываний, в итоге через полгода проект или обрастает костылями для удовлетворения этого линтера, или его просто выкидывают.
Это потому что статические анализаторы кто-то внедрил для галочки, чтоб записать строчку в резюме, в «достижения на прошлой работе».
Новые правила статического анализа вносятся аккуратно. На еженедельном митинге предлагается внедрить новое правило/конвенцию в коде, которое должно форсироваться статическим анализатором. Выслушивается конструктивная критика команды. Люди обычно начинают сопротивляться, надо немного надавить. Новое правило внедряется на тестовый период. Через неделю две все недовольные должны кинуть примеры кода когда срабатывание было ложным. И это самое интересное…
Часто оказывается что «ложные» срабатывания (которые приводятся примером неработающей проверки) половина команды посчитает чистейшим, калиброванным говнокодом. И внезапно недовольные оказывается в клинче, ведь другие девелоперы такой код начинают критиковать прямо на этой разборке. И часто по итогам разборки такой код решают переписать а проверку оставить.
Иногда действительно есть ложное срабатывание в одном место и тогда используется магическое слово которое выключает статическую проверку на конкретном фрагменте кода. Ну и иногда правило дейсвительно оказывается непригодным и его отбрасывают.
Вот так работают хорошие команды со статическими анализаторами.