第二回 自作スクリプト発表会【PHP】

[0001 1様 2006/08/21(月) 08:31:27 ID:???]

自分が書いたPHPスクリプトを発表するスレ。

初心者が書いた入門スクリプト、ベテランが書いたすげースクリプトを発表してね。
予めスクリプトの内容を明記してURLを貼り付けるのがよろしいかと。
ﾏﾀｰﾘ進行でいきませう。


【PHP】自作スクリプト発表会
　　http://pc8.2ch.net/test/read.cgi/php/1017456911/all


Warning: 発表の場は、学びの場。クレクレ君は他所へ。

[0067 nobodyさん 2008/11/16(日) 16:05:49 ID:???]

>>66
失礼しました。

[0068 nobodyさん 2008/11/17(月) 03:02:43 ID:DKgH+i/4]

じゃあ、
【PHP】下らねぇ質問はID出して書き込みやがれ 76
http://pc11.2ch.net/test/read.cgi/php/1226517332/
このスレッドを１時間に１回監視して、
書き込みがあったら、
番号、名前、さげ、年月日時間、ID、内容
をテキストファイルに追記していく。
アボーンがあったら、何番がアボーンになったと記録する。
そんな、PHPスクリプトを自作してここで発表して下さい。
お願いします！！

[0069 nobodyさん 2008/11/17(月) 04:09:29 ID:???]

>>68
それって、スレを丸ごとファイルに保存するのと何か違うの？？

[0070 ６８ 2008/11/17(月) 05:05:35 ID:DKgH+i/4]

>>69
・PHPの技術力が違う
・エレガントさが違う
それに一番大事なのは、
１日１回、スレを丸ごと保存し、古いのを捨てる、ってやり方があるけど、
それだと、アボーンに何が書いてあったかわからないのです。
よろしくお願いします！！

[0071 nobodyさん 2008/11/17(月) 05:12:05 ID:???]

>>70
要するにあぼーんされた部分を残しときゃいいってこと？
あぼーん以外を保存しときゃあとから参照できるような気がするけど・・・

■番号、名前、さげ、年月日時間、ID、内容をテキストファイルに追記していく。
■アボーンがあったら、何番がアボーンになったと記録する。

↑この2つはフォーマットが全然違うと思うけど、そのへん何か考えてます？

あと、1時間に1回って処理をしようと思ったらcronとか使うと思うけど大丈夫ですか？

[0072 ６８ 2008/11/17(月) 05:22:04 ID:???]

>>71
> 要するにあぼーんされた部分を残しときゃいいってこと？
そうです。

> ■番号、名前、さげ、年月日時間、ID、内容をテキストファイルに追記していく。
> ■アボーンがあったら、何番がアボーンになったと記録する。
>
> ↑この2つはフォーマットが全然違うと思うけど、そのへん何か考えてます？
いえ、考えてません。

> あと、1時間に1回って処理をしようと思ったらcronとか使うと思うけど大丈夫ですか？
cronを知らないのでなんとも言えません。
ただ、winXPのパソコンを２４時間立ち上げておくことは可能です。

[0073 nobodyさん 2008/11/17(月) 05:30:09 ID:???]

>>72
http://lcdata.ddo.jp:8080/2ch/1156116687/68/
↑ここで作ってます。
7時までにできなかったら仕事に出動orz
ニートになって趣味プログラミングに没頭したいなぁ。

[0074 ６８ 2008/11/17(月) 05:34:40 ID:???]

>>73
おお、よろしくお願いします。

[0075 nobodyさん 2008/11/17(月) 05:52:05 ID:???]

メアドがテスト用にあぼーんになってたことに気づいたorz

>>74
ごめん、あぼーんって動作テストできないってことに気づいた。
肝心のあぼーん部分がほとんど検証できてないです。

一応ソース↓
http://lcdata.ddo.jp:8080/2ch/1156116687/68/index.phps

1．自分のPCにWeb鯖を立てる。
2．↑のスクリプトを設置する。
3．ブラウザから何度かアクセスして、以下のCSVファイルを吐くことを確認する。
　　（1）現在のスレの内容（ただしあぼーん以外/あぼーんされたレスは「あぼーん」って文字が追加される）
　　（2）前回のアクセスから増えているレスの内容
4．cronが使えない環境みたいなので、「コントロールパネル＞タスク」を使って1時間ごとにアクセスするようにがんばる。

スクリプトがまともに動くなら、CSVファイルからあぼーんされたレスを
抽出するスクリプトを組むのは簡単だと思う。

[0076 nobodyさん 2008/11/17(月) 06:02:37 ID:???]

ごめん、あぼーんのとこミスしてた。
たぶんもう大丈夫だと思います。
ソースは↑のやつを更新しました。

[0077 ６８ 2008/11/17(月) 06:48:24 ID:???]

>>75
おお、有り難うございます。すごいですね。
なにはともあれ、1226517332_new.txtというファイルが出来て、そこにデータが全部入ってるみたいです。感謝、感謝です。
ただ、↓のエラーがどっちゃりでてます。

Notice: Undefined offset: 1 in C:\php\ThreadKeep.php on line 34
Notice: Undefined index: abone in C:\php\ThreadKeep.php on line 72

[0078 ６８ 2008/11/17(月) 06:52:49 ID:???]

$new_csv[$num]["mail"] = $matches[1]; 　　これが３４行目です。
if (!$value["abone"]) { 　　　　　　　　　これが７２行目です。

[0079 nobodyさん 2008/11/17(月) 06:57:43 ID:???]

>>77
Noticeなのでエラーではなく通知ですね。
PHPの設定で、エラーを拾うレベルが高すぎるんだと思います。
php.iniの「error_reporting」の箇所を↓に修正して、Noticeがでないようにできます。

error_reporting = E_ALL & ~E_NOTICE

エラーはふつうに表示されるのでご安心を。
ホントはNoticeも出ないように作るべきなんだろうけど時間切れです、すいません。

[0080 ６８ 2008/11/17(月) 07:05:46 ID:???]

>>79
了解しました。有り難うございました。感謝、感謝です。

[0081 nobodyさん 2008/11/17(月) 12:27:58 ID:???]

>>65はどうなったの？

[0082 nobodyさん 2008/11/17(月) 20:12:35 ID:???]

>>81
まだ作りはじめたとこで、アップローダのアもできてない状態です。
おもっくそスレ違いで申し訳なかったです。

[0083 nobodyさん 2008/11/17(月) 22:47:09 ID:???]

>>80
あぼーんされたレスだけを別ファイルに保存するようにしてみた。
ソースは同じとこです。
Noticeが出るのは放置してます。

[0084 nobodyさん 2008/11/18(火) 00:00:51 ID:???]

ファイルアップローダ程度でDB使うのってどう思いますか？
できるだけ環境を選ばず設置できるようにしたいので使いたくないんですが、
コーディングの煩雑さとパフォーマンスの誘惑に負けそうです。
僕もそうだったけど、「DBって何？聞いたことあるけど」って人は多いですよね。
敷居下げるならDBなしなのかなぁ・・・。

[0085 nobodyさん 2008/11/18(火) 02:18:16 ID:???]

ファイルベースが特段煩雑とも思わんが・・・。
素人向けのレンタルスペースでも動くスクリプトを配布したいんなら無しでしょうね。
ただPHPが動くレンタルサーバでDBが入ってないとこなんてあんの？知らないけど。
どちらも押さえときたいなら両方作って配布するとか。

[0086 nobodyさん 2008/11/18(火) 07:15:17 ID:???]

>>85
なるほど。
でもやっぱSELECTの手軽さは魅力だと思います。
DBばっか使ってるからファイルベースだとCSVしか思いつかなくて・・・。
ただ、アップローダは自鯖ってのが基本だと思ってます。
XAMPPとか使ってとりあえずDB入れたけど、使い方はさっぱりって人は多いと思うなぁ。
ってことでファイルベースでやってみます。
どうでもいいけど、前スレ見てたら以前もアップローダ作ってたこと思い出したw
完成したんだっけかなぁ・・・。

[0087 >>65 2008/11/24(月) 21:25:23 ID:???]

ものすごくどうでもいいことですが、
ルータを買ったらDDNSに対応してたので、URLを変更しちゃいました。
>>65のアップローダは↓で作成中です。
ttp://lcdata.ath.cx:8080/2ch/upload/

[0088 nobodyさん 2008/11/25(火) 20:18:05 ID:???]

>>87
見れねーよ？
完成したん？

[0089 >>65 2008/11/25(火) 22:54:07 ID:???]

>>88
すいません、またまたURLを変更しました。
ttp://lcdata.ath.cx/2ch/upload/
こっちでやってます。
まだ作りかけです。
なんでポート80番が使えないんだろ？
とか思ってたんですが、スカイプが使ってることに気づいたのでorz

[0090 ６８ 2008/11/26(水) 15:39:46 ID:yZjcfnV8]

// レスごとに分解
$thread = @file_get_contents($url);
preg_match_all("/<dt>(.*)/", $thread, $matches);
$res = $matches[1];
if ($res) {
foreach ($res as $key => $value) {
// レス番号
preg_match("/^(\d*)/", $value, $matches);
$num = $matches[1] - 1;
$new_csv[$num]["num"] = $matches[1];
// メールアドレス
preg_match("/<a href=\"mailto:([^\"]*)/", $value, $matches);
$new_csv[$num]["mail"] = $matches[1];
// 名前
preg_match("/<b>([^<]*)/", $value, $matches);
$new_csv[$num]["name"] = $matches[1];
// 名前
preg_match("/<b>([^<]*)/", $value, $matches);
$new_csv[$num]["name"] = $matches[1];
// 時刻
preg_match("/(<\/a>|<\/font>)：(.[^I]*)ID:/", $value, $matches);
$new_csv[$num]["time"] = $matches[2];
// ID
preg_match("/(ID:)(.*?)<dd>/", $value, $matches);
$new_csv[$num]["id"] = $matches[2];
// レスの内容
preg_match("/<dd>(.*)/", $value, $matches);
$new_csv[$num]["res"] = $matches[1];
$abone = preg_match("/あぼ.*?ん<\/b><\/a>：あぼ.*?ん<dd>/", $value, $matches);
if ($abone) {
$new_csv[$num]["abone"] = true;
}}}

[0091 ６８ 2008/11/26(水) 15:45:19 ID:yZjcfnV8]

>>90は>>75さんのhttp://lcdata.ddo.jp:8080/2ch/1156116687/68/index.phpsのソースの一部分です。
で、
preg_match_all("/<dt>(.*)/", $thread, $matches);
この行で1レスづつ、配列に入れる。
で、
$res = $matches[1];
この行の意味がわからないです。
$matchesの変数、$matches[1]の変数、$resの変数のそれぞれの違いがわからないのです。

[0092 >>65 2008/11/26(水) 22:59:01 ID:???]

>>90-91
いやあの晒されると恥ずかしいですが。
特にそこ適当なんでw
http://jp2.php.net/manual/ja/function.preg-match-all.php
意味はマニュアルに書いてあるとおりです。
どっか動きがおかしいのを修正したいのか、改造したいのか、単純に意味がわからなかったのか、
コードが汚すぎて読みにくいって指摘なのか、PHPを覚えてみようと思ってるのか、
いったいどれでしょうか？

ふと思い出したけど、php.iniのerror_reportingを書き換えなくても、
----------------------------------------
<?php
error_reporting(0);　　←これ追加
----------------------------------------
だけでNoticeは出なくなると思います。

[0093 >>65 2008/11/26(水) 23:04:25 ID:???]

超適当に説明しておくと・・・
まず、このスレのソースコードを見てみてください。
各レスが「<dt>〜」って1行になってるのがわかると思います。
まず、$matchesにその各レスが入ります。
$matches[1]には、「"/^(\d*)/"」などの1番目のカッコの中身が入ります。
そんでもって、さらにそのレスに対してpreg_matchをすることによって、
レス番号とか名前とかを切り出してるんです。
$matches[1]をわざわざ$resって変数に入れてるのは特に意味はありません。

[0094 ６８ 2008/11/26(水) 23:56:10 ID:yZjcfnV8]

>>92
有り難うございます。　確かに、http://jp2.php.net/manual/ja/function.preg-match-all.php に
↓が書いてありました。　すみません。

>$matches[0] はパターン全体にマッチした文字列の配列
>$matches[1] は第 1 のキャプチャ用サブパターンにマッチした文字

> どっか動きがおかしいのを修正したいのか
動きは、ちゃんと動いてくれて感動しました。

>単純に意味がわからなかったのか、
はいそうです。

>PHPを覚えてみようと思ってるのか
そうです、これがメインで1行づつ読み解かしてもらってます。

> ふと思い出したけど、php.iniのerror_reportingを書き換えなくても、
> ----------------------------------------
> <?php
> error_reporting(0);　　←これ追加
> ----------------------------------------
> だけでNoticeは出なくなると思います。
有り難うございます。

[0095 nobodyさん 2008/11/27(木) 10:46:47 ID:QDCpWLvT]

俺はこれ
error_reporting(E_ALL);

面倒だけど変数をまとめて設定する癖がつくから後々楽だぞ

[0096 >>65 2008/11/27(木) 20:06:40 ID:???]

>>95
僕も余裕があればデバッグ中はそうするように心がけてみます。
PHPしか知らないから、変数の宣言とか型とかいい加減な扱いするクセついちゃったなぁ。

[0097 nobodyさん 2008/11/27(木) 20:12:41 ID:???]

開発機のphp.iniで設定しときゃいいじゃん

[0098 >>65 2008/11/27(木) 20:18:28 ID:???]

>>97
そうですね。
他に何に使ってるわけでもないWeb鯖だし、それでいいと思います。
僕はいつも、↓みたいな感じでログを流しながら作業してます。
tail -f /apache/log/error_log
Windowsで似たようなことってできるのかなぁ。
Cygwinとか試してみよう。

[0099 nobodyさん 2009/05/19(火) 20:17:57 ID:D1418Uic]

【設置場所】　http://u9.getuploader.com/kiren/
【ソース】　http://u9.getuploader.com/kiren/download/1/chat.7z
【プログラミング経験】　3ヶ月
【希望評価スタイル】　悪い部分を指摘して欲しい
【自分の性格】　またーり
【備考】
表示部分はJQUERYで、サーバーの部分はPHPで作りました。どうも保守性が悪いような気がするので、
悪い部分があったら指摘してほしいです。

[0100 nobodyさん 2009/05/21(木) 01:39:47 ID:???]

悪い部分って？

[0101 nobodyさん 2009/05/21(木) 10:06:33 ID:???]

>>99
HTMLのエスケープはPHP側でやったほうがいいよ
あとちゃんと見てないからよくわからんが、今HTTPヘッダで取ってくるログを判別してるみたいだけど、
それもPHP経由で重複したデータを取ってこないように。

[0102 nobodyさん 2009/05/30(土) 11:37:25 ID:???]

>99
暇だから見てみるか。
とりあえず3秒見たところだが、enter.phpにXSSな。htmlspecialchars()かけろ。

[0103 nobodyさん 2009/05/30(土) 12:28:09 ID:???]

load_namefile(&$a, $b)は、$aをそのまま返せばいい。参照渡しで値を返すと可読性が激しく低下する。返り値$iはcount($a)に等しいはずだ。

変数名が手抜きすぎる。$aとか$bとか$cとか$vとかなんの拷問だ。
append_namefile($name, $last_talk_time, $room_number)にするだけで10,000倍読みやすくなる。foreach()とかで使う一時変数も同じ。
「保守性の高いコード」はアーキテクチャが生むんじゃなくて、変数名やインデントなどのミクロな部分が生むんだ。神は細部に宿る。

$_POST["state"]での処理の振り分けは、明らかにifではなくcaseにすべき箇所。

名簿ファイルのロック処理が半端。flock()を信用するならbegin_lock()は不要。
信用しないならflock()が正常に動く事を前提としているbegin_lock()も同じく信用ならない事になるので意味がない。
信用しないならmkdirロックを使うべき。SQLiteかなんかを使ったほうが手っ取り早いが。

flock($fp, LOCK_UN)してはいけない。「鍵を開けた後、ファイルを閉める」までの間にファイルが破壊される。
単に閉じれば安全な形でロックが開放される。

[0104 nobodyさん 2009/05/30(土) 12:29:19 ID:???]

メイン処理（$_POST["state"]を見て実行される場所）のひとつひとつを関数ないしクラスにした方が、可読性が向上する。
意図をそのまま関数名にしろ。例えば最初のブロックは、そのまま読むと
「ロックを取り、ユーザー名簿を取得し、名簿に書いてある数字がいまの時刻より$_keep_time小さかったら名簿を削除し、名簿を再度書き出す」
わけだが、お前はトップレベルのコードを読んでる時はそうは思考しないだろ？
「長時間退席しているユーザーを蹴る」とか、そういう粒度でこの処理を見るはずだ。ならこの処理は「kick_afk_users()」と関数化すべきなんだ。
トップレベルでやってる処理は多岐に渡るんだから、詳細はいったん別の場所においておくべき。
kick_afk_users();
if($_POST["state"] == "enter"){
　enter_room();
}else if($_POST["state"] == "quit"){
　quit_room();
}else if($_POST["state"] == "sid"){
　if(!in_room($name)){
　　 enter_room();
　}
　rotate_log();
　add_talk();
とかなってれば、この部分をざっと見るだけでおおざっぱな処理の流れが見て取れるようになる。

[0105 nobodyさん 2009/05/30(土) 16:37:24 ID:???]

>>103
99じゃないけど
> flock($fp, LOCK_UN)してはいけない。
これは以外だった。ちょっと調べてみよ。

[0106 【23.9m】 電脳プリオン 2012/05/06(日) 15:00:42.23 ID:???]

　　∧＿∧
　 （　・∀・）　　　　　　　　　　　　人　ｶﾞｯ
　 （　　　　つ―-‐-‐-‐-‐-‐○ < 　>__Λ∩
　 人　Y ノ.　　　　　　　　　　　 　Ｖ｀Д´）/
　 し（＿）　　　　　　　　　　　　　　　 　　/ 　←>>21

[0107 島本町で凄惨なイジメを受けて廃人になった方へ 2014/08/09(土) 16:53:26.10 ID:eoXmXjRk]

＞大阪府三島郡島本町の小学校や中学校は、暴力イジメ学校や。
島本町の学校でいじめ・暴力・脅迫・恐喝などを受け続けて廃人になってしもうた僕が言うんやから、
まちがいないで。僕のほかにも、イジメが原因で精神病になったりひきこもりになったりした子が何人もおる。
教師も校長も、暴力やいじめがあっても見て見ぬフリ。イジメに加担する教師すらおった。
誰かがイジメを苦にして自殺しても、「本校にイジメはなかった」と言うて逃げるんやろうなあ。
島本町の学校の関係者は、僕を捜し出して口封じをするな

子供の時に受けた酷いイジメの体験は、一生癒えない深い傷になる
「暴力とイジメと口裏合わせと口封じ」の町やそういう町に巣食うヤクザ・チンピラ・ゴロツキ・不良・
いじめっ子・殺人鬼・ダニ・ノミ・シラミなどを監視して非難するのは暮らしやすい町を作るのに必要だ

[0108 nobodyさん 2017/12/30(土) 13:59:10.80 ID:YhlYw6jg]

誰でも簡単にネットで稼げる方法など
参考までに、
⇒　『半藤のブブイウイウレレ』 というサイトで見ることができます。

グーグル検索⇒『半藤のブブイウイウレレ』

AO0UH9L8HP

[0109 nobodyさん 2019/05/09(木) 02:21:04.26 ID:HHcYDMUs]

phpについて役立つ情報とか
http://mevius.5ch.net/test/read.cgi/tech/1557329831/l50

7H9

[0110 nobodyさん 2023/09/25(月) 03:18:49.75 ID:???]

(||ﾟДﾟ)ﾋｨｨｨ!(ﾟДﾟ||)