search

Vitosh / PythonHB

Python code.
3 stars 5 forks source link

Feedback по crawler-a #1

Open RadoRado opened 9 years ago

RadoRado commented 9 years ago

Първо, евала! Положил си усилия и си получил резултат. Надявам се, че си научил и нови неща покрай тази задача :)

Сега, feedback:

  1. Кода не е PEP8. Функциите не почват с главни букви. Моля те, оправи го това - мини целия код през PEP8.
  2. Ей тоя default аргумент - https://github.com/Vitosh/PythonHB/blob/master/Crawling/crawlbg.py#L15 - ще ти изиграе лоша шега. Ето защо - http://docs.python-guide.org/en/latest/writing/gotchas/#mutable-default-arguments - прочети го добре това. Често срещана грешка.
  3. Generate_Table прави твърде много неща. А и името не е хубаво. Тази функция просто crawl-ва. Не трябва да се занимава и с писане в база. Loose coupling!
  4. https://github.com/Vitosh/PythonHB/blob/master/Crawling/crawlbg.py#L41 - честно, има хора, които ако видят този код, ще те изгорят ритуално. Тези хора са по-страшни и от Мелисанра! ( http://www.comicbookmovie.com/images/users/uploads/8073/burningmen1.jpg )
  5. https://github.com/Vitosh/PythonHB/blob/master/Crawling/crawlbg.py#L46 - всички статуси, които са по-големи и равни от 200 и по-малки от 300 са OK статус кодове. Ако някой ти върне 201, ще го изпуснеш.
  6. https://github.com/Vitosh/PythonHB/blob/master/Crawling/crawlbg.py#L55 - за > 6 виж коментара по-горе за Мелисандра! Ритуално изгаряне!

Като за начало, това е :+1:

Vitosh commented 9 years ago

OK thanks :) CU tomorrow :)

2015-05-03 21:58 GMT+03:00 Radoslav Georgiev notifications@github.com:

Първо, евала! Положил си усилия и си получил резултат. Надявам се, че си научил и нови неща покрай тази задача :)

Сега, feedback:

  1. Кода не е PEP8. Функциите не почват с главни букви. Моля те, оправи го това - мини целия код през PEP8.
  2. Ей тоя default аргумент - https://github.com/Vitosh/PythonHB/blob/master/Crawling/crawlbg.py#L15
  3. ще ти изиграе лоша шега. Ето защо - http://docs.python-guide.org/en/latest/writing/gotchas/#mutable-default-arguments
  4. прочети го добре това. Често срещана грешка.
  5. Generate_Table прави твърде много неща. А и името не е хубаво. Тази функция просто crawl-ва. Не трябва да се занимава и с писане в база. Loose coupling! 4. https://github.com/Vitosh/PythonHB/blob/master/Crawling/crawlbg.py#L41
  6. честно, има хора, които ако видят този код, ще те изгорят ритуално. Тези хора са по-страшни и от Мелисанра! ( http://www.comicbookmovie.com/images/users/uploads/8073/burningmen1.jpg ) 5. https://github.com/Vitosh/PythonHB/blob/master/Crawling/crawlbg.py#L46
  7. всички статуси, които са по-големи и равни от 200 и по-малки от 300 са OK статус кодове. Ако някой ти върне 201, ще го изпуснеш. 6. https://github.com/Vitosh/PythonHB/blob/master/Crawling/crawlbg.py#L55
  8. за > 6 виж коментара по-горе за Мелисандра! Ритуално изгаряне!

Като за начало, това е [image: :+1:]

— Reply to this email directly or view it on GitHub https://github.com/Vitosh/PythonHB/issues/1.