配列の数だけループ

foreachで配列の数だけループできたんですね。試しに簡略化したプログラムを書いて実行…。
<?php
$arr = array('Apple', 'Google', 'Microsoft');
foreach($arr as $filename) {
var_dump($filename);//配列の値
}
foreach($arr as $kye => $filename) {
var_dump($kye);//配列のキー
}
結果。
string(5) "Apple" string(6) "Google" string(9) "Microsoft" int(0) int(1) int(2)
単純化してみてやっと理解できたような…。
という事は…。
さくらのレンタルサーバのOSのバージョンアップでphpの動作が変わった: STP^3
お絵かき掲示板交流サイトのトップページのコードを書きましたが…。
<?php
$arr = array();
foreach(glob('cgi/{foo,bar,baz}/index.html', GLOB_BRACE) as $filename) {
$arr[$filename] = $filename;
}
$i = 1;
foreach($arr as $key => $value){
var_dump($key);
if($i >= 3){break;}
$i++;
}
?>
配列の数が3でカウンターが3になったらbreak…。
エラーメッセージはでないし実行結果も問題ないのですが、この人配列の数だけループする事を知らないんじゃない?とphpの基礎がわかっている方は思ったでしょうね。

カウンターを回してbreakする必要があるとしたら例えば配列の数が30で上位3件でループを終了したい時ですね…。

他にも…。

php Brackets 変数 シングルクオート

書き間違いがあればエラーになるから文法がわからなくても何とかなる…ではまずいのでコードを調べて無駄な動作を減らし、メモリを消費する変数がループ外に残らないようにunset。

お絵かき掲示板交流サイトのトップページの負荷がこれで軽く?
<?php
$time_start = microtime(true);
(ここに処理が入る)
$time = microtime(true) - $time_start;
echo "{$time} 秒";
?>
書き直し前の速度を計測するのを忘れていました…。
書き直し後に計測してみたところ、0.097969055175781 秒…でした。

POTI-board用テンプレートPINKの動的パレットを修正しました。

動的パレットスクリプトのMatrix関連の不具合を修正しました。

発生していた問題。

お絵かき掲示板 POTI−board 動的パレットスクリプト

モノクロを「現在」で「取得」。

お絵かき掲示板 POTI−board 動的パレットスクリプト

「取得」した書式が正しくないので正しく色が「セット」されない。

お絵かき掲示板 POTI−board 動的パレットスクリプト

書式を修正して「セット」すれば正しく色が入るが「あいうえお」というパレット名が「あいうえ」になる。
14番目のパレットの色が正しくセットされない。

POTI-boardの動的パレットスクリプトはテンプレートに組み込まれているので、修正されたテンプレートに入れ替えればこれらの不具合は解消します。

POTI-board用テンプレート PINKは、lot.190516で対応しました。

お絵かき掲示板交流サイトの動的パレットの不具合も解消していますのでMatrix機能をぜひ使ってみてください。
パレットをインポートしたり、エクスポートしたりする事ができる便利な機能です。

さくらのレンタルサーバのOSのバージョンアップでphpの動作が変わった

サーバのOSのバージョンがあがっただけなのにお絵かき掲示板交流サイトの掲示板の並び方が変わってしまいました。
phpのバージョンは7.2のままです。
XAMPPでは意図通りの動作をするし、さくらのレンタルサーバでも昨日まで問題ありませんでした。
メンテナンスの内容。
FreeBSDのアップデートに伴う変更点 (2019/4/9 更新) – さくらのサポート情報

レンタルサーバのOSである「FreeBSD」のバージョンを9.x系から11.x系へ変更します。
問題が発生したプログラム。
<?php
$arr = array();
foreach(glob('cgi/{foo,bar,baz}/index.html', GLOB_BRACE) as $filename) {
$arr[$filename] = $filename;
}
$i = 1;
foreach($arr as $key => $value){
var_dump($key);
if($i >= 3){break;}
$i++;
}
?>
これまでは、
string(18) "cgi/foo/index.html" string(18) "cgi/bar/index.html" string(18) "cgi/baz/index.html"
foo,bar,baz だったのに、

string(18) "cgi/bar/index.html" string(18) "cgi/baz/index.html" string(18) "cgi/foo/index.html"
bar,baz,foo の順になってしまいました。
foreach(glob('cgi/{foo,bar,baz}/index.html', GLOB_BRACE) as $filename) {
$arr[$filename] = $filename;
}
このコードでは期待通りに並ばないので書き直す事にしました。
<?php
$arr = array('cgi/foo/index.html', 'cgi/bar/index.html', 'cgi/baz/index.html');
for($i = 0; $i <= 2; $i++){
$filename = $arr[$i];
var_dump($filename);
}
?>
XAMPPでも、さくらのレンタルサーバでも、
string(18) "cgi/foo/index.html" string(18) "cgi/bar/index.html" string(18) "cgi/baz/index.html"
が返ってくるようになりました。

POTI-board改のメンテナンスに参加した経験が無かったら書き直せなかったかも…。

お絵かき掲示板 ネコ娘 paintBBS NEO
お絵かき掲示板はこちら。

POTI-boardのエラーを減らすために 1のログの書式の見直し

POTI-board改のエラーを根絶できた…と思っていたのですが…。

POTI-board 1のログの書式の見直し

続きから描く時に番号を1にするとWarningが出ました。
if(file_exists(IMG_DIR.$ctim.$cext)){//画像が無い時は処理しない
$dat['picfile'] = IMG_DIR.$ctim.$cext;
$size = getimagesize($dat['picfile']);
中略
}
file_exists()で、画像が無い時は処理しないようにした筈…。
var_dump(IMG_DIR.$ctim.$cext); //string(4) "src/"
あるのはディレクトリだけのようです…。
var_dump(file_exists(IMG_DIR.$ctim.$cext)); //bool(true)
画像が無い時はfalseの筈…。
PHP: file_exists - Manual

file_exists − ファイルまたはディレクトリが存在するかどうか調べる
src/というディレクトリが存在しているかどうか調べていた…ようです。

パターンを変えてみると…。

POTI-board 1のログの書式の見直し
string(17) "src/1547552231446" bool(false)
falseになりました。
src/1547552231446というファイルは存在しないので…。

自動生成される1のログにタイムスタンプが入っていないので、ディレクトリがあるかどうかのチェックになり、ディレクトリがあるので画像がなくてもtrueになって処理続行。ディレクトリは画像ではないのでWarningがでる…。
POTI-boardのエラーを減らすためにE_NOTICE削減編: STP^3

設置時にできる[1]を編集すると、その後の書き込みでWarningが発生するので修正しました。
//文字列を整数に
$ltime = (int)$ltime;
(int)をつければエラーはでなくなるかもしれませんが、こちらも1のログにタイムスタンプが入っていない事が原因で発生していたWarningなので、この作業の時に1のログの書式を修正していれば良かったのですが…。
$time = time();
$tim = $time.substr(microtime(),2,3);
$testmes="1,".$now.",".DEF_NAME.",,".DEF_SUB.",".DEF_COM.",,,,,,,".$tim.",,,\n";
他のログ番号と同じ形式のタイムスタンプが1のログに入るように書き換えました。
(int)で整数にしなくても、Warningが発生しない事を確認しました。
続きから描くの番号を1にしてもWarningが発生しない事を確認しました。

この変更は、POTI-board改 v1.51.1 に反映されました。

お絵かき掲示板はこちら。

extract()の使用を全廃

poti-boardというPHPスクリプトをサイトに設置したいのですが、設置手順通... - Yahoo!知恵袋

このスクリプトはPHP5であるにも関わらず extract($_POST); extract($_GET); extract($_COOKIE); と、register_globals同等のことを自らやっているので、相変わらず脆弱性はあります。
extract($_POST); extract($_GET); extract($_COOKIE);による変数の取得をやめて、個別に変数を取得するようにしました。
POTI-board改 v1.50.0で対応ずみです。

残るはextract($_SERVER);でしたが…。
var_dump(count($_SERVER));で確認すると存在している配列の数47…。

変数として使っていれば配列のキーが変数名になっているはずなので1個ずつgrepで確認。
extract()を使っているのに、POSTかどうか調べているだけでした。
$REQUEST_METHOD = ( isset($_SERVER["REQUEST_METHOD"]) === true ) ? ($_SERVER["REQUEST_METHOD"]): "";
に書き直しました。

外部から取得する変数の取得方法を三項演算子に統一。
変数が未定義なら空文字列になるようになりました。
POTI-boardのエラーを減らすために $mode編: STP^3

//未定義エラー対策
if (isset($mode)){
switch($mode){
と書けば未定義エラーはでなくなると書きましたが…。
$modeが未定義にならなくなったので、isset()で確認する必要がなくなりました。
かなりの数のisset()を削減する事ができました。

この変更はPOTI-board改 v1.51.0に反映されました。

お絵かき掲示板はこちら。

php7.3のcompact()のエラーを修正

Notice、Warning、Deprecatedのエラーを少しずつ減らし…、ついにエラーのでないPOTI-boardができた…と思ったら、php7.3でcompact()のエラーがでてしまいました。

compact()に未定義変数が入るとNoticeが発生…。

compact()の手前でisset()で判定して空文字列にすればエラーはでなくなりますが、isset()だらけになるので、変数取得時のif文を見直す事に…。
if(USE_ANIME){
if(file_exists(PCH_DIR.$time.'.pch')){
$pch = $time.$ext;
}
elseif(file_exists(PCH_DIR.$time.'.spch')){
$pch = $time.$ext.'&amp;shi=1';
}
else{
$pch="";
}
}
else{
$pch="";
}
もしアニメを保存しない設定の時は、$pchを空文字列に。.pchファイルがなくて.spchファイルも無い時は、$pchを空文字列に。

値が入らない時は空文字列になるので、compact()に未定義変数が入るのを回避できます。
if文とelseを組み合わせれば未定義変数を減らす事ができるという基本的な事がやっとわかりました。

この変更はPOTI-board改 v1.50.6に反映されました。

お絵かき掲示板はこちら。

POTI-boardのエラーを減らすために 1のログファイルの書式編

POTI-boardの設置時にできる1のログファイルのコンマ数不足がが原因でNoticeが発生していたので…。
$testmes="1,".$now.",".DEF_NAME.",,".DEF_SUB.",".DEF_COM.",,,,,,,,,,\n";
コンマを8つ付け足しました。

お絵かき掲示板 POTI-board Notice削減

これだけでていたNoticeが…。

お絵かき掲示板 POTI-board Notice削減

でなくなりました。

記事を削除するとNoticeがでるので…。
for($i = 0; $i < $countline; $i++){
if($line[$i]){
list($no,,,,,,,$dhost,$pass,$ext,,,$tim,,) = explode(",",$line[$i]);

}
else{
$no=$dhost=$pass=$ext=$tim="";
}
$line[$i]が空文字の時には処理しないようにしました。

お絵かき掲示板 POTI-board Notice削減

記事を削除した時に発生していたNoticeが…。

お絵かき掲示板 POTI-board Notice削減

でなくなりました。

この変更はPOTI-board改 v1.50.5 lot.190101に反映されました。

お絵かき掲示板はこちら。

formの入れ子を回避する

<input>: 入力欄 (フォーム入力) 要素 | MDN

form 属性によって、入力欄を文書内のどこに置いても、文書内の他の場所にあるフォームと関連付けることができます。
formの入れ子になりそうなところは、早めにformのタグを閉じてformにidをつけて、inputにformのidを入れてやれば…。
<form id="foo" action="hoge" method="POST">
中略
</form>
<!--formのタグを閉じる-->
<!--別のformここから-->
<form action="fuga" method="POST">
中略
</form>
<!--別のformここまで-->
<!--最初のformのinput-->
<input form="foo" type="text">

id fooのformのinput。
ただしIE11では動作しないようです。

POTI-board改のテンプレート、POTI-board用テンプレート PINKの管理画面のセキュリティ対策のため、GETで管理者パスワードを取得していたテキストリンクをjavaScriptでPOSTに書き換えて、formの入れ子回避のためformにidを付けたのですが…。

やはりIE非対応…。
管理者が記事を削除するときに、Chrome・Firefox・Edgeを使えばすむ事なので、セキュリティリスクの回避を優先しました。

ブラウザのアドレスバーに管理者パスワードが表示されている状態で、著作表示のurlをクリックしたら…。
アクセスログに管理者パスワードが入ったurlが残るかもしれません。

noreferrerタグをいれてあるので、Chromeなら大丈夫かもしれないのですが…。IEはどうだったか…。
POTI-boardに限った問題ではなくて、relmにも同じ問題がありました。

テンプレートで対応できるのであれば、フタをしておいたほうがいいのでIE非対応でも…。

ゲストが投稿を編集・削除するときに使うメンテナンスフォームはIEでも問題なく動作するので、管理者が管理画面を操作する時にChrome・Firefox・Edgeを使えばすむことです。

formの入れ子を回避する

変更前のformタグの範囲。

formの入れ子を回避する

その中にjavaScriptを使ったform。

formの入れ子を回避する

formの入れ子になってしまうので、早めにformのタグを閉じて、idを付ける。

formの入れ子を回避する

inputにform属性を指定してformタグの範囲外から入力。
POTI-board用テンプレート PINK を配布します。: STP^3

セキュリティ対策済みのPOTI-board用テンプレート。

NEOを使うのチェックを外すとしぃペインター標準版が起動するテンプレートの配布も開始しましたので、よろしくお願いいたします。

POTI-boardの$delに不正な値が入らないようにする

他の変数にはスパム無効化関数を使う事ができたのですが、配列の$delをHTMLネームエンティティにエンコードすると動作しなくなるので…。
$del = ( isset($_POST["del"]) === true ) ? ($_POST["del"]): "";

if(is_array($del)){
$countdel=count($del);
for($i = 0; $i < $countdel; $i++){
if(!ctype_digit($del[$i])){//数字のみ
$del="";
}
}
}
else{
$del="";
}
$delが配列かどうか調べて配列なら配列の数だけ中身を確認し、数字以外の値が入っていたら空文字に置換、配列では無かった時も空文字に置換。入力フォームが改変されていない状態で、かつ数字が入力された時以外は$delが空文字になるようにしました。

お絵かき掲示板 POTI-board $delに不正な値が入らないようにする

$delが配列なのは、複数の投稿を選択して一括削除するためです。
投稿を2つ削除する時に、$delに入る配列は2つ。

お絵かき掲示板 POTI-board $delに不正な値が入らないようにする

数字以外をPOSTしてみます。

お絵かき掲示板 POTI-board $delに不正な値が入らないようにする

削除ボタンを押しても削除されず、$delに空文字が入りました。

お絵かき掲示板 POTI-board $delに不正な値が入らないようにする

入力欄の、name="del[]"から[]をとって、name="del"にして配列以外の文字列をPOSTしてみます。

お絵かき掲示板 POTI-board $delに不正な値が入らないようにする

配列かどうかチェックしているので、$delが空文字になりました。

お絵かき掲示板 POTI-board $delに不正な値が入らないようにする

name="del[a]"に書き換えて連想配列をPOSTしてみます。
$_POST["del"]で値を取得した直後に、var_dump($del["a"]);、var_dump($del);で確認すると…。
$del["a"]には配列が入りますが、$delは配列ではなくなります。
if(is_array($del)){
//中略
}
else{
$del="";
}
より下にvar_dump($del["a"]);、var_dump($del);を移動して確認すると…。
$delは配列ではないので空文字になります。
$del["a"]も空文字になります。

あとは、問題なく操作できる事を編集、削除を繰り返して確認…。

この変更は、POTI-board改 v1.50.3に反映されました。

お絵かき掲示板交流サイトはこちら。

POTI-boardのセキュリティを高めるために

poti-boardというPHPスクリプトをサイトに設置したいのですが、設置手順通... - Yahoo!知恵袋

このスクリプトはPHP5であるにも関わらず
extract($_POST);
extract($_GET);
extract($_COOKIE);
と、register_globals同等のことを自らやっているので、相変わらず脆弱性はあります。
PHP: extract - Manual

警告
ユーザーの入力、例えば $_GET や $_FILES のような、 信頼できないデータに extract() を使用しないでください。
今後も使い続けるために書き直す事にしました。
といっても、何がGETで何がPOSTなのかわからないところから検索して調べて…ですから、この人POSTとGETの違いがわからないのに掲示板をいじっていたの?と思われても仕方がない状態…からの書き直しとなりました。

extract($_POST); の問題点
extract($_POST);を使っていたら…。Chromeのデベロッパーツールでフォームの入力欄を変更するだけで任意の変数を設定できてしまいます。
ここでは、useneoの筈のところを改変してusenoeにしています。

extract($_POST); の問題点
var_dump($_POST);で確認してみると…。usenoe が on。
extract($_POST);なら、$usenoe=true;に。
そうならないように、あらかじめ設定した変数しか入らないように書き直しました。

extract($_GET); の問題点
extract($_GET);はもっと大変で、ブラウザのアドレスバーへの入力で任意の変数を設定できてしまいます。
また、?res=105 のような数字が入る事を期待している箇所にhtmlを入力すると入ってしまいます。

extract($_GET); の問題点
HTMLやjavaScriptが入力されたら、HTMLネームエンティティにエンコードして無害化するように書き直しました。
一部対応しきれなかったので穴がありますが、URLパラメータを入力するだけでは変数を設定できなくなりましたからかなりの改善…だと思います。

効率よく短くまとめる事ができるはずなのですが、技術も知識もないのでGitHub - sakots/noe: noe-board, the new oekaki BBS for php7のコードを拝借して、必要な変数を一個づつ追加していきました。
3行だった箇所が100行以上に膨れ上がりましたが、処理速度は速くなりました。

計測は、PHPの処理速度を計測 - Qiitaで…。
$usercode = ( isset($_COOKIE["usercode"]) === true ) ? ($_COOKIE["usercode"]): false;//falseならuser-codeを発行
//user-codeの発行
//if(!isset($usercode)){
if(!$usercode){//falseなら発行
$usercode =
動作確認に何日もかかった箇所。
user-codeのcookieが取得できない…。
isset($_COOKIE["usercode"])のところで未定義なら空文字となり、空文字でも定義ずみだからuser-codeを発行しないとなっていたただけでした。

もし未定義ならfalse、falseならuser-codeを発行するようにしました。
この箇所が今回の書き直しの最大の難関…でした。

そして、php5.2で動作確認してみると…文字化け。
$buf = charconvert(HtmlTemplate::t_buffer($template,$dat),'utf-8');
文字コードをutf-8に統一する時に4に置換したのがいけなかったようです。utf-8で出力するように書き直したところ、php5.2からphp7.2まで正常に動作するようになりました。

php4では動作しなくなっているので、動作するphpのバージョンを
if(phpversion()>="5.2.0"){
としました。

そして、@を使ったエラー制御があまりにも多かったので、@を外しました。
@を外したところディレクトリが存在しているからディレクトリを作れないというWarningが発生。
もし、ディレクトリが無かった時は作るとしました。
@mkdir(IMG_DIR,0777);chmod(IMG_DIR,0777);
if(!is_dir(realpath(IMG_DIR))){
mkdir(IMG_DIR,0777);chmod(IMG_DIR,0777);
}
に書き直し。

書き直した行数があまりにも多いので、プログラムを書くのが大変だった…でも間違いではないのですが、どちらかというと動作確認に時間がかかって大変…でした。

今でもメンテナンスされていて、誰でも開発に参加できるお絵かき掲示板のスクリプトはnoeを除くとPOTI-board改だけです。誰でも設置できるお絵かき掲示板の最後の砦になってしまいました。

この変更はPOTI-board改 v1.50.0に反映されました。

お絵かき掲示板交流サイトはこちら。

POTI-boardの処理速度をあげるために

PHP: for - Manual

for($i = 0; $i < count($people); ++$i) {
$people[$i]['salt'] = mt_rand(000000, 999999);
}
このコードは実行速度が遅くなることでしょう。 というのも、配列のサイズを毎回取得しているからです。 サイズが変わることはありえないのだから、これは簡単に最適化することができます。 配列のサイズを変数に格納して使うようにすれば、 何度も count() を呼ばずに済むのです。
他のfor文では変数に格納して処理しているのに、その他のfor文で同じ処理を変数に格納しないで行っているのは…整理したほうがいいので…。
//ここでcount($tree)を使うとループの回数分countする。$counttree = count($tree);で定義ずみ。
for($i = 0; $i < $counttree ; $i+=PAGE_DEF){
//中略
}
変数に置き換える事ができる箇所がそのほかにもあったので書き換えました。
念のため、
var_dump($counttree);
var_dump(count($tree));
が同じ結果になる事を確認しました。
var_dump($counttree);
var_dump(count($tree));
if($counttree > $next){
$dat['next'] = PHP_SELF.'?mode=catalog&amp;page='.$next;
}
POTI-board カタログモード var_dump($counttree); var_dump(count($tree));
どちらも(int)554。
はじめから
$counttree=count($tree);
for($i = 0;$i<$counttree;$i++){
//中略
}
となっている箇所もありました。

お絵かき掲示板交流サイトのPOTI-boardの該当箇所を書き直しました。

POTI-board改にこれからプルリクエスト…。

お絵かき掲示板はこちら。

POTI-boardのエラーを減らすために メール通知クラス編

POTI-boardの、メール投稿通知機能に使われているnoticemail.incというメール通知クラスから非推奨のエラーがでていました。
potiboard.phpに
ini_set( 'error_reporting', E_ALL );
を追加して、すべてのエラーを表示するように設定して調べた筈では?
投稿の直後に一瞬ちらっとみえるエラーが気になったので、「画面を切り替えます」の表示秒数を1秒から10秒に変更して確認してみたところ
Deprecated: Non-static method noticemail::send() should not be called statically in C:\xampp\htdocs\hoge\test\potiboard.php on line 1276
htmltemplate.incで発生していたのと同じエラーでした。
static 宣言されていないメソッドに対する static 呼び出しが非推奨となりました。 将来的には削除される見込みです
static 宣言すればエラーはでなくなるのですが…。本当にそれでいいのかどうかは、phpのバージョンがあがってみないとわからないという…。
とりあえずstatic 宣言して、E_DEPRECATEDが発生しないようにする事はできましたが…。実は動きませんだと困りますね…。
public static function send($data,$usemb="1"){
public staticをつけただけ…です。

そして、Noticeはもうでなくなった筈だったのですが…。

お絵かき掲示板 POTI-board 画像なし二重投稿の時に発生するNoticeお絵かき掲示板 POTI-board 画像なし二重投稿の時に発生するNotice

画像なしにチェックが入っていて二重投稿のエラーになる時にNoticeが発生していました。
if(isset($com)){
if(isset($textonly) && $textonly){//画像なしの時
$dest="";
}
switch文の手前だったので、画像なし=textonlyの時は画像に関する変数に空文字がはいるようにしました。
しかし、今度は…。

お絵かき掲示板 POTI-board 画像なしの時に発生するNoticeお絵かき掲示板 POTI-board 画像なしの時に発生するNotice
画像なしにチェックをいれてテキストの投稿に成功した時にNoticeが発生…。
たしか対策済みだった筈…と思って該当行をみてみると、
if(!isset($dest)){
$dest=$ext=$W=$H=$chk="";
}
もし、$destが未定義なら変数を空文字にとしていたため、$destに空文字がはいって定義済みの空文字となり、他の変数が空文字にならなくなっていました。

画像なしにチェックが入っている…という段階で、画像に関する変数が空文字になっても問題ない筈なので、
if(isset($textonly) && $textonly){
$dest=$ext=$W=$H=$chk="";
}
に書き換えました。

お絵かき、画像アップロード、レスでお絵かき、レス画像アップロード。
画像なしテキストのみ、レス画像なしテキストのみの組み合わせで1時間ほどテストして、お絵かき掲示板交流サイトのPOTI-boardに組み込みました。

この変更は、POTI-board改 v1.45.6 lot.181202に反映されました。

POTI-boardのログファイルの上限を2MBに

ぷち的ココロ♪

総レス数がまだ2800を超えた程度なのに、親記事のナンバーの低いスレが消えてしまっていました。
親記事ナンバー+5000レスをスレの消える寿命に設定するにはどうしたらいいのでしょうか?

スクリプトをちょっと見ただけなので間違いかもしれませんが
バッファに1000000byte読み込む処理があったので
logファイルの上限がおそらく約1Mbyteだと思います。
その上限を超えたので2800でレスが消えたのではないでしょうか
POTI-boardのログファイルの上限の1000000byteは15年前に設定されたものです。
パソコンに搭載されているメモリはこの15年で…。
今ではメモリ4GB…それでも少ないという時代になりましたが、15年前はハードディスクが8GBだったり…。
2倍の2MBに拡張しても問題なさそうな気がしたので、2倍の2000000byte…だと2MB以下になってしまうので該当箇所を
$buf=fread($fp,2097152);
$buf=fread($tp,2097152);
に書き換えました。
5MBのログに書き込み処理を行うと、2MBになる事を確認しました。

お絵かき掲示板交流サイトで使っているPOTI-boardのログの上限は5MBですが動作が重くなるようならログをリセットして運用するつもりです。
配布するスクリプトも5MB…だと、無料のサーバなどで問題がでるかもしれませんし、動作が重くなるかもしれません。
2倍の2MBならという判断です。

お絵かき掲示板 POTI-board Notice削減お絵かき掲示板 POTI-board Notice削減
if(isset($savepng)){
$dat['savetypes'].= "<option value='PNG'".$savepng.">PNG</option>\n";
}
if(isset($savejpeg)){
$dat['savetypes'].= "<option value='JPEG'".$savejpeg.">JPEG</option>\n";
}
保存形式をAUTO、PNG、JPEGから選択するテンプレートは今はほとんど使われていません。NEOでは何を選択してもPNGになるのであまり意味がありません。しかし、しぃペインターでは有効な設定なので、テンプレートに選択項目が無くてもNoticeが出ないように書き直しました。

お絵かき掲示板 POTI-board Notice削減お絵かき掲示板 POTI-board Notice削減
// if(!$com&&!@is_file($dest)) error(MSG008,$dest);

if(!$com&&!@is_file($dest)) error(MSG008);
画像が無い事を確認してエラー画面を表示しているのに、画像に関する処理をしようとしてNoticeがでていたので書き直しました。
<!--(// しぃで反映されない上にお絵かき画面でパレット変更できるのでコメント化
パレット<select name=palette><option value="" selected> デフォルト</option>
{$palette}
</select>
)-->
v1.32の MONO WHITEでコメントアウトされ、そのほかのテンプレートでも使用されていない機能でNoticeがでていたので、
//v1.32のMONO WHITEでコメントアウト、対応テンプレートが無いパレット選択用データ(selectタグ用option配列)
// $lines = file(PALETTEFILE);
// foreach ( $lines as $line ) {
// $line=preg_replace("/[\t\r\n]/","",$line);
// list($pid,$pname,) = explode(",", $line);
//if(isset($palette)){
// $dat['palette'] .= '<option value="'.$pid.'">'.CleanStr($pname)."</option>\n";
//}
// }
該当箇所をコメントアウトしました。

お絵かき掲示板 POTI-board Notice削減お絵かき掲示板 POTI-board Notice削減
if(isset($textonly) && $textonly){//画像なしの時
$upfile=$upfile_name=$pictmp=$picfile="";
if(!isset($resto)){$resto="";}
}
画像アップロード画面で、画像なしにチェックを入れた時に、Noticeが発生していたので、textonlyの時は画像に関する変数を空文字に。
また、変数が定義されていてtrueの時は…という処理をifの入れ子にしていましたが&&でつなぐだけで良かったので書き直しました。

お絵かき、レスでお絵かき、画像アップロード、レス画像アップロード、文字だけ…。
数時間テストしましたがとりあえずNoticeなし。
ただし、設置時にできる[1]を削除しないとNoticeが出ます。
ログの形式が[1]だけ異なっている事が原因なので、それを修正できればでなくなる筈なのですが…。
そして、管理画面で記事を削除するとNoticeが発生…。
2行程度ですが…。

この変更は、POTI-board改 v1.45.5 lot.181129に反映されました。

お絵かき掲示板はこちら。

POTI-boardのエラーを減らすために 動画再生編

PaintBBS NEOが動画の記録と再生に対応したので、絵板に動画へのリンクが増えて来ました。

動画を見ようとしてクリックすると…。

POTI-board お絵かき掲示板の改造

動画ファイルが存在しないというWarningが発生する事例が何件が…。
リロードしても動画ファイルは見当たらないまま…。

POTI-boardの動画再生のurlは画像のファイル名と紐付けられているので、続きから描いて画像のファイル名が変われば動画のurlも変わってしまいます。

つまり、掲示版を開いて動画のリンクをクリックしたタイミングによっては動画のファイルが無いかもしれません。
Google検索結果から動画のリンクを開いた時にも、動画がなくてWarningが発生していました。
if(file_exists($pchfile)){//動画が差し換えられていた時
$datasize = filesize($pchfile);
$size = getimagesize($picfile);
if(!$sp) $sp = PCH_SPEED;
$picw = $size[0];
$pich = $size[1];
$w = $picw;
$h = $pich + 26;
if($w < 200){$w = 200;}
if($h < 226){$h = 226;}
}
else{
$w=$h=$picw=$pich=$datasize="";
}
file_existsでファイルの存在を確認してから処理するようにしました。

POTI-board お絵かき掲示板の改造
動画が無い時は表示しないだけなのですが…。

POTI-board お絵かき掲示板の改造

.htaccessで、キャッシュの有効期限を過去にすればキャッシュされなくなるので問題ないのですが、投稿してもリロードしないと画面が更新されない、これはバグだ…と言われてしまったりもしているので、urlパラメータを追加してキャッシュを表示しないように。

さくらのブログの管理画面から記事を見るのリンクをクリックした時にurlのあとにつく、
?1542988451と同じ形式のurlパラメータです。
投稿するたびに変化するurlパラメータで、ブラウザに別のurlと認識させて、キャッシュを表示しないようにします。
<Files ~ "\.html?$">
<IfModule mod_headers.c>
Header set Pragma no-cache
Header set Cache-Control no-cache
Header set Expires "Thu, 01 Dec 1994 16:00:00 GMT"
</IfModule>
</Files>
.htaccessの使い方がわかっている方なら、これをコピペしてPOTI-boardのフォルダに入れるだけでHTMLファイルをキャッシュしないようにする事ができます。
サーバによってはうまくいかない場合もあるようですがさくらのレンタルサーバはこの設定で問題ありませんでした。
.htaccessでキャッシュの有効期限を過去にしたほうが、見た目もスマートだし掲示版を開いた時にキャッシュが表示される事もありません。
.htaccessで設定できない場合には有効かもしれない機能の追加…でした。

この変更は、POTI-board改 v1.45.3 lot.181122に反映されました。

お絵かき掲示板はこちら。

POTI-boardのエラーを減らすために 設置直後のNotice編

お絵かき掲示板交流サイトに4種類の掲示版を設置していましたが、PaintBBS NEOの描画アニメに対応できて、TwitterやFacebookに投稿した画像を表示できる仕組みも利用できるのはPOTI-boardだけだったので、6つの絵板をすべてPOTI-boardにしました。

そうなると…。
お絵かき掲示板の改造で注力するべきは、POTI-board…という事でNoticeを減らすための作業をすすめる事に。
前回の作業で、大量のNoticeはでなくなった筈だったのですが…。

POTI-board お絵かき掲示板の改造

まだログが20行存在していないのに過去の20行の二重投稿をチェックしようとしてNoticeが発生。
// 連続・二重投稿チェック (v1.32:仕様変更)
if($countline >= 21){//存在する行数のみチェック
$chkline=20;
}
else{
$chkline=$countline-1;
}
// for($i=0;$i<20;$i++){
for($i=0;$i<$chkline;$i++){
ログファイルが何行あるのかチェックして存在する行の数だけループするように書き換えました。

POTI-board お絵かき掲示板の改造

二重投稿のチェックが有効で、かつNoticeが出ない事を確認。
設置時にできる[1]のログの書式が他のものとは異なるため、[1]が残っているとNoticeが2行発生しますが、画面が埋まるほどのNoticeが発生する訳ではないので、許容範囲…ですね。

POTI-board お絵かき掲示板の改造

まだログが200行存在していないのに、過去の200行の画像重複をチェックしようとしてNoticeが発生。
// アップロード処理
if(isset($dest)){//未定義エラー対策
$countline=(count($line));
if($countline >= 201){//存在する行数のみチェック
$chkline=200;
}
else{
$chkline=$countline-1;
}
if($dest&&@file_exists($dest)){
// for($i=0;$i<200;$i++){ //画像重複チェック
for($i=0;$i<$chkline;$i++){ //画像重複チェック
こちらも、ログファイルが何行あるのかチェックして存在する行の数だけループするように書き換えました。

POTI-board お絵かき掲示板の改造

画像重複のチェックが有効で、かつNoticeが出ない事を確認しました。
// if($tree[$i]==""){
//空文字ではなく未定義になっている
if(!isset($tree[$i])){
また、カタログモードの時に、Noticeが発生していたのでしらべてみたら、空文字ではなく、未定義になっていたので書き直しました。

この変更は、POTI-board改 v1.45.3 lot.181122に反映されました。

お絵かき掲示板はこちら。