自作CGIを評価するスレ (672ﾚｽ)
上下前次1-新
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に対してそこまでしなくていいのに
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
上下前次1-新書関写板覧索設栞歴
あと 212 ﾚｽあります
ｽﾚ情報 赤ﾚｽ抽出 画像ﾚｽ抽出 歴の未読ｽﾚ AAｻﾑﾈｲﾙ
ぬこの手 ぬこTOP 0.010s