ruby - 我可以将方法用作 lambda 吗？

Question

我有一个接口(interface)，它定义了一组条件。它是将与其他模型一起使用的几个此类接口(interface)之一。

这些条件将由消息队列处理程序调用以确定警报的完整性。所有警报调用都是相同的，因此我试图通过将条件抽象到它们自己的方法中来稍微干燥入队调用(我质疑方法是否是正确的技术)。我认为通过这样做我将能够测试这些条件中的每一个。

class Loan
  module AlertTriggers
    def self.included(base)
      base.extend           LifecycleScopeEnqueues

      # this isn't right
      Loan::AlertTriggers::LifecycleScopeEnqueues.instance_method.each do |cond|

        class << self
          def self.cond
            ::AlertHandler.enqueue_alerts(
              {:trigger => Loan.new}, 
              cond
            )
          end
        end

      end
    end
  end

  module LifecycleScopeEnqueues
    def student_awaiting_cosigner 
        lambda { |interval, send_limit, excluding|
          excluding ||= ''
          Loan.awaiting_cosigner.
            where('loans.id not in (?)', excluding.map(&:id) ).
            joins(:petitions).
            where('petitions.updated_at > ?', interval.days.ago).
            where('petitions.updated_at <= ?', send_limit.days.ago) 
        }
    end
  end

我考虑过替代方案，其中每个方法都充当一个范围。沿着这条路，我不确定如何让 AlertHandler 成为 interval、send_limit 和 excluding，它在调用它时传递给 block /过程。

---

有人向我(离线)建议范围是 lambda，因此可能是更合适的解决方案 - 根据@BorisStitnicky 的推论，钳子可以用作锤子，但是不应该。我也愿意接受这方面的答案。

Answer 1

您知道，这可能不是您寻求的答案，但我会尽力而为。像这样回答比删除评论更好。在您的代码中，您正在做一些非常不寻常的事情。首先，您在一个类中定义一个模块。我以前从未做过或见过这个，以至于我点击 irb 来尝试一下。其次，您正在定义一个返回 lambda 的方法。这让我想起了我刚刚学习 Ruby 时一直在做的事情。 Lambda 有非常特殊的应用，通常应尽可能避免使用这种形式。如果你想像这样使用 lambda，至少将它们分配给一个变量，或者更好的是，一个常量:

STUDENT_AWAITING_COSIGNER = lambda { |interval, send_limit, excluding|
  # do your SQL magic
}

我无法理解您的词汇:特别是，我不确定您所说的“范围”是 Ruby 范围还是其他类型的范围。

但就我个人而言，我认为您根本不应该使用 lambda。我敢说，你的代码需要的不仅仅是干一点。我认为你不应该在类中设置子命名空间。你为什么不使用例如。一个实例变量？此外，公共(public)类方法只是蛋糕上的一颗樱桃。首先解决没有它们的问题，然后您可以决定添加它们以使您的界面更方便。总而言之，我会简单地按照这些思路做一些事情:

class Loan
  attr_reader :alerts

  def initialize( whatever_options )
    @alerts = Array( whatever_options[ :alerts ] )
  end

  def check_alerts
    @alerts.each &:test
  end
end

# Then I would set up an alert class:
class Alert
  def test( interval, send_limit, excluding = '' )
    Loan.awaiting_cosigner.
      where('loans.id not in (?)', excluding.map(&:id) ).
      joins(:petitions).
      where('petitions.updated_at > ?', interval.days.ago).
      where('petitions.updated_at <= ?', send_limit.days.ago) 
  end
end
# I used your prescription statically, if you have different kind
# of alerts, you would have to make the class sufficiently flexible
# to handle them all.

# and then I would eg. supply alerts to a Loan upon instantiation
# (this can be also done later, if you make it so)
my_little_loan = Loan.new( alerts: Alert.new )
# and when the time comes to check whether the alerts alert or not:
my_little_loan.check_alerts

显然，这只是我谦虚地认为应该在 Ruby 中简单地解决这类问题的概述。您需要付出自己的努力，让它在您的特定复杂情况下为您所用。