ボタンが表示されない

Question

前提

応募ボタンとメッセージボタンを条件分岐で表示させようとしていたのですが、
応募ボタンが出現しません。
ifでtrueならメッセージボタンを、falseなら応募ボタンを表示するようにしていたのですが、
応募ボタンだけが表示されません。

該当のソースコード

def show
    if user_signed_in?
    @plan = Plan.find(params[:id])
    @room = Room.all.find_by(plan_id: @plan.id)
    if @room
      @alreadyRoom = true
    else
      @alreadyRoom = false
    end
    
    @currentUserEntry=Entry.where(user_id: current_user.id)
    @userEntry=Entry.where(user_id: @plan.user_id)
    unless @plan.user.id == current_user.id
      @currentUserEntry.each do |cu|
        @userEntry.each do |u|
          if cu.room_id == u.room_id then
            @isRoom = true
            @roomId = cu.room_id
          end
        end
      end
      if @isRoom
      else
        @room = Room.new
        @entry = Entry.new
      end
    end
    end
  end
  
  def new
    @plan = Plan.new
    if @plan.user == current_user
      redirect_to '/'
    end
  end

!インデントの修正10/08/23:01

     <div class="plan-content-btn">
        <% if @isRoom == true %>
          <%= @isRoom %>
          <%= @alreadyRoom %>
          <% if @alreadyRoom == true%>
            <p class="user-show-room"><a href="/rooms/<%= @roomId %>"class="btn btn-primary btn-lg">メッセージ</a>
          <% end %>
        <% else %>
        　<% if @alreadyRoom == false%>
            <%= form_for @room do |f| %>
              <%= fields_for @entry do |e| %>
                <%= e.hidden_field :plan_id, value: @plan.id %>
                <%= e.hidden_field :user_id, value: @plan.user.id %>
              <% end %>
                <%= f.submit "応募", class:"btn btn-primary btn-lg user-show-chat"%>
            <% end %>
          <% end %>
        <% end %>
      </div>

!追記
1.@isRoom, @alreadyRoom = true　：　メッセージボタンを表示
2.@isRoom, @alreadyRoom = false ：　応募ボタンを表示
になっているかを表示させて試してみたのですが、2の条件が@isRoomのほうがtrueと表示されていました。

Answer 1

html のインデントが狂ってるから気が付かないかもですが <%= f.submit "応募", が実行されるのは
@plan.user.id == current_user.id ではなく
かつ @isRoom == true ではなく
かつ @alreadyRoom == false な場合
です。
m.ts10806 さんの言われるように、 @isRoomがtrueなのでは？

この様に（プログラムする人にとって）条件が複雑な場合は頭で考えずに 目で考えられるようにしてみるよ良いです
どういう場合に @isRoom @alreadyRoom が true、false になるのか を整理し、
２ｘ２の４つの マス==組み合わせ 毎に どういう風にしたいのか、を書いて見る。
そのとおりのcodeになっているか 確認する

もしかしたら @plan.user.id == current_user.id も含めて ８通りかも？

Comments

[退会済みユーザー]

インデントの修正とボタン条件の表示の条件を
[e-mail] == current_user.idではない
2.@isRoom == true ではない
3.@alreadyRoom == trueではない
という状態を作成してみたのですが、
Encountered a syntax error while rendering template: check <% if user_signed_in? %>というエラーが発生したのですが、調べたところボタンの箇所の<% else %>が問題っぽいのですが、<% else %>は重ねていいのでしょうか？
コードはインデントを整えて質問の箇所に書き直してみました。

[winterboum]

ということの前に
[e-mail] @isRoom @alreadyRoom が true、false の組み合わせ毎にどうしたいのか を整理
したものを見せてください。
でないと codeのどこが悪いのか、がわかりません

[退会済みユーザー]

すみません。組み合わせ的には
@plan.user.idなし、@isRoom, @alreadyRoom = true　：　メッセージボタンを表示
@plan.user.idなし、@isRoom, @alreadyRoom = false ：　応募ボタンを表示
@plan.user.idなし、@isRoom = tue @alreadyRoom = false ：　何も表示しない
@plan.user.idなし、@isRoom = false @alreadyRoom = rue ：　何も表示しない
というような組み合わせにしたいです。

[退会済みユーザー]

すみません。
もしかしたら([e-mail] == current_user.idではない)の記述はplans/show.html.erbの最初に
<% if user_signed_in? %>
<% if current_user == @plan.user %>
で分岐させており、いらなかったかもしれません。

[winterboum]

>すみません。組み合わせ的には
ということですと、 show.html の方は合っているようにみえます。
ということは、@isRoom, @alreadyRoom が 思う値で渡っていない ということかも

<div class="plan-content-btn">
<% if @isRoom == true %>
の間に @isRoom, @alreadyRoom を表示させてみては？

[退会済みユーザー]

すみません質問でも更新して添付させて頂いたのですが
1.@isRoom, @alreadyRoom = true　：　メッセージボタンを表示
2.@isRoom, @alreadyRoom = false ：　応募ボタンを表示
になっているかを表示させて試してみたのですが、2の条件が@isRoomのほうがtrueと表示されていました。

[winterboum]

ということは、controller の Show を読み直してください。
今度は
「@isRoom, @alreadyRoom の組み合わせでどう表示するか」
ではなく
「@isRoom, @alreadyRoom はそれぞれどういう時にどういう値になるのか」
を整理して、そのとおりになっているか をみてください

これらを決めるのはどういう項目なのか、をまず整理できているか、ですね

[退会済みユーザー]

if cu.room_id == u.room_id && @alreadyRoom thenにしたら
応募ボタンが出現しました。
そして押してみたところ、無事作成でき、これで掲載ごとにDM作成できるようになりました。
ありがとうございます！

Answer 2

コードが一部しかないのでなんとも言えませんが@isRoomがtrueなのでは？

インデントをきちんと整理されたほうが良いと思います。
あと、デバッグを。

Comments

[退会済みユーザー]

plans_controller.rbのshowページのコードを全部載せてみたのですが@isRoomをcontrollerのほうで修正する必要があるということでしょうか？

[m.ts10806]

小手先でどうにかするのではなく一つ一つのロジックに意味を持たせてください。