PHPで、コメント内の２パターンの置換処理について

Question

他のQ&Aのところで質問させてもらってたのですが、最終的な回答がつかずで、少し急いでいるためこちらで質問させてください。
やりたいことは、掲示板の書き込まれたコメント内に、URLが入っていた場合自動リンクにする処理と、同じコメント内に、 ＞○○さん という文字列が入っていた場合この文字列を太字にする
ということがやりたくて関数を書いたのですが、URLの方はちゃんと反映されるのですが、太字にする方が出来ません。エラーメッセージは出ていません。（ちなみに、○○部分に ＞ も さん も入ることはないのでそこの細かいチェックは必要ないです）
コードについてですが、

mySQLに保存しているコメントをSQL文で呼び出して$mojiretu2_2 というのに代入しています。

###該当のソースコード

if (!function_exists('change2')) {
function change2($mojiretu2_2){ 
$keyword_start = '＞';
$keyword_end = 'さん';
 if(preg_match("/(http|https):\/\/[-\w\.]+(:\d+)?(\/[^\s]*)?/",$mojiretu2_2)){
   preg_match_all("/(http|https):\/\/[-\w\.]+(:\d+)?(\/[^\s]*)?/",$mojiretu2_2,$pattarn2);
     foreach ($pattarn2[0] as $key2=>$val2){
      $replace2[] = "<a href='".$val2."' target='_blank' style='font-size:0.9em;'>".$val2."</a>";

}
       $mojiretu2_2 = str_replace($pattarn2[0],$replace2,$mojiretu2_2);

} else if(preg_match("/'.preg_quote($keyword_start).'.*'.preg_quote($keyword_end).'/",$mojiretu2_2)){
   preg_match_all("/'.preg_quote($keyword_start).'.*'.preg_quote($keyword_end).'/",$mojiretu2_2,$match);
foreach ($match[0] as $key3=>$val3){
$futoji[] = "<span style='font-weight:bold;'>".$val3."</span>";
}
       $mojiretu2_2 = str_replace($match[0],$futoji,$mojiretu2_2);

}
return $mojiretu2_2;
}
}
     $mes2_2 = change2($mojiretu2_2);

###試したこと
関数を２つ別のに分けて、
$mes2_2 = change2($mojiertu2_2).change3($mojiretu2_2);
としてみたら、文字列が太字にはなったのですが、URLの自動リンクがきかないのと、２重書き込みみたいなことになってしまいました。

よろしくお願いいたします。

Answer 1

if ( !function_exists( 'h' ) ) {
	function h( $str ) { return htmlspecialchars( $str, ENT_QUOTES, 'UTF-8' ); }
}

function change2( $text ){ 
	$pattern_url = "/(http|https):\/\/[-\w\.]+(:\d+)?(\/[^\s]*)?/";
	if( preg_match( $pattern_url, $text ) ) {
		preg_match_all( $pattern_url, $text, $much_url );
		foreach ( $much_url[ 0 ] as $key => $val ){
			$replace1 = "<a href='".h( $val )."' target='_blank' style='font-size:0.9em;'>".h( $val )."</a>";
			$text = str_replace( $val, $replace1, $text );
		}
	}
	$pattern_name = "/＞.*さん/";
	if ( preg_match( $pattern_name, $text ) ) {
		preg_match_all( $pattern_name, $text, $much_name );
		foreach ( $much_name[ 0 ] as $key => $val ){
			$replace2 = "<span style='font-weight:bold;'>".h( $val )."</span>";
			$text = str_replace( $val, $replace2, $text );
		}
	}
	return $text;
}
$mes2_2 = change2( $mojiretu2_2 );

こうするとかでどうでしょうか。（すいませんが未テストです）

function changeX( $text ){ 
	$text = preg_replace( "/(http|https):\/\/[-\w\.]+(:\d+)?(\/[^\s]*)?/", "<a href='$0' target='_blank' style='font-size:0.9em;'>$0</a>", $text );
	$text = preg_replace( "/＞.*さん/", "<span style='font-weight:bold;'>$0</span>", $text );
	return $text;
}
$mes2_2 = changeX( $mojiretu2_2 );

こうするとか。（未テストです）

Comments

[mari.rinn]

＞kei344さん

２パターンも考えてくださりどうもありがとうございました！
どちらもコピペしてやらせていただいたのですが、どちらも最初に該当のものが出てくるところ（太字にするのがURLのコメントより先になってるのでそこで）で、こちらのエラーが出て、それ以降のコメントはまったく表示されない形となりました。

Fatal error: Cannot redeclare h() (previously declared in C:\xampp\htdocs\waiwai\wg.php:228) in C:\xampp\htdocs\waiwai\wg.php on line 228

この２２８行というのは、頭のfunction h( $str ) { return htmlspecialchars( $str, ENT_QUOTES, 'UTF-8' ); }
この行になります。

今回は、takasima20さんに教えていただいたコードでうまく出来たので問題解決しましたが、kei344さんが書いてくださったものはどちらもかなり短いのでビックリしました。ここまで簡潔に出来るものなんですね。ちょっと自分でも後でこれを調べてみたいと思います。

大変勉強になりました。ありがとうございました！
また何かありましたらどうぞよろしくお願いいたします！

[kei344]

ああ、すいません h()を外したら問題ないと思います。（ちょっと修正しました）

> ここまで簡潔に出来るものなんですね。
preg_match_all はそれぞれ抜き出した項目に処理をかけたいときに使うので、置き換えるだけであれば preg_replace でいける気がします。（いろいろ書き方はあると思います）

ひとまず解決お疲れ様です！

[mari.rinn]

＞kei344さん

最後までどうもありがとうございました！ちゃんと出来ました！
置き換えるだけなら～　というのはなるほどです！

なんかせっかく書いていただいたのにこれをボツにするのももったいないので、
今のとは別の掲示板で使わせていただきます（；^ω^）
ほんとにどうもありがとうございました～！

また何かありましたらよろしくお願いいたします｜！

Answer 2

elseがいらないんじゃない？

Comments

[mari.rinn]

＞te2jiさん

ご回答をつけてくださりどうもありがとうございました！
やってみたのですがやはりURLの方だけがきいてて、太字の方は変化なしでした。

今回は、takasima20さんに教えていただいたやり方で無事にうまく出来ましたので良かったです。
どうぞ、また何かありましたらよろしくお願いいたします！

[退会済みユーザー]

条件式の正規表現もおかしかったんですね。
"/" . preg_quote($keyword_start) . '.*' . preg_quote($keyword_end) . "/"
に置き換えると動きます。
ただ、同じ文字列2回が出現すると、多分意図する結果が得られなくなりそうですね。
＞さしすせそさん
＞さしすせそさん
＞さしすせそさん
こんな文章だと、複数回置換が走ってしまう＾＾；

[退会済みユーザー]

kei344 さんのだとうまくいくのでそちらを採用されたほうが良いかも

[mari.rinn]

＞te2jiさん

追記のお返事をくださりどうもありがとうございます！
書いてくださったこと、あとで試してみたいと思います。

＞ただ、同じ文字列2回が出現すると、多分意図する結果が得られなくなりそうですね
そうなんですか？
一応今回は、そういうパターンになることはまずないと思われる掲示板なので、今の形でとりあえずはいってみたいと思います。

ほんとにどうもありがとうございました！
またよろしくお願いいたします～！

Answer 3

こんな感じ? 動かしてないんでミスってたらごめん。

if (!function_exists('change2')) {
  function change2($mojiretu2_2) { 
    preg_match_all("/(http|https):\/\/[-\w\.]+(:\d+)?(\/[^\s]*)?/", $mojiretu2_2, $pattarn2);
    $replace2 = array();
    foreach ($pattarn2[0] as $key2=>$val2) {
      $replace2[] = "<a href='" . $val2 . "' target='_blank' style='font-size:0.9em;'>" . $val2 . "</a>";
    }
    $tmp = str_replace($pattarn2[0], $replace2, $mojiretu2_2);

    $keyword_start = '＞';
    $keyword_end = 'さん';
    preg_match_all("/" . preg_quote($keyword_start) . '.*' . preg_quote($keyword_end) . "/", $tmp, $match);
    $futoji = array();
    foreach ($match[0] as $key3=>$val3) {
      $futoji[] = "<span style='font-weight:bold;'>" . $val3 . "</span>";
    }
    $ret = str_replace($match[0], $futoji, $tmp);

    return $ret;
  }
}
$mes2_2 = change2($mojiretu2_2);

Comments

[mari.rinn]

＞takasima20さん

早速にご回答くださりどうもありがとうございます！
コピペさせていただいてやってみたのですが、URLの方も太字の方もどちらもきかない状態となります。

なんかでもあと少しな感じもするので、書いていただいたことを元に、自分でもあれこれやってみようと思います。
どうもありがとうございました。

[takasima20]

あ、$retにセットするとこミスってた。
修正しときました。
(^_^;

[mari.rinn]

＞takasima20さん

度々どうもありがとうございます！
コピペさせていただいてやってみましたが、やはり文字の方は何も変化なく、またURLの方は、いくつか入ってるURLの一番最初のものだけが自動リンク付で表示されるのですが、それ以降のものはURLごと表示が消えてしまいます。
お手数おかけしてすみません。

[takasima20]

置換内容はみてなかったんで修正してみました。

[mari.rinn]

＞takasima20さん

ありがとうございます！無事にうまく出来ました～～～！！！
お忙しいところ、何度もほんとにすみませんでした。
最後までめんどう見ていただき、またコピペするだけで良いように書いていただき、
とっても感謝です！　ほんとにどうもありがとうございました！

また何かありましたらよろしくお願いいたします！