Нормален ли такой код?

Question

[voproser45654]

#include <windows.h>
#include <stdio.h>
#include <conio.h>

char map[10][20];
int x = 5, y = 5;
char key;

int main()
{
	for (;;)
	{
		sprintf(map[0], "##########");  // Создание карты
		for (int i = 1; i < 9; ++i) // Создание карты
			sprintf(map[i], "#        #");
		sprintf(map[9], "##########");  // Создание карты
		map[y][x] = '@'; // Символ игрока
		system("cls"); // Очищение экрана
		for (int i = 0; i < 10; ++i)
		{
			printf("%s\n", map[i]); // Отрисовка карты
		}
		key = getch();
		switch (key)
		{
			case 'w':  // Проверка имени клавиши
				if (map[y - 1][x] != '#') // Если следущая клетка в этом направлении не занята, уменьшить переменную
					--y;
				break;
			case 'a':
				if (map[y][x - 1] != '#')
					--x;
				break;
			case 's':
				if (map[y + 1][x] != '#')
					++y;
				break;
			case 'd':
				if (map[y][x + 1] != '#')
					++x;
				break;
		}
	}
}

Comments

[Михаил Бондаренко]

лучше описывать что делает код и закомментировать его для более удобного чтения

Чтобы ответить на вопрос нужно знать задачу этого кода.

[voproser45654]

Михаил Бондаренко, делает коробку в которой игрок может перемещатся

Answer 1

[rPman]

'магические константы' - что такое 9 или 10?, мне то понятно но чтобы их понимать нужно изучать код, размеры области нужно определить в переменные или константы, приучаться надо с самого начала к тому чтобы такого не было. То же самое с символами кнопок управления, определи для них переменные, пусть сначала это будут константы, заданные в коде, зато позже будет проще сделать изменяемые настройки.

Отсюда кстати баг, ширина у тебя при определении буфера карты определена как 20 а при заполнении экрана - 'стенками' забита строкой в 10 (хотя ожидалось что будет еще один цикл, заполняющий посимвольно строку).

Правильно - определить буфер как содержимое стенок # и заполнить его однократно вне игрового цикла (тут можно и другие препятствия расставить, например сделать лабиринт), не важно константами или алгоритмом.

А в бесконечном цикле просто выводить содержимое map и уже во время вывода проверять x,y положения 'игрока' и выводить его на экран вместо символа карты. Достоинство такого подхода - карта может состоять не только из символов стенок но и любых других, проходимых.

ну и красивости раскидай по функциям вывод экрана и обработку клавиш управления, так же внутри вызова каждой клавиши проверку на стенку вынеси в вызов отдельной функции (в параметрах - символ карты), возвращает boolean, является ли символ проходимым, благодаря этому позже проще будет добавить другие символы стенок

Следующим шагом x,y определи структурой, если продолжать дальше, в идеологии объектного программирования (да я вижу что это С, но простой вызов метода у структуры/класса сделать можно) в этой структуре определи функции, описывающие действия над персонажем (обработка действий - там функция что разбирает нажатые клавиши, только по логике нав ход должны подаваться не клавиши а константы, определяющие действие, вывод на экран - вернет символ, проверку проходимости точки на карте, т.е. стены, и т.п.) тогда будет проще добавить к примеру второго игрока или NPC и т.п.

Comments

[voproser45654]

Это не баг, просто я забыл что у меня он на 20 символов)

[rPman]

voproser45654, это и есть красивый пример бага из-за использования чисел вместо определенных констант
представь было бы наоборот? ты определил бы буфер в 10 символов а рисовал бы 20

[voproser45654]

rPman, насколько понял, то организовывать программы лучше так?

#include <....h>

const ... = ...;
int ... = ...;

void update()
{
    // code
}

void draw()
{
    // code
}

int main()
{
    update();
    draw();
}

[rPman]

Следующим шагом x,y определи структурой, если продолжать дальше, в идеологии объектного программирования (да я вижу что это С, но простой вызов метода у структуры/класса сделать можно) в этой структуре определи функции, описывающие действия над персонажем (обработка действий - там функция что разбирает нажатые клавиши, только по логике нав ход должны подаваться не клавиши а константы, определяющие действие, вывод на экран - вернет символ, проверку проходимости точки на карте, т.е. стены, и т.п.) тогда будет проще добавить к примеру второго игрока или NPC и т.п.

Answer 2

[jcmvbkbc]

Нормален ли такой код?

В каком смысле "нормален"? Он понятен, но не очень хорошо организован и делает какие-то с виду лишние действия.

Comments

[voproser45654]

Вот про это я и говорю. Чувствую что тут говно-код, но не могу понять где

[jcmvbkbc]

тут говно-код, но не могу понять где

- #include заголовков, которые не нужны (ты используешь только stdio)
- глобальные переменные которые по смыслу локальны (char key -- точно)
- весь код в main вместо того, чтобы поделить его на логически законченные части
- магические константы 9 и 10
- ты полностью перерисовываешь карту в памяти на каждом цикле, хотя меняются всего два поля
- корректность работы кода опирается на корректность заполнения карты, это немного легкомысленно

[voproser45654]

jcmvbkbc, getch без conio.h не работает и system() без windows.h тоже

[jcmvbkbc]

voproser45654,
system() -- часть stdlib.h, к windows.h отношения не имеет.
conio.h -- нестандартный пережиток ms-dos. getch() есть в curses.h

[Дмитрий Королев]

voproser45654, можно использовать терминальные последовательности для управления содержимым, будет красочнее