BugTrack/2273

AutoLinkが有効の時も、recent.dat の処理の高速化を†

• ページ: BugTrack2
• 投稿者: 名無しさん
• 優先順位: 低
• 状態: 提案
• カテゴリー: その他
• 投稿日: 2007-09-16 (日) 14:35:45
• バージョン:

メッセージ†

BugTrack2/196#q089d030 のために、BugTrack2/151やlib/file.php を覗いていたのですが、put_lastmodified() で実行しているAutoLink の処理は実質この部分だけみたいです。

	$pages = get_existpages();

（ここの間に、recent.dat を再生成する処理と、それを基にRecentChanges を更新する処理が入る）

	if ($autolink){
		autolink_pattern_write(CACHE_DIR . PKWK_AUTOLINK_REGEX_CACHE,
			get_autolink_pattern($pages, $autolink));
	}

つまり、recent.dat を作り直す部分はAutoLink の処理と関係が無い*1ので、lastmodified_add() をこんな感じに変えてはどうでしょうか。

cvs:lib/file.php (1.86) をベースに

 function lastmodified_add($update = '', $remove = '')
 {
	global $maxshow, $whatsnew, $autolink;

-	// AutoLink implimentation needs everything, for now
-	if ($autolink) {
-		put_lastmodified(); // Try to (re)create ALL
-		return;
-	}
+	if (PKWK_READONLY) return; // Do nothing

-	if (($update == '' || check_non_list($update)) && $remove == '')
-		return; // No need
+	if (($update == '' || check_non_list($update)) && $remove == '') {
+		if ($autolink) {
+			$pages = get_existpages();
+			autolink_pattern_write(CACHE_DIR . PKWK_AUTOLINK_REGEX_CACHE,
+				get_autolink_pattern($pages, $autolink));
+		}
+		return;
+	}

（中略）

	// Recreate
	ftruncate($fp, 0);
	rewind($fp);
	foreach ($recent_pages as $_page=>$time)
		fputs($fp, '-' . htmlspecialchars(format_date($time)) .
			' - ' . '[[' . htmlspecialchars($_page) . ']]' . "\n");
	fputs($fp, '#norelated' . "\n"); // :)

	flock($fp, LOCK_UN);
	fclose($fp);
+
+	if ($autolink) {
+		$pages = get_existpages();
+		autolink_pattern_write(CACHE_DIR . PKWK_AUTOLINK_REGEX_CACHE,
+			get_autolink_pattern($pages, $autolink));
+	}
 }

if (PKWK_READONLY) が追加されているのは、put_lastmodified() にはこれがあるのに、何でlastmodified_add() には無いんだろう、という理由だけです。

本当は、if ($autolink) の中の処理を関数化した方がいいかなとも思うのですが、いい関数名が思いつかないのでこのままです（autolink_pattern_write() はAutoLinkだけでなく、AutoAliasのためにも使ってますから）。

recent.dat の処理だけ全ファイルを（必ず）開く必要がなくなったとして、どこまで効果があるのかは未知数なんですが*2、提案しておきます。

関連†

• BugTrack2/151

コメント†

• コメントありがとうございます。時間が無いので、全然関係ないかもしれないことを書きます。そういえば更新したページ名がAutoLinkにマッチしている時、何か無駄な処理をキャンセルする余地があるんじゃなかったっけなぁ・・・zzz *3 -- henoheno 2007-09-19 (水) 00:19:03
  • BugTrack2/81 のような基本ロジックのことは無視して、get_autolink_pattern() を呼び出す時の事を考えると、
    AutoLink のリストを更新する必要があるのは、(1)追加・更新されたページがリストに載っていない時、(2)削除されたページがリストに載っている時、のどちらかが該当している時だけに絞れるのは確かですね。
    今の仕組みだとページ名が「アルファベットのみの構成」と「それ以外」の2種類リストがあるので、上の(1)(2)と合わせると4回はpreg_match() 実行する必要がでてしまいますけど（さらにエラーの可能性も*4）。
    後は、$nowikiname やページ名の長さといったget_autolink_pattern() の中で判定していることを、リストを更新しなくてもいいようにとここで除外判定するのかという問題や、put_lastmodified() にも$update と$remove を渡してこの判定処理を入れるのかという問題があります。
    ここまでくると、判定用関数にまとめたほうが良さそうにみえます。BugTrack2/196と一緒に料理するほうがいいのかな？2つともrecent.dat 更新の呼び出し側ですが、lib/file.php がらみなんで。 -- 2007-09-19 (水) 21:17:37

• BugTrack
• 開発版
  • リポジトリ
  • 開発日記
• :CategoryDev
  • PukiWiki2
  • PukiWiki/1.5
  • PukiWiki/1.4
  • PukiWiki/1.3
  • メーリングリスト
  • 開発談義
  • 雑談
• 開発者向け情報
  • Books
  • プラグイン

最新の16件