自作CGIを評価するスレ (672ﾚｽ)
上下前次1-新
434: ぼるじょあ ◆ySd1dMH5Gk [（＾＾）] 03/08/02 04:52 ID:??? AAS
AA省
435(1): 03/08/06 01:58 ID:u4hkzo1r(1/2) AAS
まだ評価していただけますか？
436: 03/08/06 02:05 ID:??? AAS
どうぞ
437(1): 435 03/08/06 02:10 ID:u4hkzo1r(2/2) AAS
それでは。
外部ﾘﾝｸ[zip]:www.42ch.net
昔作った掲示板です。今ではもうこんなにかけません。
438: 03/08/06 02:44 ID:??? AAS
>>437
ぱっとみですが、スコープや、名前空間を理解していらっしゃるのでよく勉強されてると思われます。
私の趣味的にはグロブを使わないようにするともっとよさげかと・・・
439(1): 03/08/06 03:48 ID:??? AAS
外部ﾘﾝｸ[zip]:www.42ch.net
良かったらお願いします。
使い方はcgi呼び出して?ってリンク見ると大体書いてあります。
典型的な我流でCGIしか書いたことの無い人のソースだとは思います。
非常識な部分があればご指摘頂きたいです。
#普段タブ使っているのですが、環境に依存しそうなので
#アップの際に、全てスペースに変換しました。
#個人的な情報も消してあります。ご理解ください。
440(1): 向こうの472=475 03/08/06 10:05 ID:??? AAS
>>439
コーディングスレの469さんか。(w
名前空間の扱いに振り回されてる感じがする。
基本的にパッケージ名や定数のハードコーディングは避けた方がいいし、
依存し合うものを別ファイルに分けると見通しが悪くなるだけだよ。
どうしても分けたいのなら各パーツはもっと役割分担を
はっきりさせて汎用性を上げ、OO なスタイルで組むが吉。
441(1): 03/08/06 10:38 ID:??? AAS
>>440
ここ過疎なので人すくなくてばれるだろうとは思ってました。。。
分けるのは、基本的に改造向けではじめました。
（HTML部分を分ける、とか。）
あと、機能追加のたびに追加したりとかって感じです。
最初はあまり深く考えないで分けていたんですが
最近えらい悩むようになってしまって
つきつめるとOOPにするしかないような気もしてきてしまします。
次になにか0から書くことがあったら、OOで書きたいとは思います。
これは今ある程度完成してしまってますし
省4
442(1): 441 03/08/06 10:58 ID:??? AAS
しかもコーディングスレであとで言われたとこ
直す前のうｐしてた ｳﾜｰﾝ
443: 03/08/12 15:36 ID:QPay4kE6(1) AAS
>>442
age
444: 山崎 渉 [（＾＾）] 03/08/15 22:37 ID:??? AAS
AA省
445: [ ] 03/08/30 22:39 ID:??? AAS
52 名前：nobodyさん ：03/08/30 20:46 ID:???
>>50
自作CGIを評価するスレ
2chｽﾚ:php
446: 03/08/31 01:55 ID:6GeG39LL(1/3) AAS
Blogの試作公開
外部ﾘﾝｸ[zip]:ex.1000gex.net
common.phpで初期設定。
排他制御も何も入れてないがとりあえず動いた。
これからトラックバックを盛り込むけど難儀するだろうなぁ
447: 03/08/31 02:19 ID:??? AAS
何？評価して欲しいの？
ただの宣伝？
448: 03/08/31 02:27 ID:6GeG39LL(2/3) AAS
ダメ出しきのん。
SAFE_MODEでさっそく引っかかったが
449: 03/08/31 02:54 ID:??? AAS
微妙な物作るな
450: ガイキチ厨房 [age] 03/08/31 06:44 ID:??? AAS
PHPでカウンタを作ってみたのですが、どなたかご意見下さい。
外部ﾘﾝｸ[txt]:popup3.tok2.com
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に対してそこまでしなくていいのに
上下前次1-新書関写板覧索設栞歴
あと 219 ﾚｽあります
ｽﾚ情報 赤ﾚｽ抽出 画像ﾚｽ抽出 歴の未読ｽﾚ AAｻﾑﾈｲﾙ
ぬこの手 ぬこTOP 0.031s