оптимизируют код рубин рельсы

Question

Я делаю это в моей функции редактирования, обновления, уничтожить

@post = Post.friendly.find(params[:id]) 
if @post.user_id == current_user.id 

там какой-нибудь способ оптимизировать и сделать одну функцию, вы использовали, чтобы это нравится:

before_action :set_user_post, only: [:edit, :update, :destroy] 

private 
def set_user_post 
    @post = current_user.posts.find_by(id: params[:id]) 
end 

Но добавьте драгоценный камень friendly_id и измените его следующим образом:

private 
def set_user_post 
    @post = current_user.posts.friendly.find(id: params[:id]) 
end 

Но это дает мне ошибку. ошибка:

Mysql2::Error: Unknown column 'id.id' in 'where clause': SELECT `posts`.* FROM `posts` WHERE `posts`.`user_id` = 1 AND `id`.`id` = 'primero' LIMIT 1 

Answer 1

Метод find находит запись по идентификатору, и вы можете передать только один идентификатор или массив идентификаторов в качестве входных данных. Но в вашем методе вы передаете хэш. Измените метод set_user_post, как

def set_user_post 
    begin 
    @post = current_user.posts.friendly.find(params[:id]) 
    rescue ActiveRecord::RecordNotFound 
    redirect_to root_path, alert: 'You are knocking at the wrong door' 
    end 
end 

Если пользователь пытается получить доступ к пост другого пользователя, метод find возбудит исключение, потому что мы находим посты в current_user. Мы можем спасти исключение и перенаправить пользователя на домашнюю страницу с предупреждающим сообщением .

EXTRA

Вы также можете найти запись, используя find_by метод и передать его хэш. Этот метод используется для поиска записи с любым полем (а не только с id). Он возвращает первую запись, которая соответствует запросу.

Например,

def set_user_post 
    @post = current_user.posts.friendly.find_by(id: params[:id]) 
end 

Это делает то же самое, но разница в том, что find поднимает ActiveRecord::RecordNotFound исключение, если запись не существует с заданным идентификатором, а find_by возвращает nil, если запись не существовать.

Для получения дополнительной информации см. http://apidock.com/rails/ActiveRecord/FinderMethods/find