first project. code reviews?

I've written a parser for OFX 1.x (these are the files you can download from most banks detailing recent account activity). I'm not sure I've written it well. This is my first serious thing in Haskell. (Kudos to the writers of the guide for how to make a new Hackage project! It's brilliant.) http://patch-tag.com/r/ofx1/pullrepo

On Fri, Jul 10, 2009 at 2:08 PM, Jared Jennings
I've written a parser for OFX 1.x (these are the files you can download from most banks detailing recent account activity).
I'm not sure I've written it well. This is my first serious thing in Haskell.
At a quick glance I can give some stylistic feedback: * I like to have at least 1 blank line between functions * Add type signatures for top level definitions, this helps later on * More haddocks next to function definitions :) I didn't read your code deeply enough to comment on the way you used things, named stuff, built abstractions, etc. Jason
participants (2)
-
Jared Jennings
-
Jason Dagit