<?php…?>は、どこに書く？

Question

PHPを出力するとき、スペースが勝手に入って困ります。

###スペースが入る出力
次のように書いたとき
「未入力」だけを出力したいのに
「　　　未入力　　　」みたいな感じで出力されます。

<p>
<?php
	$test = xprofile_get_field_data('test' ,bp_get_member_user_id());
	if( !empty($test) ){
		echo $tset;
	}else{
		echo '未入力';
	}
?>
</p>		

###正しい出力
下記のようにすれば
「未入力」だけ出力されるのですが、
なんかコードの見た目がイヤです。

<p></p>タグとか縦にそろっててほしいです。 ```php <p><?php $test = xprofile_get_field_data('test' ,bp_get_member_user_id()); if( !empty($test) ){ echo $tset; }else{ echo '未入力';}?></p> ```

そこで質問です。
###質問
どのようにすれば、スペースも入らず、コードの見た目も良くなるのでしょうか？
それとも２番目の方がコードの見た目は良いと考えられているのでしょうか？

Comments

[m.ts10806]

xprofile_get_field_data()の出所を書かないと解決につながる回答つきませんよ。単なるPHPだけじゃないですよね、これ。何が影響するか分からないので環境情報は正しく全て提示してください。

[退会済みユーザー]

https://teratail.com/users/matsuzakaq

Answer 1

なげやり

<?php
$test = xprofile_get_field_data('test' ,bp_get_member_user_id());
echo sprintf("<p>%s</p>", empty($test) ? '未入力' : $test);
?>

のようにタグごとだしてしまうとか

テンプレートエンジンの利用（おすすめ）でしょうか

Comments

[退会済みユーザー]

なるほど。そうですね。見やすいです。テンプレートエンジン初耳です。勉強になりました。

[退会済みユーザー]

ところで「%s」というのは、どういう働きをするものなのでしょうか？

[退会済みユーザー]

http://php.net/manual/ja/function.sprintf.php

Answer 2

計算と表示が同じ箇所で実施されているため、見通しの悪い状況になっています。
見やすいコードにするためには分離するのが良いです。
表示箇所では、変数のみ使用するのがきれいです。

<?php
function h($str)
{
return htmlspecialchars($str, ENT_QUOTES, 'UTF-8');
}

$result = '';
$test = xprofile_get_field_data('test' ,bp_get_member_user_id());
if( !empty($test) ){
    $result = $tset;
}else{
    $result = '未入力';
}
?>

<p><?php echo h($result) ?></p>  

もう一歩進めると、テンプレートエンジンの利用となります。

<P>を縦に揃えるのは、CSS で調整が必要なので微妙。

Comments

[退会済みユーザー]

>表示箇所では、変数のみ使用するのがきれい
そういうものでしたか！知りませんでした。ありがとうございます。

Answer 3

テンプレートエンジンを利用してHTMLとPHPを分離するのはどうでしょうか。

Answer 4

<p><?php...と言うような記述も出来るけど、<?php...?>とあるようにphpコード全体がタグであると考えると、<p>タグの内容を出力したい場合は<p>タグ自身もコード内で出力した方が想定外を生みにくい気がする。
ぶっちゃけ<!DOCTYPEから</html>まで全部<?php?>で囲んで出力してしまっても良い訳だし。

Comments

[退会済みユーザー]

なるほど。ありがとうございます。