[Boo]https://gitlab.com/tsfan/futaba-phpソースコードを開示しています。修正する案など募集中。[Boo]初めまして。ソースコード拝見しました。HTMLとしての書き出しにエスケープが足りて無さそうな感じだったりがあって、ちょっと危険な香りがします。今まで「ファイルが壊れた」という話は出ていない様なので、安全性には問題無いと思うのですが、データファイルをテキストで保持するよりは、SQLiteでもいいのでDBを使った方が楽そうに思います。[Boo]>>6レンタルサーバーが昔ながらのファイルアップロードする形式なのか、データベースが使えるのか。それともVPSなのかにもよるよね。[Boo]>>7今時のPHPはSQLiteがデフォルトで使えます。サーバ屋がわざわざ使えない様に自前でコンパイルし直していれば別ですが、普通そんな事はしないですから。根幹部分の変更なので、改造というよりも同じインターフェイスの別物を新規に作る感じになりそうですね。誰も手を出さずに、PHP3の頃のコードが残っているんでしょう。[Boo]軽微なタイプミス(=動作にさほど影響は無い)等は、発見の都度修正で良いでしょう。現時点で対象とすべきPHPのバージョンは7.3(または7.2)でしょう。拡張された三項演算子を使うとか、filter系で値は取得するのがベターじゃないかと。参照渡しが多用されているのは「メモリの節約になる」という思いでしょう。CoWをが効いているので変更する予定の無い物は(パフォーマンス的にも)無意味ですよね。「差が無いなら変更しなくても良い」かもしれませんけど、何が変更されるのかが明確になるので、戻り値を受け取る方が良い様に思います。先々の修正が楽になる(と思う)ので、クラス化とオートロードの採用が良いのではないでしょうか?ファイル数が増えるという弊害はあるでしょうけど。[Boo]ちょびっとした修正なら、このままでも改修できると思います。新しい機能を追加していくにつれて、レガシー部分をどうするか?っていう課題も膨らんでいくのでUIだけ同じにして、裏側のphpをモダンにしていくことが必要かもしれません。futabaの画像掲示板、1phpファイルで動く簡単な仕組みになっているので簡単な仕組みを踏襲していくかLaravelとかのフレームワークを使って構築するような手順にしてしまうか、悩みどころです。UIだけ同じにして、裏側のphpをモダンにしていく僕もそれが良いと思います。Laravel(評判は良い様ですけど)などのフレームワークを入れてしまうとそれに対するメンテナンスも必要にもなるし、PHPが速くなったとはいえ現状よりも複雑にするのは得策ではない様に思います。とそれ以外の分離、機能のクラス化、オートロード、シングルトンなデータストレージ、位でそれなりにモダンで、この先も使える様なものになるんじゃないかと思います。id属性値を付けておいて『IDを指定した入力フィールドに』がハッキリして良いんじゃないかと。実行させようとするものは他に無いだろうから、無名関数で十分だと思います。特定のキーを探す関数も、指定したキーで場所を特定すればいいのだから冗長な部分をカットしてはどうかと思います。それらを書いてみるとこんな感じ。してまでヘッダに押し込まなくても外部ファイルにした方が良いのかなと思います。[Boo]関数名や変数名は冗長に付ける主義なので、長いですよね。管理者じゃないユーザの出来る事はこんな感じ?システムもUMLもチャンと理解してないから...いろいろ指摘してください。表示する都度、DBアクセスしない「静的ページ=DBに保存したデータを必要としないページ」なら、その様な内容で記述するだけでしょうから特に問題は無いかと。広告掲載・注意文などと合わせて、簡易なテンプレート式にするのが良いだろうと思います。class="thread"><?=$thread;?></div>とか、ループ処理とかを素のPHPで書くだけで実現できますし。'../template/hoge.html';の様にインクルードしてもいいと思います。良ければ、ファイルの保存時にMD5をファイル名にすれば確認が容易だと思います。ないと、どこかにハッシュ値を記録しないといけないですから。いうファイル名を嫌がる人も居ますけどね。[Boo]マイナス評価ってのは、そういうファイル名が嫌って事?今の様なファイル名だと、DBに記録する他無いから、余分なコストが掛かるって事だけど?それに、何らかの条件で検索しなければならないならDBで保持する必要性もあるけど、ハッシュ値だけしか無いなら理由としては薄いと思う。