STP^3

配列の最後の値を取得するコードの速度を比較

サーバの負荷が気になるので、お絵かき掲示板交流サイトのトップページのphpのコードを書き直しているところです…。

ディレクトリ内の画像から最新の画像を1枚表示したいだけなのに、ディレクトリ内に1000枚画像があったら配列の数が1000個…それをタイムスタンプで逆順でソートしてまた配列に格納…。

ファイル名の数字が同じ桁数で新しいほど数値が増えるのなら、globでディレクトリ内の画像を呼びした時点でソートがかかっているのでタイムスタンプを使った処理をしなくても…。

その処理を省略するだけで、少なくとも00.1秒は速くなるようですね。サーバの処理速度にもよりますが…。

ふと思ったのですが…。

ディレクトリに1000枚の画像が存在していたらglobで古い画像から順番に配列に格納されるので、最新の画像は1000番目になりますね…。

foreach(glob('cgi/poti/src/{*.png,*.jpg}', GLOB_BRACE) as $filename) {
}

foreachで配列の数だけループするけれど何もしない。しかしループが終わったあとに残る$filenameは配列の最後の値…。（必ずそうなる訳では無いようですが…）

$filenameに1000枚の画像のファイル名が入るので処理速度は期待できない…。

そこで…

$filename=glob("cgi/poti/src/{*.jpg,*.png}", GLOB_BRACE);
$filename=$filename[count($filename)-1];

最新の画像が表示されるので結果は同じ。ループしていないのでforeachを使うより軽い…はずだったのですがcountが重いのか…。

$filename=glob("cgi/poti/src/{*.jpg,*.png}", GLOB_BRACE);
$filename=array_slice($filename, -1)[0];

と

foreach(glob('cgi/poti/src/{*.png,*.jpg}', GLOB_BRACE) as $filename) {
}

の処理速度がほぼ同じ…。

むしろforeachでループして最後に残った値を取得する下側の書き方のほうが処理時間が短くて…。

$filename=$filename[count($filename)-1];

が一番遅い…。

$filename=glob("cgi/poti/src/{*.jpg,*.png}", GLOB_BRACE);
$filename=end($filename);

配列の最後の値を取得したいのならendという関数が…と思いましたがなぜかあまり速くない。

一番速かったのは…。一番遅いと思っていたforeachで配列を最後まで読み込んで最後に残った値…でした。

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

タグ：PHP 配列お絵かき掲示板

posted by さとぴあ at 2019/05/30 15:49 | お絵かき掲示板の改造

配列の数だけループ

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件でループを終了したい時ですね…。

他にも…。

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

お絵かき掲示板交流サイトのトップページの負荷がこれで軽く？

<?php
$time_start = microtime(true);
(ここに処理が入る)
$time = microtime(true) - $time_start;
echo "{$time} 秒";
?>

書き直し前の速度を計測するのを忘れていました…。
書き直し後に計測してみたところ、0.097969055175781 秒…でした。

タグ：PHP 配列お絵かき掲示板

posted by さとぴあ at 2019/05/23 23:20 | お絵かき掲示板の改造

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

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

発生していた問題。

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

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

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

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

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

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

タグ：テンプレートお絵かき掲示板 POTI-board 動的パレットスクリプト

posted by さとぴあ at 2019/05/18 06:45 | お絵かき掲示板の改造