One of my rails controllers is horribly overcrowded with a bunch of methods that link to static web pages.
Controller
def adventure
end
def cooking
end
def dancing
end
def programming
end
def reading
end
def running
end
def sports
end
def writing
end
For each of the above actions I have seperate view files and routes defined for them.
View Files
adventure.html.erb
cooking.html.erb
dancing.html.erb
programming.html.erb
reading.html.erb
running.html.erb
sports.html.erb
writing.html.erb
Routes
match '/adventure', to: 'pages#adventure', via: 'get'
match '/cooking', to: 'pages#fighting', via: 'get'
match '/dancing', to: 'pages#first_person_shooter', via: 'get'
match '/programming', to: 'pages#programming', via: 'get'
match '/reading', to: 'pages#reading', via: 'get'
match '/running', to: 'pages#running', via: 'get'
match '/sports', to: 'pages#sports', via: 'get'
match '/writing', to: 'pages#writing', via: 'get'
It is quite embarrassing to have code that horrible. Is there any way I can optimize that?
Pages
controller? The fact that a similarly named.erb
file exists? \$\endgroup\$