javascriptとPHPで同じ文字列なら関数を実行しないようにしたい

Question

PHPでmysqlにあるデーターを取得します

mysqlの中身(5つのレコードがあります)
果物
ゲーム
果物
ゲーム
ゲーム

PHPのforeachですべて取得する後その個数分だけhtmlのselectのoption属性を増やしたいのですが
その際に同じ文字列(ゲームならゲーム)をの時だけoptionを増やしたくないです

その為if文で文字列が一緒なら実行しないにしているのですが文字列が一緒じゃなくてもoptionが追加されません
そもそもPHP内で<<<EOMを使ってjavascriptのIF文は使えないのでしょうか...

 <select name="type1" id="type1" class="type1" onchange="categoliOption()">
  <option value="not">選択してください</option
   </select>

   <?php

ini_set('display_errors', "On");

$stmt = $dbh->prepare('SELECT * FROM type WHERE id=?');
$stmt->bindValue(1,$id,PDO::PARAM_INT);
$stmt->execute();

foreach($stmt as $rec){

$type1 = $rec["type1"];
$type1 = json_encode($type1);


$stt = <<<EOT

<script type="text/javascript">

var type1 = $type1; 



if(type1 == lastType1 ){ 



}else{

  function addOption1() {
    var select = document.getElementById("type1");
    var option = document.createElement("option");
    option.text = type1;
    option.value = type1;
    select.appendChild(option);
    
    var lastType1 = type1;   <------lastType1変数にoptionを追加した時のtype1を入れて次回のforeachのif文で使用する
  addOption1();
 

}


}

</script>

EOT;

echo $stt;
}

?> 

Comments

[KoichiSugiyama]

HTML部分はphpのファイル内に書かれていないのでしょうか？

[nguyenseiji]

はい
PHP領域の外になります

[m.ts10806]

＞javaacriptとPHPで同

「javaacript」というものはありません。
そういうところを雑にしてしまうとまともに動くプログラムは出来ませんよ。本当に。

[thyda.eiqau]

ご提示のHTML (<select>部分) とPHPは別ファイルという意味ですよね。HTMLファイルはどのようにPHPファイルを読み込んでいるのですか？

[m.ts10806]

<option>も正しく閉じれていませんし、まともなHTML作れない状態では動くはずがないです。

[nguyenseiji]

ご指揮的ありがとうございます

スペルミスが多いのは自覚しています...
気お付けます

[m.ts10806]

「スペル」という認識だからでは？
あくまで「そういう文字列の羅列」ですよ。
間違いない手順が癖づかないままでは同じ間違いをおかしつづけます。
「気を付ける」では改善は見込めません（これまでの質問の傾向から、一切それが見られない）

[m.ts10806]

指摘うけてなお、修正されないので、「気を付ける」というのも言葉だけなのかなと。

[m.ts10806]

タグからコピペすれば確実と思いませんか。まぁそれでいいと思うならそのやり方続けたらいいです。手でやることが増えれば増えるほど不具合は増えるものです。

Answer 1

解決されたようですが、要点はそれでいいとしても提示された範囲のコードを見るかぎり (JavaScriptの実装内容も含めて) 本当にそれで大丈夫か？と思わなくもないので、その範囲をカバーするコードを提示しておきます。

<?php
$pdo = new PDO('sqlite::memory:', '', '', [
    PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION,
]);
$pdo->exec("
    CREATE TABLE type(id INTEGER, type1 TEXT);
    INSERT INTO type values(1, '果物');
    INSERT INTO type values(1, 'ゲーム');
    INSERT INTO type values(1, '果物');
    INSERT INTO type values(1, 'ゲーム');
    INSERT INTO type values(1, 'ゲーム');
");
$stmt = $pdo->prepare('SELECT DISTINCT type1 FROM type WHERE id=?');
$stmt->bindValue(1, 1, PDO::PARAM_INT);
$stmt->execute();
// ここまで検証用コード

// ここから実装部分
$rows = $stmt->fetchAll(PDO::FETCH_COLUMN, 0); // この 0 は実際の環境によって違う
$json = json_encode($rows);

$stt = <<<EOT
<script>

JSON.parse(`{$json}`).forEach(val => {
  let option = document.createElement('option');
  option.text = val;
  option.value = val;
  document.getElementById('type1').appendChild(option);
});

</script>
EOT;
echo $stt;

?>

ただ、デザインとしては、HTMLとPHPが別れているのであれば、PHPはたんにAPIとして実装して、HTMLファイル側からAjaxで取得するようにしたほうが綺麗だと思います。

<select name="type1" id="type1" class="type1" onchange="categoliOption()">
  <option value="not">選択してください</option>
</select>
<script>
fetch(`getType1.php?id=${id}`).then(res => res.json()).then(json => {
  json.forEach(val => {
    let option = document.createElement('option');
    option.text = val;
    option.value = val;
    document.getElementById('type1').appendChild(option);
  });
});
</script>

<?php
// getType1.php
header('Content-Type: application/json');

$id = filter_input(INPUT_GET, 'id', FILTER_VALIDATE_INT);
if($id === false) {
  http_response_code(400);
  exit(json_encode([
    'message' => '`id` is not found in query or invalid format.',
  ]));
}

$pdo = new PDO('sqlite::memory:', '', '', [
    PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION,
]);
$pdo->exec("
    CREATE TABLE type(id INTEGER, type1 TEXT);
    INSERT INTO type values(1, '果物');
    INSERT INTO type values(1, 'ゲーム');
    INSERT INTO type values(1, '果物');
    INSERT INTO type values(1, 'ゲーム');
    INSERT INTO type values(1, 'ゲーム');
");
$stmt = $pdo->prepare('SELECT DISTINCT type1 FROM type WHERE id=?');
$stmt->bindValue(1, 1, PDO::PARAM_INT);
$stmt->execute();
// ここまで検証用コード

// ここから実装部分
$rows = $stmt->fetchAll(PDO::FETCH_COLUMN, 0); // この 0 は実際の環境によって違う
echo json_encode($rows);

Answer 2

よくわからないけど
SELECT DISTINCT type1 FROM type WHERE id=?

Comments

[nguyenseiji]

DISTINCT で解決しました

ありがとうございます!
また
EOT;
echo $stt;
をforeachループの外に記載していました。。。