ループ処理中に起こったエラーをスキップして処理を継続したい場合、どのようにコードを書けばよいでしょうか？

Question

実現したいこと

クラウドファンディングのようなサイトをrailsで開発・運営しております。

Event（企画） を応援したContributor（寄付者）の寄付金額の手数料のカラムを、
コマンド入力で一気にアップデートしたいです。
しかし、Contributorに紐づいているEventが削除されているとエラーが発生してしまいます。
（Eventが削除されていても、Contributorは削除されておりませんため）

Eventが削除されていても、同じアップデートを継続させたいのですが、
どのようにコーディングすればよいのでしょうか？
大変お忙しい中恐縮ですが、教えて頂けますと大変嬉しいです。

開発の前提

クラウドファンディングのようなサイトをrailsで開発・運営しております。

発生している問題・エラーメッセージ

undefined method `created_at' for nil:NilClass

該当のソースコード

models/concerns/Fee_update.rb

module Fee_update
  class Fee_up
    def self.fees
      Contributor.preload(:event).find_each do |contributor|
        if contributor.event.created_at < Date.new(2021,8,1)
          fee_stripe = contributor.contribution_amount * 0.036
          fee_we = contributor.contribution_amount * 0.214
          fee_user =  contributor.contribution_amount * 0.75
        else
          fee_stripe = contributor.contribution_amount * 0.036
          fee_we = contributor.contribution_amount * 0.264
          fee_user =  contributor.contribution_amount * 0.7
        end
        contributor.update!(fee_stripe:fee_stripe, fee_we:fee_we, fee_user:fee_user)
      end    
    end
  end
end

実行コマンド

heroku run rails runner Fee_update::Fee_up.fees

カラム
Contributor

event_id #こちらによって、Eventと紐づいています
fee_stripe 
fee_we
fee_user

試したこと

https://style.potepan.com/articles/30566.html#Ruby-2
こちら等、色々見まして、
begin
resque
end
を使用すればよいことは分かってきたのですが、
if内でどう処理をすれば、コマンドの実行が継続されるのかが分からず、途方に暮れております。

大変お忙しい中申し訳ありませんが、教えて頂けますと本当に助かります。
どうぞよろしくお願い申し上げます。

Comments

[hatsu]

Eventが存在しないContributorは処理をしない、でもいい仕様でしょうか？
それならばeventが存在しない場合はスキップしたり、で解決可能と思いましたが。

[akikko]

ありがとうございます！取り急ぎ、それでも大丈夫です！
どのようなコーディングの仕方になるものでしょうか？
すいません、お忙しい中ご回答頂きまして本当にありがとうございます。

Answer 1

eventが存在しないときにするスキップする場合は、
each文の中で next if contributor.event.blank?のような条件文を書いてあげると、contributor.event.blank? が true の時にnextが実行され、7行目以降は処理されず次のcontributorの処理が始まるようになります。
以下例です。

module Fee_update
  class Fee_up
    def self.fees
      Contributor.preload(:event).find_each do |contributor|
        // NOTE: ここでcontributor.eventがblankの時には空にする
        next if contributor.event.blank?
        if contributor.event.created_at < Date.new(2021,8,1)
          fee_stripe = contributor.contribution_amount * 0.036
          fee_we = contributor.contribution_amount * 0.214
          fee_user =  contributor.contribution_amount * 0.75
        else
          fee_stripe = contributor.contribution_amount * 0.036
          fee_we = contributor.contribution_amount * 0.264
          fee_user =  contributor.contribution_amount * 0.7
        end
        contributor.update!(fee_stripe:fee_stripe, fee_we:fee_we, fee_user:fee_user)
      end    
    end
  end
end

そのほか、事前に Contributor.where.associated(:event),find_each do |contributor|のように、そもそもeventが紐づいているデータのみをeachするようにしておくとかもありそうです。(DBのリレーションが正確には不明なため補足でこちらは書いております。

Comments

[akikko]

こんなに短時間の間に、的確なアドバイスを下さいまして本当にありがとうございます！！
なるほど、ループ処理前か処理後のどちらでも可能なのですね！！
ありがとうございます、大変勉強になります。
早速頂きました内容にてトライいたします！！

[akikko]

無事、設定することが出来ました！！大変ありがとうございました！！