Если не будет вечно поддержки и при этом трудно подменить реализацию, то на качество кода точно будут обращать внимание.
Однако этот класс даже скопипасть можно, если че-то помрёт.
Если программист колличество строк кода это партянка? а хрен там, это очень даже объектный код. Просто в нём заложено много логики, а без этой логики увы никак,и размещать её где то нужно. Поэтому контроллер длинный. Вот как называются переменные и функции это да, можно это считать конструктивной критикой