rails のビューファイルで Javascript が動きません

Question

Ruby on rails のscaffold で、ブログサイトのような物を作ろうとしています。
blog>entry>comment とモデルを作り、entry#show のビューファイルにcomment の一覧表が表示されるようにしました。
そのコメントの属性に「status」というものを用意し、「approve」「unapprove」のどちらかに該当するようにします。
ビューファイルから「unapprove」を選択すると「approve」に変わり、「approve」をクリックすると「unapprove」に戻りコメントの内容が非表示になるようにしたい、と思っています。

そこで、以下のように Javascript の「addEventListener」を使って実現しよう、と思ったのですが、Ruby とJavascript のコードが混在するせいか上手く動きません。
どうすればよいでしょうか？

・・・
<table class = "table table-striped">
<thead class = "thead-light">
<%# <th> Title</th> %>
<th> Body</th>
</thead>
<tbody>
<% @entry.comments.each do |comment| %>
<tr>
<td id = "comment">
<%= comment.body %>
</td>
<script type="text/javascript">
$(function_1(){
if (<%= comment.status %> == "unapproved"){
$('td#comment').text("（承認待ち）");
}
});
</script>
<script>
document.getElementByID('status').addEventListener('click',
function(){
<% if comment.status == "approved" %>;
<% comment.status = "unapproved" %>;
<% else %>;
<% comment.status = "approved" %>;
<% end %>;
});
</script>
・・・

Answer 1

全般におかしいです。

• まず、<% ...%>はRuby的に何も出力しませんので、addEventListenerでセットした関数の処理内容は完全に空になります。
• そして、ループ内で<td id = "comment">を生成しているため、同じIDが複数発生してしまいます。
• さらに、document.getElementByIDは、そもそも最後のdが小文字でないといけませんし、それを無視するにしても、同じIDはHTML内で1つですので、全部のイベントが同じエレメントにセットされてしまうことになります。

「ループ内で<script>を生成する」というようなアクロバティックな構造は取らず、HTMLにclassやdata-*で情報を指定して、それをJavaScriptから読み取って制御する、というような形態にしたほうがいいでしょう。

Comments

[potaufew]

ありがとうございます。ただ、「HTMLにclassやdata-*で情報を指定して、それをJavaScriptから読み取って制御する」の箇所でつまづいています・・・。