On Tue, 14 May 2013 11:22:27 +0200, Giacomo Tesio <giacomo@tesio.it> wrote:

Hi, I'm trying to improve a small haskell program of mine.

:

Some remarks:

0) Use hlint (available on Hackage) for improvement suggestions
1) You don't have to write the module heading in Main.hs, it is not a library (why export main?)
2) Change "print" to "putStrLn" if you want to display messages without quotes
2) switchArgs is only partially defined, add something like:
     switchArgs [x] = putStrLn $ "Unknown tool: " ++ x
3) Use shorter lines, for example change:

  importTrades outDir csvFile = transformFile csvFile (foldTradingSample.getTickWriteTrades)   (saveTradingSamples outDir)

to:

  importTrades outDir csvFile =
    transformFile
      csvFile
      (foldTradingSample.getTickWriteTrades)
      (saveTradingSamples outDir)

4) It is considered good practice, to write the function
   composition operator between spaces (change f.g to f . g)

I have analyze your software further to see how sufficient laziness can be reached.

Regards,
Henk-Jan van Tuyl


--
Folding@home
What if you could share your unused computer power to help find a cure? In just 5 minutes you can join the world's biggest networked computer and get us closer sooner. Watch the video.
http://folding.stanford.edu/


http://Van.Tuyl.eu/
http://members.chello.nl/hjgtuyl/tourdemonad.html
Haskell programming
--

_______________________________________________
Beginners mailing list
Beginners@haskell.org
http://www.haskell.org/mailman/listinfo/beginners