class TwitterProfile < ActiveRecord::Base

  def send_status_update(status_update)
    if publish?
      client = Twitter::Client.new( :oauth_token => authentication.token,
                           :oauth_token_secret => authentication.secret)
      client.update(status_update.to_twitter_string)
    end
  rescue Exception => e
    logger.info "Error publishing to twitter: #{e.to_s}"
  end
end

Есть модель StatusUpdate и наблюдатель, который размещает их в Twitter в after_create. Я иногда получаю следующее исключение:

NameError (undefined local variable or method `e' for #<TwitterProfile:0x00000004e44ab8>):
app/models/twitter_profile.rb:23:in `rescue in send_status_update'
app/models/twitter_profile.rb:18:in `send_status_update'
app/models/status_update_observer.rb:6:in `block in after_create'
app/models/status_update_observer.rb:4:in `after_create'
app/models/workout_observer.rb:5:in `after_update'
app/controllers/frames_controller.rb:76:in `update'
app/controllers/application_controller.rb:24:in `call'
app/controllers/application_controller.rb:24:in `block (2 levels) in <class:ApplicationController>'
app/controllers/application_controller.rb:10:in `block in <class:ApplicationController>'

Что мне здесь не хватает?

4
Dean Brundage 28 Авг 2011 в 22:34

2 ответа

Лучший ответ

У меня есть одна вещь, которую я знаю, и одна это просто безумная догадка.

Я знаю, что вам не нужно вызывать to_s для общего выражения #{}; это произойдет автоматически. Но это не вредит.

Мое безумное предположение состоит в том, что ваш тестовый пример на самом деле не запускает опубликованный вами код. Что произойдет, если вы измените e на f?

Я должен отметить, что спасение самого исключения обычно плохая идея. Вы должны спасти RuntimeError или StandardError на самом высоком уровне, и желательно что-то более конкретное. Вы можете получить довольно странные ошибки при спасении Exception, потому что вы мешаете потокам и событиям уровня интерпретатора.

2
DigitalRoss 28 Авг 2011 в 20:09

Вам не хватает блока begin в предложении begin / rescue.

-2
Andy Waite 28 Авг 2011 в 19:25