piotr_majewski_magic 0.0.1 → 0.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 642fea55869de823824c5e99ddfc05c507accdec58e748a747e91ff2fe0f296d
-  data.tar.gz: 319f695786b0ab39013930735a6ef5a9324bd8d9a6b574b4f6d0fb44d72a3428
+  metadata.gz: a14d3c066a0bf4c520e2ec0850ecd34ba99767f2a17658114d808e0a63241e61
+  data.tar.gz: d3e2ca74fd6f3f9a0d37f51e201157afcee5d40ddf1118957811c3041aefee69
 SHA512:
-  metadata.gz: 5b3c439a7b259903abae0d0f44f8db94f198dbad02baf005ec583a92293c9486b3a0294d8f94337c3df1dcc2aad278cb10b4aa57d1aa3886fc717a61eb273be8
-  data.tar.gz: 74a0851d96fbb1dc45e838c9522c0a6ac05d1a331d07f7e4274ac1cd283c9132c9c4f23572d79b46d5431627f61b44f500a60b080c20798b29f252d3006fc1ee
+  metadata.gz: 1bb49e447b525acdc35f32ea9880103808d6bb6d1fcf3fcec1610601253a579b0c480a8835ef403528fa2f95803812d332abf31a7c52273df64177673042bf08
+  data.tar.gz: 03dc6ced9fa57fac243d2c9e38f8eb39312e10fc04a537945516af8d7b45d33e4d0300c1c7c26c3ebfcec6f3a008f5ca56a2a88d5a27e95028a12996f558a32d

data/bin/query_mtg

@@ -1,6 +1,6 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
 
-require_relative '../lib/query_mtg.rb'
+require_relative '../lib/query_mtg'
 
 QueryMtg.call(ARGV)
-

data/lib/cards_manager.rb

@@ -0,0 +1,52 @@
+# frozen_string_literal: true
+
+require_relative 'query'
+
+class CardsManager
+  InvalidValue = Class.new(StandardError)
+  UnknownAttribute = Class.new(StandardError)
+
+  attr_reader :query, :cards_source
+
+  def initialize(query, cards_source)
+    @query = query
+    @cards_source = cards_source
+  end
+
+  def requested_cards
+    return cards_source if query.empty?
+
+    group_and_select(query, cards_source)
+  end
+
+  private
+
+  def group_and_select(query, cards)
+    raise UnknownAttribute unless cards.first.key?(query[0][:attribute])
+
+    selected_cards = matching_cards(query[0], cards)
+    new_query = query[1..]
+
+    return selected_cards if new_query.empty?
+
+    selected_cards.map.with_object({}) do |(attribute_value, cards), hash|
+      hash[attribute_value] = group_and_select(new_query, cards)
+    end
+  end
+
+  def matching_cards(filter, cards)
+    cards.group_by { |card| card[filter[:attribute]] }
+         .select { |key, _| match?(filter[:value], key) }
+  end
+
+  def match?(cli_value, source_value)
+    return true if cli_value == Query::Group
+    return cli_value == source_value if [cli_value, source_value].all? { |val| val.is_a?(String) }
+    return cli_value.to_i == source_value if source_value.is_a?(Numeric)
+
+    cli_value.tally == source_value&.tally
+  rescue NoMethodError
+    message = "`#{cli_value}`: Type of provided value doesn't match type of card attribute"
+    raise InvalidValue, message
+  end
+end

data/lib/cards_source.rb

@@ -0,0 +1,39 @@
+# frozen_string_literal: true
+
+require 'json'
+require_relative 'mtg_api_response'
+
+class CardsSource
+  def self.get(no_cache = nil)
+    new(no_cache).get
+  end
+
+  def get
+    JSON.parse(cards_source)
+  end
+
+  private
+
+  attr_reader :no_cache
+
+  def initialize(no_cache)
+    @no_cache = no_cache
+  end
+
+  def cards_source
+    return File.read(cache_file_path) if File.file?(cache_file_path) && !no_cache
+
+    cache_cards
+    File.read(cache_file_path)
+  end
+
+  def cache_cards
+    File.open(cache_file_path, 'w') do |file|
+      file.write(MtgApiResponse.results)
+    end
+  end
+
+  def cache_file_path
+    File.expand_path('../cache/cards.json', __dir__)
+  end
+end

data/lib/mtg_api_request.rb

@@ -0,0 +1,61 @@
+# frozen_string_literal: true
+
+require 'typhoeus'
+require 'typhoeus/adapters/faraday'
+require 'json'
+
+class MtgApiRequest
+  URL = 'https://api.magicthegathering.io/v1/cards'
+  PARALLEL_COUNT = 40
+
+  attr_reader :results
+
+  def initialize
+    @results = []
+  end
+
+  def make
+    while unfetched_pages.any?
+      @temp_results = []
+      connection.in_parallel do
+        unfetched_pages.sample(PARALLEL_COUNT).each do |page|
+          fetch_page(page)
+        end
+      end
+
+      process_results(@temp_results)
+    end
+  end
+
+  private
+
+  def unfetched_pages
+    @unfetched_pages ||= (1..total_count).to_a
+  end
+
+  def total_count
+    response = Faraday.get(URL)
+    (response.headers['total-count'].to_f / 100).ceil
+  end
+
+  def connection
+    @connection ||= Faraday.new(url: URL) { |faraday| faraday.adapter :typhoeus }
+  end
+
+  def fetch_page(page)
+    url = URL + "?page=#{page}"
+    puts "Fetching: #{url}"
+    response = connection.get(url)
+
+    @temp_results << [page, response]
+  end
+
+  def process_results(temp_results)
+    temp_results.each do |page, result|
+      next unless result.status == 200
+
+      unfetched_pages.delete(page)
+      results << JSON.parse(result.body)
+    end
+  end
+end

data/lib/mtg_api_response.rb

@@ -0,0 +1,14 @@
+# frozen_string_literal: true
+
+require_relative 'mtg_api_request'
+
+class MtgApiResponse
+  def self.results
+    request = MtgApiRequest.new
+    request.make
+
+    request.results
+           .flat_map { |page| page['cards'] }
+           .to_json
+  end
+end

data/lib/query.rb

@@ -0,0 +1,45 @@
+# frozen_string_literal: true
+
+require 'forwardable'
+
+class Query
+  extend Forwardable
+
+  RepeatedFilter = Class.new(StandardError)
+  Group = Class.new
+
+  attr_reader :argv
+
+  def initialize(argv)
+    @argv = argv
+  end
+
+  def build
+    @build ||= argv.map.with_object([]) do |argument, query_options|
+      query_element = parse_argument(argument)
+
+      if query_options.find { |el| el[:attribute] == query_element[:attribute] }
+        message = "#{query_element[:attribute]}: Every filter can be passed only once"
+        raise RepeatedFilter, message
+      end
+
+      query_options << query_element
+    end
+  end
+
+  def_delegators :build, :empty?
+
+  private
+
+  # TODO: pick a divider (something that is unlikely to show up)
+  # potential problem with imageUrl values
+  # NOTE: should allow escaping `#`
+  def parse_argument(argument)
+    return { attribute: argument, value: Group } unless argument.match?('=')
+
+    attribute, value = argument.split('=')
+    value = value&.match?('#') ? value.split('#') : value
+
+    { attribute: attribute, value: value }
+  end
+end

data/lib/query_mtg.rb

@@ -1,34 +1,39 @@
 # frozen_string_literal: true
 
-require 'pry'
+require 'json'
+require_relative 'cards_manager'
+require_relative 'query'
+require_relative 'cards_source'
+
 class QueryMtg
-  def self.build_query(argv)
-    new(argv).build_query
+  def self.call(argv)
+    new(argv).call
   end
 
-  # rubocop:disable Metrics/MethodLength
-  def build_query
-    argv.map.with_object({}) do |argument, query_options|
-      attribute, value = argument.split('=')
-
-      if query_options[attribute.to_sym]
-        query_options[attribute.to_sym][:values] << value
-      else
-        query_options[attribute.to_sym] = {
-          priority: @grouping_priority, values: [value]
-        }
-        @grouping_priority += 1
-      end
-    end
+  def call
+    puts requested_cards
+  rescue Query::RepeatedFilter, CardsManager::InvalidValue, CardsManager::UnknownAttribute => e
+    puts e.message
   end
-  # rubocop:enable Metrics/MethodLength
 
   private
 
-  attr_reader :argv
+  attr_reader :no_cache, :argv
 
   def initialize(argv)
+    @no_cache = argv.delete('--no-cache')
     @argv = argv
-    @grouping_priority = 0
+  end
+
+  def requested_cards
+    CardsManager.new(query, cards_source).requested_cards
+  end
+
+  def query
+    Query.new(argv).build
+  end
+
+  def cards_source
+    CardsSource.get(no_cache)
   end
 end

metadata

@@ -1,17 +1,17 @@
 --- !ruby/object:Gem::Specification
 name: piotr_majewski_magic
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.0.2
 platform: ruby
 authors:
 - Piotr Majewski
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-06-30 00:00:00.000000000 Z
+date: 2021-07-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: typhoeus
+  name: faraday
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
@@ -25,7 +25,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '1.4'
 - !ruby/object:Gem::Dependency
-  name: faraday
+  name: typhoeus
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
@@ -47,6 +47,11 @@ extensions: []
 extra_rdoc_files: []
 files:
 - bin/query_mtg
+- lib/cards_manager.rb
+- lib/cards_source.rb
+- lib/mtg_api_request.rb
+- lib/mtg_api_response.rb
+- lib/query.rb
 - lib/query_mtg.rb
 homepage: http://rubygems.org/gems/piotr_majewski_magic
 licenses: