自作CGIを評価するスレ (672ﾚｽ)
上下前次1-新
301(6): 03/06/03 10:20 ID:??? AAS
>>291
とりあえずソース、見せてもらった。
極悪に汚い・・・。
全部、もう少し見やすいように手入れた。
とりあえず、手入れて思った事。
１、goto文なんか使うな！
２、スペース２個じゃなく、タブ使っとけ。
３、引数などのコンマの後は、スペース一つ入れる事。
４、なんでもかんでも、ケツにif文を置かない！
５、ソース汚いから、自分でも気付いてないんだろうが・・・
省11
302(1): 03/06/03 12:55 ID:??? AAS
>>301
＞２、スペース２個じゃなく、タブ使っとけ。
だけおかしい。
タブを使うと4TAB<->8TABの違いでずれるから空白（個数は問わないが普通4個）で。
303(1): 03/06/03 13:01 ID:??? AAS
何を言っとるんだ？ >>288を読め
304: 03/06/03 13:17 ID:??? AAS
>>303
「TABだけ」ならいいんだがな。空白と混じってるとおかしくなる。
いちいちTABだけなのを確認するより最初から空白だけの方がいいだろ。
TABなんて今時ディスクの節約にもならん。
どっちにしろ（例えば）SUNもMOZDEVもタブ不使用規約になってんだから
合わせた方がいいんでないかい？
305: 03/06/03 13:20 ID:??? AAS
間違えた。SUNは推奨してるだけだった。以下引用
＞インデントの単位として，4個分の半角スペースを推奨する．
＞そのインデントが実際に半角スペースによって行われるかタブによって行われるかについては，これを規定しない．
＞タブはすべて（4個ではなく）8個分の半角スペースでなければならない．
eclipseだったかな。曖昧記憶でｽﾏｿ
306(1): 298 03/06/03 14:58 ID:??? AAS
自分が思ったのは
「CGIスクリプトしか書いたことない人にありがちなソースだな。」
ってこと。
リファクタリングとか色々なことを勉強してみては？
307: 301 03/06/03 15:12 ID:??? AAS
>>302
タブは、インデント以外に使ってないから、
どんなタブ幅でもずれてないはずだけど・・・
ずれてる？？？
308: 291 03/06/03 19:31 ID:??? AAS
291です。評価ありがとうございます。
>>301さん
わざわざ直してくださって、本当にありがとうございます！
指摘されたのに気をつけて書き直してから、また評価をお願いしに来ます。
でも、もう…直せるところが無いかもしれませんが(汗)
>>306さん
そのとうりです。自分でCGIを作ってみたくなったので、perlを勉強しだしました。
色々なことを勉強するのもいいですけど、虻蜂取らずにはなりたくないので…。
309(2): 03/06/04 00:00 ID:??? AAS
>>285
ソース、今見てるけど・・・
えらくコメントの少ない・・・って言うか、コメントの無いソースだな・・・
わざと、コメント消したの？
それとも、素でこれ？
素でコメント無いなら、コメント入れていった方が良いと思う。
310(1): 03/06/04 10:29 ID:??? AAS
>>309
これ以上コメント書かれたら、読みづらくないですか？
と、素な意見。
311: 03/06/04 10:41 ID:??? AAS
>>310
う〜ん、本体（？）の方はコメントあるけど、
モジュールの方にコメントがないかな。
312: 385 ◆T98LkrO.aM 03/06/04 12:48 ID:??? AAS
>>309さん
基本的にコメントを一切書かないのです。
ただ、配布とか考えると書いた方がいいかなと思って、今回はちょっと書いてみました。
やっぱ少ないですかね..
313: 385 ◆T98LkrO.aM 03/06/04 12:52 ID:??? AAS
別にコメント書かないとかいう変なこだわりがあるわけじゃなくて、
ただ、自分はコメントなくてもすぐ分かるので人に見せる予定がないものに
コメント書かなくてもいいかなと..今は勉強中ということですし。
ただ、今回は見てもらおうとここに載せたわけなので、
モジュールの方にもコメントを書くべきだったと思いました..すみません。
314: 285 ◆T98LkrO.aM 03/06/04 12:54 ID:??? AAS
うお、385じゃなくて、285でした。
315(1): 03/06/04 13:20 ID:??? AAS
「未来の自分は他人です」という言葉を贈っておこう。
316(3): 298 03/06/04 14:07 ID:??? AAS
コメント無しでもいいような
わかりやすいクラス名、変数名、関数名をつけるのが基本です。
コメントが無いとわからないってのは
それだけ汚いソースだということです。
317: 285 ◆T98LkrO.aM 03/06/04 14:59 ID:??? AAS
>>315 >>316なるほど.
自分は今、就職希望の会社からCのソース(50-500行,10枚くらい)を渡されて、
この説明を書いてこいとか言われているのですが、コメントなくても
関数名, 変数名が分かりやすいし、うまい具合にサブルーチンに分けてあるので
コメント少ないですが、かなり読みやすいと思いました。
まあ、漏れのはコメントないと読みにくいかもしれませんが..
自分で見ても読みにくいな、と思ったらコメントでカバーするようにしてみます。
318(1): 301 03/06/04 19:23 ID:??? AAS
>>316
# ■■■ 俺の意見 ■■■
俺は、とにかくコメント入れるのに賛成派。
ルーチンの名前とか、変数の名前に意味を持たすのは当たり前だけど、
コメントあれば、コメントだけ読めば、何の処理してるのかわかるし。
# ■■■ ブロック毎にコメントを入れる ■■■
ブレスで囲って無くても、処理の単体が生まれる。
それ毎に、コメントを入れておけば、一行のコメント読めば数行理解できる。
# ■■■ ルーチンの使い方のコメントは重要！ ■■■
とくに、ルーチンの使い方のコメント。
省10
319: 301 03/06/04 19:40 ID:??? AAS
そうそう、掲示板の書き込みでも、
結構その人のセンスが出る。
変な所で改行してる人や、
長文なのに空白行入れず、
だらだらとメリハリの無い書き込みする人とか。
そういう人って、プログラムも汚いんだよね・・・
320: 285 ◆T98LkrO.aM 03/06/04 20:33 ID:??? AAS
はい、たしかに後で読む人がいるとすれば、
コメントは書いたほうがいいですね。
読む側と見るとありがたいものなので、自分でも書くようにします。
コメントが無いために間違った認識を受けたり、
重要な処理を 「これいらねぇんじゃね？」
とか思われて消されること減るはずだし。
上下前次1-新書関写板覧索設栞歴
あと 352 ﾚｽあります
ｽﾚ情報 赤ﾚｽ抽出 画像ﾚｽ抽出 歴の未読ｽﾚ AAｻﾑﾈｲﾙ
ぬこの手 ぬこTOP 0.017s