自作CGIを評価するスレ (672レス)
1-
抽出解除 レス栞

1
(4): Premature end of script headers : /this/thread 03/04/05 12:47 ID:tIET0nhf(1) AAS
自分の作ったCGIをみんなに評価してもらうスレだよ。言語は問わないよ。
よほどプログラムが短くないかぎりはアプロダにでもアップしてね。
4
(8): 03/04/05 15:00 ID:??? AAS
PHPもOKなの?
17
(3): これ自信作だぜ 03/04/05 22:52 ID:qMi3a/lA(1/2) AAS
外部リンク:oasis.minidns.net
41
(3): 03/04/06 02:32 ID:??? AAS
かなり恥ずかしいのですが、他人の意見を一度でも聞いてみたくて…
外部リンク:kabocha.org
↑のサイトの
外部リンク[html]:kabocha.org
↑のページに
山口_BBS2.00b1
↑って言う私のCGIがあるのですが、いかがな物でしょうか。

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

ああ、ついに2ちゃんねるに晒してしまったよ…
冷静に考えると恥ずかしいけど…、プロの方(?)に見てもらいたい気持ちが上のようだ…。
省1
71
(31): 03/04/06 04:34 ID:??? AAS
>>70
余りいい設計じゃないなぁ・・・
subは位置に依存しないものであるべき
79
(7): 75 03/04/06 05:00 ID:??? AAS
まぁ意識しろって言われたら
漏れならこう書くんだけど

sub saitama{
 foreach my $value (@_){
  my ($saitama,$hoge) = split(/,/,$value);
  if($saitama == $hoge){ return 1; }
 }
 return 0;
}

78のコードと比べるとどっちのほうが
省3
84
(4): 03/04/06 05:10 ID:??? AAS
AA省
163
(3): ◆YHmsETNOS2 03/04/12 20:17 ID:??? AAS
再び。。。

すみませんがちょっと漏れのスクリプトを見てください。
前回のソートの練習に加え、ページの分割もしてみますた。
いちおう漏れの思っている動作しているのですが,突っ込むところがあれば,後学のために色々教えてください。

なお、前回指摘された “myとかの使い方” は。。。。。(^^A

ソースはこちら,
外部リンク[txt]:members.tripod.co.jp
使ったデータはこちら
外部リンク[txt]:members.tripod.co.jp

要 cgi-lib.pl です。
省1
168
(6): 03/04/12 23:21 ID:??? AAS
>>163
途中でダレたからかなり適当ですがいぢってみますた。
外部リンク[txt]:boobar.hp.infoseek.co.jp
211
(25): [3025] 03/04/24 04:27 ID:??? AAS
ども。ちょっとスレ違いかもしれないけど、お邪魔します。
CGIじゃないんだけど、Perlスクリプト内のコメントを削除するスクリプトを作りました。
使い方は、同封してるテキスト読んでも分かるけど、D&DでOKな、結構便利な感じです。

外部リンク[lzh]:isweb25.infoseek.co.jp

評価お願いします。
また、改造してくれてアップしてくれる人は、
それ見て勉強したいと思いますのでお願いします。
223
(18): 03/04/24 16:48 ID:??? AAS
外部リンク[zip]:isweb25.infoseek.co.jp
実は211のやつ見て無いんだが
書いて見たよ
先頭のタブとかは消さないようにした
一応自分の書いたスクリプトではきれいに
コメント部は消えた模様
265
(4): 03/05/25 22:00 ID:mSA7mvhe(1) AAS
自作新着情報cgiを作ったんだけど
評価して!
外部リンク[cgi]:fine.tok2.com
外部リンク[zip]:fine.tok2.com

携帯で見れるようにしたいんだけどどうしても文字化けしちゃうんだよね
279
(4): 03/06/01 18:08 ID:k+3UFbse(1) AAS
今度は大丈夫!!!!
評価して!
外部リンク[cgi]:fine.tok2.com
外部リンク[zip]:fine.tok2.com
外部リンク[cgi]:fine.tok2.com
288
(3): 03/06/02 18:26 ID:??? AAS
>>285
全然読んでないけど、インデントで空白とTABを使い分けられると
結構困る。
漏れはいつも4TABだから、8TAB & 4インデントだとずれる。
インデントを全部TABにしとくと、TAB幅に依存しなくていいよ。
291
(4): 03/06/02 20:36 ID:xwllfUbL(1) AAS
指摘バシバシお願いします。
外部リンク[cgi]:cgi.tripod.co.jp
(ソース)
外部リンク[txt]:members.tripod.co.jp
298
(3): 03/06/02 21:59 ID:??? AAS
>>291
感想。
ソース汚くて読む気しない。
終わり。
301
(6): 03/06/03 10:20 ID:??? AAS
>>291
とりあえずソース、見せてもらった。
極悪に汚い・・・。
全部、もう少し見やすいように手入れた。

とりあえず、手入れて思った事。
1、goto文なんか使うな!
2、スペース2個じゃなく、タブ使っとけ。
3、引数などのコンマの後は、スペース一つ入れる事。
4、なんでもかんでも、ケツにif文を置かない!
5、ソース汚いから、自分でも気付いてないんだろうが・・・
省11
316
(3): 298 03/06/04 14:07 ID:??? AAS
コメント無しでもいいような
わかりやすいクラス名、変数名、関数名をつけるのが基本です。
コメントが無いとわからないってのは
それだけ汚いソースだということです。
368
(6): 03/06/22 07:46 ID:??? AAS
スレの趣向とは少し外れるんですが、
下記のロック、アンロックはどうでしょうか?

# lock routine
sub lock {
 $retry = 5;

 if (-e $lockex_file) {
  $mtime = (stat($lockex_file))[9];

  if ($mtime + 600 < time) {
   rename($lockex_file, $lock_file) || &error("lock error");
  }
省15
371
(16): 03/06/24 13:54 ID:01cZwzPj(1/5) AAS
>>368
このルーチンだと穴がある。経験則だけど、アクセスが殺到すると簡単に壊れる。
説明するのめんどいので、

外部リンク[htm]:www.din.or.jp

この辺りでも読んでみて。

>>370
好みだと思う。

個人的には>>368>>369>>370もループの最中にreturnやら&errorで関数の
外に飛んでるので気持ち悪い(これも好みの問題)。
省8
1-
スレ情報 赤レス抽出 画像レス抽出 歴の未読スレ AAサムネイル

ぬこの手 ぬこTOP 1.223s*