-
Notifications
You must be signed in to change notification settings - Fork 0
Matchup Algorithm has flaws #2
Copy link
Copy link
Open
Description
AkeelAli
opened on May 5, 2011
Issue body actions
- the first iteration places repeating quotes (this is because the first iteration has only 1 left circular shift, and hence, the right quote appears in the next matchup)
- Storing all the quotes in the session is not a good idea (possiblity of cookie overflow). It may be possible to have a scheme of indexes to follow, but such an approach would present a challenge (how will we update the previous matchup's attributes if we don't store the order? maybe we can pass previous indexes that battled and which one?)
- If you're going to keep using session, make sure to add a timer (so that it doesn't last too long, and end up overflowing as the user stores more and more stuff in the session)
- If you want to randomize, http://stackoverflow.com/questions/2752231/random-record-in-activerecord
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels