ｐｈｐ出力時にデータが重複する問題について

Question

参照元ファイルのデータを出力したいと思っています。
ところが。
下記のファイル構成では、出力結果がこうなってしまうのです。

【出力結果】
一部表示するときだけのやつその１
一部表示するときだけのやつその１
一部表示するときだけのやつその２

上のような「一部表示するときだけのやつその１」の重複をなくしたいのですが、下記のファイル構成では何がいけないか…ご指摘頂けませんでしょうか？

【参照元ファイル】

<?php if ($class_zenbu == 'zenbu'): ?>
<span class="zenbu">
全部表示するときだけのやつその１
</span>
<?php endif; ?>

<?php if ($class_ichibu1 == 'ichibu1'): ?>
<span class="ichibu">
一部表示するときだけのやつその１
</span>
<?php endif; ?>

<?php if ($class_zenbu == 'zenbu'): ?>
<span class="zenbu">
全部表示するときだけのやつその２
</span>
<?php endif; ?>

<?php if ($class_ichibu2 == 'ichibu2'): ?>
<span class="ichibu">
一部表示するときだけのやつその２
</span>
<?php endif; ?>

<?php if ($class_zenbu == 'zenbu'): ?>
<span class="zenbu">
全部表示するときだけのやつその３
</span>
<?php endif; ?>

【出力ファイル】

<?php
$class_ichibu1 = 'ichibu1';
include('nakami.php');
?>

<?php
$class_ichibu2 = 'ichibu2';
include('nakami.php');
?>

よろしくお願い致します。<(_ _)>

Answer 1

同じPHPスクリプト: nakami.php を2回インクルード(＝2回実行)してるんですから、2回出力されるのは当たり前だと思うのですが･･･？

シチュエーションがわからないのですが、1回だけ include するのはだめなんでしょうか？

$class_ichibu1 = 'ichibu1';
$class_ichibu2 = 'ichibu2';
include('nakami.php');

またいただいたコードをを実行すると大量の Notice エラーが出ています。開発環境では php.ini を編集して必ずエラーをすべて表示する設定にしましょう。

PHPエラー出力レベルを php.iniに設定する - Qiita

もしくはPHPコード内で error_reporting() を実行し、一時的にエラーを表示させることも可能です。

<?php
error_reporting(-1);
$class_ichibu1 = 'ichibu1';
$class_ichibu2 = 'ichibu2';
include('nakami.php');

Comments

[退会済みユーザー]

ありがとうございます。

>1回だけ include するのはだめなんでしょうか？
実は出力ファイルにはすでにテーブルがあり、そのtdの中にたまに「一部表示するときだけのやつその１」などを表示したいのです。なので、１回にまとめることができなそうでして。。

>2回出力されるのは当たり前
そうなのですね。笑
初期化という処理が必要だと別の方からご教示いただいたので、今そのやり方を調べています。

>大量の Notice エラー
おお、なんということでしょう。。
これは初期化とやらをすれば解決できるのでしょうか？

ま、なんにせよ、初期化というのにチャレンジしてみますね(^_-)-☆

[退会済みユーザー]

下記のように【出力ファイル（初期化版）】によって質問は解決できました！！

ところで、ご提示のエラー出力がされないようなのですが、どちらを見ればいいのでしょうか？
開発ツールを見てもわかりませんでした。
尚、Qiitaの方は難しくてやっておりません。

【出力ファイル（初期化版）】
<?php
error_reporting(-1);
$class_ichibu1 = 'ichibu1';
include('nakami.php');
$class_ichibu1 = array();
?>

<?php
error_reporting(-1);
$class_ichibu2 = 'ichibu2';
include('nakami.php');
$class_ichibu2 = array();
?>

[miyahan]

> tdの中にたまに「一部表示するときだけのやつその１」などを表示したい
なるほど。そういうときは、やりたいことそのもの（例：条件に応じてメッセージを表示させる処理を共通化したい）を書いて質問したほうが「こうした方がいいんじゃない？」という回答がつきやすいと思います。

> これは初期化とやらをすれば解決できるのでしょうか？
はい。変数は必ず初期化(宣言)するようにしてください。常時 notice エラーが出ないように作っておけば、もしスペルミス等をしてしまったときにエラーが出てバグにいち早く気づくことが出来ます。

> ご提示のエラー出力がされない
"PHP Notice: Undefined variable: class_zenbu " みたいなエラーが表示されませんか？？

[退会済みユーザー]

仰るとおりですね。素人が下手にポイントを切り取って質問しちゃダメですね。笑

スペルミスに気づけるのは心強いです！いつもそれで数時間経っています。変数の初期化とともに覚えておきます。ありがとうございます。

エラーはやはり表示されないようですね。出力されたHTMLと、開発ツールのコンソールとを見てみましたが、Noticeという文字は検索できませんでした。(・・?

まぁ、そういうこともあります、、かね。笑

[miyahan]

じゃあもう一行追加してみてくれますか？

error_reporting(-1);
ini_set("display_errors", 1);

ただこれを全部のPHPファイルに書くのは大変ですし、本番環境ではOFFにする必要がある（エラーが他人に見えてしまうといろいろとマズい）ので、できれば php.ini を編集する方法を検討してみてください。

[退会済みユーザー]

むう、特に変わった様子はないですね？不思議です。
念のためぜんぶ書いておきます。

■参照元ファイル（初期化版）
<?php if ($class_zenbu == 'zenbu'): ?>
<span class="zenbu">
全部表示するときだけのやつその１
</span>
<?php endif; ?>

<?php if ($class_ichibu1 == 'ichibu1'): ?>
<span class="ichibu">
一部表示するときだけのやつその１
</span>
<?php $class_ichibu1='';　?>
<?php endif; ?>

<?php if ($class_zenbu == 'zenbu'): ?>
<span class="zenbu">
全部表示するときだけのやつその２
</span>
<?php endif; ?>

<?php if ($class_ichibu2 == 'ichibu2'): ?>
<span class="ichibu">
一部表示するときだけのやつその２
</span>
<?php $class_ichibu2='';　?>
<?php endif; ?>

<?php if ($class_zenbu == 'zenbu'): ?>
<span class="zenbu">
全部表示するときだけのやつその３
</span>
<?php endif; ?>

■出力ファイル（エラー版）
<?php
error_reporting(-1);
ini_set("display_errors", 1);
$class_ichibu1 = 'ichibu1';
include('nakami.php');
?>
<?php
error_reporting(-1);
ini_set("display_errors", 1);
$class_ichibu2 = 'ichibu2';
include('nakami.php');
?>

[退会済みユーザー]

ちなみにwordpressです。まぁなんとかなったので、あまりお気になさらずです☆たびたびのご返信、ありがとうございます！
今回は直接的な解決に結びついた「初期化せよ」とご指摘頂いたmotuoさんをベストアンサーにさせて頂きます。（いろいろ改善案くださったのにスミマセン！）

Answer 2

$class_ichibu1に'ichibu1'という値が入ったまま、

<?php
$class_ichibu2 = 'ichibu2';
include('nakami.php');
?>

が呼ばれているのが原因です。

例えば、nakami.phpの最後の処理で変数内を初期化するなどの対応をしましょう。

Comments

[退会済みユーザー]

ありがとうございます。チャレンジしてみます♪

[退会済みユーザー]

できました♪♪
こうですね！

<?php
$class_ichibu1 = 'ichibu1';
include('nakami.php');
$class_ichibu1 = array();
?>

<?php
$class_ichibu2 = 'ichibu2';
include('nakami.php');
$class_ichibu2 = array();
?>

初期化、おぼえておきます。<(_ _)>

[motuo]

悪くはないのですが…$class_ichibu1がStringに見えるので$class_ichibu1=''が良いのでは？
個人的にはnakama.phpでまとめて後処理した方が、あちこちにロジックが分散しないので良いと思います。
あと、miyahanさんが指摘している通り、このままでは$class_zenbuが定義されていない場合(いきなりifで)、 Undefined variableとしてNoticeが出ます。
そもそも、この作りが良いのかどうかは分かりませんが、issetを使うなどしてその値が存在しているかどうかnakama.php側でチェックした方が良いかもしれません。

[退会済みユーザー]

おお、そうなのですね！ありがとうございます。

どれどれ…
…なるほど。nakami.phpでこのように↓書いても、重複は起きませんでした。こちらにまとめておくようにします。

<?php if ($class_ichibu1 == 'ichibu1'): ?>
<span class="ichibu">
一部表示するときだけのやつその１
</span>
<?php $class_ichibu1='';　?>
<?php endif; ?>

あれま。ifってこういう使い方じゃないのですね。うーん(*´Д｀)
調べてみます！

[motuo]

いや、ifの使い方は合っています。ただ、$class_zenbuが設定されていない場合($class_zenbu=''とかの初期化が、一切されて無い)、Undefined variableとしてNoticeが出ます。というだけのお話しです。

[退会済みユーザー]

あ、なるほど。５つ全て初期化しないといけないのですね！
miyahanさんへのコメントでは「一部のやつ」の２つしか初期化していなかったので、「全部のやつ」の３つもやっておきます。
何度もお付き合いいただきありがとうございます!!<(_ _)>