1 Тема від generation

  • generation
  • generation
  • Користувач
  • Поза форумом
  • Звідки: c. Hell
  • Дата реєстрації: 02.11.2014
  • Повідомлень: 90
    • Репутація: 7

Тема: Парсер rabota.ua

Добрий день всім.
Отже, створив я легкий та зручний(як на мене) парсер сайту rabota.ua .
Мені потрібно щоб ви вказали на помилки в коді і на архітектуру коду(і як її поліпшити).
Код розміщений на github:

▼Прихований текст

2 Відповідь від ADR Востаннє редагувалося ADR (14.03.2017 02:53:10)

  • ADR
  • ADR
  • Replace Group
  • Поза форумом
  • Дата реєстрації: 28.01.2013
  • Повідомлень: 540
    • Репутація: 254

Re: Парсер rabota.ua

Назва змінної self._pages не відповідає її значенню. По факту це останні сторінка, а не кількість.

3 Відповідь від generation Востаннє редагувалося generation (14.03.2017 12:53:30)

  • generation
  • generation
  • Користувач
  • Поза форумом
  • Звідки: c. Hell
  • Дата реєстрації: 02.11.2014
  • Повідомлень: 90
    • Репутація: 7

Re: Парсер rabota.ua

У мене виникло одне запитання. Чи є така структура даних на пайтоні щоб мала функції collections.OrderedDict та підримувала зрізи? Я не дуже бажаю писати костилі. Я планую з отриманих даних вирізати URL адреси, але якось з регулярними виразами код не дуже красивий стає(бо я використовую list() для збереження всієї інформації), а OrderedDict не підтримує дану функцію. Є варіант використовувати два різні списки, але як на мене це також костиль.
P.S. Все таки написав костиль з двома списками. Але якщо хтось знаю подібну библіотеку то напишіть, будь ласка.

4 Відповідь від generation

  • generation
  • generation
  • Користувач
  • Поза форумом
  • Звідки: c. Hell
  • Дата реєстрації: 02.11.2014
  • Повідомлень: 90
    • Репутація: 7

Re: Парсер rabota.ua

ADR написав:

Назва змінної self._pages не відповідає її значенню. По факту це останні сторінка, а не кількість.

Так, я розумію. Просто завтикав. Дякую) А щодо ООП є якісь недоліки? Мені здається що щось криво, але я не розумію що.