We improved the code for our dice rolling web app by adding support for all possible combinations of number of dice and type.
We refactored the app.rb code with 1 addition and deleting 42 delitions!
#.rb
get("/dice/:number_of_dice/:how_many_sides") do
  @num_dice = params.fetch("number_of_dice").to_i
  @sides = params.fetch("how_many_sides").to_i
  @rolls = []
  @num_dice.times do
    die = rand(1..@sides)
    @rolls.push(die)
  end
  erb(:flexible)
end
.erb
<h1><%= @num_dice %>d<%= @sides %></h1>
<ul>
  <% @rolls.each do |a_roll| %>
    <li>
      <%= a_roll %>
    </li>
  <% end %>
</ul>
    
Top comments (0)