Найти - Пользователи
Полная версия: Как подготовить проект к код-ревью?
Начало » Python для новичков » Как подготовить проект к код-ревью?
1 2
WhiteTiger
Всем привет. Написал относительно не большой проект на питоне 2. Питон учил именно из-за этой разработки и серьёзного опыта кодинга на нём у меня нет. Из-за этого код проекта имеет явно не лучший вид и появилось у меня желание заказать код-ревью, дабы получить грамотных советов по работе с питоном и улучшению своего проекта.
Но тут встал следующий вопрос - как подготовить проект к код-ревью? Нужно ли схематичное описание работы проекта? Если да, то на сколько подробное? Нужно ли нашпиговать код комментариями?
Буду благодарен за любую информацию.

P.S. Проект представляет собой консольную утилиту. 330Кб кода.
JOHN_16
WhiteTiger
если вы через месяц другой посмотрите на произвольный кусок кода и не сможете понять что он делает и/или как он работает - значит что то с ним не так=)
330кб Питон кода? ЧТо это за утилита такая. Выкладывайте код на каком нибудь ресурсе, кто нибудь да глянет.
ZerG
да. На каком нить гитхабе
ayb
Нормальная такая “утилитка” на 6000-7000 строк кода.
noob_saibot
ayb
Нормальная такая “утилитка” на 6000-7000 строк кода.
Ну оооооочень подробные комментарии.
doza_and
Вы лучше напишите для чего утилитка. Может кому интересно будет.
А так буду банален. Должно быть как минимум:
http://pep8.ru/doc/pep8/
pylint
sphinx или аналогичная документация (т. е. человеческий текст об использовании а не комментарии к функциям).
Возможность одним из стандартных способов установить вашу утилиту.

Мне тоже интересно что это за утилита на 330 килобайт текста. Трудно даже представить откуда такой объем.
4kpt_III
Да любой даже маленький проект по печати документов с GUI если не разносить по модулям потянет на 6000 строк теста. Просто, видимо, на модули не разбивался а все лепилось в один мега файл.
doza_and
4kpt_III
Просто, видимо, на модули не разбивался
Может и разбивался автор не пишет.
4kpt_III
по печати документов с GUI
Так вроде консольная утилита. Но даже для GUI программы 300 Кб это много.
Rodegast
> Ну оооооочень подробные комментарии.

Вот этого как раз и не нужно.

> Да любой даже маленький проект по печати документов с GUI если не разносить по модулям потянет на 6000 строк теста.

Не. Гуй увеличит число строк процентов на 10-15 не больше. На маленьком проекте 6000 строк точно не будет.
ZerG
Автор вобще что-то не пишет. Тут пока полемика
This is a "lo-fi" version of our main content. To view the full version with more information, formatting and images, please click here.
Powered by DjangoBB