土屋つかさのテクノロジーは今か無しか

fukken 2017/05/19 22:13 その場合、
var isFadeinCompleted = alpha >= 1.0f;
とするべきです（Fowler「説明用変数の導入」）。あるいはis...を関数としてもよい。
「alphaが1以上である」と「フェードインが完了している」は、同じ意味だとしても関心事の詳細度が異なります。
Fowlerが紹介したKent Beckによるテキストエディタのコード
function hilight( area ){
reverse( area );
}
が有名ですね。コンピュータにとっては完全に無意味なコードですが、「ハイライトする」ことと「領域を反転表示する」ことは別である（前者が意図であり、後者は実装詳細である。後者は呼び出し側から隠蔽されねばならない）という意図を持つコードです。
「コメントは不要である」という場合は、普通はこのような実装を行うことを前提としています。

fukken 2017/05/19 22:15 なお、「意図」を示すコードが時に必要になる、という点は否定しません。
「シンプルに書くと(a+b)/2に過ぎないけど、桁溢れ防止のためにややこしい書き方をしているよ」みたいなケースは多々あるからです。

fukken 2017/05/19 22:16 「意図」を示すコード　＞　「意図」を示すコメント、のミス。

t_tutiya 2017/05/19 22:25 >fukkenさん
コメントありがとうございます。実は下書きの段階では条件をメソッド化したパターンも書いていたのですが「自分ではここはメソッド化しないな」と思い削っていました。その理由はまだ言語化できないのですが、仮に

if(フェードインが完了している)

と、識別子をマルチバイトで書ける環境ではメソッド化するかもしれません（これはまた別の話ですが）。
ツイッター上でも同じ指摘を頂いたので、後日こちらについて補足記事を書こうと思います−。

スパム対策のためのダミーです。もし見えても何も入力しないでください