потому берется сонар и настраиваются правила под себя)

Согласен)

слишком блекбоксно и гибкость теряется. уж лучше тогда тот же jib.

У нас например свой стайлгайд принят в части неймингов

И ещё в чем то, уже забыл чо там ещё есть

опа. У нас ещё не придумали

т.е. стайлгайд на меня спихнули )))))

придумать

я знаю, что надо настраивать правила, но это я настрою через чекстайл. На время сгодится. Мне главное понять: в каких случаях final во входной параметр лучше не пихать ??

если ты кодишь бекенд на спринге

Вообще никогда

Это синтаксический мусор, от него толку 0

во всех

странно, что чекстайл это воспринмает как эррор

Потому что там стоят правила которые нравятся создателям плагина, а не те которые нужны вам

ясно. Ох и свинью они подложили. А в Java конвенциях, что в формате pdf я выложил про параметры мало чего есть

но про final ни слова. Поэтому поверю. Редкий случай когда final там вообще нужен

Ну почитай гугловый стайлгайд

Он неплохой, хотя там было кое что с чем я не согласен

Единственный смысл делать аргумент final в том, чтобы исключить случайное, неумышленное переопределение параметра в коде, которое было бы багом, а не задуманным поведением. В этом плане final помогает, т.к. прямо на этапе компиляции не даст этого сделать. Это его плюс.

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

Так что по совокупности PROS/CONS для себя я сделал вывод, что final в аргументах (как в общем-то и в локальных переменных) не нужен. Но это лишь личное мнение. У других людей оно может отличаться, и это нормально.

Главное, принять что-то одно и дальше везде делать одинаково в пределах одного проекта.