adapter

lib/newsletter/adapters/json.rb

@@ -0,0 +1,15 @@
+module Newsletter
+  module Adapters
+    class Json
+      attr_reader :source
+
+      def initialize(source)
+        @source = source
+      end
+
+      def parse
+        JSON.parse(source)
+      end
+    end
+  end
+end

lib/newsletter/adapters/xml.rb

@@ -0,0 +1,19 @@
+module Newsletter
+  module Adapters
+    class Xml
+      attr_reader :source
+
+      def initialize(source)
+        @source = source
+      end
+
+      def parse
+        xml = Nokogiri::XML source
+        title = xml.xpath('//item/title')[0].children[0].text
+        body = xml.xpath('//item/body')[0].children[0].text
+
+        { 'title' => title, 'body' => body }
+      end
+    end
+  end
+end

lib/newsletter/content.rb

@@ -3,27 +3,14 @@ class Content
     attr_reader :title, :body
 
     def self.parse(source, format)
-      content = if format == :json then parse_json(source)
-                elsif format == :xml then parse_xml(source)
-                end
+      adapter = Newsletter::Adapters.const_get(format).new(source)
+      content = adapter.parse
       new(content['title'], content['body'])
     end
 
     def initialize(title, body)
       @title = title
       @body = body
     end
-
-    def self.parse_json(source)
-      JSON.parse(source)
-    end
-
-    def self.parse_xml(sourse)
-      xml = Nokogiri::XML sourse
-      title = xml.xpath('//item/title')[0].children[0].text
-      body = xml.xpath('//item/body')[0].children[0].text
-
-      { "title" => title, "body" => body }
-    end
   end
 end

spec/anything_spec.rb

@@ -4,6 +4,8 @@
 require 'pry'
 
 require 'newsletter/content'
+require 'newsletter/adapters/json'
+require 'newsletter/adapters/xml'
 
 module Newsletter
   describe Content do
@@ -14,7 +16,7 @@ module Newsletter
         )
       end
 
-      subject { Content.parse(json, :json) }
+      subject { Content.parse(json, :Json) }
 
       it 'parses the title' do
         expect(subject.title).to eq('Hello world!')
@@ -32,7 +34,7 @@ module Newsletter
         )
       end
 
-      subject { Content.parse(json, :xml) }
+      subject { Content.parse(json, :Xml) }
 
       it 'parses the title' do
         expect(subject.title).to eq('Hello world!')