Выход за пределы массива

[NFXrus]

Вот вообщем то сама задачка

#include <iostream> // cin, cout
#include <clocale>  // поддержка русского языка
#include <conio.h>  // _getch()
#include <stdlib.h> // поддержка функции очистки экрана system("cls")

using namespace std;

int main()
{

setlocale(LC_CTYPE,"Russian"); // установка русского языка


char act;    // переменная для хранения ввода пользователя
int ch;      // ввод пользователя приведённый к типу int
int x = 0;   // две переменные для хранения координат пользователя
int y = 0;
char map[15][20]; // двумерный массив для хранения игровой карты


// блок инициализации массива
// внешний цикл отвечает за строки
for (int i = 0; i< 15; i++)
{
	// внутренний цикл отвечает за столбцы
	for (int j = 0; j< 20; j++)
	{
		map[i][j] = ' ';
	}
}

map[0][0] = 'Т'; // помещаем игровой персонаж в начало координат
while (1)
{
	system("cls"); // вызов функции очистки экрана

	for (int i = 0; i < 15; i++)
	{
		for (int j = 0; j < 20; j++)
		{
			cout << map[i][j]; // в циклах поэлементно выводится массив
		}
		cout << "\n"; // переход на новую строку после каждого "ряда"
	}

	act = _getch();
	ch = static_cast<int>(act);
	{
		if (ch == -32)
		{}
		else if (ch == 27)
		{break;}
		else if (ch == 75)
			{map[x][y] = ' '; // "Убираем" объект со старой координаты
			y--;             // изменяем координаты
			map[x][y] = 'Т'; // "рисуем" объект на новом месте
		}
		else if (ch == 77)
			{map[x][y] = ' ';
			y++;
			map[x][y] = 'Т';}
		else if (ch == 80)
			{map[x][y] = ' ';
			x++;
			map[x][y] = 'Т';}
		else if (ch == 72)
			{map[x][y] = ' ';
			x--;
			map[x][y] = 'Т';}
	}
} // конец цикла while

	_getch();
	return 0;
}

Сейчас в программе возможна запись в несуществующие элементы массива. Нужно сделать проверку на выход за пределы карты. Вообщем, чтоб когда "Т" крайнюю точку (15:20), нужно оставался на месте и не выходил за пределы поля(когда нажимаешь вниз - он выходит за пределы и выскакивает ошибка). Помогите пожалуйста=) я пол дня ломаю голову=))

[Alex11223]

Используйте тег [CODE] (значок # ) для сохранения форматирования и подсветки кода в сообщении. Кнопка Правка снизу справа.

[NFXrus]

Походу я встрял=))

[Alex11223]

Что-то не очень понял, что надо сделать.

Сейчас в программе возможна запись в несуществующие элементы массива. Нужно сделать проверку на выход за пределы карты.

Надо запретить ввод некорректных данных? Ну, например, можно сделать ввод в цикле пока не будут введены правильные данные.

[NFXrus]

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

Что-то не очень понял, что надо сделать.

Надо запретить ввод некорректных данных? Ну, например, можно сделать ввод в цикле пока не будут введены правильные данные.

Попробуйте скопировать код к себе в программу и запустить его, "Т" принимает значение (0;0) это

map[0][0] = 'Т';

. По сторонам он двигается как положено, но выходит за пределы массива размером 15х20, т.е. исчезает, если нажать влево или вверх в локации (0;0); вверх, в локациях (0;y); а внизу, при выходе за пределы - вовсе выдает ошибку. Мне лишь надо чтоб "Т" не выходил за пределы массива(т.е. оставался на месте при нажатии влево или вверх в локе (0;0) и т.п.).

[onewho]

добавь функцию

bool validCoords(int x, int y, int n, int m) {
return x>=0 && x<=n-1 && y>=0 && y<=m-1;
}

и собсно проверяй ей в каждом условии.
как-то так.

[_Alerter_]

Просто добавь условия проверки на выход за пределы массива. Например, если увеличиваешь y:

else if (ch == 77 && y + 1 < 20)
{
     map[x][y] = ' ';
     y++;
     map[x][y] = 'Т';
}

Если уменьшаешь, то сравнивай с 0:

y - 1 >= 0

Аналогично с x

[NFXrus]

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

Просто добавь условия проверки на выход за пределы массива. Например, если увеличиваешь y:

else if (ch == 77 && y + 1 < 20)
{
     map[x][y] = ' ';
     y++;
     map[x][y] = 'Т';
}

Если уменьшаешь, то сравнивай с 0:

y - 1 >= 0

Аналогично с x

Помогло, но все равно ограничения x с низу нету, как выходил, так и выходит, причем размер предела по вертикали 15, а не 20, как устанавливалось..

[_Alerter_]

Вроде всё работает

            if (ch == -32)
            {}
            else if (ch == 27)
            {break;}
            else if (ch == 75 && y - 1 >= 0)
            {map[x][y] = ' ';
            y--;             
            map[x][y] = 'Т'; 
            }
            else if (ch == 77 && y + 1 < 20)
            {map[x][y] = ' ';
            y++;
            map[x][y] = 'Т';}
            else if (ch == 80 && x + 1 < 15)
            {map[x][y] = ' ';
            x++;
            map[x][y] = 'Т';}
            else if (ch == 72 && x - 1 >= 0)
            {map[x][y] = ' ';
            x--;
            map[x][y] = 'Т';}

[NFXrus]

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

Вроде всё работает

            if (ch == -32)
            {}
            else if (ch == 27)
            {break;}
            else if (ch == 75 && y - 1 >= 0)
            {map[x][y] = ' ';
            y--;             
            map[x][y] = 'Т'; 
            }
            else if (ch == 77 && y + 1 < 20)
            {map[x][y] = ' ';
            y++;
            map[x][y] = 'Т';}
            else if (ch == 80 && x + 1 < 15)
            {map[x][y] = ' ';
            x++;
            map[x][y] = 'Т';}
            else if (ch == 72 && x - 1 >= 0)
            {map[x][y] = ' ';
            x--;
            map[x][y] = 'Т';}

Спасибо большое=)))
Я вот тут

 else if (ch == 77 && y + 1 < 20)
            {map[x][y] = ' ';
            y++;
            map[x][y] = 'Т';}
            else if (ch == 80 && x + 1 < 15)

перепутал 20 и 15. Поменял - все заработало=))