Форум программистов
 

Восстановите пароль или Зарегистрируйтесь на форуме, о проблемах и с заказом рекламы пишите сюда - alarforum@yandex.ru, проверяйте папку спам!

Вернуться   Форум программистов > C/C++ программирование > Общие вопросы C/C++
Регистрация

Восстановить пароль
Повторная активизация e-mail

Купить рекламу на форуме - 42 тыс руб за месяц

Ответ
 
Опции темы Поиск в этой теме
Старый 03.08.2012, 12:18   #1
Nicolas_C++
 
Регистрация: 03.08.2012
Сообщений: 4
По умолчанию Прокомментируйте код

прокомментируйте пожалуйста код. Жду ваших замечаний и пожеланий.
p.s. Знаю что не писал ф-ю проверки ввода.
Вложения
Тип файла: txt race.txt (14.8 Кб, 135 просмотров)
Nicolas_C++ вне форума Ответить с цитированием
Старый 03.08.2012, 12:52   #2
Nicolas_C++
 
Регистрация: 03.08.2012
Сообщений: 4
По умолчанию

Код мой. Мне действительно очень важно ваше мнение!
Особенно вопросы читабельности и использование глобальных переменных.
Nicolas_C++ вне форума Ответить с цитированием
Старый 03.08.2012, 13:05   #3
blacktener
Пользователь
 
Регистрация: 15.12.2010
Сообщений: 78
По умолчанию

посмотрел я на код
бросилось в глаза что переменные класса обьявлены public, это не очень хорошо, лучше обьявлять protected чтобы допускать меньше ошибок.
кстати попробуй какое то время ставить скобочки вот так :
Код:
void superFunc(int a, int b) {
    if (a > b) {
        return a;
    }
    return b;
}
может тебе так понравится больше. Лично мне так намного проще читать код.
А так в целом код хороший) продолжай писать
blacktener вне форума Ответить с цитированием
Старый 03.08.2012, 13:31   #4
Nicolas_C++
 
Регистрация: 03.08.2012
Сообщений: 4
По умолчанию

Спасибо blacktener, учту.
Nicolas_C++ вне форума Ответить с цитированием
Старый 03.08.2012, 13:45   #5
Krok27
Форумчанин
 
Аватар для Krok27
 
Регистрация: 08.07.2010
Сообщений: 505
По умолчанию

2Nicolas_C++
Если смотреть с позиции Си - то все хорошо, процедурное программирование и так далее с плохой сопровождаемостью и расширением.

А так, если посмотреть с точки зрения С++ - рефакторинг с выделением классов, полиморфностью и инкапсуляцией.
Читабельность и сопровождаемость увеличиться в разы.
Удачи.
Знающий не говорит, говорящий не знает (С) Лао Цзы
Krok27 вне форума Ответить с цитированием
Ответ


Купить рекламу на форуме - 42 тыс руб за месяц



Похожие темы
Тема Автор Раздел Ответов Последнее сообщение
Прокомментируйте код 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