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

Обещаный консольный графический Тетрис (исходники и релиз) - C++

Войти
Регистрация
Восстановить пароль
Другие темы раздела
C++ Сортировка каталогов и файлов http://www.cyberforum.ru/cpp-beginners/thread445272.html
Нужно создать программу выполняющую сортировку каталогов и файлов различными методами (напр.: по типу, дате создания, имени, расширению и т.д)
C++ C++ Упорядочить все сверхпростые числа по возростанию.Найти к-ое сверхпростое в етом порядке. C++ Упорядочить все сверхпростые числа по возростанию.Найти к-ое сверхпростое в етом порядке. Сверхпростое число - простое число номер которого в списке простых чисел упорядоченым по возростанию... http://www.cyberforum.ru/cpp-beginners/thread445269.html
C++ Аналоговые часы используя библиотеку SDL.
Здравствуйте, нужно написать программу которая будет показывать правильное время на аналоговых часах с тремя стрелками с помощью библиотеки SDL. Можете пожалуйста подсказать как рисуются стрелки в...
C++ Найти самое длинное слово в файле
Есть програма но в ней нужно дописать функцию которая бедет, получать название файла, а возращает позицию самого длинного слова и самое длинное слово выводит на екран. #include <iostream> #include...
C++ разьясните http://www.cyberforum.ru/cpp-beginners/thread445253.html
Почему у меня неработают эти команды в шыгфд ыегвшщ 2008? clrscr(); setcursortype(_NOCURSOR); delay(1000); gotoxy(1,3); БИБЛИОТЕКИ Я ПОДКЛЮЧИЛ! постоянно выбивает ошибку что идентификатор...
C++ Найти самое длинное слова в файле. Есть програма но в ней нужно дописать функцию которая бедет, получать название файла, а возращает самое позицию самого длинного слова и самое длинное слово выводит на екран. #include <iostream>... подробнее

Показать сообщение отдельно
ForEveR
В астрале
Эксперт С++
7978 / 4737 / 321
Регистрация: 24.06.2010
Сообщений: 10,543
Завершенные тесты: 3
13.02.2012, 09:21
IFree Host, Запускать не запускал - смотрю код.

1)
C++
1
2
CBrett(void);
~CBrett(void);
void в параметрах абсолютно не имеет смысла для С++.
2)
C++
1
CConsolenGrafik BrettScreen;
Определен в глобальной видимости. Почему бы не сделать статической переменной класса?
3)
C++
1
2
3
4
        for (char j=0, y = 1+y_location; j<10 , y < y_location+21 ; j++, y=y+2 )
        {
            BrettScreen.Farben(x_location,x_location,y,y+1,0,4);
        }
Слишком много магических чисел. Исходник ими так и пестрит.
4) Файл SpielFeld.cpp - дофига свитчей + опять же две переменные объявлены в глобале, хотя можно сделать статик полями класса. TetrisFiguren - та же фигня.
5)
C++
1
void main()
А вот за такое человека пишущего на С++ надо убивать.

Вообщем - везде функции, не принимающие параметров зачем-то обозначены принимающими void. Магические числа. Неиспользование статических членов класса там, где это явно нужно.
Абсолютное отсутствие константных функций. void main(). Не зачтено.
0
 
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2017, vBulletin Solutions, Inc.
Рейтинг@Mail.ru