Уведомления

Группа в Telegram: @pythonsu

#1 Дек. 1, 2016 11:58:19

frpaul
От:
Зарегистрирован: 2009-08-21
Сообщения: 29
Репутация: +  0  -
Профиль   Отправить e-mail  

Slacker Tracker - "классный журнал" для преподавателя

Ничего похожего не нашел (Moodle и прочие “мостры” - не в счет), поэтому решил сделать для себя такой журнал. Я не профессиональный программист, так что не судите строго.

python2 + PyGTK + sqlite3

Честно признаюсь, под Виндоуз запускать не пробовал.

Проект на гитхабе



Офлайн

#2 Дек. 1, 2016 12:07:50

FishHook
От:
Зарегистрирован: 2011-01-08
Сообщения: 8312
Репутация: +  568  -
Профиль   Отправить e-mail  

Slacker Tracker - "классный журнал" для преподавателя

frpaul

User has to create a set of text files: student_list.txt, event_list.txt accoring to the default ones.
нахрена?



Офлайн

#3 Дек. 1, 2016 13:00:51

frpaul
От:
Зарегистрирован: 2009-08-21
Сообщения: 29
Репутация: +  0  -
Профиль   Отправить e-mail  

Slacker Tracker - "классный журнал" для преподавателя

Еще не запилил меню, чтобы делать это через GUI. Меня самого вариант с текстовыми файлами пока устраивает, но я понимаю необходимость изменений.



Офлайн

#4 Дек. 1, 2016 13:10:34

FishHook
От:
Зарегистрирован: 2011-01-08
Сообщения: 8312
Репутация: +  568  -
Профиль   Отправить e-mail  

Slacker Tracker - "классный журнал" для преподавателя

frpaul
И вы думаете, что в таком виде ваше поделие будет хоть кому-то интересно?



Офлайн

#5 Дек. 1, 2016 13:27:55

ZerG
Зарегистрирован: 2012-04-05
Сообщения: 2627
Репутация: +  61  -
Профиль   Отправить e-mail  

Slacker Tracker - "классный журнал" для преподавателя

Я непример вобще не понял что это и зачем
На гите даже примеров с гуи нету (скринов)



Влодение рускай арфаграфией - это как владение кунг-фу: настаящие мастира не преминяют ево бес ниабхадимости

Офлайн

#6 Дек. 1, 2016 14:00:36

frpaul
От:
Зарегистрирован: 2009-08-21
Сообщения: 29
Репутация: +  0  -
Профиль   Отправить e-mail  

Slacker Tracker - "классный журнал" для преподавателя

ZerG
Я непример вобще не понял что это и зачемНа гите даже примеров с гуи нету (скринов)

Вы на гите много таких страниц видели?
Выложу позже. Пока вот скриншот:

https://yadi.sk/i/9Av_DhWUzv4MF

И да, там русский Readme есть



Отредактировано frpaul (Дек. 1, 2016 14:02:26)

Офлайн

#7 Дек. 1, 2016 14:12:04

frpaul
От:
Зарегистрирован: 2009-08-21
Сообщения: 29
Репутация: +  0  -
Профиль   Отправить e-mail  

Slacker Tracker - "классный журнал" для преподавателя

FishHook
frpaulИ вы думаете, что в таком виде ваше поделие будет хоть кому-то интересно?

Что еще не так? Я пришел сюда за конкретными замечаниями, а не за ушатом помоев. Здесь ведь не клон LOR'а?



Офлайн

#8 Дек. 1, 2016 15:17:42

JOHN_16
От: Россия, Петропавловск-Камчатск
Зарегистрирован: 2010-03-22
Сообщения: 3292
Репутация: +  221  -
Профиль   Отправить e-mail  

Slacker Tracker - "классный журнал" для преподавателя

1) str.py - str как то вот пересекается с встроенным типом str - это не хорошо.
2) обилие за комментированного отладочного и/или устаревшего кода в master ветке - это не хорошо
3) python 2 - все таки считается устаревшим
4) стиль кода местами не соответствует PEP8 (например такие длинные строки - ну это вот прям глаза режет)
5) нету единого стиля кода по самому проекту
6) вот эта переменная итак глобальная так как определена на уровне модуля и, судя по дальнейшему использованию, вы не понимаете как в питоне работает global
7)

 if new_text == "Late" or new_text == "L":
new_text = "L"
ничего не смущает? А если внимательно посмотреть? зачем делать операции вида а = а ?
8) А что если в имени вкладки будет строка, позволяющая провести SQL Injection аттаку ? будет с вашей БД все что угодно
9) Основной файл под 3к строчек, с несколькими базовыми классами. Не кажется вам что было бы логичнее разнести код на отдельные модули?
10) судя по другим файлам в проекте, я не углублся, но могу предположить что у Вас дублируется функционал в следствии плохой архитектуры приложения в целом.
Вот вам мой комментарий, диагональным взглядом



_________________________________________________________________________________
полезный блог о python john16blog.blogspot.com

Отредактировано JOHN_16 (Дек. 1, 2016 15:18:59)

Офлайн

#9 Дек. 1, 2016 15:48:32

FishHook
От:
Зарегистрирован: 2011-01-08
Сообщения: 8312
Репутация: +  568  -
Профиль   Отправить e-mail  

Slacker Tracker - "классный журнал" для преподавателя

frpaul
Что еще не так?
Дружище, ты семь лет уже на форуме, ты до сих пор не понял, какая ветка для каких целей служит?
Если нужна инспекция кода - выкладывай его в новичков, будут конструктивные замечания по коду. А здесь “Python проекты”, соответственно оценивается проект как таковой, и как таковой твой проект кроме тебя никто никогда использовать не будет и никому он не нужен.



Офлайн

#10 Дек. 1, 2016 17:47:02

frpaul
От:
Зарегистрирован: 2009-08-21
Сообщения: 29
Репутация: +  0  -
Профиль   Отправить e-mail  

Slacker Tracker - "классный журнал" для преподавателя

JOHN_16
1) str.py - str как то вот пересекается с встроенным типом str - это не хорошо.2) обилие за комментированного отладочного и/или устаревшего кода в master ветке - это не хорошо3) python 2 - все таки считается устаревшим4) стиль кода местами не соответствует PEP8 (например такие длинные строки - ну это вот прям глаза режет)5) нету единого стиля кода по самому проекту6) вот эта переменная итак глобальная так как определена на уровне модуля и, судя по дальнейшему использованию, вы не понимаете как в питоне работает global7)
Спасибо! Постараюсь учесть. А что касается бранчей - разработку лучше вести не в master, а потом мержить? Насчёт глобальных переменных все верно - я обычно стараюсь вообще ими не пользоваться.



Офлайн

Board footer

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

Powered by DjangoBB

Lo-Fi Version