[Haskell-cafe] Soliciting feedback on my lens json parsing tutorial (please don't share yet!)

Geraldus heraldhoi at gmail.com
Sun Jan 22 21:06:46 UTC 2017


Also, after you've defined isHkOrLU function you use it only once (in the
same code block), however there are few use cases of "naked" (`elem` ["HK",
"LU"]) in the rest part of article.  Would it make sense to substitute them
with defined alias?

пн, 23 янв. 2017 г. в 0:37, Will Yager <will.yager at gmail.com>:

> Looks good! I would suggest putting near the beginning a note to the
> effect of "Note: This may look very verbose now, but I am going to explain
> how to make these statements idiomatic and compact further down. I'm just
> being explicit now." Otherwise, I would worry about people being scared off
> by the fact that the parenthesized, non-eta-reduced version is much longer
> and harder to read than the Python version.
>
> Maybe also compare the short version to the Python version, just to wrap
> up the tutorial?
>
> Cheers,
> Will
>
> On Jan 22, 2017, at 05:27, Cody Goodman <codygman.consulting at gmail.com>
> wrote:
>
> Hi All, I wrote a lens json parsing tutorial here:
> https://github.com/codygman/concise-json-parsing-in-haskell
>
> I'd appreciate any feedback. My approach tries to use a prior
> understanding of Python into understanding lens enough to do something
> useful. Perhaps I can include links to tutorials that cover more of the
> theoretical foundations as well.
>
> Any criticism and/or ideas is welcome!
>
> Thanks,
>
> Cody Goodman
>
> _______________________________________________
> Haskell-Cafe mailing list
> To (un)subscribe, modify options or view archives go to:
> http://mail.haskell.org/cgi-bin/mailman/listinfo/haskell-cafe
> Only members subscribed via the mailman list are allowed to post.
>
> _______________________________________________
> Haskell-Cafe mailing list
> To (un)subscribe, modify options or view archives go to:
> http://mail.haskell.org/cgi-bin/mailman/listinfo/haskell-cafe
> Only members subscribed via the mailman list are allowed to post.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.haskell.org/pipermail/haskell-cafe/attachments/20170122/d5e1f0b3/attachment.html>


More information about the Haskell-Cafe mailing list