自作CGIを評価するスレat PHP自作CGIを評価するスレ - 暇つぶし2ch■コピペモード□スレを通常表示□オプションモード□このｽﾚｯﾄﾞのURL■項目テキスト450:ガイキチ厨房 03/08/31 06:44 PHPでカウンタを作ってみたのですが、どなたかご意見下さい。 http://popup3.tok2.com/home/program/cgi-bin/uploader/source/File:0011.txt 451:nobodyさん 03/08/31 11:38 こちら素人だが、ざっと読んで気づいた点を。 script書き始めなら上々の滑り出しだと思う。 ■L38の $ip のダブルクウォートは意図不明。 ■” と ’ が混在してるけど、このスクリプトで使われてる ” は上記をのぞいて ’ に統一可能。 ■L41は、$newdata = $newcont . '<>' . $_SERVER['REMOTE_ADDR'] ; と書けば十分なのでは。 ■関数名とレフトブラケットの間に半角スペースがあったり無かったりする。これは趣味だから どう書いてもいいと思うが、普通は入れない。制御語（ifとかforとか）の後には入れる。そうすることで 関数と制御構造を見分けやすくする。カラーリング機能のないエディタではこの方が見やすい気もする。 いずれにせよ、どっちかに統一したほうがよいと思われ。 ■切り詰めを忘れている。ので、現データが 13<>127.127.127.127 で上書きデータが、14<>25.25.25.25 だった 場合、14<>25.25.25.257.127 に。書き込んだ後で、ftruncate(); を使ってファイルを切り詰める必要あり。 次ページ最新レス表示レスジャンプ類似スレ一覧スレッドの検索話題のニュースおまかせリストオプションしおりを挟むスレッドに書込スレッドの一覧暇つぶし2ch