-
Notifications
You must be signed in to change notification settings - Fork 18
Strict拡張を使用する際の注意点 #168
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Strict拡張を使用する際の注意点 #168
Conversation
✍️ あとexampleでやるべきは
|
exampleはここまでで、そろそろ本文に取りかかろう |
📝 メモ: おまけで紹介したい: https://github.com/ndmitchell/spaceleak |
I mistook the reason why (error Error in MyTuple2) is called: it is because MyTuple is strict in its arguments, not because the pattern matching is strict #167 (comment)
Finding better example was hard...
「BangPattern をつけてはいけないところは, Strict 拡張でも lazy を明示的に指定しなくてはいけないので,ここら辺も NoStrict の時と注意点が同じと言う感じですね」の例としてfoldrの例を分ける
@mizunashi-mana @minoki 大変長らくお待たせしました。ようやくWIPを外したのでお手すきの際にレビューいただきたく! 🙏 |
Just to move the article to the top of the article list. I'm not sure when it'll really be published!
IMO: 好みの問題だと思いますが,一文一文が頻繁に改行されていると,一々目線を先頭に持っていかなければいけなくてとても読みづらいので,パラグラフは改行なしでまとまっている方が良いと思いました |
なるほど。個人的には改行するのを好んでやってたので、悩ましいですね... 😕 |
試しにすべての改行を消してみました。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
いくつかコメントしました.
なお,個人的には Strict
拡張は有効にすべき派で,理由としてはこれらのミスより Strict
を有効にしない場合に bang を書き忘れて起こるスペースリークの方がはるかに致命的な場合が多いからですね
それは別途気になるのでいつか記事にしていただけるとありがたいです! 🙏 |
@mizunashi-mana @minoki 一通り直せたはずです。 |
Update the date to publish Fix link to `StrictData` Set the link to the section of case 2 Add note about my point of view And several other update of expressions
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
幾つかコメントしました
@minoki @mizunashi-mana 直しました! ✅ |
私の見た感じでは、公開しても問題ないレベルに達したと思います。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
ありがとうございました!最終確認が終わったのでマージします! |
Fix: #167