Отправка электронных писем в выбранные пользователем даты

Мое приложение включает людей, заходящих на домашнюю страницу, вводящих название сделанной ими ставки (для развлечения), их адрес электронной почты, дату, на которую они хотели бы получить напоминание, и некоторые подробности о ставке. Я использую драгоценный камень всякий раз, когда это делается один раз в день.

bet.rb

class Bet < ActiveRecord::Base
  attr_accessible :details, :email, :name, :reminder, :sent

  # Sends user a reminder if current_date is equal to the reminder date of the bet
  def check_bet
    current_date = Time.now.strftime("%Y-%m-%d").to_s
    @bets = Bet.all
    @bets.each do |bet|
    BetMailer.bet_reminder(bet).deliver and bet.sent = true and bet.save! if bet.reminder.to_s == current_date
    end 
  end
end

schedule.rb

every :day, :at => '5:00pm' do
  runner "Bet.check_bet"
end

bet_mailer.rb

class BetMailer < ActionMailer::Base
  default from: "from@example.com"

  def bet_reminder(bet)
    @bet = bet

    mail to: bet.email, subject: bet.name + " Reminder"
  end
end

Я успешно отправлял электронные письма, когда текущая дата равна дате, на которую они хотели получить напоминание (напоминание). Чтобы проверить это, я вошел в консоль rails, выбрал конкретный объект Bet и запустил на нем метод check_bet, используя:

1.9.2p320 :013 > Bet.last.check_bet
  Bet Load (2.8ms)  SELECT "bets".* FROM "bets" ORDER BY "bets"."id" DESC LIMIT 1
  Bet Load (0.9ms)  SELECT "bets".* FROM "bets" 
   (0.2ms)  BEGIN
   (0.3ms)  COMMIT
   (0.2ms)  BEGIN
   (0.3ms)  COMMIT
   (0.2ms)  BEGIN
   (0.2ms)  COMMIT
 => [#<Bet id: 3, name: "Newsroom", email: "email@email.com", reminder: "2013-07-30", details: "Mac and Will are going to get back together.", sent: false, created_at: "2013-07-29 17:23:13", updated_at: "2013-07-29 17:23:13">, #<Bet id: 4, name: "Testing", email: "email@email.com", reminder: "2013-07-29", details: "This is a test", sent: true, created_at: "2013-07-29 18:38:42", updated_at: "2013-07-29 20:17:34">, #<Bet id: 5, name: "Cheaper iPhone", email: "email@email.com", reminder: "2013-07-29", details: "I bet Dad that there will be a cheaper iphone in th...", sent: true, created_at: "2013-07-29 20:39:33", updated_at: "2013-07-29 20:50:14">, #<Bet id: 6, name: "My grades", email: "email@email.com", reminder: "2013-07-29", details: "My grades this year will be > 84% average", sent: true, created_at: "2013-07-29 20:56:18", updated_at: "2013-07-29 21:14:21">]

После того, как терминал завершит вышеупомянутое, мой почтовый ящик заполнен всеми объектами Bet, у которых есть памятка = current_date. Это доказывает, что настройки SMTP работают, и код в моих представлениях работает нормально.

Однако, когда я пытаюсь запустить метод check_bet на всех объектах ставок, я получаю неопределенную ошибку метода:

1.9.2p320 :016 > Bet.all.check_bet
  Bet Load (1.8ms)  SELECT "bets".* FROM "bets" 
NoMethodError: undefined method `check_bet' for #<Array:0x007ffdd2c8c6c0>
    from (irb):16
    from /Users/bvlaar/.rvm/gems/ruby-1.9.2-p320@rails3tutorial/gems/railties-3.2.13/lib/rails/commands/console.rb:47:in `start'
    from /Users/bvlaar/.rvm/gems/ruby-1.9.2-p320@rails3tutorial/gems/railties-3.2.13/lib/rails/commands/console.rb:8:in `start'
    from /Users/bvlaar/.rvm/gems/ruby-1.9.2-p320@rails3tutorial/gems/railties-3.2.13/lib/rails/commands.rb:41:in `<top (required)>'
    from script/rails:6:in `require'
    from script/rails:6:in `<main>

'

Кроме того, когда я запускаю команду 'bundle exec всякий раз' в терминале, кажется, что ничего не отправляется.

Источник

2 ответа

Решение

Bet.all возвращает Arrayдля которого check_bet метод не определен. Вызывать check_bet в каждом случае Betнужно сделать Bet.all.each {|bet| bet.check_bet} или уборщик Bet.all.each(&:check_bet) (кивком @Ryan Bigg).

См. Также http://guides.rubyonrails.org/active_record_querying.html, для некоторых соображений производительности.

Источник

  1. всякий раз, когда это просто драгоценный камень, который помогает вам создавать задачи cron, он не запускает никаких запланированных задач для вас, crontab делает.

  2. Я думаю, что будет лучше запустить свой планировщик в час, как это

    every :hour do
  runner "Bet.check_bet"
end

  3. Затем в своем классе модели Bet вы можете сделать это:

    class Bet < ActiveRecord::Base
  attr_accessible :details, :email, :name, :reminder, :sent

  # Sends user a reminder if current_date is equal to the reminder date of the bet
  def self.check_bet
    current_date = Time.now.strftime("%Y-%m-%d").to_s
    self.where(:reminder => current_date).each do |bet|
      BetMailer.bet_reminder(bet).deliver # later you can use some background job gem like sidekiq, rescue or delayed_job to send email, otherwise the performance might be a issue
      bet.update_attribute(:sent, true)
    end
   end 
 end
end

Спасибо

Источник
Другие вопросы по тегам