[Haskell-fr] Demande de revue de code
Gautier DI FOLCO
gautier.difolco at gmail.com
Thu Apr 3 09:00:15 UTC 2014
Le 3 avril 2014 10:44, Sylvain Henry <hsyl20 at gmail.com> a écrit :
> Pour l'immutabilité, je montrerais les Lenses.
>
Bonne idée
> Eviter d'utiliser des listes partout, notamment dans DayStmt, WeekStmt et
> MonthStmt, vu que le nombre de champs est fixe.
>
Pas forcément, tu ne peux avoir que les relevés de 40 jours par exemple, ce
qui fais que tu as un mois et une semaine incomplète.
> Utiliser des lenses ici serait pas mal, surtout pour réécrire la fonction
> finale. Si je comprends bien elle fait une sélection et une réduction, donc
> parfait avec des lenses.
>
> Je ne suis pas convaincu par le Monoid, une simple fonction de réduction
> suffirait :
> toStats :: [Temperature] -> Statistics
> Là c'est compliqué inutilement je trouve.
>
Du coup tu perds la notion de groupes de relevés (jour, semaines, mois) et
tu te retrouve à gérer le "conflit" °C/°F en même temps que le calcul des
stats.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.haskell.org/pipermail/haskell-fr/attachments/20140403/ef33bde0/attachment.html>
More information about the Haskell-fr
mailing list