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

Поиск:

Ответ в темуСоздание новой темы Создание опроса
> Спешу похвастаться! С удовольствием приму поздравления))) 
V
    Опции темы
PyAlexey
Дата 27.9.2006, 12:09 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Шустрый
*


Профиль
Группа: Участник
Сообщений: 57
Регистрация: 19.9.2006
Где: Россия, Нижний Но вгород

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



ser_gen, а ещё лучше — в Epytext Markup Language (epydoc).
А так, пока нет времени смотреть исходники.
PM   Вверх
pythonwin
Дата 27.9.2006, 13:14 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
****


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

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



Отлично!
может J2A что-то мощное предложит?
PM WWW GTalk Jabber   Вверх
proDiva
Дата 27.9.2006, 13:14 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Шустрый
*


Профиль
Группа: Участник
Сообщений: 130
Регистрация: 19.9.2006
Где: КЧР, Россия

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



Цитата(PyS @  27.9.2006,  12:04 Найти цитируемый пост)
Думаю, что proDiva, будет не против.

Для полноты архива не хватает пробного xml-файла..... чтобы проверить работу программы.

Добавлено @ 13:15 
Цитата(proDiva @ 27.9.2006,  13:14)
Цитата(PyS @  27.9.2006,  12:04 Найти цитируемый пост)
Думаю, что proDiva, будет не против.

Для полноты архива не хватает пробного xml-файла..... чтобы проверить работу программы.

Вот выкладываю.

Это сообщение отредактировал(а) proDiva - 27.9.2006, 15:03
PM MAIL ICQ   Вверх
PyS
Дата 27.9.2006, 13:18 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Шустрый
*


Профиль
Группа: Участник
Сообщений: 149
Регистрация: 21.8.2006
Где: г. Алматы (Казахс тан)

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



Цитата(proDiva @  27.9.2006,  16:14 Найти цитируемый пост)
Присоединённый файл  A9C0H91M.O0X 2,12 Kb

Это xml-файл?
PM MAIL   Вверх
PyAlexey
Дата 27.9.2006, 13:26 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Шустрый
*


Профиль
Группа: Участник
Сообщений: 57
Регистрация: 19.9.2006
Где: Россия, Нижний Но вгород

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



Цитата(PyS @  27.9.2006,  13:18 Найти цитируемый пост)
Это xml-файл? 

Да, со расширением )).
PM   Вверх
PyS
Дата 27.9.2006, 13:46 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Шустрый
*


Профиль
Группа: Участник
Сообщений: 149
Регистрация: 21.8.2006
Где: г. Алматы (Казахс тан)

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



Цитата(proDiva @  27.9.2006,  16:14 Найти цитируемый пост)

Присоединённый файл  A9C0H91M.O0X 2,12 Kb

Спасибо!
PM MAIL   Вверх
PyAlexey
Дата 27.9.2006, 14:05 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Шустрый
*


Профиль
Группа: Участник
Сообщений: 57
Регистрация: 19.9.2006
Где: Россия, Нижний Но вгород

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



proDiva
Короче, посмотрел я код. Как здесь уже говорили, лучше было бы использовать другой модуль для dbf. Также стоило использовать нормальный модуль для xml. Далее. Зачем везде писали путь для файлов? К тому же такие: ‘c:/XML_Editor/’. Надо было использовать переменные, а пути делать относительными.

Честно говоря, мне не понятно, Вы первый раз пишите на Питоне или программу вообще?
PM   Вверх
Vach
Дата 27.9.2006, 14:51 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Опытный
**


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

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



proDiva, какой у тебя wx стоит (или версия другая или я того...)? + при кодировки не cp1251 запустить не получается (мой случай)
PM MAIL ICQ   Вверх
proDiva
Дата 27.9.2006, 15:10 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Шустрый
*


Профиль
Группа: Участник
Сообщений: 130
Регистрация: 19.9.2006
Где: КЧР, Россия

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



Цитата(PyAlexey @  27.9.2006,  14:05 Найти цитируемый пост)
Честно говоря, мне не понятно, Вы первый раз пишите на Питоне или программу вообще? 

Первый раз на питоне
а в замечаниях вы правы........ я говорила, что не уверена в рациональности
насчет путей, исправлю....... просто я делала все по частям, а потом собрала в кучу....... и получилось, что везде прописаны пути)) (мне бывший коллега тоже все время говорил, не вбивать пути...... как видите, продолжаю)
а с xml я работаю впервые и выбрала вариант, который смогла разобрать по частям.

Цитата(Vach @  27.9.2006,  14:51 Найти цитируемый пост)
какой у тебя wx стоит (или версия другая или я того...)?

версия 2.6
PM MAIL ICQ   Вверх
J2A
Дата 27.9.2006, 18:47 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Бывалый
*


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

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



Для первого опыта вполне удачно.

1. Не нужно импортировать os.path, он доступен и при импорте просто os
2. Не рекомендуется делать "from foo import *", следует либо "from foo import Foo, Bar", либо "import foo". Почему: не знаешь точно какие переменные импортировала. Так кстати и быстрей.
3. По стилю все же рекомендуется классы называть с большой буквы, т.е. не "class foo", а "class Foo"
4. Переформатирования слишком сложны (класс dataload). Можно сделать гораздо проще. Например так: 
а) переформатирование даты '.'.join(reversed('2006-05-25'.split('-'))) (можно для увеличения читабельности разбить по переменным).
б) переформатирования дробного "%.2f" % value
в) выравнивание строки пробелами влево/вправо: mystr.ljust(n), аналогично и .rjust
г) ну и я бы не оформлял как класс (никакого преимущества нет, т.к. не использует ни один атрибут), а оформил как функции. С учетом вышеперечисленных примеров, выделять в функцию стоит лишь переформатирование даты, остальное и так в одну конструкцию smile
5. Не стоит городить огород с if .. elif .. elif .. elif .. else. Возможно, проще будет записать словарем.
6. Всё же лучше разбить по модулям. 70кб-исходник тяжко читать, да и реиспользовать потом проще будет.

Извини, дальше меня не хватило...

Это сообщение отредактировал(а) J2A - 27.9.2006, 19:20
--------------------
Be easy, stay cool
PM MAIL WWW Jabber   Вверх
proDiva
Дата 27.9.2006, 19:31 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Шустрый
*


Профиль
Группа: Участник
Сообщений: 130
Регистрация: 19.9.2006
Где: КЧР, Россия

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



Цитата(J2A @  27.9.2006,  18:47 Найти цитируемый пост)
Извини, дальше меня не хватило...

На долго же тебя хватило smile  Нужно это все записать, а то не запомню все сразу.
Я про это и говорила, что не знаю готовых функций для того или иного действия, поэтому громоздила громадный код. По всем этим мелочам неудобно новые темы открывать и мучать людей своими вопросами smile 

Я знаю, что все можно было написать проще и удобнее. Но я все равно довольна, что я ее доделала, а главное - что она работает! Потому что когда я начинала ее писать, я питоновский код впервые в глаза увидела. Как наш зав.кафедрой на лекциях говорил: "Есть прорыв! Идем дальше!"
Так что, интеллигентная мужская компания, продолжайте учить дальше smile 

PM MAIL ICQ   Вверх
PyAlexey
Дата 27.9.2006, 20:12 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Шустрый
*


Профиль
Группа: Участник
Сообщений: 57
Регистрация: 19.9.2006
Где: Россия, Нижний Но вгород

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



Кстати, проверьте свой код на вшивость: http://pychecker.sourceforge.net/
PM   Вверх
J2A
Дата 27.9.2006, 20:36 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Бывалый
*


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

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



Цитата(PyAlexey @ 27.9.2006,  23:12)
Кстати, проверьте свой код на вшивость: http://pychecker.sourceforge.net/

Проверьте pylint'ом и ужаснитесь smile это не к коду ProDiva или PyAlexey, а ко всем smile

Это сообщение отредактировал(а) J2A - 27.9.2006, 20:44
--------------------
Be easy, stay cool
PM MAIL WWW Jabber   Вверх
pythonwin
Дата 28.9.2006, 07:39 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
****


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

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



Цитата(J2A @  28.9.2006,  03:36 Найти цитируемый пост)
Проверьте pylint'ом и ужаснитесь smile это не к коду ProDiva или PyAlexey, а ко всем smile

Это что такое?
PM WWW GTalk Jabber   Вверх
pythonwin
Дата 28.9.2006, 08:21 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
****


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

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



Цитата(J2A @  28.9.2006,  03:36 Найти цитируемый пост)
роверьте pylint'ом и ужаснитесь smile

А можешь показать пример использования? smile

Это сообщение отредактировал(а) pythonwin - 28.9.2006, 08:21
PM WWW GTalk Jabber   Вверх
Страницы: (3) Все 1 [2] 3 
Ответ в темуСоздание новой темы Создание опроса
1 Пользователей читают эту тему (1 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема | Python: Общие вопросы | Следующая тема »


 




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


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

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