Refactor рубин код, основанный на админа присутствует или нет

Question

Я хочу, чтобы реорганизовать этот код:

def self.find_posts(page, per_page, author_id, user) 
    if user.admin? 
    paginate(page: page, per_page: per_page) #fetches on the basis of per_page 
    else 
    paginate(page: page, per_page: per_page).where(author: author_id) #Based on author_id 
    end 
end 

Post.paginate(page: page, per_page: per_page) повторяется два раза, так что я хотел, чтобы удалить его.

Вот что я пробовал:

def self.find_posts(page, per_page, author_id, user) 
    paginate(page: page, per_page: per_page) #fetches the no of per_page 
    if user.admin? 
    #not getting what should I write here 
    else 
    where(author: author_id) #Based on author_id 
    end 
end 

Этот код также не работает, который я знаю, что paginate не последнее предложение. Я мог бы использовать return, но почему-то я не понимаю, как это сделать.

Answer 1

Я не уверен, что его необходимо реорганизовать. Но только, чтобы дать вам варианты:

Вы можете сделать это (или до создания хэш)

paginate(page: page, per_page: per_page).where(user.admin? ? {} : {author: author_id}) 

или добавить объем по умолчанию на свой пост

Answer 2

Я бы написать:

def self.find_posts(page, per_page, author_id, user) 
    posts = user.admin? ? Post.all : Post.where(author: author_id) 
    posts.paginate(page: page, per_page: per_page) 
end