Как добавить разные классы блокам?

Question

[lavezzi1]

Здравствуйте!
Возможно ли сделать так, что бы в конечном коде, добавлялся определенный класс к отправителю и получателю, чтобы стилизовать по разному эти блоки. Спасибо.

.wrapper
  .container     
    - if @over_ten
      = link_to 'Show Previous', '?m=all'
    .ui.segment
      - @messages.each do |message|
        - if message.body
          - user = User.find(message.user_id)
          .item
            .content
              .header
                %strong= user.first_name
                = message.message_time
              .list
                .item
                  %i.right.triangle.icon
                  = message.body
    = form_for [@conversation, @message], html: {class: "ui reply form"} do |f|
      .field
        = f.text_area :body, class: "form-control"
      = f.text_field :user_id, value: current_user.id, type: "hidden"
      %div
        = f.submit "Add Reply", class: "ui blue labeled submit icon button"
      %div
        = link_to "Delete conversation", conversation_path(@conversation), :method => :delete, :data => { :confirm => 'Are you sure?' }

Answer 1

[thepry]

определенный класс к отправителю и получателю

Отправитель - это message.user, а получатель кто?

- @messages.each do |message|
- if message.body
- user = User.find(message.user_id)

Так делать очень плохо.

Comments

[lavezzi1]

У меня есть отправитель и получатель. Вот так выглядят таблицы:
class CreateConversations < ActiveRecord::Migration
def change
create_table :conversations do |t|
t.integer :sender_id
t.integer :recipient_id
t.timestamps
end
end
end

class CreateMessages < ActiveRecord::Migration
def change
create_table :messages do |t|
t.text :body
t.references :conversation, index: true
t.references :user, index: true
t.boolean :read, :default => false
t.timestamps
end
end
end

[thepry]

lavezzi1: Не ясно где они на странице отображаются, чтобы им стили добавлять. Типа .message style="#{ message.user == message.conversation.sender ? 'sender' : 'recipient'}"

[lavezzi1]

thepry: Я понимаю, что так делать плохо, но знания на данный момент, позволяют писать только так) Возможно вы мне поможете пепеписать этот блок?

[lavezzi1]

thepry: Единственное, что у меня получилось это вот -
.item{:class => ("sender" if message.user == message.conversation.sender)}
Впринципе и этого достаточно, но хотелось бы вас послушать.

[lavezzi1]

thepry: Все, решился вопрос)
%div{:class => (message.user == message.conversation.sender ? 'sender' : 'recipient')}

Но я все равно хотел бы узнать у вас как переписать тот код лучше.

[thepry]

lavezzi1: Во-первых, стоит использовать связи belongs_to в сообщении и получать пользователя так: message.user. Во-вторых, у тебя на каждое сообщение создается запрос к базе данных для получения пользователя - это называется «n+1 problem» - вот тут есть примеры решения guides.rubyonrails.org/active_record_querying.html

Answer 2

[Станислав]

.item class=(условие ? 'sender' : 'recipient' )
На выходе будет