Tuesday, April 24, 2012

1: Programming Review

Project description.
Concerns
  • line 21: I'm not sure if this is the best way to check the command line inputs and save them.
  • line 27: I feel like there should be a way to combine this with line 24? like or die in perl 
  • I can't believe the there aren't any Math.sum Math.average or Math.median functions but I do like the way you can easily insert methods into a core class ( line 1 - 11 )
  • line 31 - 36: I swear I've heard of an cleaner way to parse in a file, but my solution works 
  • line 40: I'm not really sure what ensure does
  • line 2: inject(:+) ??? what the crap? this is cool, but not intuitive at all. Someone please explain
  • I could make it a little more object oriented by making a print result method ... but meh
Let me know what you think. 

No comments:

Post a Comment