|
|
Регистрация Восстановить пароль |
Повторная активизация e-mail |
Регистрация | Задать вопрос |
Заплачу за решение |
Новые сообщения |
Сообщения за день |
Расширенный поиск |
Правила |
Всё прочитано |
|
Опции темы | Поиск в этой теме |
26.07.2017, 12:26 | #1 |
Форумчанин
Регистрация: 13.07.2012
Сообщений: 357
|
Можно ли улучшить этот код?
Доброго времени суток.
Собственно сабж) если кому ни будь не будет лень это запускать) https://github.com/TinoAjato/WFWS2 К сожалению в сообщение 6 классов не поместились. Всего несколько вопросов, можно ли улучшить этот код и как? |
26.07.2017, 13:28 | #2 |
Старожил
Регистрация: 12.01.2011
Сообщений: 19,500
|
Да конечно можно.
Не особо разбирался что он делает, но например из общего
А, ну и еще выкинуть Java и взять Kotlin
Ушел с форума, https://www.programmersforum.rocks, alex.pantec@gmail.com, https://github.com/AlexP11223
ЛС отключены Аларом. |
26.07.2017, 13:48 | #3 |
Форумчанин
Регистрация: 13.07.2012
Сообщений: 357
|
Ага, спасибо.
1. Значит с точки зрения проектирования всё сделано правильно? 2. Так много комментариев это же хорошо. Ну, если не покусан белками-истеричками. А по поводу писать понятно - сложно выдумывать названия переменных, особенно если их много. А если почти одинаковых классов дофига, то всё же лучше использовать 1 абстрактный с реализацией половины общих методов, а другую половину реализовать в каждом конкретном? Или так можно не всегда(по хорошему)? Последний раз редактировалось Artsiom; 26.07.2017 в 14:10. |
26.07.2017, 14:51 | #4 |
Старожил
Регистрация: 12.01.2011
Сообщений: 19,500
|
я ж говорил, что не смотрел особо это, даже не в курсе какая задача решалась.
Плохо, они могут не соответствовать действительности, их надо обновлять (и компилятор не заставит) и т.п., удобнее просто сразу читать понятный код. Ну если речь конечно не о документации публичного API или действительно не очевидных вещах (оптимизации и т.п.) Зависит от ситуации, может "дофига классов" вообще не нужны.
Ушел с форума, https://www.programmersforum.rocks, alex.pantec@gmail.com, https://github.com/AlexP11223
ЛС отключены Аларом. |
26.07.2017, 16:09 | #5 | |
Форумчанин
Регистрация: 13.07.2012
Сообщений: 357
|
Цитата:
Есть родительское окно на нем располагаются дочерние. Дочерние(сейчас работаю над справочниками - 15 штук, вот тут и добавил абстрактный класс) все как под копирку(почти). Различие только в наполнении компонентами(поле ввода, комбобокс, радикнопка) модальных окон этих справочников. А на гите только вырезанная часть из основной проги. Кажется я таки смог понять, зачем нужен абстрактный класс. Код:
Последний раз редактировалось Artsiom; 26.07.2017 в 16:15. |
|
Похожие темы | ||||
Тема | Автор | Раздел | Ответов | Последнее сообщение |
Можно ли перевести этот код с С++ на Pascal? | lixod96 | Паскаль, Turbo Pascal, PascalABC.NET | 0 | 28.04.2014 16:15 |
На каком языке этот код и можно ли его переделать? | thedoctor | Общие вопросы C/C++ | 8 | 11.01.2014 14:18 |
Как можно улучшить код для синтаксической подсветки? (Delphi 7)) | munthrekosh | Общие вопросы Delphi | 5 | 24.03.2012 08:29 |
Как можно улучшить код | @ndreiKO | Помощь студентам | 5 | 23.08.2011 04:21 |
Как можно улучшить эффективность и качество ПП? | Evgenii | Помощь студентам | 5 | 29.01.2009 16:22 |