WhiteTiger
Окт. 27, 2015 11:18:55
Всем привет. Написал относительно не большой проект на питоне 2. Питон учил именно из-за этой разработки и серьёзного опыта кодинга на нём у меня нет. Из-за этого код проекта имеет явно не лучший вид и появилось у меня желание заказать код-ревью, дабы получить грамотных советов по работе с питоном и улучшению своего проекта.
Но тут встал следующий вопрос - как подготовить проект к код-ревью? Нужно ли схематичное описание работы проекта? Если да, то на сколько подробное? Нужно ли нашпиговать код комментариями?
Буду благодарен за любую информацию.
P.S. Проект представляет собой консольную утилиту. 330Кб кода.
JOHN_16
Окт. 27, 2015 11:47:44
WhiteTiger
если вы через месяц другой посмотрите на произвольный кусок кода и не сможете понять что он делает и/или как он работает - значит что то с ним не так=)
330кб Питон кода? ЧТо это за утилита такая. Выкладывайте код на каком нибудь ресурсе, кто нибудь да глянет.
ZerG
Окт. 27, 2015 12:08:22
да. На каком нить гитхабе
ayb
Окт. 27, 2015 14:52:46
Нормальная такая “утилитка” на 6000-7000 строк кода.
noob_saibot
Окт. 27, 2015 15:36:12
ayb
Нормальная такая “утилитка” на 6000-7000 строк кода.
Ну оооооочень подробные комментарии.
doza_and
Окт. 27, 2015 18:10:20
Вы лучше напишите для чего утилитка. Может кому интересно будет.
А так буду банален. Должно быть как минимум:
http://pep8.ru/doc/pep8/pylint
sphinx или аналогичная документация (т. е. человеческий текст об использовании а не комментарии к функциям).
Возможность одним из стандартных способов установить вашу утилиту.
Мне тоже интересно что это за утилита на 330 килобайт текста. Трудно даже представить откуда такой объем.
4kpt_III
Окт. 27, 2015 18:19:41
Да любой даже маленький проект по печати документов с GUI если не разносить по модулям потянет на 6000 строк теста. Просто, видимо, на модули не разбивался а все лепилось в один мега файл.
doza_and
Окт. 27, 2015 19:13:39
4kpt_III
Просто, видимо, на модули не разбивался
Может и разбивался автор не пишет.
4kpt_III
по печати документов с GUI
Так вроде консольная утилита. Но даже для GUI программы 300 Кб это много.
Rodegast
Окт. 27, 2015 19:52:28
> Ну оооооочень подробные комментарии.
Вот этого как раз и не нужно.
> Да любой даже маленький проект по печати документов с GUI если не разносить по модулям потянет на 6000 строк теста.
Не. Гуй увеличит число строк процентов на 10-15 не больше. На маленьком проекте 6000 строк точно не будет.
ZerG
Окт. 27, 2015 20:19:04
Автор вобще что-то не пишет. Тут пока полемика