Skip to content

Convert rss to json#10

Open
God1-ike wants to merge 12 commits intoulstu:masterfrom
God1-ike:feature
Open

Convert rss to json#10
God1-ike wants to merge 12 commits intoulstu:masterfrom
God1-ike:feature

Conversation

@God1-ike
Copy link
Copy Markdown

@God1-ike God1-ike commented Mar 6, 2020

No description provided.

# frozen_string_literal: true

module InputFormat
def self.format(data)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PARSERS = ['JsonParser', 'RssParser', 'AtomParser']

Salikov/main.rb Outdated
end

@input_format = InputFormat.format(data)
puts @input_format
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

parsed_data = @input_format.parse(data)

@@ -0,0 +1,10 @@
# frozen_string_literal: true

module BaseParsers
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

BaseParser - в единственном числе.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

хорошо, исправлю

@@ -0,0 +1,3 @@
module DescSorter
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Не доделал (

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

да, AscSorter тоже не работает. Возник вопрос сортировка должна быть только по дате?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Да, только по дате.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

хорошо, спасибо

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants