В контроллере более 1000 строк кода. Это правильно?
Размер файла ещё не говорит о том, что в нём что-то не правильно.Контроллер содержит бизнес логику. Проверку введенных данных и т.д.
А вот это уже не хорошо. Бизнес логику я бы убрал в модели, а проверки в фильтры и валидаторы.Может разбить контроллер на несколько частей?
Если экшены можно логически разделить на несколько независимых групп, то почему бы и нет. Ещё иногда удаётся объединить несколько экшенов в один.
Вам все верно ответили выше, но у меня появилось одно нехорошее предположение. Вы сказали, что это контроллер админки, а вы случайно не положили в него один методы по управлению всеми сущностями сайта? Если да, то это конечно тоже не смертельно, но желательно разделить. Ну и неплохо было бы взглянуть на этот контроллер, хотя бы на имена методов.