1

以下の if else ステートメントを短縮するにはどうすればよいですか。

現在、2 つのステートメントの唯一の違いは最初の行です。accordion-body および accordion-body.collapse.

- if @office.to_i == office.id
        .accordion-body{id: "collapse-#{index}"}
          .accordion-inner 
            - office.positions.each_with_index do |position, index|
              #measures-accordion.accordion
                .accordion-group
                  .accordion-heading
                    .row-fluid
                      .span2{class: "offices-#{position.status}"}= position.status
                      .span1= position.results.first.value
                      .span2= position.results.first.value_date_time.blank? ? '' : Date.parse(position.results.first.value_date_time)
                      .span5= position.name
      - else
        .accordion-body.collapse{id: "collapse-#{index}"}
          .accordion-inner 
            - office.positions.each_with_index do |position, index|
              #measures-accordion.accordion
                .accordion-group
                  .accordion-heading
                    .row-fluid
                      .span2{class: "offices-#{position.status}"}= position.status
                      .span1= position.results.first.value
                      .span2= position.results.first.value_date_time.blank? ? '' : Date.parse(position.results.first.value_date_time)
                      .span5= position.name

上記のコードは機能します。しかし、それを以下に短縮すると、私のロジックは機能しません

    - if @office.to_i == office.id
      .accordion-body{id: "collapse-#{index}"}
    - else
      .accordion-body.collapse{id: "collapse-#{index}"}
      .accordion-inner 
        - office.positions.each_with_index do |position, index|
          #measures-accordion.accordion
            .accordion-group
              .accordion-heading
                .row-fluid
                  .span2{class: "offices-#{position.status}"}= position.status
                  .span1= position.results.first.value
                  .span2= position.results.first.value_date_time.blank? ? '' : Date.parse(position.results.first.value_date_time)
                  .span5= position.name
4

1 に答える 1

2

繰り返されるコードをパーシャルに移動することをお勧めしますが、それなしで実行したい場合は、次のことを試してください。

    .accordion-body{id: "collapse-#{index}", class: (@office.to_i == office.id) ? "" : "collapse" }
      .accordion-inner 
        - office.positions.each_with_index do |position, index|
          #measures-accordion.accordion
            .accordion-group
              .accordion-heading
                .row-fluid
                  .span2{class: "offices-#{position.status}"}= position.status
                  .span1= position.results.first.value
                  .span2= position.results.first.value_date_time.blank? ? '' : Date.parse(position.results.first.value_date_time)
                  .span5= position.name
于 2013-03-04T17:36:17.557 に答える