Форум программистов, компьютерный форум CyberForum.ru
Наши страницы

ООП - C++

Войти
Регистрация
Восстановить пароль
Другие темы раздела
C++ Односвязные списки c++ http://www.cyberforum.ru/cpp-beginners/thread147907.html
Надо написать функцию, которая вставляет в список элемент со значением n после первого элемента со значением m. Реализовать как односвязный список(используя динамические структуры). Должны быть...
C++ Работа с текстовыми файлами Создать программу, которая обрабатывает файл в последовательном режиме. Данные надо считывать по одному символу (нельзя считывать сразу весь файл или строку) Дан не пустой текстовый файл.... http://www.cyberforum.ru/cpp-beginners/thread147901.html
строки и множества в С C++
Вывести в обратном порядке третье слово
C++ Матрицы в С++
Здравствуйте, форумчане...я, уповая на ваше помощь, прошу помочь мне с программой... Смысл задачи: Дана символьная матрица. Характеристикой строки назовем длину строки. Определить номер строки,...
C++ вывести элементы массива целых чисел http://www.cyberforum.ru/cpp-beginners/thread147887.html
вывести элементы массива целых чисел, которые находится между max и min
C++ Функция которая сравнивает два целых числа и возвращает результат сравнения в виде знака: >, < или = Используя функцию, которая сравнивает два целых числа и возвращает результат сравнения в виде знака: >, < или =, вычислить количество равных среди введенных трех чисел x, у, t. подробнее

Показать сообщение отдельно
rangerx
1933 / 1542 / 141
Регистрация: 31.05.2009
Сообщений: 2,912
22.06.2010, 13:38
Цитата Сообщение от _Eldar_ Посмотреть сообщение
вот код (принимаю критику):
Цитата Сообщение от _Eldar_ Посмотреть сообщение
C++
1
2
3
 for(int i = 0; i < 16; ++i)
    if (i < 15) arr.push_back(i + 1);
        else arr.push_back(-1);
Хм...

Цитата Сообщение от _Eldar_ Посмотреть сообщение
C++
1
2
3
4
while(!game_field.Validate_End()){
    game_field.Show();
    game_field.Move();
}
1) Где в программе комментарий, что надо именно так использовать данный класс(т.е. вызывать методы в таком порядке)?
2) Зачем вообще этому всему быть открытым?

Цитата Сообщение от _Eldar_ Посмотреть сообщение
C++
1
int num_X; // индекс пустого элемента (-1)
Вместо этого комментария лучше бы здесь было имя переменной говорящее само за себя.

Цитата Сообщение от _Eldar_ Посмотреть сообщение
C++
1
std::vector<int> arr; // Игровое поле
То же самое насчёт имени и комментария + не вижу никакого смысла в использовании вектора в данной программе.

Цитата Сообщение от _Eldar_ Посмотреть сообщение
C++
1
2
3
int temp = arr[num_field];
 arr[num_field] = arr[num_X];
 arr[num_X] = temp;
Используй std::swap вместо этого.

Плюс ко всему в коде куча "магических чисел"(15,16,4).

P.S. Лично я бы не стал использовать ООП для написания Пятнашек
0
 
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2017, vBulletin Solutions, Inc.
Рейтинг@Mail.ru