Уведомления

Группа в Telegram: @pythonsu

#1 Авг. 12, 2014 10:45:20

ruslanpa
Зарегистрирован: 2014-08-12
Сообщения: 3
Репутация: +  0  -
Профиль   Отправить e-mail  

code review

Добрый день

Начинаю изучать python, сделал небольшой проект, но нуждаюсь в code review, что делаю не правльно, буду благодарен за любую критику.

https://github.com/ruslanpa/currency-cash

Спасибо.

Отредактировано ruslanpa (Авг. 12, 2014 10:49:42)

Офлайн

#2 Авг. 12, 2014 10:49:49

doza_and
От:
Зарегистрирован: 2010-08-15
Сообщения: 4138
Репутация: +  253  -
Профиль   Отправить e-mail  

code review

Надо написать в readme для чего это все нужно.



Офлайн

#3 Авг. 12, 2014 11:01:34

ruslanpa
Зарегистрирован: 2014-08-12
Сообщения: 3
Репутация: +  0  -
Профиль   Отправить e-mail  

code review

doza_and
Надо написать в readme для чего это все нужно.
Да, спасибо, подправил

Офлайн

#4 Авг. 12, 2014 11:33:51

4kpt_II
От: Харьков
Зарегистрирован: 2013-10-24
Сообщения: 999
Репутация: +  58  -
Профиль   Отправить e-mail  

code review

1. Посмотрите в сторону Flask-Classy.
2. PEP8.
3. Проще:

organization = Organization(item['title'], item['address'], item['phone'])

organization = Organization(**item)

4. Зависимости.

Офлайн

#5 Авг. 12, 2014 12:17:06

ruslanpa
Зарегистрирован: 2014-08-12
Сообщения: 3
Репутация: +  0  -
Профиль   Отправить e-mail  

code review

4kpt_II
1. Посмотрите в сторону Flask-Classy.2. PEP8.3. Проще:
Спасибо, буду смотреть

Офлайн

Board footer

Модераторировать

Powered by DjangoBB

Lo-Fi Version