Pourquoi ma vue avec if ne fonctionnera pas correctement?

Question

Je reçois cette sortie. le nombre affiché en sortie est post.user.id.
La bulle devrait être commutée d'avant en arrière s'il s'agissait d'un identifiant différent.
Mais il juge que c'est la même personne même si ce n'est pas le cas.
Pourquoi?

Courant de sortie

2   Good, you?   > Person B 
1   How have you been? > person A 
2   What's up?   > person B 
Person A < Hello!!!!    1 

Je n'étais pas cette sortie

2   Good, you?   > Person B 
Person A < How have you been?  1 
2   What's up?   > person B 
Person A < Hello!!!!    1 

Voir

<% who = "" %> 
<% @posts.each do |post| %> 

    <tr id="<%= dom_post_id(post) %>"> 

     <% if post.user.id == who %> 
      <td><%= post.user.nickname if post.user %></td> 
      <td><div class="bubble me"><%= post.body %></div></td> 
      <td><%= post.user.id %></td> 
     <% else %> 
      <td><%= post.user.id %></td> 
      <td><div class="bubble you"><%= post.body %></div></td> 
      <td><%= post.user.nickname if post.user %></td>  
     <% end %> 
    </tr> 
    <% who = post.user.id %> 
<% end %> 

Answer 1

Vous devrait factoriser votre code comme ceci:

<% user_id_for_left_side = @posts.first.try(:user_id) %> 
<% @posts.each do |post| %> 

    <tr id="<%= dom_post_id(post) %>">  
    <% if post.user_id == user_id_for_left_side %> 
     <td><%= post.user.nickname %></td> 
     <td><div class="bubble me"><%= post.body %></div></td> 
     <td><%= post.user_id %></td> 
    <% else %> 
     <td><%= post.user_id %></td> 
     <td><div class="bubble you"><%= post.body %></div></td> 
     <td><%= post.user.nickname %></td>  
    <% end %> 
    </tr> 

<% end %> 

Pourquoi utiliser post.user_id au lieu de post.user.id? Car faire post.user_id utilise moins de ressources que post.user.id.

Pourquoi supprimer if post.user? Il est implicite que chaque publication appartient à un utilisateur (vous devez avoir une validation de présence sur le modèle Post pour l'attribut user_id). Cela signifie qu'un poste a toujours un utilisateur associé, pas besoin de vérifier sa présence.

Answer 2

<% who = -1 %> 
<% @posts.each do |post| %> 

    <% if who == -1 %> 
     <% who = post.user.id %> 
    <% end %> 

    <tr id="<%= dom_post_id(post) %>"> 

     <% if post.user.id == who %> 
      <td><%= post.user.nickname if post.user %></td> 
      <td><div class="bubble me"><%= post.body %></div></td> 
      <td><%= post.user.id %></td> 
     <% else %> 
      <td><%= post.user.id %></td> 
      <td><div class="bubble you"><%= post.body %></div></td> 
      <td><%= post.user.nickname if post.user %></td>  
     <% end %> 
    </tr> 

<% end %> 

Le bloc ajouté if fixera le who variable à l'identifiant de l'utilisateur du premier message de la rencontre de la boucle, et ensuite ne pas fixer à nouveau. À chaque itération de la boucle après cela, post.user.id sera who chaque fois que le message aura cet identifiant, et le surnom sera sur la gauche. Pour les messages avec un identifiant différent, le pseudo sera sur la droite.