外部ファイルの読み込みの条件分岐

Question

前提・実現したいこと

素人の質問で恐縮です。
現在外部に予め用意したテキストがあり、それを読み込みたいのですが
一応読み込みは出来ましたが望んでいる箇所が解消ができず困っております。

●環境について
分かる範囲で記載します。
PHPバージョン 7.1
OS windows7
サーバー heteml
ブラウザ Chrome

例
http://example.com/directory/index.php?id=aaa
だった場合、予め用意しているテキストファイル名がaaa.txtのファイルが読み込まれるという形です。

実在しないファイルだっ場合は空のテキストファイルを表示させたいです。

発生している問題・エラーメッセージ

実現しないファイルがあった場合にブラウザ上では空白となるが、ソースを覗くと404のソースが埋め込まれている。

●ソースを覗くについて
こちらはChromeの"view-source:https://example.com を使っています。

該当のソースコード

ちなみに下記の none.txtは空のテキストファイルです。
雑な書き方で恐縮です。そもそもこの書き方であっているかもわかりません。

<?php
	$id = isset($_GET['id']) ? htmlspecialchars($_GET['id']) : null;
	if ($id){$data = file("http://example.com/$id.txt");}
	else {$data = file("http://example.com/none.txt");}
	srand(time());
	foreach ($data as $cont) {
	$text = mb_convert_encoding($cont,"utf8","utf8");
	echo $text;
}
?>

拙い説明で恐縮ですが宜しくお願いします。

Comments

[退会済みユーザー]

こんにちは。
コード拝見しました。
まずインデントを整えましょう！
恐らくみなさん読む気にもなりません。

[m.ts10806]

＞素人の質問で
「初心者アイコン」をつけるだけで良いです

[m.ts10806]

また、ドメインは自身が保有しているものでないのでしたら例示ドメイン(example.comなど)を利用してください。
環境情報も追記してください。PHPのバージョン、導入先のOSなど

[m.ts10806]

あと「ソースを覗くと」とはどういう行為でしょうか。具体的に記載してください

[ken1_takagi]

回答ありがとうございます。

●環境について
分かる範囲で記載します。
PHPバージョン 7.1
OS windows7
サーバー heteml
ブラウザ Chrome

●ソースを覗くについて
こちらはChromeの"view-source:https://example.com"を使っています。
※意味が伝わってますでしょうか？

[m.ts10806]

もろもろ含めて質問本文更新していただければと。

[m.ts10806]

「回答を求めています」はいいのですが、既に質問日に回答していますので、そちらご確認ください。

Answer 1

http://example.com/directory/index.php?id=aaa

だった場合、予め用意しているテキストファイル名がaaa.txtのファイルが読み込まれるという形です。
実在しないファイルだっ場合は空のテキストファイルを表示させたいです。

空のテキストファイルを表示させる必要がわかりませんが、こんな感じでしょうか。

$id = filter_input( INPUT_GET, "id" );
$response = "";
if ($id) {
  $response = @file_get_contents("http://example.com/{$id}.txt");
}
if ($response) {
  $response = mb_convert_encoding($response,"UTF-8");
}
echo $response;

Answer 2

色々と細かいところをツッコミ。

• htmlspecialchars()は出力時に行いましょう。画面に出力する情報をエスケープするものなので、今のように取得時にしてはいけません。

参考：サニタイズ/入力値検証/エスケープの考え方

• 直接$_POSTや$_GETを参照するのではなくfilter_input()を使いましょう

参考：filter_input関数を使って簡潔なコードを書こう

• 「ファイルが存在しない場合」があるのでしたらfile_exits()を利用したほうが良さそうです。PHPが実行されるサーバー内にあるファイルを参照するのでしたら特に。
• file()は失敗したらfalseを返すので念のため結果がfalseかどうか確認してください。

「ブラウザ上では空白となるが」とありますが、エラー表示OFFにしていて空白なのか何も出力指示をしていないから空白なのかパッと見わからないので失敗した場合でも何か出力するようにしてください。

• mb_convert_encoding()使うのはいいのですけど、utf8→utf8って変換してないから意味なさそうです。

もし全角文字列が入ってくるのでしたら対象のテキストファイルとプログラムファイルの文字コードにあわせて適宜対応入れてください。

    $text = mb_convert_encoding($cont,"utf8","utf8");
    echo $text;

//その場で出力するなら変数に受けなくても良いのでは

Answer 3

該当のソースコード

<?php //$_GET['time']が存在していれば if(isset($_GET['id'])){ //idの条件がある場合 //echo 'IDが入力されました<br>'."\n"; //idがあった場合にパスを参照する。パスはサーバーパスを入力。 $filename = '/example/'.$_GET['id'].'.txt'; if (file_exists($filename)) { include($filename); } //ファイルがない場合の記述 日本語表記はダブルクォーテーションの間に入れておく。 else { echo ""; } } //idの条件がない場合 else{ echo 'てきすとなし'; } ?>

Comments

[m.ts10806]

回答も質問と同じくマークダウン使えます