ソース読んで分からないことだけ書いてほしいよね。for ( i = 0; i < 3; i ++ ) // 3回繰り返すとかは要らないけど、引数に何が渡ってくるのかとかは書いてほしい。
例えば// 状態の有無で変更$status = $_POST['status'];if ($status == 1) {} else {}$email = $_POST['email'];ていう並びのソースがあったとして、このままでは$emailの部分まで含まれるように感じるから、// メールアドレス$email = $_POST['email'];というように分かりきったコメントを入れることはある。
Re:3 ん、どゆこと?
Re:2 引数に何が渡って来てるかが分かる変数名を付けてくれれば、それで充分。
コメントあろうがなかろうが、3年前に自分が書いたコードでも解読に手間取るw
//わかった
Re:6 3年前はさすがにもう。。。1週間でも他人が書いたコードになるからなー。
Re:6 3年前と言わず、昨日の俺とは既に赤の他人との戦い。マジで死活問題。ギットハブのイシュートラッカーが無いと詰んでる。「誰がやった?お前の責任、いやそれ君のことでしょ、君が変更しろと行ったじゃないの?」でよくもめるから、イシュートラッカー引用してとりあえず黙らせて責任の所在をはっきりさせとくとか、よくあるよ。少しの変更でもとにかくイシュー発行させて紐付けしとく。
Re:3 たぶん、最初のコメントの範囲がどこまでを指してるかわかりにくいから、区切りを入れるために意味のないコメントを入れることがあるって意味だろうけど、例が悪すぎるw- 区切りを入れたいだけなら、意味のないコメントはいらず、//を挟めば事足りる。- 最初のコメも、関数化して何をしてるかわかる関数名にすればコメントそものもが不要。- 趣旨からずれるが、$_POSTをそのまま代入するようなことはありえない。
Re:9 職場ギスギスしてるんだろうな〜
Re:10 $_POSTをそのまま代入するのは有り得ないってのは、なぜなのですか?
Re:12 クロスサイトスクリプティングでしょ。てか例にいちいちセキュリティやら効率やら言い出したら、例になんないと思うのですが。さすがにそこは「空気読んでください」で良いと思うのに、とかくID:NrynCd みたいに1から10まで指摘したがり屋がいるので、コメントも細かくなるのです。神経質な人のせいです。
Re:13 クロスサイトスクリプティングの対策って出力時だよね?
Re:10 最初のコメントの範囲がどこまでか分からなくなるから、途中に別のコメント挟むってことだけど、コメントがネストすることもあり得るのでは?てか、コメント多い時点でクソコード感ぱねーけど
Re:13 例が悪いっていう趣旨なんだから空気読めよ
Re:16 でも自分は例を出して説明できないんだろ?ただ批判するだけだもんね。
Re:13 空気読めと自分で言いつつ、空気読めの意味わからんのか。本題からずれてるから、そこひっぱってどうするんだって話だろ。「趣旨からずれる」ってあらかじめ断ってるんだから、それ以上引っ張んなよ。指摘したお前が一番どーでもいいこと引っ張って趣旨ずらしてるんだから世話ね~わ
17 件の回答
ソース読んで分からないことだけ書いてほしいよね。
for ( i = 0; i < 3; i ++ ) // 3回繰り返す
とかは要らないけど、引数に何が渡ってくるのかとかは書いてほしい。
例えば
// 状態の有無で変更
$status = $_POST['status'];
if ($status == 1) {
} else {
}
$email = $_POST['email'];
ていう並びのソースがあったとして、このままでは$emailの部分まで含まれるように感じるから、
// メールアドレス
$email = $_POST['email'];
というように分かりきったコメントを入れることはある。
Re:3
ん、どゆこと?
Re:2 引数に何が渡って来てるかが分かる変数名を付けてくれれば、それで充分。
コメントあろうがなかろうが、3年前に自分が書いたコードでも解読に手間取るw
//わかった
Re:6
3年前はさすがにもう。。。
1週間でも他人が書いたコードになるからなー。
Re:6
3年前と言わず、昨日の俺とは既に赤の他人との戦い。マジで死活問題。
ギットハブのイシュートラッカーが無いと詰んでる。「誰がやった?お前の責任、いやそれ君のことでしょ、君が変更しろと行ったじゃないの?」でよくもめるから、イシュートラッカー引用してとりあえず黙らせて責任の所在をはっきりさせとくとか、よくあるよ。少しの変更でもとにかくイシュー発行させて紐付けしとく。
Re:3
たぶん、最初のコメントの範囲がどこまでを指してるかわかりにくいから、区切りを入れるために意味のないコメントを入れることがあるって意味だろうけど、例が悪すぎるw
- 区切りを入れたいだけなら、意味のないコメントはいらず、//を挟めば事足りる。
- 最初のコメも、関数化して何をしてるかわかる関数名にすればコメントそものもが不要。
- 趣旨からずれるが、$_POSTをそのまま代入するようなことはありえない。
Re:9
職場ギスギスしてるんだろうな〜
Re:10 $_POSTをそのまま代入するのは有り得ないってのは、なぜなのですか?
Re:12
クロスサイトスクリプティングでしょ。
てか例にいちいちセキュリティやら効率やら言い出したら、例になんないと思うのですが。
さすがにそこは「空気読んでください」で良いと思うのに、とかくID:NrynCd みたいに
1から10まで指摘したがり屋がいるので、コメントも細かくなるのです。神経質な人のせいです。
Re:13
クロスサイトスクリプティングの対策って出力時だよね?
Re:10
最初のコメントの範囲がどこまでか分からなくなるから、途中に別のコメント挟むってことだけど、コメントがネストすることもあり得るのでは?
てか、コメント多い時点でクソコード感ぱねーけど
Re:13
例が悪いっていう趣旨なんだから空気読めよ
Re:16
でも自分は例を出して説明できないんだろ?ただ批判するだけだもんね。
Re:13
空気読めと自分で言いつつ、空気読めの意味わからんのか。
本題からずれてるから、そこひっぱってどうするんだって話だろ。
「趣旨からずれる」ってあらかじめ断ってるんだから、それ以上引っ張んなよ。
指摘したお前が一番どーでもいいこと引っ張って趣旨ずらしてるんだから世話ね~わ