Задержка ActiveResource по запросу json в бесплатной базе - PullRequest
Новая
       86

Задержка ActiveResource по запросу json в бесплатной базе

0 голосов
/ 10 марта 2012

У меня есть установка ActiveResource для использования API freebase в json, и он должен работать нормально, за исключением того, что возвращение json freebase приводит к взрыву ActiveResource. 

NoMethodError: undefined method `collect!' for #<Hash:0x007fd674831dd0>

Как определить собственный анализатор jsonисправить все, что не так? 

class Freebase < ActiveResource::Base
  self.site = "https://www.googleapis.com/"
  self.format = :json

  def self.search(word)
    self.find(:all, :from => "/freebase/v1/search/", :params => { :query => word })
  end

   #https://www.googleapis.com/freebase/v1/search?query=nirvana

  #Freebase.get('search', :query => 'nirvana')

end

Возвращается Json:

https://www.googleapis.com/freebase/v1/search?query=nirvana

{"status":"200 OK","result":[{"mid":"/m/05b3c","name":"Nirvana","notable":{"name":"Belief","id":"/religion/belief"},"lang":"en","score":67.540009},{"mid":"/m/0b1zz","name":"Nirvana","notable":{"name":"Musical Artist","id":"/music/artist"},"lang":"en","score":64.311432},{"mid":"/m/092bf5","name":"Buddhism","notable":{"name":"Religion","id":"/religion/religion"},"lang":"en","score":33.647118},{"mid":"/m/02_6qh","name":"Nirvana","notable":{"name":"Film","id":"/film/film"},"lang":"en","score":30.068491},{"mid":"/m/01h89tx","name":"Nirvana","notable":{"name":"Musical Album","id":"/music/album"},"lang":"en","score":27.799274},{"mid":"/m/01rn9fm","name":"Nirvana","notable":{"name":"Musical Group","id":"/music/musical_group"},"lang":"en","score":27.445602},{"mid":"/m/015k7","name":"Gautama Buddha","notable":{"name":"Deity","id":"/religion/deity"},"lang":"en","score":24.129679},{"mid":"/m/01rkx5","name":"Mahayana Mahaparinirvana Sutra","lang":"en","score":22.359026},{"mid":"/m/03d7q7v","name":"Nirvana","lang":"en","score":21.034473},{"mid":"/m/055ym7w","name":"Nirvana bootleg recordings","notable":{"name":"Musical Album","id":"/music/album"},"lang":"en","score":19.241596},{"mid":"/m/0122_j","name":"Nevermind","notable":{"name":"Musical Album","id":"/music/album"},"lang":"en","score":18.366383},{"mid":"/m/04n7mt","name":"Nirvana fallacy","lang":"en","score":17.212397},{"mid":"/m/0484q","name":"Kurt Cobain","notable":{"name":"Musician","id":"/m/09jwl"},"lang":"en","score":16.594929},{"mid":"/m/027_k8j","name":"Nirvana","lang":"en","score":16.336584},{"mid":"/m/0285c","name":"Dave Grohl","notable":{"name":"Musician","id":"/m/09jwl"},"lang":"en","score":16.115103},{"mid":"/m/068shv","name":"Smells Like Nirvana","notable":{"name":"Musical Album","id":"/music/album"},"lang":"en","score":15.350652},{"mid":"/m/01kq85c","name":"Manic Nirvana","notable":{"name":"Musical Album","id":"/music/album"},"lang":"en","score":15.275189},{"mid":"/m/0437sc","name":"Lithium","notable":{"name":"Composition","id":"/music/song"},"lang":"en","score":14.637386},{"mid":"/m/055kh1","name":"Mechanus","lang":"en","score":14.621847},{"mid":"/m/01f1vf","name":"Lucifer","notable":{"name":"Fictional Character","id":"/fictional_universe/fictional_character"},"lang":"en","score":13.504528}],"cursor":20,"cost":11,"hits":3104}

Использование Rails 3.2.1

ОБНОВЛЕНИЕ:

Нашел эту проблему, но все еще не уверен, как ее преодолеть.https://github.com/rails/rails/issues/2318

Ответы [ 2 ]

1 голос
/ 10 марта 2012

Проблема в том, что возвращаемый JSON сформирован не так, как ожидает ActiveResource. ARes не ожидает всех возвращаемых метаданных, а только ожидает, что находится в results части ответа.

Если говорить прямо, вы возвращаетесь: 

{"status":"200 OK","result":[{"mid":"/m/05b3c","name":"Nirvana","notable":{"name" ...

Но ARes хочет: 

[{"mid":"/m/05b3c","name":"Nirvana","notable":{"name"...

Самое простое (и, возможно, самое грязное) решение, которое я могу предложить, - это перезаписать частный метод ActiveResource :: Base # find_every в вашей модели, например, так: 

class Freebase < ActiveResource::Base

  self.site = "https://www.googleapis.com/"
  self.format = :json

  def self.search(word)
    self.find(:all, :from => "/freebase/v1/search/", :params => { :query => word })
  end

  private

  def self.find_every(options)
    begin
      case from = options[:from]
      when Symbol
        instantiate_collection(get(from, options[:params]))
      when String
        path = "#{from}#{query_string(options[:params])}"
        instantiate_collection(format.decode(connection.get(path, headers).body['result']) || [])
      else
        prefix_options, query_options = split_options(options[:params])
        path = collection_path(prefix_options, query_options)
        instantiate_collection( (format.decode(connection.get(path, headers).body['result']) || []), prefix_options )
      end
    rescue ActiveResource::ResourceNotFound
      # Swallowing ResourceNotFound exceptions and return nil - as per
      # ActiveRecord.
      nil
    end
  end

end

Единственное изменение, которое я сделал, - это вызовы .body метода теперь .body['result']. С этим дополнением #instantiate_collection теперь будет получать массив, как и ожидалось, а не хэш.

В конечном счете, хотя это и должно устранить ошибку, я не знаю, что это решит все ваши проблемы, так как вам могут понадобиться некоторые данные, которые теряются при использовании этого метода. Мое предложение было бы отказаться от ActiveResource, если вы можете и использовать что-то вроде RestClient и построить свою модель вокруг этого. Другая причина такого подхода заключается в том, что семантика в том виде, в каком она существует сейчас, нарушена Вы не получите коллекцию бесплатных баз. Вы используете Freebase API для получения результатов поиска.

0 голосов
/ 11 марта 2012

Я последовал совету Дерека и отказался от ActiveResource ...

для всех остальных, кто в итоге использовал Фарадей и Faraday_Middleware 

class Freebase

  def self.search(id)
    connection = Faraday.new 'https://www.googleapis.com/freebase/v1' do |conn|

      conn.adapter Faraday.default_adapter
      conn.use FaradayMiddleware::ParseJson
      #conn.use Faraday::Response::Mashify
    end

    response = connection.get do |req|
      req.url('search', :query => id, :limit => 10)#, :filter => '(any namespace:/wikipedia/en_id namespace:/authority/imdb/title)')#  #:filter => '(any namespace:/wikipedia/en_id namespace:/authority/imdb/title namespace:/authority/netflix/movie)'  , :with => 'commons'
    end

  end

Возможно, вы можете использовать несколько встроенныхв модулях, чтобы сделать методы стиля rails из возвращенного хэша, но я еще не понял эту часть.ActiveResource, кажется, хорош только в других приложениях Rails или RESTFUL-маршрутах, которые идеально имитируют его.

...