Rails - Active Record "RuntimeError: не может изменить замороженную строку", как-то связанный с конструктором форм? - PullRequest
3 голосов
/ 28 февраля 2012

Когда я отправляю форму для обновления моих моделей, я получаю эту ошибку в браузере:

ActiveRecord::StatementInvalid in CoachesController#update

RuntimeError: can't modify frozen String: INSERT INTO "availabilities" ("coach_id", "created_at", "day", "hour", "updated_at") VALUES (?, ?, ?, ?, ?)

Консоль рельсов говорит следующее:

(0.0ms)  begin transaction
Binary data inserted for `string` type on column `day`
SQL (0.5ms)  INSERT INTO "availabilities" ("coach_id", "created_at", "day", "hour", "updated_at") VALUES (?, ?, ?, ?, ?)  [["coach_id", 14], ["created_at", Mon, 27 Feb 2012 21:59:05 UTC +00:00], ["day", "Monday"], ["hour", 20], ["updated_at", Mon, 27 Feb 2012 21:59:05 UTC +00:00]]
RuntimeError: can't modify frozen String: INSERT INTO "availabilities" ("coach_id", "created_at", "day", "hour", "updated_at") VALUES (?, ?, ?, ?, ?)
(0.1ms)  rollback transaction
Completed 500 Internal Server Error in 25ms

ActiveRecord::StatementInvalid (RuntimeError: can't modify frozen String: INSERT INTO "availabilities" ("coach_id", "created_at", "day", "hour", "updated_at") VALUES (?, ?, ?, ?, ?)):
app/models/coach.rb:93:in `block (2 levels) in update_general_availability'
app/models/coach.rb:92:in `each'
app/models/coach.rb:92:in `block in update_general_availability'
app/models/coach.rb:91:in `each'
app/models/coach.rb:91:in `update_general_availability'
app/controllers/coaches_controller.rb:25:in `update'

После долгих экспериментов я нашел, как обойти эту ошибку, но не то, почему я ее получаю.

У меня есть две модели: Coach и Availabilities, с has_many и belongs_to ассоциациями. Это схема для таблицы доступности:

# Table name: availabilities
#  id         :integer         not null, primary key
#  coach_id   :integer
#  day        :string(255)
#  hour       :integer

Он хранит день недели и час в течение дня, когда тренер свободен.

Я написал два метода в модели Coach, чтобы легче было справляться с еженедельной доступностью тренера. Они используют вложенную хэш-таблицу, поэтому вы можете узнать, свободен ли тренер в данный момент времени. (Напр .: general_availability["Thursday"]["12"] #=> true)

#coach.rb
class Coach < ActiveRecord::Base
  ...

  # Creates a hash table mapping day and hour to true if available then, false otherwise
  # Form is general_availability["day"]["hr"]. Per Availability model, "0" = midnight, and
  # day of the week is of the form "Monday" or "Tuesday".
  def general_availability
    h = Hash.new()
    %w(Monday Tuesday Wednesday Thursday Friday Saturday Sunday).each { |day| h[day] = Hash.new(false) }

    self.availabilities.each do |a|
      h[a.day][a.hour.to_s] = true
    end

    return h
  end

  # Takes a hash table of the kind returned by general_availability and updates
  # this coach's records in the Availabilities table
  def update_general_availability(ga_hash_table)
    self.availabilities.clear

    ga_hash_table.each do |day, hrs|
      hrs.each do |hr, val|
        self.availabilities.create({day: day, hour: hr.to_i})
      end
    end
  end

Это часть для отображения еженедельной доступности тренера в виде таблицы. Каждый день / час ячейка - это флажок, который тренер может установить или снять, чтобы указать, свободны ли они.

<!-- availabilities/_scheduler.html.erb -->
<h2>General Availability</h2>
Please check the times below that you would generally be available for a training session.
<table class="table" id="availabilities_table">
  <tr>
    <th>Time</th>
    <% days_of_the_week.each do |day| %> 
      <th><%= day %></th> 
    <% end %>
  </tr>
  <% (6..21).each do |hr| %>
    <tr>
      <td><%= format_as_time hr %></td>
      <% days_of_the_week.each do |day| %>
        <% is_checked = @general_availability[day][hr.to_s] %>
        <td class="availabilities_cell">
          <%= check_box_tag "availability[#{day}][#{hr}]", true, is_checked, :class => 'availabilities_check_box' %>
        </td>
      <% end %>
    </tr>
  <% end %>
</table>

Это контроллер:

# coaches_controller.rb
...
def edit
  @coach = current_user.coach
  @general_availability = @coach.general_availability
end

def update
  @coach = Coach.find(params[:id])

  @coach.update_attributes(params[:coach])  
  if @coach.save
    @coach.update_general_availability(params[:availability])
    redirect_to @coach
  end
  # ...
end

Это линия

@coach.update_general_availability(params[:availability])

, который вызывает ошибку.

Теперь вот мой вопрос. Почему это представление вызывает вышеуказанную ошибку?

<!-- edit.html.erb version 1 -->
<h1><%= @coach.user.first_name %>  </h1>
<%= form_for @coach, :html => { :multipart => true } do |f| %> 

  <%= f.label :profile_photo %> 
  <%= f.file_field :profile_photo %> 

  <div class="field">
    <%= f.label :phone_number %>
    <%= f.text_field :phone_number %>
  </div>  

  ... More Form Fields Here ...

  <%= render 'availabilities/scheduler' %>
  <%= f.button %> 

<% end %> 

Пока этого представления нет?

<!-- edit.html.erb version 2 -->
<h1><%=  @coach.user.first_name %>  </h1>
<%= form_for @coach, :html => { :multipart => true } do |f| %> 

  <%= f.label :profile_photo %> 
  <%= f.file_field :profile_photo %> 

  <div class="field">
    <%= f.label :phone_number %>
    <%= f.text_field :phone_number %>
  </div>  

  ... More Form Fields Here ...

  <%= f.button %> 

<% end %> 

<%= form_for @coach, :class => "form-vertical" do |f| %>
  <%= render 'availabilities/scheduler' %>
  <%= submit_tag "Update Schedule" %>
<% end %>

Обратите внимание, что в первом случае частичное находится внутри формы построителя формы, тогда как во втором случае частичное отображается как его собственное form_for ниже.

Часть, которая выскакивает у меня из журнала, который я вставил выше, это строка:

Binary data inserted for `string` type on column `day`

, который не появляется, когда форма работает (например, в версии 2 формы). Это кажется важным, но я не знаю, что это значит или почему это происходит.

Большое спасибо!

1 Ответ

2 голосов
/ 29 февраля 2012

Разобрался. Рубиновые ключи хеш-таблицы заморожены. Так что мои параметры выглядели так:

params[:availability][:Thursday][:10] = "true"

Когда мой update_general_availability метод сделал это:

self.availabilities.create({day: day, hour: hr.to_i})

:day было "Thursday", но адаптер SQLite понял, что он имеет Encoding::ASCII_8BIT (он же «двоичный»), и попытался сделать encode! 'utf-8'. Однако, поскольку он был заморожен, это вызвало ошибку Runtime Frozen String. Проблема была решена путем добавления этих строк в метод update_general_availability:

day = day.dup
hr = hr.dup

Теперь, поскольку они дублируются, а не сами хеш-ключи, они могут быть закодированы в utf-8.

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...