![]() |
Модераторы: Daevaorn |
![]() ![]() ![]() |
|
aspolyakov |
|
|||
Новичок Профиль Группа: Участник Сообщений: 29 Регистрация: 13.9.2008 Где: Russia, Urals, Ye katerinburg Репутация: нет Всего: нет |
Хеллоу! Я хотел бы услышать комменты к своей софтине: http://sourceforge.net/projects/mydbf2mysql/
Любые, и по стилю, и рекомендации, советы. |
|||
|
||||
alex_smirnov |
|
|||
Бывалый ![]() Профиль Группа: Участник Сообщений: 194 Регистрация: 22.6.2007 Где: Санкт-Петербург Репутация: 2 Всего: 4 |
Не могу заценить. Не являюсь пользователем dbf формата.
Но по коду: блин :(, * ограничение на длинну строки в 79 символов, где? * заворачивание минимальных кусков кода в try except, это явно не про вас ![]() * окружение мат. знаков пробелами * if (current[0] == right_current): , что тут лишнее? и так по всему коду *- метод run c двенадцатимерной вложенностью вообще убил * код после класса, для меня сейчас представил манную кашу на завтрак ;) * а ещё cp1251... * ... Из всех тудух нужно на первое место поставить: # - To refactor and optimize some code. Не то, чтобы оптимизировать, а переписать с минимальной вложенностью и максимальной ясностью. Надо помнить Less kode = less bugs, и http://en.wikipedia.org/wiki/KISS_principle Желаю больших успехов! ![]() ЗЫ: если хочется делать действительно открытое и качественное ПО о качестве кода надо думать постоянно, кстати, и о документации тоже. |
|||
|
||||
aspolyakov |
|
|||
Новичок Профиль Группа: Участник Сообщений: 29 Регистрация: 13.9.2008 Где: Russia, Urals, Ye katerinburg Репутация: нет Всего: нет |
Что там лишнее? Пробел перед скобкой? Как надо было оформить код после класса? Это сообщение отредактировал(а) aspolyakov - 26.1.2009, 10:08 |
|||
|
||||
alex_smirnov |
|
|||
Бывалый ![]() Профиль Группа: Участник Сообщений: 194 Регистрация: 22.6.2007 Где: Санкт-Петербург Репутация: 2 Всего: 4 |
Лишние круглые скобки вокруг логического выражения.
Код после класса, который подразумевает под собой main лучше оформлять так:
|
|||
|
||||
aspolyakov |
|
|||
Новичок Профиль Группа: Участник Сообщений: 29 Регистрация: 13.9.2008 Где: Russia, Urals, Ye katerinburg Репутация: нет Всего: нет |
поправил, пересмотрел итд. Гляньте снова(последний релиз)
https://sourceforge.net/project/showfiles.p...lease_id=657118 |
|||
|
||||
alex_smirnov |
|
|||
Бывалый ![]() Профиль Группа: Участник Сообщений: 194 Регистрация: 22.6.2007 Где: Санкт-Петербург Репутация: 2 Всего: 4 |
Ну что тут скажешь... вложенность кода уменьшилась, теперь не 12, а 9-10 (как смог, посчитал). Легче не стало.
Ну как бы main был выделен, но дальше как был монолитный трудночитаемый код, так и остался. Да, немного поправлена пунктуации и кодинг стайл, но этого маловато будет. Следующий шаг к улучшению (даже если что-то не работает ;)), это выделение сущностей. Попробуйте всё это невероятно линейное чудо с ямами разделить на функции или даже классы. Подсказка, начинать с ям. |
|||
|
||||
![]() ![]() ![]() |
1 Пользователей читают эту тему (1 Гостей и 0 Скрытых Пользователей) | |
0 Пользователей: | |
« Предыдущая тема | Python: Общие вопросы | Следующая тема » |
|
По вопросам размещения рекламы пишите на vladimir(sobaka)vingrad.ru
Отказ от ответственности Powered by Invision Power Board(R) 1.3 © 2003 IPS, Inc. |