POTI-boardのエラーを減らすためにE_NOTICE削減編

Noticeが投稿時に300行以上発生していたので修正しました。
また、設置時にできる[1]を編集すると、その後の書き込みでWarningが発生するので修正しました。

数値にNULLが入るので、(int)で処理

[1]を編集すると、連続投稿の時間を計算している箇所にNULLが入りWarningが発生するので…。
//文字列を整数に
$ltime = (int)$ltime;
if(RENZOKU && $time - $ltime < RENZOKU){error(MSG020,$dest);}
if(RENZOKU2 && $time - $ltime < RENZOKU2 && $upfile_name){error(MSG021,$dest);}

数値にNULLが入るので、(int)で処理

空文字だったところがint(0)になりました。

画像にWarningが出ていますが、var_dump()で変数を確認すると出る箇所なので仕方がありません。通常の使用では発生しません。

投稿時に大量のNoticeが発生するのを修正投稿時に大量のNoticeが発生するのを修正

投稿時に大量のNoticeが発生。
ログがあまり多くなかった時にはあまり気にならなかったのですが、運営中のレス数が1600件ほどあるPOTI-boardではレスを送信するたびに300行以上のNoticeが発生しました。
// レス記事一括格納
if(isset($rres)){//未定義エラー対策
$dat['oya'][$oya]['res'] = $rres[$oya];
}

レスがついていないスレッドの数だけエラーが出るので、未定義の場合は処理しないように書き直しました。
その他、画像アップロードの時と、お絵かきの投稿の時と、文字だけの投稿の時に、それぞれ未定義になる変数が存在していたので空文字を入れるようにしました。
//未定義エラー対策
if(isset($picfile)){//お絵かきの時
$upfile=$upfile_name="";
if(!isset($resto)){$resto="";}//レスではなかった時
}
elseif(isset($upfile)){//画像アップロードの時
$pictmp=$picfile="";
if(!isset($resto)){$resto="";}
}
else{//文字だけの時
$upfile=$upfile_name=$pictmp=$picfile="";
if(!isset($resto)){$resto="";}
}
regist($name,$email,$sub,$com,$url,$pwd,$upfile,$upfile_name,$resto,$pictmp,$picfile);

htmltemplate.incの79行目で発生するNoticeも投稿時にくりかえし出てきます。
まず、var_dump()で変数の内容を確認。

投稿時に大量のNoticeが発生するのを修正
//未定義エラー対策
if(isset($match[1])){
$ind=$this->getIndex($match[1],$multilabels);
}
else{
$ind=$this->getIndex("",$multilabels);
}
($match[1]))が未定義の時は、代わりに空文字が入るようにしました。
この変更を行っても$indに入る文字列は同じである事を確認しました。

気を使った点…。
変数に空文字を入れて対処した事で動作がおかしくなっていないか?
画像アップロードに必要な変数に空文字が入れば画像がありませんとなるし、お絵かきの投稿の時に必要な変数に空文字が入ればやはり画像がありませんになってしまいます。

新規スレッドに画像アップロード、お絵かき、文字だけの投稿。
レスで画像アップロード、お絵かき、文字だけの投稿。
続きから描く、動画表示。
ひととおりテストして、運営している絵板サイトにも導入してトラブルが起きていない事を確認したので、公開版の候補にする事にしました。
それから…。
salt パラメータは必須ではありませんが、これを省略すると crypt() が作るハッシュが弱いものになってしまいます。 PHP 5.6 以降は、このパラメータを省略した場合に E_NOTICE が発生するようになりました。
time()をsaltに使い差し換え時の動作に問題がないことを確認しました。
//差し換え時の認識コード追加
if($type=='rep'){
// $repcode = substr(crypt(md5($no.getenv("REMOTE_ADDR").$pwd.date("Ymd", time()))),-8);
$repcode = substr(crypt(md5($no.getenv("REMOTE_ADDR").$pwd.date("Ymd", time())),time()),-8);
この変更は、POTI-board改v1.45.2 lot.181110に反映されました。

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

この記事へのコメント

コメントを書く
お名前:

メールアドレス:

ホームページアドレス:

コメント:

※ブログオーナーが承認したコメントのみ表示されます。