Skip to content

Conversation

@vladbahmat
Copy link

No description provided.

@AlexeiBuzuma AlexeiBuzuma added the [Deadline] Iteration 1-2 This is a marker for first and second iterations deadline. label Nov 17, 2019
print(open(file, 'r').read())


def news_parsing():
Copy link
Collaborator

Choose a reason for hiding this comment

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

К сожалению, не работает

# rss-reader https://news.tut.by/rss/geonews/minsk.rss
Traceback (most recent call last):
  File "/usr/local/bin/rss-reader", line 8, in <module>
    sys.exit(main())
  File "/usr/local/lib/python3.8/site-packages/rssreader/cmd.py", line 5, in main
    rssreader.main()
  File "/usr/local/lib/python3.8/site-packages/rssreader/rssreader.py", line 60, in main
    news_parsing()
  File "/usr/local/lib/python3.8/site-packages/rssreader/rssreader.py", line 22, in news_parsing
    making_log(1, "Opened URL for news reading, URL: %s" % args.source)
NameError: name 'args' is not defined


def filewrite(self):
for index in range(len(self.title)):
fp=open("feeddata.txt","a")
Copy link
Collaborator

Choose a reason for hiding this comment

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

Обычно работа с файлами идет через контекстные менеджеры, очень советую их использовать :)

Copy link
Author

Choose a reason for hiding this comment

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

Исправил, спасибо!

self.links=[]
self.datalist=[]

def feed_find(self):
Copy link
Collaborator

Choose a reason for hiding this comment

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

Советую добавить тут докстринги. Сложновато понять, что это функция делает

@vladbahmat vladbahmat changed the title [WIP] Bakhmat Vladislav Bakhmat Vladislav Dec 1, 2019
@AlexeiBuzuma
Copy link
Collaborator

Выдает следующую ошибку при попытке установить пакет (из-за того, что пропущена запятая в зависимостях)

ERROR: Could not find a version that satisfies the requirement fpdf==1.7.2lxml==4.4.1 (from rssreader==1.3) (from versions: 1.7, 1.7.2)
ERROR: No matching distribution found for fpdf==1.7.2lxml==4.4.1 (from rssreader==1.3)

@AlexeiBuzuma
Copy link
Collaborator

При попытке сгенерировать новости в pdf создается только картинка, но самого pdf файла нет

@vladbahmat
Copy link
Author

При попытке сгенерировать новости в pdf создается только картинка, но самого pdf файла нет

такой расклад существует при условии если текст содержит кирилицу или некие некодируемые символы, вроде я описал такое в readme

@AlexeiBuzuma
Copy link
Collaborator

  1. при работе с файломи в пайтоне обычно используются контекстные менеджеры (ключевое слово with). А данном решении местами встречается открытие файла без контекстного менеджера (и без закрытия этого файла). Например файл rssreader.py строка 150
  2. есть много магических чисел и в целом очень странных ситуаций. Например строка 147 в файле rssreader.py. В данном случае объект image является итерируемым, более того, мы от него отрезаем часть в начале и чать в конце. У меня нет идей, для чего это нужно, понять это достаточно сложно.
  3. функция feed_find достаточно сложна для понимания. Возможно есть смысл ее каким-то образом разбить на несколько маленьких
  4. функция fileread. Функциональность с маппингом названий месяцев уже готова и ее можно использовать :) В данной ситуации может очень сильно помочь модуль datetime.
  5. в решении нет юнит-тестов
  6. не хватает документации к коду. Все функции достаточно сложные, было бы очень хорошо иметь подробное описание, что происходит внутри.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants