The Author Online Book Forums are Moving

The Author Online Book Forums will soon redirect to Manning's liveBook and liveVideo. All book forum content will migrate to liveBook's discussion forum and all video forum content will migrate to liveVideo. Log in to liveBook or liveVideo with your Manning credentials to join the discussion!

Thank you for your engagement in the AoF over the years! We look forward to offering you a more enhanced forum experience.

Martin Sojka (2) [Avatar] Offline
#1
Great book, Richard. Just finished it. Love the way you try to provide quick solutions first and then refactor them. Leads to those a-ha moments smilie

Just 3 comments/questions:

1. Seems like on p 148 you removed
, subscriptions = \_ -> statusChanges SetStatus

2. Wouldn't it be better to refactor "Hue", "Ripple" and "Noise" to Union type instead of strings?
3. Do you plan to add some info about breaking larger files/apps into separate modules and best practices?

Thanks.
rtfeldman (60) [Avatar] Offline
#2

1. Seems like on p 148 you removed
, subscriptions = \_ -> statusChanges SetStatus



The page numbers vary depending on which version you're looking at - could you give me chapter/listing number instead? smilie


2. Wouldn't it be better to refactor "Hue", "Ripple" and "Noise" to Union type instead of strings?


That wouldn't really get us anything in this case; they still need to be turned into strings anyway for how they're used. smilie


3. Do you plan to add some info about breaking larger files/apps into separate modules and best practices?


Yep! Particularly in Chapters 8 and 9.
Martin Sojka (2) [Avatar] Offline
#3
1. SENDING INITIALIZATION DATA TO PROGRAMWITHFLAGS