Python-сообщество

ruslanpa · Авг. 12, 2014 10:45:20

Добрый день

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

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

Спасибо.

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

doza_and · Авг. 12, 2014 10:49:49

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

ruslanpa · Авг. 12, 2014 11:01:34

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

Да, спасибо, подправил

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

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

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

organization = Organization(**item)

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

ruslanpa · Авг. 12, 2014 12:17:06

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

Спасибо, буду смотреть

Python-сообщество

Уведомления

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

code review

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

code review

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

code review

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

code review

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

code review

Board footer