search

liam-m / flippd

A lightweight VLE for the "flipped classroom"
1 stars 0 forks source link

Refactor answer array #13

Closed liam-m closed 8 years ago

liam-m commented 9 years ago

In main.rb

Used throughout quiz.erb and quiz_complete.erb with array indexing which lowers the clarity of the code.

Also an example of primitive obsession (poor cohesion) but justifiable at the moment as it's not widely used and used only in tightly coupled areas of the code so will change for the same reasons anyway.

Plan is to use a symbol based dictionary with descriptive keys, to improve clarity.

liam-m commented 8 years ago

Still outstanding - @ReverateNM @Lixquid

liam-m commented 8 years ago

Could the template method be used?