探検

自作CGIを評価するスレ

1Premature end of script headers : /this/thread
垢版 |
03/04/05 12:47ID:tIET0nhf
自分の作ったCGIをみんなに評価してもらうスレだよ。言語は問わないよ。
よほどプログラムが短くないかぎりはアプロダにでもアップしてね。
03/04/05 12:58ID:???
乙
03/04/05 14:40ID:???
ソース貼り用掲示板
http://do.sakura.ne.jp/~junkroom/cgi-bin/megabbs/lounge/index.html

あぷろだはどこ使うよ？
03/04/05 15:00ID:???
PHPもOKなの？
03/04/05 15:02ID:???
>>4
>>1 くらい読め
03/04/05 15:32ID:???
CGI版のPHPならOKてことだな>>4
Cで書くCGIスレも統合するか、ゴミだし。
03/04/05 15:51ID:???
WebProgならなんでもいいんじゃネーノ？
03/04/05 17:41ID:???
CGIならいいんだろ？
03/04/05 18:07ID:???
>>5

モジュール版のPHPについて聞いたことぐらい気づけ。
03/04/05 18:09ID:???
>>9
あーなるほど、スマン。
っつーかそれもアリに決まってるじゃねーかよーヽ(TДT)ﾉ
11マリモーマ ◆r6PDGT6Xd2
垢版 |
03/04/05 18:33ID:4Hc1kYlW
僕の作った　CGIは　どう？　http://777.or.nu/cgi.html
03/04/05 18:43ID:???
絵に描いたような一直線ソースだな
03/04/05 18:57ID:???
もうちょいスコープとか意識したほうがいいんじゃない？
殆どグローバル変数・・・
03/04/05 18:59ID:???
・どっかからコピーした部分以外で使われる変数が全てグローバル
・関数名から内容が想像できない(re,fot,ur,re2,wr2lok,co.gc等 sub ge {} は結構引いた）
・スタイルに一貫性がない
・$FORMの扱いがｷﾓｲ
・symlinkロックしてるが解除はどこでやってんだ？
03/04/05 19:05ID:???
>>14
5番目訂正しとく。
・symlinkロックしてないから解除はやらなくてもいいんだけどロックしないってことか。

sub lok{
if($lock == 0){return;}
local($wait) = 4;
while (!symlink(".", $lock)) {
if ($wait-- <= 0) {
&error("$(B%m%C%/%U%!%$%k$,B8:_$7$F$^$9!#(B");
}
sleep 1;
}
}


$ grep '$lock' ch/*
ch/sub.cgi: if($lock == 0){return;}
ch/sub.cgi: while (!symlink(".", $lock)) {

これじゃあ$lockは常に0だよ・・・
まあ$lockに値を入れたところで解除できんから1回しか書き込めんのだけどね。
16マリモーマ ◆r6PDGT6Xd2
垢版 |
03/04/05 21:18ID:4Hc1kYlW
なるほど　いろいろなアドバイス　サンクス
17これ自信作だぜ
垢版 |
03/04/05 22:52ID:qMi3a/lA
http://oasis.minidns.net/
18これ自信作だぜ
垢版 |
03/04/05 23:35ID:qMi3a/lA
tail -f /var/log/httpd/accesslog
見たけど誰もアクセスして来ないや。
皆臆病者だね。
03/04/05 23:45ID:???
お前がきもいから人気無いだけだろ。とんだ勘違い野郎だな。
204
垢版 |
03/04/06 00:23ID:t3fZoblF
よかったら批判ください。
もうちょいコード直してから、URL晒します。
03/04/06 00:26ID:???
元から批判が欲しいとは
224
垢版 |
03/04/06 00:28ID:t3fZoblF
批判以外もらってもどうしようもない..
ほめてもらったら何も意味がないっす。
03/04/06 00:31ID:???
>>17
いったけどエラー起こしとる罠
03/04/06 00:39ID:???
>>17
ソース見たらDOCTYPEがXHTMLだったけどぱっと見ても明らかに間違えまくってたのでチェックしてみた。
チェックの結果は以下のとおりです。

http://oasis.minidns.net/ を XHTML1.0 Transitional としてチェックしました。
19個のエラーがありました。このHTMLは 4点です。タグが 9種類 14組使われています。文字コードは Shift JIS のようです。

先頭の数字はエラーのおおまかな重要度を 0〜9 で示しています(減点数ではありません)。少ない数字は軽く、9 になるほど致命的です。0 は減点対象外のごく軽度のエラーで (グレイのかっこつき) でメッセージされています。

9: line 1: このXML宣言は正しくありません。 → 解説 23
6: line 5: <html> には xmlns 属性が必要です。 → 解説 86
2: line 5: <html> には lang 属性を指定するようにしましょう。 → 解説 116
2: line 5: <html> には xml:lang 属性を指定するようにしましょう。 → 解説 116
03/04/06 00:39ID:???
0: line 6: (<head>〜</head> 内に <link rev="made" href="mailto:〜" /> が含まれていません。) → 解説 121
0: line 6: (<head>〜</head> 内に <link rel="next" href="〜" /> などのナヴィゲーション用のリンクが含まれていません。) → 解説 122
1: line 6: <p> と </p> の間が空です。 → 解説 51
1: line 6: <body> での色指定が不完全です。text 属性も含めるようにしましょう。 → 解説 149
0: line 7: (<input type="text"> には tabindex 属性を指定するようにしましょう。) → 解説 166
0: line 7: (<input type="file"> には tabindex 属性を指定するようにしましょう。) → 解説 166
1: line 7: <input type="text"> には value 属性で初期値となるテキストを指定しておきましょう。 → 解説 160
0: line 7: (<input type="text"> には tabindex 属性を指定するようにしましょう。) → 解説 166
0: line 7: (<input type="submit"> には tabindex 属性を指定するようにしましょう。) → 解説 166
1: line 7: <input type="text"> には value 属性で初期値となるテキストを指定しておきましょう。 → 解説 160
0: line 7: (<input> には accesskey 属性を指定するようにしましょう。) → 解説 167
0: line 7: (<input> には accesskey 属性を指定するようにしましょう。) → 解説 167
0: line 7: (<input> には accesskey 属性を指定するようにしましょう。) → 解説 167
0: line 7: (<input> には accesskey 属性を指定するようにしましょう。) → 解説 167
9: line 7: </html> の後にまだ何かテキストがあります。 → 解説 231
264
垢版 |
03/04/06 00:40ID:t3fZoblF
なぜか、2重カキコデスカ!?と
274
垢版 |
03/04/06 00:41ID:t3fZoblF
HTMLがDOCTYPE無し、気分によって大文字小文字が混じっている
のはNGですか？
03/04/06 00:45ID:???
あ、ゴメン。エラーのせいで最後の方が減点食らってるのね。
本来のドキュメント（最初の</html>まで）でチェックしてみた。

XHTML1.0 Transitional としてチェックしました。
19個のエラーがありました。このHTMLは 8点です。タグが 9種類 14組使われています。

先頭の数字はエラーのおおまかな重要度を 0〜9 で示しています(減点数ではありません)。少ない数字は軽く、9 になるほど致命的です。0 は減点対象外のごく軽度のエラーで (グレイのかっこつき) でメッセージされています。

9: line 1: このXML宣言は正しくありません。 → 解説 23
6: line 5: <html> には xmlns 属性が必要です。 → 解説 86
2: line 5: <html> には lang 属性を指定するようにしましょう。 → 解説 116
（以下略）
03/04/06 00:47ID:???
>>27
別にいいんじゃねぇの？>>17は自信作らしいから突っ込まれてるだけで。
304
垢版 |
03/04/06 00:53ID:t3fZoblF
ほう、今自分で見ててすでに数箇所突っ込みどころがあったよ。
もう少し。
314
垢版 |
03/04/06 00:58ID:t3fZoblF
ついでにオレはここの4ではない。
03/04/06 00:59ID:???
評価して欲しいやつはトリップつけれや
03/04/06 01:04ID:???
>>31
ややこしいぞｗ
じゃあPHPじゃないのか？
344 ◆R.zOhOehsw
垢版 |
03/04/06 01:29ID:t3fZoblF
PERLですが、まずいですか？
03/04/06 01:34ID:???
いえ、さっさと貼っちゃってよん。
03/04/06 01:41ID:???
　　　______________
　／:＼.＿＿＿_＼
　|:￣＼(∩´∀｀)　＼　　＜先生！こんなのがありました！
　|:在　 |:￣￣ U￣:|
http://saitama.gasuki.com/aomori/
374 ◆R.zOhOehsw
垢版 |
03/04/06 01:56ID:t3fZoblF
BBS: http://hex51.darktech.org/cgi-bin/the/bbs.pl
SRC: http://hex51.darktech.org/the.txt

言語: PERL
ネタ元: http://thebbs.jp/

糞ほど批判いただければ幸いです。
書き込みまくっても糞スレたてまくっても結構。
03/04/06 02:04ID:???
>>37
Perlの文法的な間違いはないみたいだけど、処理がいまいちな部分が
散見される。
効率を考えればもうちょい工夫したほうがいいだろう。
394 ◆R.zOhOehsw
垢版 |
03/04/06 02:08ID:t3fZoblF
意見どうも。
もっとアルゴリズムを勉強しれってことですね？
03/04/06 02:22ID:???
・myが気になる。
・一行書き出すのにヒアドキュメント使うのは何故？
・かと思えばqq連発で使ってたり…

・１．入力 ２．処理 ３．出力　という風に処理を分けよう
・自作自演防止はハッシュとかじゃ駄目なのかい？
・FORMの処理が2回書いてあったりして無駄


・HEADリクエストは面白いね。
03/04/06 02:32ID:???
かなり恥ずかしいのですが、他人の意見を一度でも聞いてみたくて…
http://kabocha.org/jelfe/
↑のサイトの
http://kabocha.org/jelfe/cgi/ybbs.html
↑のページに
山口_BBS2.00b1
↑って言う私のCGIがあるのですが、いかがな物でしょうか。

ただし「携帯のEZ専用」なので皆様の中には意味を理解しかねる出力があるかもしれません。


ああ、ついに２ちゃんねるに晒してしまったよ…
冷静に考えると恥ずかしいけど…、プロの方（？）に見てもらいたい気持ちが上のようだ…。

変なところあったらけなしてくれてもけっこうです、むしろけなして下さい。
03/04/06 02:35ID:???
myもいいんだけど、実質グローバルと同じだな。
処理毎に｛　｝で囲んで使わなきゃ意味ないだろ。
434 ◆R.zOhOehsw
垢版 |
03/04/06 02:47ID:t3fZoblF
ご意見どうも。
>・一行書き出すのにヒアドキュメント使うのは何故？
エディタで折り返すのが気になるので..
>・かと思えばqq連発で使ってたり…
Emacsなので、ヒアドキュメントだと表示する特殊によって
自動のタブがおかしくなるのです..

全部個人的なことですね..修正します。

myに関しては、ちょっと調べてみます。
その他の意見に関しても、もう一度よく考え直してみます。
444 ◆R.zOhOehsw
垢版 |
03/04/06 02:49ID:t3fZoblF
>特殊
特殊文字、記号です。
03/04/06 02:50ID:???
>>43
cperl-mode使ってる？

myは変数のスコープを局所範囲に閉じ込めるためのもので、
mainパッケージでmy使ってもmainパッケージ全体から読み書きできてしまうので
いやーんな感じ
03/04/06 02:53ID:???
>>41
2193行目　$error_name = @_[0];　→　$error_name = $_[0];でいいだろ
あちこちの変数、もうちょっとスコープを考えてレキシカルに
するとかしたほうがいい。
問題ないけど、読み取りopenに　<　をつけたりつけなかったりなどしてるのを
書き方統一したら？
他の部分なんかでも統一させる書き方をくせにしたほうが可読性良くなるし。
吐いてるhdmlはすぐにテストできないから見ていない。
474 ◆R.zOhOehsw
垢版 |
03/04/06 03:00ID:t3fZoblF
>>45ｻﾝ
Emacsは、普通にそのまま.plとして読んだ感じで使ってます..
mainで、my使うと、別の関数から書きかえれないのでは？
それは意味ないのでしょうか？
新着レスの表示
レスを投稿する

ニューススポーツなんでも実況