briankeane/pl2 — pull request #1 by briankeane (2014-06-16 20:21:17+00:00)
Title: My Issues

Hey Y'all,

Thanks so much for checking this out for me -- I really really appreciate it. Please take your time -- I've got tons of areas to work in so I won't get held up at all.

Here's some of the things I sort of know intuitively could be improved but I don't know how to go about fixing:
1) station.rb feels big, but I'm not sure where I'd put any of the methods except for where they are
2) station.update_estimated_airtimes, make_log_current, and generate_playlist all step through the playlist in similar ways, but the way they modify the playlist is each so different I'm not sure how to refactor this code so it's not repeated. I'm guessing it needs 'passed-blocks' or 'yields' but how to execute one this complicated is a little over my head. No biggie though -- they all 3 work fine.
3) station.rb and my database talk to each other -- a lot. Especially in these:
station -- generate_playlist, update_estimated_airtimes, make_log_current
in_memory.db -- insert_spin

Anyways, I am obviously a beginner here (don't get me wrong... a pretty good beginner, but still a beginner) so if I'm missing some more glaring problems than these feel free to concentrate on whatever you think is important.

Anyways, thank you SO much! I owe you many beers.

Talk to you soon,
-Brian