Форум программистов, компьютерный форум 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
1908 / 1517 / 139
Регистрация: 31.05.2009
Сообщений: 2,876
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. Лично я бы не стал использовать ООП для написания Пятнашек
 
Текущее время: 06:34. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2016, vBulletin Solutions, Inc.
Рейтинг@Mail.ru