статья, как написать игру Собиралка

[pufystyj]

Я пытался написать статью для programmersforum.ru, но код вышел, потому что иногда (т.е. в некоторых его частях) кривоват (может глюки ток у меня, т.к. непонятно что у меня творится на компе, опера виснеьт, компилятор тоже...) и я просто выставляю игру и к ней код (хорошо закоментирован). тому кому не нравится пусть сделает замичание, или пусть промолчит. Сама игра банальная, собираем фрукт.

П.С пожалуйста ставьте + . и все спасиба в верхних коментариях!

воть

вот код под MinGW (переделывать под VS почти не прийдется, главное либы подключить):



вот сама игра:
Воть

вложение - это исходник, я просто заного открывал оперу

[Blade]

Код ужасный.
Функция main() не должна выполнять тех действий, которые выполняет у вас.
Функции принимают слишком большое число аргументов
Имена аргументов совершенно не отражают их смысла
Половина функций работает с double, половина с float, не понятно зачем

void plods (double *x1,double *x2,double *y1,double *y2, int *v)
...
void movement (float *x1,float *x2,float *y1,float *y2, int *v, double *x, double *y, int *l)

Всюду магические числа:

float mex1 = -1.0;
float mex2 = -0.8;
float mey1 = -1.0;
float mey2 = -0.8;

Полное отсутствие констант
Ужасные логические условия

(*x1 > 1.0 || *y1 > 1.0 || *v == 1 || *x1 < - 1.0 || *y1 < - 1.0)

Отсутствие какой-либо логической структуры программы
Комментарии ни сколько не информативны.
Орфография на двойку. И те причины, которые вы описали в файле, не дают вам право на орфографические ошибки

[pufystyj]

но код вышел

но код вышел кривой (мышка скочила и случайно выделила слово)

Сообщение от Blade

Код ужасный.
Функция main() не должна выполнять тех действий, которые выполняет у вас.
Функции принимают слишком большое число аргументов
Имена аргументов совершенно не отражают их смысла
Половина функций работает с double, половина с float, не понятно зачем

void plods (double *x1,double *x2,double *y1,double *y2, int *v)
...
void movement (float *x1,float *x2,float *y1,float *y2, int *v, double *x, double *y, int *l)

Всюду магические числа:

float mex1 = -1.0;
float mex2 = -0.8;
float mey1 = -1.0;
float mey2 = -0.8;

Полное отсутствие констант
Ужасные логические условия

(*x1 > 1.0 || *y1 > 1.0 || *v == 1 || *x1 < - 1.0 || *y1 < - 1.0)

Отсутствие какой-либо логической структуры программы
Комментарии ни сколько не информативны.
Орфография на двойку. И те причины, которые вы описали в файле, не дают вам право на орфографические ошибки

Функции принимают слишком большое число аргументов

знаю... нужно исправлять, но потом

Имена аргументов совершенно не отражают их смысла

ну это кому как...

Половина функций работает с double, половина с float, не понятно зачем

просто был глюк в rand() исправить удалось, а доубле забыл убрать

float mex1 = -1.0;
float mex2 = -0.8;
float mey1 = -1.0;
float mey2 = -0.8;

из-за того же глюка.


ставь -

вот что я хотел спросить, у Вас не бывает глюков когда картинку плода не выводит?

[Blade]

Сообщение от pufystyj

вот что я хотел спросить, у Вас не бывает глюков когда картинку плода не выводит?

Не знаю, я это не пробовал запускать

[NiCola999]

функция main должна быть примерно такой:

int main(int argc, char** argv) {
    glutInit(&argc, argv);
    glutInitWindowSize(W, H);
    glutInitWindowPosition(0,800);
    glutInitDisplayMode(GLUT_SINGLE | GLUT_RGB | GLUT_DOUBLE | GLUT_DEPTH);
    glutCreateWindow("3D Shapes");
    glutReshapeFunc(resize);
    glutDisplayFunc(display);
    glutMouseFunc(mouse);
    glutKeyboardFunc(key);
    glutIdleFunc(idle);    
    glutMainLoop();
    return EXIT_SUCCESS;
}

код ужасен

[pufystyj]

Сообщение от NiCola999

функция main должна быть примерно такой:

int main(int argc, char** argv) {
    glutInit(&argc, argv);
    glutInitWindowSize(W, H);
    glutInitWindowPosition(0,800);
    glutInitDisplayMode(GLUT_SINGLE | GLUT_RGB | GLUT_DOUBLE | GLUT_DEPTH);
    glutCreateWindow("3D Shapes");
    glutReshapeFunc(resize);
    glutDisplayFunc(display);
    glutMouseFunc(mouse);
    glutKeyboardFunc(key);
    glutIdleFunc(idle);    
    glutMainLoop();
    return EXIT_SUCCESS;
}

код ужасен

ок буду старатся идти к лучшому

П.С я хотел как лучше, специально исходники выставил

[Сtrl]

я хотел как лучше, специально исходники выставил

Это ведь хорошо! Предлагаю заняться улучшением. Сейчас я поищу, что можно изменить в коде, и составлю список.
UPD: Составил. (ссылка)

[pufystyj]

Сообщение от Сtrl

Это ведь хорошо! Предлагаю заняться улучшением. Сейчас я поищу, что можно изменить в коде, и составлю список.

спасиба за поддержку... займусь обезательно.
вот часть того что нужно сделать:

переделать функцию маин;
сделать меньше аргументов в некоторых функциях (но как?);
сделать нормальные коменраии;

[Сtrl]

Как я понял, пишете вы на чистом C, так что, к сожалению, помощь моя вряд ли будет эффективной. Но могу однозначно предложить кое-что:

1. Ни в коем случае не хранить координаты как число с плавающей точкой (double, float). Потому что поступая так, вы ставите себе под ноги очень больно бьющие грабли - погрешности округления.
2. Координаты X и Y хранить в одной структуре, вроде такого:
   

   struct Coord
   {
     int X;
     int Y;
   };

3. Проверку диапазона можно поместить в функцию:
   

   bool ApproximatelyEqual(int value1, int value2, int eps);

   Хотя на C++ вместо int я бы использовал шаблоны.
4. Как уже было сказано - избавиться нужно от всех магических чисел.

[pufystyj]

Сообщение от Сtrl

Как я понял, пишете вы на чистом C, так что, к сожалению, помощь моя вряд ли будет эффективной. Но могу однозначно предложить кое-что:

1. Ни в коем случае не хранить координаты как число с плавающей точкой (double, float). Потому что поступая так, вы ставите себе под ноги очень больно бьющие грабли - погрешности округления.
2. Координаты X и Y хранить в одной структуре, вроде такого:
   

   struct Coord
   {
     int X;
     int Y;
   };

3. Проверку диапазона можно поместить в функцию:
   

   bool ApproximatelyEqual(int value1, int value2, int eps);

   Хотя на C++ вместо int я бы использовал шаблоны.
4. Как уже было сказано - избавиться нужно от всех магических чисел.

ок, где-то через час займусь этим.

с числами float и double, у меня просто из-за глюка... буду исправлять!
спасиба