自作CGIを評価するスレ (672レス)
上
下
前
次
1-
新
301
(6)
: 03/06/03 10:20 ID:???
AA×
>>291
外部リンク[zip]:isweb25.infoseek.co.jp
[240|
320
|
480
|
600
|
100%
|
JPG
|
べ
|
レス栞
|
レス消
]
301: [sage] 03/06/03 10:20 ID:??? >>291 とりあえずソース、見せてもらった。 極悪に汚い・・・。 全部、もう少し見やすいように手入れた。 とりあえず、手入れて思った事。 1、goto文なんか使うな! 2、スペース2個じゃなく、タブ使っとけ。 3、引数などのコンマの後は、スペース一つ入れる事。 4、なんでもかんでも、ケツにif文を置かない! 5、ソース汚いから、自分でも気付いてないんだろうが・・・ ファイルロックの掛け忘れがある。 6、同じく、ソース汚いから気付いてないんだろうが・・・ ちゃんと明示的にファイルハンドルを閉じてない所がある。 とくに、「4」のケツにif文は、うんざり。 処理内容自体は、読んでない。 人に評価してほしいなら、 自分だけが読めるソースを書くんじゃなく、 もう少し読みやすいように書く工夫をしよう。 ttp://isweb25.infoseek.co.jp/computer/pcqa-2ch/cgi-bin/img/1141.zip ↑ここに、手入れたソース置いてるから。 まだ、俺が手入れたソースの方が読みやすいと思う。 http://medaka.5ch.net/test/read.cgi/php/1049514428/301
とりあえずソース見せてもらった 極悪に汚い 全部もう少し見やすいように手入れた とりあえず手入れて思った事 1文なんか使うな! 2スペース2個じゃなくタブ使っとけ 3引数などのコンマの後はスペース一つ入れる事 4なんでもかんでもケツに文を置かない! 5ソース汚いから自分でも気付いてないんだろうが ファイルロックの掛け忘れがある 6同じくソース汚いから気付いてないんだろうが ちゃんと明示的にファイルハンドルを閉じてない所がある とくに4のケツに文はうんざり 処理内容自体は読んでない 人に評価してほしいなら 自分だけが読めるソースを書くんじゃなく もう少し読みやすいように書く工夫をしよう ここに手入れたソース置いてるから まだ俺が手入れたソースの方が読みやすいと思う
上
下
前
次
1-
新
書
関
写
板
覧
索
設
栞
歴
あと 371 レスあります
スレ情報
赤レス抽出
画像レス抽出
歴の未読スレ
AAサムネイル
ぬこの手
ぬこTOP
0.026s