Eine bessere Möglichkeit, Routen einzurichten? Die Schienen

Question

Gibt es einen besseren Weg, um Setup diese beiden Routen unter:

post '/song:id/favorite', to: 'songs#favorite', as: :favorite 
    delete '/song:id/favorite', to: 'favorites#destroy', as: :destroy 

Hier sind alle meine Routen:

Rails.application.routes.draw do 
    root "home#index" 
    get '/login', to: 'sessions#new' 
    post '/login', to: 'sessions#create' 
    delete '/logout', to: 'sessions#destroy' 

    post '/song:id/favorite', to: 'songs#favorite', as: :favorite 
    delete '/song:id/favorite', to: 'favorites#destroy', as: :destroy 

    namespace :admin do 
    resource :dashboard, only: [:show] 
    end 

    resources :users, only: [:new, :create] 
    resources :playlists 
    resources :genres 
    resources :albums 
    resources :artists 
    resources :playlist_songs 
    resources :favorites 
    resources :songs 

end 

Answer 1

etwas Versuchen Sie, wie dieses

resources :songs do 
    member do 
    post 'favorite', to: 'songs#favorite', as: :favorite 
    delete 'favorite', to: 'favorites#destroy', as: :destroy 
    end 
end 

Answer 2

Diese Aktionen don gehören nicht wirklich zum Song Controller. Wenn Sie darüber nachdenken, wird für einen Song keine CRUD-Aktion ausgeführt (Sie erstellen, bearbeiten oder löschen keinen Song).

Ich würde ein Konzept namens Nested Resources (http://blog.8thcolor.com/en/2011/08/nested-resources-with-independent-views-in-ruby-on-rails/ - das ist ein tolles Intro) verwenden.

Grundsätzlich würde ich meine Routen strukturieren:

resources :songs do 
    resources :favourites, only: [:create, destroy] 
end