ruby on rails Wheneverの使い方

Question

Ruby on Railsで開発しているのですが、データベースと比較して、時間が来たら自動で削除するということをしたいと思っています。
調べたところ、wheneverというgemを使えばできるようだったのですが、やり方がよくわからず、うまく反映しません。

`schedule.rb`

require File.expand_path(File.dirname(__FILE__) + "/environment")

rails_env = Rails.env.to_sym
rails_root = Rails.root.to_s

# environment は設定しないと production になってしまう
set :environment, rails_env
set :output, "#{rails_root}/log/cron.log"


every 2.minutes do
  runner 'Hoge.hoge'
end

`lib/hoge.rb`

def hoge
    @tweets = Tweet.all
    @tweets.each do | tweet |
      if tweet.updated_at < Time.now
        tweet.destroy
      end
    end
  end

bundle exec whenever --update-crontabを実行すると
[write] crontab file updatedとなり、
crontab -l　では

 Begin Whenever generated tasks for: /Users/xxxx/projects/xxxx/config/schedule.rb at: 2018-02-28 14:50:45 +0900
0,2,4,6,8,10,12,14,16,18,20,22,24,26,28,30,32,34,36,38,40,42,44,46,48,50,52,54,56,58 * * * * /bin/bash -l -c 'cd /Users/xxx/projects/xxx && bundle exec bin/rails runner -e development '\''Hoge.hoge'\'' >> /Users/xxx/projects/xxx/log/cron.log 2>&1'

となっており、できているのかと思ったのですが、何も起きません。

lib/hoge.rbというファイル名がおかしいのか、
そもそもの記述が違うのかもわかっていないので、
詳しい方是非お願いします！！

Answer 1

whenever はとっても簡単に cron できて非常に便利なのですが，いくつかハマりポイントがあるんですよね。

crontab -l の結果を見ると，schedule.rb の書き方などは問題なさそうですね。

ちょっと気になるのが実行ユーザーです。
cron はユーザーごとなので，sudo をつけずに crontabl -l で出てきた，ということは，whenever -w（もしくは whenever -i）したときも sudo をつけていなかった，ということですか？

root じゃなくてサーバーのログインユーザーでそのタスクを実行するのでよいのかが問題です。
たとえば sudo でないとデータベースの更新ができないため失敗する，ということがよくあります。

それから，whenever の使い方に気を取られて処理自体の不具合に気付かないことがあります。
問題を切り分けましょう。
今の場合，

$ rails runner -e development “Hoge.hoge"

が実行できるかを確認します。

Comments

[退会済みユーザー]

回答ありがとうございます。
sudoなのですが、一度もつけずにできていました。
また、rails runner -e development “Hoge.hoge"
なのですが、ターミナル上で実行しても、>が出て、次の実行を待つ状態になっていますが、
どういうことでしょうか？

[退会済みユーザー]

あともう一点なのですが、
```
$ bundle exec whenever
0,2,4,6,8,10,12,14,16,18,20,22,24,26,28,30,32,34,36,38,40,42,44,46,48,50,52,54,56,58 * * * * /bin/bash -l -c 'cd /Users/xxx/projects/xxx && bundle exec bin/rails runner -e development '\''Hoge.hoge'\'' >> log/crontab.log 2>&1'

## [message] Above is your schedule file converted to cron syntax; your crontab file was not updated.
## [message] Run `whenever --help' for more options.
```
## [message] Above is your schedule file converted to cron syntax; your crontab file was not updated.
といった感じにシンタックスエラーのようなものが出ている気がします、、こちらはデフォルトなんでしょうか？

[退会済みユーザー]

ダブルクォーテーションのエラーでした。

Running via Spring preloader in process 87923
Please specify a valid ruby command or the path of a script to run.
Run 'bin/rails runner -h' for help.

undefined method `hoge' for Hoge:Module
と出て来ていたのでこちらでのエラーということはわかりました。

[退会済みユーザー]

メゾットを消してモジュール内にコードを書き込むと動きました。
メゾット内で動かしたたかったんですが、一応解消はできそうです。ありがとうございます。

[scivola]

「Above is your schedule file converted to cron syntax」は「上記はあなたの schedule ファイルを cron の構文に変換したものです」というような意味で，正常です。
続く「your crontab file was not updated.」も，意訳すれば「また whenever セットしたけど，既にセットされているのと同じだったから crontab ファイルに変化は無いよ。」という意味で，正常です。

[scivola]

hoge が Hoge の特異メソッドとして書けてないのかも？
lib/hoge.rb の全貌を見せていただくと分かるかもしれません。

[退会済みユーザー]

module Hoge
def hoge
@tweets = Tweet.all
@tweets.each do | tweet |
if tweet.updated_at < Time.now
tweet.destroy
end
end
end
end

[退会済みユーザー]

直書きでで申し訳ありません。
このようにmoduleの中にdef hogeを書いていました。

[scivola]

それだと Hoge.hoge の形では呼べません。
def hoge
を
def self.hoge
に変えると，Hoge モジュールの特異メソッドになるので Hoge.hoge の形で呼べるようになります。

[scivola]

hoge メソッド内で @tweets とインスタンス変数を使っていますが，メソッド内だけで使うものはローカル変数にしましょう。

というか，
Tweet.where("updated_at < ?", Time.now).destroy_all
でいいんじゃないかと思います。

Time.now は Time.zone.now にすべきかもしれませんが，ちょっと知識不足で判断つきません。

where の部分は outdated なんて名前のスコープをモデルに定義しちゃうといいですね。

でもって，古い Tweet を削除するメソッドは lib/hoge.rb なんかに書かないで，Tweet モデルの特異メソッドとして定義するとよさげ。この処理は Tweet モデルの管轄だろうからです。

[退会済みユーザー]

ありがとうございます！！
できました！！！