Q&A
面倒だからと必要な手順を飛ばしていては何も進みませんよ。あと失礼ながら文章が乱雑で要件がくみ取りにくいので、要件を整理して文章を調整して読みやすくしてください。何度も指摘しているかと思いますが、質問は編集することができます。面倒だから回答がついたからと飛ばしていてはどんどん回答もつかなくなります。
前提・実現したいこと
PHP,JavaScriptで日記帳システムを作っています。
【システムの概略】
・「新規作成」ボタンを押すと一行のアコーディオンが生成される。（DBにも生成される。）
・「新規作成」されたアコーディオンはそれぞれ独立しており、「日記のタイトル」テキストボックス
をクリックしても該当の「日記の内容」しかアコーディオンが開閉されない。
・「Delete」ボタンを押すと、該当の行のアコーディオンをDBから消す。かつ画面上からも消す。
【問題点】
「新規作成」ボタンを実装中に以下の問題が発生しました。
「新規作成」を押すと変数iで新規アコーディオンが追加されます。
（※JavaScriptだと変数iが一時的にしか保持されず、ブラウザを再度立ち上げて同じことを行おうとすると、再度i = 1からになり、うまくいきそうにありません。）
【解決に向けて考えていること】
PHPのセッション変数をiに格納してみようとしました。
しかし、スクリプトの中身にPHPを入れると「新規作成」ボタンが動かなくなります。
コードのどこが間違っているのでしょうか？
####問題の個所を部分的に抜粋しました↓
<script> function add() { var div_element = document.createElement("div"); <?php $_SESSION['i'] = 1; ?> div_element.innerHTML = '<div id="accordion"><div class="card"><div class="card-header" id="headingOne"><h5 class="mb-0"><button class="btn btn-link" data-toggle="collapse" data-target="#collapse' + i +'" aria-expanded="true" aria-controls="collapse' + i +'"><textarea name="title" class="form-control" rows="1" maxlength="20" placeholder="日記のタイトルを入力"></textarea></button><div class="float-right"><a href="" class="btn btn-primary btn-lg active" role="button" aria-pressed="true">Delete</a></div></h5></div><div id="collapse' + i +'" class="collapse" aria-labelledby="headingOne" data-parent="#accordion"><div class="card-body"><div class="container"><form method="post">内容<textarea name="contents" class="form-control" maxlength="80" placeholder="日記の内容を入力"></textarea></form></div></div></div></div></div>'; var parent_object = document.getElementById("piyo"); parent_object.appendChild(div_element); <?php $_SESSION['i'] = $_SESSION['i'] + 1; ?> } </script>
発生している問題・エラーメッセージ
エラーメッセージはありません（Eclipseで検証済）
該当のソースコード
<!doctype html> <html lang="ja"> <head> <!-- Required meta tags --> <meta charset="utf-8"> <meta name="viewport" content="width=device-width, initial-scale=1, shrink-to-fit=no"> <!-- Bootstrap CSS --> <link rel="stylesheet" href="https://stackpath.bootstrapcdn.com/bootstrap/4.1.3/css/bootstrap.min.css" integrity="sha384-MCw98/SFnGE8fJT3GXwEOngsV7Zt27NXFoaoApmYm81iuXoPkFOJwJ8ERdknLPMO" crossorigin="anonymous"> <title>みんなの日記帳</title> </head> <body> <h1>みんなの日記帳</h1> <!--新規作成ボタンを左上に配置 <form method="post"> <button type="submit" name = "new_column" class="btn btn-primary" value="1">新規作成</button><hr> </form>--> <!--新規作成ボタンを左絵上に配置その２--> <div> <button onclick="add();" name="new_column" class="btn btn-primary" value="1">新規作成</button> </div> <!--HTML自動生成（BootStrapで）--> <script> function add() { var div_element = document.createElement("div"); var i = 1; div_element.innerHTML = '<div id="accordion"><div class="card"><div class="card-header" id="headingOne"><h5 class="mb-0"><button class="btn btn-link" data-toggle="collapse" data-target="#collapse' + i +'" aria-expanded="true" aria-controls="collapse' + i +'"><textarea name="title" class="form-control" rows="1" maxlength="20" placeholder="日記のタイトルを入力"></textarea></button><div class="float-right"><a href="" class="btn btn-primary btn-lg active" role="button" aria-pressed="true">Delete</a></div></h5></div><div id="collapse' + i +'" class="collapse" aria-labelledby="headingOne" data-parent="#accordion"><div class="card-body"><div class="container"><form method="post">内容<textarea name="contents" class="form-control" maxlength="80" placeholder="日記の内容を入力"></textarea></form></div></div></div></div></div>'; var parent_object = document.getElementById("piyo"); parent_object.appendChild(div_element); i = i + 1; } </script> <div id="piyo"> </div> <!--アコーディオン処理--> <div id="accordion"> <div class="card"> <div class="card-header" id="headingOne"> <h5 class="mb-0"> <button class="btn btn-link" data-toggle="collapse" data-target="#collapseOne" aria-expanded="true" aria-controls="collapseOne"> <textarea name="title" class="form-control" rows="1" maxlength="20" placeholder="日記のタイトルを入力"></textarea> </button> <div class="float-right"> <a href="" class="btn btn-primary btn-lg active" role="button" aria-pressed="true">Delete</a> </div> </h5> </div> <div id="collapseOne" class="collapse" aria-labelledby="headingOne" data-parent="#accordion"> <div class="card-body"> <div class="container"> <form method="post"> 内容<textarea name="contents" class="form-control" maxlength="80" placeholder="日記の内容を入力"></textarea> </form> </div> </div> </div> </div> </div> <?php if (isset($_POST["new_column"])) { //$kbn = htmlspecialchars($_POST["new_column"], ENT_QUOTES, "UTF-8"); session_start(); $link = mysqli_connect("localhost","root","root","diary"); if(mysqli_connect_error()){ die("DBへの接続失敗！"); } } ?> <!-- Optional JavaScript --> <!-- jQuery first, then Popper.js, then Bootstrap JS --> <script src="https://code.jquery.com/jquery-3.3.1.slim.min.js" integrity="sha384-q8i/X+965DzO0rT7abK41JStQIAqVgRVzpbzo5smXKp4YfRvH+8abtTE1Pi6jizo" crossorigin="anonymous"></script> <script src="https://cdnjs.cloudflare.com/ajax/libs/popper.js/1.14.3/umd/popper.min.js" integrity="sha384-ZMP7rVo3mIykV+2+9J3UJ46jBk0WLaUAdn689aCwoqbBJiSnjAK/l8WvCWPIPm49" crossorigin="anonymous"></script> <script src="https://stackpath.bootstrapcdn.com/bootstrap/4.1.3/js/bootstrap.min.js" integrity="sha384-ChfqqxuZUCnJSK3+MXmPNIyE6ZbWh2IMqE241rYiqJxyMiZ6OW/JmZQ5stwEULTy" crossorigin="anonymous"></script> </body> </html>
試したこと
PHPのセッション変数をJavaScriptのfunction add()関数の中に入れてみようとしましたが記述方法が間違っているのか、「新規作成」ボタンを押しても何も動作しなくなりました。
補足情報（FW/ツールのバージョンなど）
エディタ：Brackets & Eclipse
OS:windows10
よろしくお願いいたします。
また、結局丸投げがひどくなっているようにも思うので、もっと考えてプログラムを組んでください。要件の細分化ができていれば初学者であってもそこまで詰まるような内容には思えません。
ご回答、ありがとうございます。少し、お時間をください。
どういう構成でどうPOSTしているのでしょう？直感的には新規追加で入力欄を作るとどうじにｄｂに入るとは思えないのですが
ご回答ありがとうございます。まだPOSTの部分については未完成のため、これから実装する予定です。
ちなみにここは回答コメントではありません。質問内容に不足があったり不明瞭なところがあったり、見ている人からの確認事項だったり 回答するために必要な情報を「質問本文に追記してもらうための」通信欄（指摘事項）のために使われるのが通例です。なので、基本的には質問本文を編集して返してください。
承知いたしました。
回答2件
0
たぶんサーバー側で動作するPHPとクライアント側で動くhtml、CSS、JavaScriptの切り分けができていないように思うので、そのあたりをしっかり理解するところから始めたほうが良いです。
「なんとなく」「ナァナァで」やっていても余計ゴチャゴチャになるだけで成長にはつながりません。
ここ何回かの質問での進捗を見ればそれは明らかでしょう。
例えば下記のようなコード、どのような（順番で）実行されて画面に出力されるかわかりますか？
php
1<p> 2test 3<?php echo "test";?> 4</p>
PHPはサーバー側で実行されるものです。
実行した内容で何かしらechoしていたとして、それがhtmlのような文字列であったとして、
クライアント側であるWebブラウザから見ることでそのhtmlのような文字列を「HTML」として初めて解釈し、
htmlのような文字列はhtmlとしての機能を発揮できるわけです。
つまり、順番的にはどのように書いてもPHPが先に実行されている…と言っても過言ではありません。
echo やprintなど画面に出力する指示を書く場所さえ間違っていなければ、画面出力はOKです。
よく、.phpながらhtmlがずらーっと書いてあって途中で急に<?php と入り、そこの長々と処理が書いてあって最後にechoが1行書いてあるようなものをよく見ますが、htmlとして見たときに非常に読みづらいので私は好きではありません。
foreachで出力しているのならまだいいですが、何回もDB接続したり切断したり計算の処理が書いてあったり。
そういったものはたいていhtmlが想定通りできていなくてそこで詰まっています。
インデントがついていないコードに似ています。
今後もしフレームワークを使うようになってきて処理系と画面出力系の処理（コントローラとビュー）がわかれているようなもの、オブジェクト指向になってくるとおそらく今のままだとかなりの確率でアウトになると思われます。
というのを前提として、上記、ちゃんと理解されたら進んでください。
さて。
ここまでの内容が理解できていたら「試したこと」でなぜ「何も動作しなくなった」のかが見えてくるかと思います。
htmlの中に埋め込まれたJavaScriptはブラウザで解釈させる前はもともと何でしょうか？
では、それはブラウザで解釈できるような内容になっていますか？
前提部分で私は「htmlが想定通りにできていなくて詰まっている」と書きました。
ブラウザの「ソースを表示」で「正しい文章になっているか」確認しましたか？
ブラウザ開発ツールのコンソールでエラーがでていないか確認しましたか？
もしかしたら以前の質問にも書いたことがあるかもしれません。
初学者が「Hello World」ができたあとに最優先で覚えるべきは「デバッグ」です。
おおよそどの言語にもデバッグの仕方はあります。なぜか。
「自分自身で原因究明できるようにするため」と私は考えています。
teratailでもほぼ丸投げの質問に「デバッグしましたか？」と指摘や確認が入ることはよくあります（私はよく入れます）
それは「自身で確認できることがまだある段階ですよ」というのを伝えるためでです。
「面倒だから」とデバッグを抜かしてませんか？
よく考えてみてください。自分で解決できたらそれが一番早いはずです。
質問作るのにどれくらい時間がかかっているかわかりませんが、文量からすると、自分で解決できた方が確実に早い内容です。
ご検討ください。
投稿2018/10/03 11:33
総合スコア80875
0
div_element.innerHTML に下記HTMLを挿入しようとしていますが、文法違反が多数あります。
例えば、idは重複してはいけないにもかかわらずonclickのたびに id="accordion" が増えていきます。他にも、button要素はtextarea要素を内包できません。
そもそも、formの位置的に「日記のタイトル」はPHP側に送られません。
HTML
1 <div id="accordion"> 2 <div class="card"> 3 <div class="card-header" id="headingOne"> 4 <h5 class="mb-0"><button aria-controls="collapse' + i +'" aria-expanded="true" class="btn btn-link" data-target="#collapse' + i +'" data-toggle="collapse"> 5 <textarea class="form-control" maxlength="20" name="title" placeholder="日記のタイトルを入力" rows="1"></textarea></button></h5> 6 <div class="float-right"> 7 <h5 class="mb-0"><a aria-pressed="true" class="btn btn-primary btn-lg active" href="" role="button">Delete</a></h5> 8 </div> 9 </div> 10 <div aria-labelledby="headingOne" class="collapse" data-parent="#accordion" id="collapse' + i +'"> 11 <div class="card-body"> 12 <div class="container"> 13 <form method="post"> 14 内容 15 <textarea class="form-control" maxlength="80" name="contents" placeholder="日記の内容を入力"></textarea> 16 </form> 17 </div> 18 </div> 19 </div> 20 </div> 21 </div>
【<button>: ボタン要素 - HTML: HyperText Markup Language | MDN】
https://developer.mozilla.org/ja/docs/Web/HTML/Element/button
JavaScript側でも、var i = 1;の位置が関数の中にあるせいで、何度押しても #collapse1 を含むHTMLを出力してしまう仕様になっています。
いろいろなことを一度に行わず、単純な物を積み重ねることが理解につながると思います。
- 単純なHTMLにform要素とtextarea要素を置き、PHPでその値を受け取って表示する
- 受け取った物をDBに保存し、新しい順に取り出して表示する
- それぞれの日記を削除できるボタンを設置し、PHP側でDBから削除する（ここまでJavaScriptを一切使わない）
- 「画面移動なしの設計」を勉強する
- JavaScriptで項目の追加／削除を出来るようにする（データの保存は後まわし）
- 勉強した知識で実装
投稿2018/10/03 13:53
総合スコア69597
IDの重複についてはこの質問者さんはこれまでも回答いただいてますね。
大事なことであるはずですが面倒なのかうまくできないからか後回しにしてしまう癖があるように思います・・・
あなたの回答
tips
プレビュー
