元スレ[PHP]弱い掲示板の改造手伝ってください
php覧 / PC版 / みんなの評価 :
1 = :
http://www.dotup.org/uploda/www.dotup.org11328.zip.html
とりあえず作ったファイル。
興味ある人はぜひ手伝ってください…
2 = :
掲示板見てみた。
めっちゃ弱いけど自分としては「～つくろう！」で終わるやつよりはいいと思う。
がんばれ。
3 = :
表示イライラするからファイル構成変えた方が吉。
・書き込み、読み込みに分ける
・読み込みは基本Ajax
こんな感じがいいと思う。
4 = :
>>1
とりあえず、ユーザ関数作ったら？
function findAll($log_dir){
$lines = file($log_dir);
$return = array();
foreach($lines as $line){
$array = explode('<>', $line);
$return[] = $array;
}
return $return;
}
とかで、全部のログを一回配列に吐き出してみたり。
とりあえずKENTのBBSを参考にしたのか、元々Perlやってた人なのかわからんけど、
PHPでやるならもうちょっとPHPの便利な関数や書式を覚えたほうがよさげ。
6 = :
>全部のログを一回配列に吐き出して
そういう主婦の洗濯物整理みたいな下らない工夫が低レスポンスのシステムが作られるきっかけになる。
7 = :
>>4
PHPは始めたばかりです、すいません
>>5
変更しました。ありがとうございます
>>3
Ajaxでの読み込みを試してみましたが、JavaScriptの知識は少ないもので。
変数valueが文字化けしてしまいました。どうすればいいでしょうか・・・
function xmlhttp () {
var xmlhttp;
try {xmlhttp = new ActiveXObject("Msxml2.XMLHTTP");}
catch () {
try {xmlhttp = new ActiveXObject("Microsoft.XMLHTTP");}
catch (e) {xmlhttp = false;}
}
if (!xmlhttp) {xmlhttp = new XMLHttpRequest();}
return xmlhttp;
}
var xmlhttp = xmlhttp;
var node = document.getElementsByTagName("dl")[0];
var i, n;
xmlhttp.onreadystatechange = function () {
if (xmlhttp.readyState == 4) {
if (xmlhttp.status == 200) {
var value = xmlhttp.responseText;
}
}
};
xmlhttp.open("GET", "<?php echo $log ?>?<?php echo date(ymdHis) ?>", true);
xmlhttp.send();
11 = 9 :
ライブラリはあまり使いたくありませんが・・・
prototype.js使う以外に手が無いなら仕方ないですね
12 = :
Ajaxより先にMVCを分けてみたら？
13 = 9 :
MVCを分けるのが先かな？
でもスレッド式にする気はまだないしとりあえず基礎だけ作っちゃいたいな
14 = 9 :
じゃあXMLで試してみなよ
15 = 9 :
>>14
ミスった
試してみます、ですね
何が起きたのやら
16 = 9 :
http://www.dotup.org/uploda/www.dotup.org11950.zip.html
とりあえず自分的に感じるイライラ感を修正。
これからXMLたらAjaxやら放っておいて分別に入ります
17 = :
自演で自爆してるじゃんｗｗｗｗｗｗｗｗｗｗｗｗｗｗｗｗｗｗｗｗｗｗｗｗｗｗ
18 = :
>>14
できるよ。
以上。
はい、次。
20 = :
なんという名スレｗｗｗｗｗｗｗｗｗｗｗｗｗｗｗｗｗｗ
21 = :
ひどいｗｗ
22 = :
おもしれーｗｗｗ
23 :
すごい自演・・・
26 = :
ここまで全部俺の自演
27 = :
ﾜﾛﾀｗｗｗ
何がしたいんだｗ
29 :
なんだか>>1さんが複雑な事情でいなくなったみたいですが、
自分も弱い掲示板を作ったので、
http://www.dotup.org/uploda/www.dotup.org0369.zip.html
のコードレビューをしてみていただけないでしょうか？
まずは機能的にどうこうというよりも、セキュリティ的な
見落としがないか気になっています。
30 = :
あとPHPの作法というか、ファイルや関数の分け方、命名のしかたなど、
機能追加しやすいというか、保守性の高いソースコードにするには
どうしたらいいのか教えて下さい。
31 = :
すみません。ミスをしていたのに気付いたので、直したものをあげ直します。
http://www.dotup.org/uploda/www.dotup.org0372.zip.html
32 = :
Web上でソース見れるようにしてくれ
33 = :
>>32
しました。
http://d.hatena.ne.jp/nobodysan/
34 = :
とりあえずライブラリディレクトリ掘って(libとか)
データの入出力を担当するクラスを作れ。
そんで掲示板の読み込み、書き込み、削除を全てそのクラスを通してやってみろ。
他のファイルからは一切mysql関係の関数呼ぶな。
35 = :
>>34
クラス一つでデータの入出力全部ですか。
難しそうですが、やってみます。
36 :
あと、MVC意識したいなら、テンプレートとPHPの処理は極力分けれるようにしたほうがいい。
HTMLにPHPべた書きはメンテナンス面でもお勧めしない。
37 = :
クラス一つでデータの入出力全部を扱うことに挑戦してみましたが、
二つのエラーに悩まされています。
詳しくは下記に書きました。
http://d.hatena.ne.jp/nobodysan/20081002/1222935662
どなたか教えて下されば幸いです。
38 = :
マルチ乙
39 = :
>>38
マルチしてませんよ？といっても、証拠は出せませんが。
他にどのスレに書き込みがありましたか？
40 = :
お前は何を言ってるんだ
41 = :
カス乙ｗｗｗｗｗｗｗｗｗマニュアル嫁ｗｗｗｗｗｗｗｗｗｗｗ
$dbaccess = new DBAccess();
~~
$dbaccess->insert_post($name, $comment);
~ ~~
$dbaccess->show_contribution($_GET['page'], false);
~ ~~
べ、別に（ｒｙ
42 = :
>>41
カスです。マニュアル読みます。
でもありがとうツンデレなnobodyさん。
43 = :
なんでクラス一つでデータの入出力全部やらしてんの？
44 = :
はてダのアカウントまで取る姿勢は評価してやらんでもないぞ
ただコード書いたらはzip上げてくれ
もしくはSubversion使え
次はデータベースクラスでechoするの禁止にして、
純粋にデータそのものの入出力だけさせてみれ。
で、別途表示関連全部を担当するクラスを作って、
そいつがデータベースクラスから受け取ったデータを
タグやら何やら添付したり加工して表示させると。
45 = :
>>44
Subversionはサーバの準備に結構お金がかかりそうなので、
ちょっと勘弁して下さい。
とりあえず、現時点でのソースを上げておきます。
http://www.dotup.org/uploda/www.dotup.org0372.zip.html
次は表示ロジックのクラスですね。
やってみます。
47 = :
フレームワーク使ったら？
48 = :
ん～
49 = :
>>46
Smartyを使わなくても、もうちょっとやりようはあって、
bbs-admin.php----------------
$hoge = 'mozi';
$moge = 'foo';
require './template.html';
template.html-----------------
<html>
<body>
<?php echo $hoge?>
<?php echo $moge?>
</body>
</html>
とかみたいに、ファイルを分離してやるってだけでいいんじゃない。
50 = :
>>49
そんな書き方が出来たんですね。
やってみます。
みんなの評価 :
php一覧へ類似してるかもしれないスレッド
- こんにちは、PHPプロの方。教えてください。 (187) - [34%] - 2019/5/9 7:45
- PHPで作られた有名サイトってあるの？ (736) - [28%] - 2019/5/9 7:45 △
- 【PHP】Lvうｐしたいので宿題ください (264) - [28%] - 2019/5/9 7:45
トップメニューへ / →のくす牧場書庫について