Почему автор не проверяет наличие обьекта в базе перед его удалением?

Question

[therhino]

Много раз видел подобный код:

def destroy
		user = User.find params[:id]
		user.destroy!
		render nothing: true, status: 204
	end

1. Почему автор не проверяет наличие этого юзера в базе?
   
2. Что будет, если его нет? Сервер самостоятельно выкинет ошибку?
   
3. Как настроить приложение на работу с серверными ошибками? (500, 501, ...)
   
4. Как примерно было бы правильно написать данный код? Что вернуть пользователю? (это код для АПИ)
   

Answer 1

[caution]

потому что код упадет user = User.find params[:id] - на этой строке ещё и не дойдет до конца. это и есть своего рода проверка.

Comments

[therhino]

ОК, код упал. Как сообщить пользователю об ошибке?

[c10wn]

therhino: HTTP кодом?

[caution]

therhino: так, ок вылетит вот это исключение
ActiveRecord::RecordNotFound in UsersController#destroy
Couldn't find User with 'id'
и если у вас в приложении есть обработка исключений, то он увидит то, что вы хотите. В противном случае, он увидит то, что я написал выше. В случае продакшен мода это будет -
The page you were looking for doesn't exist.
В итоге мы имеем обычное 404.

Answer 2

[Семён Семёнов]

Так это ж просто примеры...

Answer 3

[Shaks]

1. Потому что, как сказал Семён Семёнов , - это просто примеры.
2. Произойдет исключение ActiveRecord::RecordNotFound
3. Вопрос не понятен, что значит настройке серверных ошибок?

Comments

[therhino]

обновил вопрос

Answer 4

[Viktor Vsk]

api.rubyonrails.org/classes/ActiveSupport/Rescuabl...