Является ли это лучшей практикой для маршрутизации в Rails?

Question

Я довольно много любитель, когда дело доходит до Rails. Я получил это, чтобы работать, однако мне кажется, что код недостаточно эффективен.

Есть ли способ ускорить это? А также это, как профессионал мог бы это сделать?

Контроллер

def mark_read 
    @topic = Topic.find(params[:id]) 
    @topic.mark_as_read! :for => current_user 
    redirect_to user_path(current_user.slug) 
end 

def mark_all_read 
    Topic.mark_as_read! :all, :for => current_user 
    redirect_to user_path(current_user.slug) 
end 

Маршруты

resources :users do 
    member do 
    post :mark_read 
    post :mark_all_read 
    end 
end 

Посмотреть

<% if current_user.id == @user.id %> 
<%= link_to "Mark all as read", mark_all_read_user_path, :method=> :post %> 

<h4> List of posts unread by you </h4> 
<% @unread.each do |topic| %> 
<% if @user.following?(Product.find(topic.product_id)) %> 
<li> <%= topic.title %> <%= link_to "Mark as read", mark_read_user_path(topic), :method=> :post %> </li> 
<% end %> 
<% end %> 

Есть ли каким-то образом, что я мог бы вызвать в контроллере без маршрута? Я чувствую, что это упростит рабочий процесс.

Answer 1

Ответ на ваш вопрос: нет. Вы не можете иметь доступ к любому в контроллере, если он не имеет маршрут, или сво CRUD генерироваться по умолчанию, например:

resources :articles 

генерирует

Controller#Action 
    articles GET /articles(.:format)   articles#index 
      POST /articles(.:format)   articles#create 
new_article GET /articles/new(.:format)  articles#new 
edit_article GET /articles/:id/edit(.:format) articles#edit 
    article GET /articles/:id(.:format)  articles#show 
      PATCH /articles/:id(.:format)  articles#update 
      PUT /articles/:id(.:format)  articles#update 
      DELETE /articles/:id(.:format)  articles#destroy