アカウント名:
パスワード:
修正前のコードにマジックナンバー的な値がずっと存在していたのが少し残念だね。
screen_priv->fake_interval = 16667;
コメントに書いてある通りで60Hz のインターバル(=interval)，つまり 1/60秒 = 16667 μ秒のことですよ
いや、1s/60 の結果なのは分かった上で書いたのだけど。コメントには
Otherwise, pretend that the screen runs at 60Hz
と書いてあるけど、16667 については明確には書いてない。だから「マジックナンバー的な値」と書いた。あんだーすたん？
それは「明確に書いている」と受け取るなぁ。俺は
あなたには前提知識がありマジックナンバーではないのかもしれないが、一般的にはマジックナンバーでしかない、という視点はコード開発ではとても重要ですよ。
プログラミングを「製造」と言って三項演算子を一律禁止するコーディング規則を作りそうなやつだな。1から10まで説明してたらコードの9割がコメントになるわ。
いちいちコメント書かなくても良いようにマクロや定数使えって話だろ？
1から10までの説明は要らないけど、コードから意図が伝わるように書かれていなかったことが問題でしょ。コンパイルする際に最適化されることを期待して、こんな風に書いていればコメント要らないし。
悪い例: screen_priv->fake_interval = 16667良い例: screen_priv->fake_interval = 1 * 1000 * 1000 / 60 // 60Hz [usec]
実務でプログラミングに関わってたら常識のハズなんですが､､やっぱりオープンソースとか「趣味で」やってるとこうなっちゃうんでしょうねぇ
少なくともマジックナンバーを使わずに下みたいにヘッダでdefineしろってのは仕事なら最初に言われるはず#define NUM_1 1#define NUM_2 2...#define NUM_10000 10000
＃ define NUM_100 10って書いてるの見つけて二度見した経験があるのでAC
#define NUM_10000までしかなかったら16667を表すのに足りないじゃん! やっぱりスーツは馬鹿だな
マクロや定数にしたからって16667が何を意味するのか説明するコメントが要らなくなるわけじゃないでしょ。そして一箇所でしか使われてないならマクロや定数にしたところで位置を移動するだけで大した意味はない。
あれ、おかしいな。#4086698にコメントしたつもりでした。どうしてここにぶら下がってるんだろう
むしろ、NUM_1000とかは酷いローカルルールなんだから読みづらいことこの上ない。直観的には全く分からないんだから、そんなコードを他人と共有するのはやめてくれ。
これは最悪のローカルルール。うん、某大手系列でやってる所あるよね。痴呆の極み。
ネタとマジが混ざりすぎて…
コメントの代わりにマクロ名や定数名使えってことだよ。そして一箇所でしか使われていなかったとしても、他で同じ数値を使いたくなったときのためにマクロや定数にすべき。16666なのか16667なのかで判断に迷うこともなくなる。
> それは「明確に書いている」と受け取るなぁ。俺は
オープンソースで「俺はわかる」と言われても。他人が読む前提が無いのかなぁ困った人だ。
「Otherwise, pretend that the screen runs at 60Hz」のコメント付きで16667と書かれてりゃわかる人は十分いるだろうし、それじゃわからない人がいるというならそう思った人が修正すりゃいいじゃん。オープンソースなんだし
スラドの猿たちの90%以上はマウント合戦に夢中で「修正前のコードに」と書かれていることに気づかない
それでわからない人はコードを弄らないで欲しいわ。
より多くのコメントがこの議論にあるかもしれませんが、JavaScriptが有効ではない環境を使用している場合、クラシックなコメントシステム(D1)に設定を変更する必要があります。
海軍に入るくらいなら海賊になった方がいい -- Steven Paul Jobs
指定可能か否かは置いといて (スコア:0)
修正前のコードにマジックナンバー的な値がずっと存在していたのが少し残念だね。
Re: (スコア:2)
コメントに書いてある通りで
60Hz のインターバル(=interval)，つまり 1/60秒 = 16667 μ秒のことですよ
Re: (スコア:0)
いや、1s/60 の結果なのは分かった上で書いたのだけど。コメントには
と書いてあるけど、16667 については明確には書いてない。
だから「マジックナンバー的な値」と書いた。あんだーすたん？
Re:指定可能か否かは置いといて (スコア:0)
それは「明確に書いている」と受け取るなぁ。俺は
Re: (スコア:0)
あなたには前提知識がありマジックナンバーではないのかもしれないが、一般的にはマジックナンバーでしかない、という視点はコード開発ではとても重要ですよ。
Re: (スコア:0)
プログラミングを「製造」と言って三項演算子を一律禁止するコーディング規則を作りそうなやつだな。
1から10まで説明してたらコードの9割がコメントになるわ。
Re: (スコア:0)
いちいちコメント書かなくても良いようにマクロや定数使えって話だろ？
Re: (スコア:0)
1から10までの説明は要らないけど、コードから意図が伝わるように書かれていなかったことが問題でしょ。
コンパイルする際に最適化されることを期待して、こんな風に書いていればコメント要らないし。
Re: (スコア:0)
実務でプログラミングに関わってたら常識のハズなんですが､､
やっぱりオープンソースとか「趣味で」やってるとこうなっちゃうんでしょうねぇ
少なくともマジックナンバーを使わずに下みたいにヘッダでdefineしろってのは仕事なら最初に言われるはず
#define NUM_1 1
#define NUM_2 2
...
#define NUM_10000 10000
＃ define NUM_100 10って書いてるの見つけて二度見した経験があるのでAC
Re: (スコア:0)
#define NUM_10000までしかなかったら16667を表すのに足りないじゃん! やっぱりスーツは馬鹿だな
Re: (スコア:0)
マクロや定数にしたからって16667が何を意味するのか説明するコメントが要らなくなるわけじゃないでしょ。
そして一箇所でしか使われてないならマクロや定数にしたところで位置を移動するだけで大した意味はない。
Re: (スコア:0)
あれ、おかしいな。#4086698にコメントしたつもりでした。
どうしてここにぶら下がってるんだろう
Re:指定可能か否かは置いといて (スコア:1)
むしろ、NUM_1000とかは酷いローカルルールなんだから読みづらいことこの上ない。直観的には全く分からないんだから、そんなコードを他人と共有するのはやめてくれ。
Re: (スコア:0)
これは最悪のローカルルール。うん、某大手系列でやってる所あるよね。
痴呆の極み。
Re: (スコア:0)
ネタとマジが混ざりすぎて…
Re: (スコア:0)
コメントの代わりにマクロ名や定数名使えってことだよ。
そして一箇所でしか使われていなかったとしても、
他で同じ数値を使いたくなったときのためにマクロや定数にすべき。
16666なのか16667なのかで判断に迷うこともなくなる。
Re: (スコア:0)
> それは「明確に書いている」と受け取るなぁ。俺は
オープンソースで「俺はわかる」と言われても。他人が読む前提が無いのかなぁ困った人だ。
Re: (スコア:0)
「Otherwise, pretend that the screen runs at 60Hz」のコメント付きで16667と書かれてりゃわかる人は十分いるだろうし、それじゃわからない人がいるというならそう思った人が修正すりゃいいじゃん。オープンソースなんだし
Re: (スコア:0)
スラドの猿たちの90%以上はマウント合戦に夢中で「修正前のコードに」と書かれていることに気づかない
Re: (スコア:0)
それでわからない人はコードを弄らないで欲しいわ。