|
|
Регистрация Восстановить пароль |
Регистрация | Задать вопрос |
Заплачу за решение |
Новые сообщения |
Сообщения за день |
Расширенный поиск |
Правила |
Всё прочитано |
|
Опции темы | Поиск в этой теме |
03.08.2012, 12:18 | #1 |
Регистрация: 03.08.2012
Сообщений: 4
|
Прокомментируйте код
прокомментируйте пожалуйста код. Жду ваших замечаний и пожеланий.
p.s. Знаю что не писал ф-ю проверки ввода. |
03.08.2012, 12:52 | #2 |
Регистрация: 03.08.2012
Сообщений: 4
|
Код мой. Мне действительно очень важно ваше мнение!
Особенно вопросы читабельности и использование глобальных переменных. |
03.08.2012, 13:05 | #3 |
Пользователь
Регистрация: 15.12.2010
Сообщений: 78
|
посмотрел я на код
бросилось в глаза что переменные класса обьявлены public, это не очень хорошо, лучше обьявлять protected чтобы допускать меньше ошибок. кстати попробуй какое то время ставить скобочки вот так : Код:
А так в целом код хороший) продолжай писать |
03.08.2012, 13:31 | #4 |
Регистрация: 03.08.2012
Сообщений: 4
|
Спасибо blacktener, учту.
|
03.08.2012, 13:45 | #5 |
Форумчанин
Регистрация: 08.07.2010
Сообщений: 505
|
2Nicolas_C++
Если смотреть с позиции Си - то все хорошо, процедурное программирование и так далее с плохой сопровождаемостью и расширением. А так, если посмотреть с точки зрения С++ - рефакторинг с выделением классов, полиморфностью и инкапсуляцией. Читабельность и сопровождаемость увеличиться в разы. Удачи.
Знающий не говорит, говорящий не знает (С) Лао Цзы
|
Похожие темы | ||||
Тема | Автор | Раздел | Ответов | Последнее сообщение |
Прокомментируйте код | ArmanPrestige | Общие вопросы C/C++ | 3 | 10.07.2012 08:21 |
Самая длинное слово в строке (прокомментируйте код) | Alexbmgn | Помощь студентам | 0 | 16.05.2012 17:57 |
Прокомментируйте маленький код | sektor2011 | Visual C++ | 0 | 12.05.2011 17:24 |
Прокомментируйте код | ZyaT-OK | Помощь студентам | 11 | 17.01.2011 21:48 |
Прокомментируйте и объясните код | Serjuk | Помощь студентам | 1 | 18.05.2010 17:56 |