Question

所以这是场景。我正在使用Rails 4和Rufus Scheduler创建一个在将来的特定时间发送SMS的应用程序。到目前为止，如果我想通过点击按钮发送短信，我就能收到短信。而且我也可以使用我自己的代码中设置的计时器发送它们，例如下面的计时器，它允许将来发送我的消息。基本上，我设置了计时器，当它关闭时，它调用我的方法发送短信。

这一切都很棒，但我需要做的最后一件事是创建一个接受用户输入的方法，并将其放在日期/时间/时区所在的位置。在我的new.html.erb中，我已经收集了所有这些信息，但我不知道如何从那里获取并将其放入调度程序操作中。

这有效：

scheduler = Rufus::Scheduler.new
      scheduler.at '2014-02-16 5:47 PM Pacific Time' do
      @sms_message.send_text_message
      end

这是我已经尝试过的，没有运气。

这是模型。

class SmsMessage

include Mongoid::Document
include Mongoid::Timestamps

field :from, type: String
field :to, type: String
field :body, type: String

field :year, type: String
field :month, type: String
field :day, type: String
field :timehour, type: String
field :timeminute, type: String
field :timezone, type: String
field :ampm, type: String

belongs_to :user

def set_time

puts year + month + day + " " + timehour + timeminute + " " + ampm + " " + timezone 

end


def send_text_message

ts = self.user.twilio_sid
tt = self.user.twilio_token

sent_to_number = to
sent_from_number = self.user.twilio_number
  message_to_send = body

@twilio_client ||= Twilio::REST::Client.new ts, tt

@twilio_client.account.sms.messages.create(

  :from => "#{sent_from_number}",
  :to => "#{sent_to_number}",
  :body => "#{message_to_send}"
)

end

end

这是控制器。

def create   
if current_user

  @sms_message = current_user.sms_messages.build(params[:sms_message].permit(:to, :from, :body))

    if @sms_message.save

      scheduler = Rufus::Scheduler.new
      scheduler.at @sms_message.set_time do
      @sms_message.send_text_message
      end

      flash[:notice] = "Your text has been scheduled!"
      redirect_to sms_messages_path

    else
      render 'new'
    end

else new_user
  redirect_to new_user_path
end

端

观点：

<div class="row">
<div class="col-md-3"></div>
<div class="col-md-6">
    <%= form_for(@sms_message) do |f| %>

        To: <%= f.text_field :to, class: "form-control" %>
        Body: <%= f.text_area :body, class: "form-control" %><br />
        <%= f.select :month, ['01-', '02-', '03-','04-',
            '05-', '06-', '07-', '08-', '09-', '10-', '11-', '12-'] %>
        <%= f.select :day, ['1', '2', '3', '4', '5', '6', '7', '8', '9',
        '10', '11', '12', '13', '14', '15', '16', '17', '18', '19', '20', '21', '22', '23',
        '24', '25', '26', '27', '28', '29', '30', '31'] %>
        <%= f.select :year, ['2014-', '2015-', '2016-', '2017-'] %>
        at <%= f.select :timehour, ['1:', '2:', '3:', '4:', '5:', '6:', '7:', '8:', '9:',
            '10:', '11:', '12:']%>
        <%= f.select :timeminute, ['01', '02', '03', '04', '05', '06', '07', '08', '09', '10',
            '11', '12', '13', '14', '15', '16', '17', '18', '19', '20', '21', '22', '23', '24',
            '25', '26', '27', '28', '29', '30', '31', '32', '33', '34', '35', '36', '37', '38',
            '39', '40', '41', '42', '43', '44', '45', '46', '47', '48', '49', '50', '51', '52',
            '53', '54', '55', '56', '57', '58', '59', '60'] %>
        <%= f.select :ampm, ['AM', 'PM'] %>
        <%= f.select :timezone, ['Pacific Time', 'Eastern Time', 'Central Time',
            'Mountain Time', 'Alaska Time', 'Hawaii-Aleutian Time', 'Samoa Time'] %><br /><br /><br />
        <%= f.submit "Send Text Message", class: "btn btn-success"  %>
    <% end %>
</div>
<div class="col-md-3"></div>

我一直试图破解这个3个多小时而没有运气。我是Rails的初学者，如果这个问题没有完美格式，请提前对不起。还在学习。当我使用“ - ”，“+”和“”时，我的方法一直出错。所以我将这些东西放在视图中。这就是为什么它看起来很奇怪。你们觉得怎么样？

Answer 1

在 app / model / sms_message.rb

中

将set_time功能更改为：

def set_time
  year + month + day + " " + timehour + timeminute + " " + ampm + " " + timezone 
end

这将return字符串而不是print。

在 app / controllers / sms_message_controller.rb 中添加：

def create   
  if current_user

    @sms_message = current_user.sms_messages.build(sms_message_params)
    if @sms_message.save

    scheduler = Rufus::Scheduler.new
    scheduler.at @sms_message.set_time do
      @sms_message.send_text_message
    end

    flash[:notice] = "Your text has been scheduled!"
    redirect_to sms_messages_path

  else
    render 'new'
  end

else new_user
  redirect_to new_user_path
end

private

def sms_message_params
  params.require(:sms_message).permit(:from, :to, :body, :month, :day, :year, :timehour, :timeminute, :ampm, :timezone)
end

Answer 2

您不应为每个用户请求创建新的rufus-scheduler实例。你最终会得到无数的rufus-scheduler线程，这些线程只有一次有用，并且只消耗内存和cpu。

更好的方法是：

a）在初始化程序中启动调度程序一次性（例如config / initializers / scheduler.rb）

b）根据需要从调度程序实例中的控制器进行调度。

所以

配置/初始化/ scheduler.rb：

require 'rufus-scheduler'

$scheduler = Rufus::Scheduler.new

在您的控制器中：

def create

  if current_user

    @sms_message = current_user.sms_messages.build(params[:sms_message].permit(:to, :from, :body))

    if @sms_message.save

      $scheduler.at @sms_message.set_time do
        @sms_message.send_text_message
      end

      flash[:notice] = "Your text has been scheduled!"
      redirect_to sms_messages_path

    else
      render 'new'
    end

  else new_user

    redirect_to new_user_path
  end
end

警告

最后，这一切都取决于您运行Rails的“服务器”。瘦和Webrick应该没问题。如果您使用Passenger或其他服务器，则必须仔细阅读该文档。其中一些服务器分叉Ruby进程，原始进程（托管您初始化的rufus-scheduler线程的进程）可能无法生存，丢失所有计划。阅读工具附带的精彩手册。

如何从Rufus调度程序（在控制器中）内部的模型中调用方法，以便根据用户输入调度时间？ Rails 4

2 个答案: