Сравнение списков хэшей полей с эквивалентными AR-объектами - PullRequest
1 голос
/ 01 мая 2010

У меня есть список хэшей, как таковой:

incoming_links = [
 {:title => 'blah1', :url => "http://blah.com/post/1"},
 {:title => 'blah2', :url => "http://blah.com/post/2"},
 {:title => 'blah3', :url => "http://blah.com/post/3"}]

И модель ActiveRecord, которая имеет поля в базе данных с некоторыми соответствующими строками, скажем:

Link.all => 
[<Link#2 @title='blah2' @url='...post/2'>,
 <Link#3 @title='blah3' @url='...post/3'>,
 <Link#4 @title='blah4' @url='...post/4'>]

Я бы хотел выполнить операции над множествами Link.all с incoming_links, чтобы я мог понять, что <Link#4 ...> не входит в набор incoming_links, а {:title => 'blah1', :url =>'http://blah.com/post/1'} не находится в Link.all установить, вот так:

#pseudocode
#incoming_links =  as above
links = Link.all
expired_links = links - incoming_links
missing_links = incoming_links - links
expired_links.destroy
missing_links.each{|link| Link.create(link)}

Дрянное решение а):

Я бы предпочел не переписывать Array#- и все такое, и я в порядке с преобразованием incoming_links в набор несохраненных Link объектов; поэтому я попытался перезаписать hash eql? и т. д. в Link, чтобы игнорировать равенство id, которое AR::Base обеспечивает по умолчанию. Но это единственное место, где такое равенство следует учитывать в приложении - в других местах требуется идентификатор по умолчанию Link # id. Есть ли какой-нибудь способ, которым я мог бы создать подкласс Link и применить перезапись hash, eql? и т. Д.?

Дерьмовый раствор б):

Другой путь, который я пробовал, - извлечь хеш-атрибуты для каждой ссылки и выполнить .slice('id',...etc), чтобы убрать хеш-значения. Но это требует написания отдельных методов - для отслеживания объектов Link при выполнении операций над множествами над хешами и написания отдельных классов Proxy для переноса хешей и ссылок incoming_links, что кажется немного излишним. Тем не менее, для меня это текущее решение .

Можете ли вы придумать лучший способ спроектировать это взаимодействие? Дополнительный кредит за чистоту.

Ответы [ 3 ]

1 голос
/ 01 мая 2010

попробуйте

incoming_links = [
 {:title => 'blah1', :url => "http://blah.com/post/1"},
 {:title => 'blah2', :url => "http://blah.com/post/2"},
 {:title => 'blah3', :url => "http://blah.com/post/3"}]

ar_links = Link.all(:select => 'title, url').map(&:attributes)

# wich incoming links are not in ar_links
incoming_links - ar_links

# and vice versa
ar_links - incoming_links

UPD

Для вашей модели Link:

def self.not_in_array(array)
  keys = array.first.keys
  all.reject do |item|
    hash = {}
    keys.each { |k| hash[k] = item.send(k) }
    array.include? hash
  end
end

def self.not_in_class(array)
  keys = array.first.keys
  class_array = []
  all.each do |item|
    hash = {}
    keys.each { |k| hash[k] = item.send(k) }
    class_array << hash
  end
  array - class_array
end

ar = [{:title => 'blah1', :url => 'http://blah.com/ddd'}]
Link.not_in_array ar
#=> all links from Link model which not in `ar`
Link.not_in_class ar
#=> all links from `ar` which not in your Link model
0 голосов
/ 03 мая 2010

Требования:

#  Keep track of original link objects when 
#   comparing against a set of incomplete `attributes` hashes.
#  Don't alter the `hash` and `eql?` methods of Link permanently, 
#   or globally, throughout the application.

Текущее решение действует с использованием метода eql? Хэша и аннотирования хэшей исходными объектами:

class LinkComp < Hash
  LINK_COLS = [:title, :url]
  attr_accessor :link
  def self.[](args)
    if args.first.is_a?(Link) #not necessary for the algorithm, 
                              #but nice for finding typos and logic errors
      links = args.collect do |lnk|
         lk = super(lnk.attributes.slice(*(LINK_COLS.collect(&:to_s)).to_a)
         lk.link = lnk
         lk
      end
    elsif args.blank?
       []
    #else #raise error for finding typos
    end        
  end
end

incoming_links = [
 {:title => 'blah1', :url => "http://blah.com/post/1"},
 {:title => 'blah2', :url => "http://blah.com/post/2"},
 {:title => 'blah3', :url => "http://blah.com/post/3"}]

#Link.all => 
#[<Link#2 @title='blah2' @url='...post/2'>,
# <Link#3 @title='blah3' @url='...post/3'>,
# <Link#4 @title='blah4' @url='...post/4'>]

incoming_links= LinkComp[incoming_links.collect{|i| Link.new(i)}]
links = LinkComp[Link.all] #As per fl00r's suggestion 
                           #this could be :select'd down somewhat, w.l.o.g.

missing_links =  (incoming_links - links).collect(&:link)
expired_links = (links - incoming_links).collect(&:link)
0 голосов
/ 01 мая 2010

Если вы переписываете метод равенства, ActiveRecord будет жаловаться до сих пор?

Разве вы не можете сделать что-то похожее на это (как в обычном классе ruby):

class Link
  attr_reader :title, :url

  def initialize(title, url)
    @title = title
    @url = url
  end

  def eql?(another_link)
    self.title == another_link.title and self.url == another_link.url
  end

  def hash
     title.hash * url.hash
  end
end

aa = [Link.new('a', 'url1'), Link.new('b', 'url2')]
bb = [Link.new('a', 'url1'), Link.new('d', 'url4')]

(aa - bb).each{|x| puts x.title}
...