记忆的类变量返回不一致的值

时间:2017-08-31 02:45:53

标签: ruby-on-rails ruby airtable

如果这个问题看起来很混乱,我真的很抱歉;我会尽力使其简明扼要。

我正在构建一个模拟ActiveRecord模型的类,但是从名为Airtable的服务获取其数据,而不是数据库。 Airtable就像Excel和数据库之间的交叉 - 它允许您创建数据电子表格,但支持不同表之间的“外键”,以便您可以在表之间链接数据。这对我正在开发的应用程序非常有用。

为了使其具有可扩展性和灵活性,我创建了一个父类AirtableModel,它定义了当类继承它时将填充的常用方法和属性。继承类的名称将帮助父方法从正确的Airtable表访问数据并检索正确的属性。相关位在下面(未提及的位是不言自明的或对问题无关紧要):

class AirtableModel
  def initialize(hash)
    hash.each do |attribute_name, attribute_value|
      attribute_value = self.class.first_value_from_arrays_with_singular_key_name(attribute_name, attribute_value)
      # ^^^ Airtable always returns references as Arrays. If the relationship is a belongs_to, we pull out the value from the Array.

      begin
        attribute_name_as_class = attribute_name.to_s.singularize.camelize.constantize
        # ^^^ Converts the attribute's name to a class constant. Used to make the generated method retrieve class records instead of ids. If the class doesn't exist, its NameError is caught below.
        instance_variable_set("@#{attribute_name}_airtable_ids", attribute_value)

        self.class.send(:define_method, attribute_name.to_sym) do
          result = attribute_name_as_class.find_all_by_airtable_id(instance_variable_get("@#{attribute_name}_airtable_ids"))
          result.length <= 1 ? result.first : result
        end
      rescue NameError
        # Triggered if `attribute_name_as_class` doesn't match an existing class
        instance_variable_set("@#{attribute_name}", attribute_value)
        self.class.send(:define_method, attribute_name.to_sym) do
          instance_variable_get("@#{attribute_name}")
        end
      end
    end
  end

  # Reaches out to Airtable to get all records for this class's table (the Airtable table matches the class name). Collects the resulting data into an array of Hashes.
  # One such hash might look like this:
  #   {
  #     'id' => <unique string ID assigned by Airtable>,
  #     'fields' => {
  #       'db_id' => <Unique integer ID. I added this to emulate a database record>,
  #       ...
  #     }
  #   }
  def self.airtable
    @airtable_records ||= AirtableService.records_from_table(table_name: "#{self}s").each.map do |raw|
      object_properties = raw['fields']
      object_properties['airtable_id'] = raw['id']
      object_properties['id'] = object_properties['db_id']

      Hash[object_properties.collect { |k, v| [k.snakecase.parameterize.underscore.to_sym, v] }]
      # ^^^ Converts parameter name to snake-case symbol, i.e. :db_id
    end
  end

  def self.all
    @all_records ||= airtable.map { |b| new(b) }
  end

  def self.find_by_airtable_id(airtable_id)
    objects = all.select { |b| b.airtable_id == airtable_id }
    raise "non unique airtable_id found" if objects.size > 1
    objects.first
  end

  def self.find_all_by_airtable_id(airtable_ids)
    [airtable_ids].flatten.map { |aid| find_by_airtable_id(aid) }
    # ^^^ Accomodates airtable_ids as an Array or a single value
  end

  def self.first
    all.first
  end

  def self.last
    all.last
  end
end

如果上述任何内容没有意义,请告诉我,我会很乐意更新。

这对我继承自AirtableModel的大多数类都有效,但是我遇到了一个特定表(FooBar)的问题,它应该像两个其他表之间的连接表一样。这看起来像这样:

[Table Foo]                   [Table FooBar]                  [Table Bar]
fooBars <==========---------> foo     bar <---------========> fooBars

他们的类定义非常简单:

class Foo < AirtableModel
end

class FooBar < AirtableModel
end

class Bar < AirtableModel
end

感谢上面的构造函数,我可以调用Foo.first.foo_bars这样的调用,并返回与此FooBar相关的所有Foo个实例的数组。这在控制台中没有问题,但我在Rails应用程序中尝试上述代码片段时遇到了问题。

foo_bars在单个控制器创建操作中被调用两次。这恰好两次调用self.all。第一次,我得到预期的结果 - @all_records等于我在Airtable中的记录数,具有适当的属性值,包括外键关系。但是,第二次输入方法时,@all_records的值将更改为空数组。调用foo_bars的对象未更改,仍包含用于查找关联的airtable_ids实例的正确FooBar@airtable_records - 来自self.airtable方法的返回值 - 仍具有相同的值。

我不确定是什么导致memoized @all_records变量改变了值。我一直在抨击它,使用调试器逐步跟踪函数调用,但我看不出是什么导致值发生变化。任何人都可以提供有关如何进一步调试的建议吗?我非常感激。

1 个答案:

答案 0 :(得分:1)

事实证明答案真是愚蠢。

all正在返回一个对象数组。在课堂的其他地方,我们有这种方法:

def self.where(filter = {})
    filtered_objects = all

    filter.each do |filter_property, filter_value|
      # filter_value = filter_value.airtable_id if filter_value.respond_to?(:airtable_id)
      filtered_objects.select! do |object|
        object_value = object.send(filter_property)

        match_check = lambda do |value|
          if object_value.is_a?(Array)
            object_value.include?(value)
          else
            object_value == value
          end
        end

        filter_value.is_a?(Array) ? filter_value.any? { |v| match_check.call(v) } : match_check.call(filter_value)
      end
    end

    filtered_objects
  end

如果filtered_objects == all,我们在select!上致电filtered_objects,会发生什么?

烨。它直接修改了对象引用。让all返回数组的.dup版本可以解决问题。