Слишком много проверок Before_Filter?

Question

Есть ли такие вещи, как «слишком много валидаторов»?

Я спрашиваю потому, что я в настоящее время используют Завещания и уже реализовали:

before_filter :authenticate_user! 

В моем Location контроллер, для представления шоу, я должен также подтвердить личность пытается просмотреть это владелец:

before_filter :authenticate_location_owner!, only: [:show] 

Теперь последнее, но не менее важное, в настоящее время моим пользователям не разрешено создавать «местоположения». Они созданы нами, администраторами на данный момент, а затем назначены.

Третий возможный before_filter валидатор:

before_filter :authorized_access?, except: [:show] 

Это понятие кластеризации мой контроллер и выглядит не совсем верно. Тем не менее, это то, что мне нужно для достижения и прекрасно работает.

Является ли это классифицированным как «хорошо» в программировании Rails?

ТИА

Answer 1

Вы используете свои before_filters в соответствующие контроллеры и соответствующим образом. Совершенно допустимо иметь несколько before_filters, и нет причин, по которым вам не нужны эти три фильтра.

Пока эти методы эффективны (например, используя Devise's current_user вместо того, чтобы каждый раз искать пользователя), я не вижу причин для улучшения вашего кода.