自作CGIを評価するスレ

自作CGIを評価するスレ (672ﾚｽ)
上下前次 1-新

451: 03/08/31 11:38 ID:??? AAS
こちら素人だが、ざっと読んで気づいた点を。
script書き始めなら上々の滑り出しだと思う。

■L38の $ip のダブルクウォートは意図不明。

■” と ’ が混在してるけど、このスクリプトで使われてる ” は上記をのぞいて ’ に統一可能。

■L41は、$newdata = $newcont . '<>' . $_SERVER['REMOTE_ADDR'] ; と書けば十分なのでは。

■関数名とレフトブラケットの間に半角スペースがあったり無かったりする。これは趣味だから
どう書いてもいいと思うが、普通は入れない。制御語（ifとかforとか）の後には入れる。そうすることで
関数と制御構造を見分けやすくする。カラーリング機能のないエディタではこの方が見やすい気もする。
いずれにせよ、どっちかに統一したほうがよいと思われ。
省2

452(1): 03/08/31 11:40 ID:??? AAS
■L29の rewind(); は意図不明。
ログファイルが壊れる（例えば 8424<>127.0.0.1420<>127.0.0.8420<>127.0.0.842のように）から
入れたのなら、アイディアとしては◎だが、スマートな処理とはいえないかと。

■あと、排他制御にバグがあるのでは。
ロックがかかっていても、fopen(); も flock(); も、 warning や fatal error （noticeも）を吐かないため、
処理が続行する。排他制御になってもならなくてもこのスクリプトは処理を最後まで実行するので、flock(); は
ちゃんと働いてない。
実際には起こらないかも知れないが、論理上、ロックがかかっているために L32 の fgets(); が失敗して
$buf に false が入ってカウンターが0に戻る可能性がある。L26でロックが取れなかった場合、
つまり前のリクエストを処理中の可能性がある場合、書き込みをスキップするかスクリプトを終了
省5

453: 03/08/31 21:50 ID:??? AAS
本7に対してそこまでしなくていいのに

454: 03/08/31 21:56 ID:6GeG39LL(3/3) AAS
ちなみにサンプルページ

外部ﾘﾝｸ:s2.arigato3.net

ハイブリッドP2Pによるトラックバックを考えてます

455: 03/09/01 00:56 ID:??? AAS
まずsageろ
話はそれからだ

456: 03/09/01 01:41 ID:??? AAS
相変わらず発言が電波過ぎる…

457(1): ガイキチ厨房 03/09/01 16:10 ID:??? AAS
>>452

詳しく指摘していただきありがとうございます。
とりあえず''に統一し、関数名とレフトブラケットの間のスペースを
制御構造のみ入れるようにしました。

また、書き込み処理を行った後に、

　// ファイルを切り詰める
　ftruncate($fp, ftell($fp));

という処理を加えました。
省4

458: ガイキチ厨房 03/09/01 18:57 ID:??? AAS
すいませんURL間違えました。正しくはこちらです。

外部ﾘﾝｸ[php]:itbtech.itboost.co.jp

459(1): 03/09/01 23:17 ID:??? AAS
Perlでも評価してもらえます？

460: 03/09/02 01:39 ID:??? AAS
>>459
>>1

461: 03/09/02 16:15 ID:??? AAS
>457
ああごめん。Manualみたらおもっくそそう書いてあった、、。
flockでとまるから一般の環境なら多分大丈夫だね。
こっちの検証スクリプトの間違いなのでflockとrewindの話しはなかったことに。
秒間50リクエストくらいでは問題なかったです（php4.3.0 apache1.3.27 RHLinux）。
ftruncate ( $fp , ftell ( $fp ) ) ; は ( $fp , strlen ( $newdata ) ) ; なんちゅう書き方もある。

462: ガイキチ厨房 03/09/02 19:51 ID:??? AAS
なるほど、参考になりました。ありがとうございます。

463(2): 03/09/03 02:03 ID:6rADPhxn(1) AAS
とある会社の部活動のHPを管理している者です。

Web上で部の共有資産の貸し出し申し込み等の管理を行えるといいなぁ、と思いいろいろとCGIを探してはいるのですがなかなか見つかりません。
探しているのは、次のようなCGIです。

１．画面で借りたい物を選択・自分の名前・貸出期間等の入力、申し込み
２．画面上では「貸し出し中」等に表記が変化（それか削除されるなど、申し込みが行えない状態に変化）
３．sendmailが使えないサーバなので、申し込みがあったことをログに吐き出し、管理人が定期的にそれをチェックし貸し出す

こういうものです。
サーバはinfoseekのiswebでPerl、Python、Ruby、PHPが使えます。

ショッピングカートCGIを活用してもいいなと思ったのですが、sendmailを利用する物がほとんどで、上記のようにログにのみはき出す物はあったことはあったのですがシェアウェア（\15K！）でした。
それほど資産の数が多いわけでもなく、貸し出し回数もそんなに多くないので、15000円もかけるわけにはいきません。
省2

464(2): 03/09/03 04:27 ID:??? AAS
>>463
>>1

465(1): sage 03/09/03 12:28 ID:Udj8Y4pI(1) AAS
>464

463です

え〜と。
スレ違いだ、ということですか…？

…すいませんでした。

466(1): 03/09/03 13:26 ID:??? AAS
sageはmail欄でｗ。
どっかに「こんなCGI探してます」的なスレあったハズだよ。

467(1): 03/09/03 13:44 ID:??? AAS
>>465
＞スレ違いだ、ということですか…？
スレ違いじゃないと思うんだったら自作CGIへのリンク貼れよ。
自作CGIを評価するスレなんだからさ。

468(1): 03/09/03 18:02 ID:??? AAS
【CGI】こんなCGI探してますver.10【素材】
2chｽﾚ:hp
Web製作板でつ。

469: 463 03/09/04 00:28 ID:??? AAS
>>464
>>467
ご迷惑をおかけしました。

>>466
>>468
そこに行ってみます。ありがとうございます

470: [age] 03/10/02 12:00 ID:??? AAS
y

上下前次 1-新書関写板覧索設栞歴

あと 202 ﾚｽあります
ｽﾚ情報赤ﾚｽ抽出画像ﾚｽ抽出歴の未読ｽﾚ AAｻﾑﾈｲﾙ

ぬこの手ぬこTOP 0.010s