Модераторы: Daevaorn
  

Поиск:

Ответ в темуСоздание новой темы Создание опроса
> Зацените программу, Нуждаюсь в критике 
:(
    Опции темы
aspolyakov
  Дата 26.1.2009, 07:47 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Новичок



Профиль
Группа: Участник
Сообщений: 29
Регистрация: 13.9.2008
Где: Russia, Urals, Ye katerinburg

Репутация: нет
Всего: нет



Хеллоу! Я хотел бы услышать комменты к своей софтине: http://sourceforge.net/projects/mydbf2mysql/
Любые, и по стилю, и рекомендации, советы.
PM MAIL WWW ICQ Skype   Вверх
alex_smirnov
Дата 26.1.2009, 09:37 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Бывалый
*


Профиль
Группа: Участник
Сообщений: 194
Регистрация: 22.6.2007
Где: Санкт-Петербург

Репутация: 2
Всего: 4



Не могу заценить. Не являюсь пользователем dbf формата.

Но по коду: блин :(, 
 * ограничение на длинну строки в 79 символов, где?
 * заворачивание минимальных кусков кода в try except, это явно не про вас smile
 * окружение мат. знаков пробелами
 * 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

Желаю больших успехов! smile 

ЗЫ: если хочется делать действительно открытое и качественное ПО о качестве кода надо думать постоянно, кстати, и о документации тоже.
PM GTalk Jabber   Вверх
aspolyakov
Дата 26.1.2009, 10:06 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Новичок



Профиль
Группа: Участник
Сообщений: 29
Регистрация: 13.9.2008
Где: Russia, Urals, Ye katerinburg

Репутация: нет
Всего: нет



Цитата

* if (current[0] == right_current): , что тут лишнее? и так по всему коду

Что там лишнее? Пробел перед скобкой?
Как надо было оформить код после класса?

Это сообщение отредактировал(а) aspolyakov - 26.1.2009, 10:08
PM MAIL WWW ICQ Skype   Вверх
alex_smirnov
Дата 26.1.2009, 11:25 (ссылка) |    (голосов:2) Загрузка ... Загрузка ... Быстрая цитата Цитата


Бывалый
*


Профиль
Группа: Участник
Сообщений: 194
Регистрация: 22.6.2007
Где: Санкт-Петербург

Репутация: 2
Всего: 4



Лишние круглые скобки вокруг логического выражения.

Код после класса, который подразумевает под собой main лучше оформлять так:

Код

if __name__ == '__main__':
    #main block class

PM GTalk Jabber   Вверх
aspolyakov
Дата 29.1.2009, 20:04 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Новичок



Профиль
Группа: Участник
Сообщений: 29
Регистрация: 13.9.2008
Где: Russia, Urals, Ye katerinburg

Репутация: нет
Всего: нет



поправил, пересмотрел итд. Гляньте снова(последний релиз)
https://sourceforge.net/project/showfiles.p...lease_id=657118
PM MAIL WWW ICQ Skype   Вверх
alex_smirnov
Дата 2.2.2009, 20:51 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Бывалый
*


Профиль
Группа: Участник
Сообщений: 194
Регистрация: 22.6.2007
Где: Санкт-Петербург

Репутация: 2
Всего: 4



Ну что тут скажешь... вложенность кода уменьшилась, теперь не 12, а 9-10 (как смог, посчитал). Легче не стало.
Ну как бы main был выделен, но дальше как был монолитный трудночитаемый код, так и остался. 

Да, немного поправлена пунктуации и кодинг стайл, но этого маловато будет.

Следующий шаг к улучшению (даже если что-то не работает ;)), это выделение сущностей. 
Попробуйте всё это невероятно линейное чудо с ямами разделить на функции или даже классы. Подсказка, начинать с ям.
PM GTalk Jabber   Вверх
  
Ответ в темуСоздание новой темы Создание опроса
1 Пользователей читают эту тему (1 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема | Python: Общие вопросы | Следующая тема »


 




[ Время генерации скрипта: 0.1074 ]   [ Использовано запросов: 21 ]   [ GZIP включён ]


Реклама на сайте     Информационное спонсорство

 
По вопросам размещения рекламы пишите на vladimir(sobaka)vingrad.ru
Отказ от ответственности     Powered by Invision Power Board(R) 1.3 © 2003  IPS, Inc.