php独自タグ

phpの文字列内に独自タグを判定し、HTMLのコードに変換する処理を実装しています。

強調文字(独自タグ)
<bold>太字</bold>

文字列内に上記のタグがある場合、HTMLコードに変換し出力しようとしております。

$html = '<link>テスト|http://yahoo.co.jp</link>aaaaaa<link>テスト2|http://google.co.jp</link>';

if ( preg_match_all('/<link>(.*)?<\/link>/', $html, $src) ) {
	if ( is_array($src[1]) ) {
		foreach ( $src[1] as $val ) {
			$arr[] = explode('|', $val);
		}
	}
}

foreach ( $arr as $key => $val ) {
	$str = $val[0];
	$url = $val[1];
	$html = preg_replace( "/<link>(.*)?<\/link>/", "<a href='$url' target='_blank'>$str</a>", $html);
}
echo $html;

上記のようにタグを変換しているのですが、複数タグだと、上書かれてしまいます。
複数タグにも対応できるようにしたいです。。

kunai

2016/10/20 01:45

何がわからないのかがわからないのですが。。preg_replaceを使うというところまでわかっているのであれば、preg_replaceでタグの書き換えを書けばよいのでは。まずご自身で「こうかな？」という形で書いてみて、想定通りに動かない場合やエラーが出る場合にその解決方法を聞くのがよろしいかと。

maisumakun

2016/10/20 01:52

本題ではないのでここに書きますが、<link>は正式なHTMLにも含まれるタグですので、勝手な方法で使わないほうがいいです。

hotta

2016/10/20 02:54

入力文字列 XXXX を変換したら YYYY になるようにしたい、という具体的な例示をしてみてください。

行動規範の内容に同意します

回答2件

ユーザが自由に記述できる<>形式の独自タグは後々困ると思います。クロスサイトスクリプティングという脆弱性はご存知でしょうか？これを避けるためには，原則的に

htmlspecialchars関数でHTMLタグをすべてエスケープする
strip_tags関数でHTMLタグをすべて消去する
HTML Purifierというライブラリで特定のタグと属性だけ許可するようにフィルタリングする

のどれかを採る必要があります。1番の方法だと巻き添えを食らってしまうので<>形式の独自タグとは共存できません。2番の方法は例外的に許可するタグを指定することもできますが，属性も含めてフィルタリングしないと危険なので意味がありません。3番のライブラリは正しいHTMLタグの使い方を想定しているので，今回のように誤った用途で使うと予期しない結果になる可能性があると思います。

私もMarkdown形式を採用すべき，に1票です。

cebe/markdown: A super fast, highly extensible markdown parser for PHP

ただ標準のMarkdownは色付き文字に対応していないので，色を付けさせるのであれば自分で定義を拡張する必要があります。もしくはHTMLタグの使用を許可し，HTML Purifierで処理することになるでしょう。GitHubやQiitaは後者ですね。

投稿2016/10/21 10:07

mpyw

総合スコア5223

miyagi1987

2016/10/22 13:27

ご回答頂きありがとうございます。また、それに伴うセキュリティの指摘や、代替案などご提示頂き助かりました。

行動規範の内容に同意します

ベストアンサー

以下のようにやればできるかと！

php
1<?php
2$html = '<link>テスト|http://yahoo.co.jp</link>aaaaaa<link>テスト2|http://google.co.jp</link>';
3
4$html = preg_replace( "/(<link>)(.*?)\|(.*?)(<\/link>)/", "<a href='$3' target='_blank'>$2</a>", $html);
5echo $html;