Ошибка в работе программы[Visual C++]

[spectrum988]

Приветствую всех участников форума.После проверки работы программы выскакивает ошибка.
Нужно удалить четные элементы одномерного массива(динамического);

 switch (n)
	 {
	 case 1:
		 cout << "Введите размерность массива: ";
		 cin >> n;
		 srand(time(NULL));
		 ptrmas = new int[n];
		 for (i = 0; i < n; i++)
		 {
			 q++;
			 ptrmas[i] = rand() % 20 + 1;
		 }
		 //--------------------------------------------
		 cout << "Для выхода из режима нажмите - 0: ";
		 cin >> k;
		 system("cls");
		 if (k == 0) { break; }
	 case 2:
		 cout << "Четные элементы массива: ";
		 for (i = 0; i < q; i++)
		 {
			 if (!(ptrmas[i] % 2 == 0))
			 {
				 yes++;
				 cout << ptrmas[i] << " ";	 
			 }
		 }
		 ptrmas2 = new int[yes];
			 for (i = 0; i < q; i++)
			 {
			 if (!(ptrmas[i] % 2 == 0))
			  {
				 ptrmas2[i] = ptrmas[i];
			  }
			 }
			 delete[]ptrmas;
			 ptrmas = new int[yes];
			 for (i = 0; i <= yes; i++)
			 {
				 ptrmas[i] = ptrmas2[i];
			 }
			 delete[]ptrmas2;
		 //-------------------------------------------------
		 cout << "Для выхода из режима нажмите - 0: ";
		 cin >> k;
		 system("cls");
		 if (k == 0) { break; }
	 
	 case 3:
		 cout << "Массив: ";
		 for (i = 0; i <q; i++)
		 {
			 cout << ptrmas[i] <<" ";
		 }
		 cout << endl;
		 //-------------------------------------------------
		 cout << "Для выхода из режима нажмите - 0: ";
		 cin >> k;
		 system("cls");
		 if (k == 0) { break; }

Безымянный.png
Может просто просмотр одномерного массива нужно сделать через функцию?Возможно ли так сделать ?

[Vanta11a]

А зачем такое условие

 if (k == 0) { break; }

? Насколько я помню, после каждого case должен быть обязательный безусловный break, т.е.

switch (i)
{
case 1: 
...
break;
case 2:
...
break;
case default:
break;
}

[p51x]

1. Ваш алгоритм разве делает заявленное?
2.

for (i = 0; i <= yes; i++)
			 {
				 ptrmas[i] = ptrmas2[i];

У этих массивов нет элемента с индексом йес...

Насколько я помню, после каждого case должен быть обязательный безусловный break

Не должен.

[spectrum988]

Дело в том,что я прелварительно создал проверку,сколько у меня получится элементов, поэтому и использовал yes;И у меня же был создан ptrmas2 c числом элементов yes,а код который вы выделили я использовал лишь ради того,чтобы поменять название динамическогода массива,чтобы я смог его потом просматрвать. Но мне кажется,что я это сделал как-то нерационально.

[p51x]

У вас проблема с выходом за границы массивов, вот и валится...

[spectrum988]

Спасибо за помощь,буду искать ошибку.

for (i = 0; i <= q; i++)
		 {
			 if (!(ptrmas[i] % 2 == 0))
			 {
				 yes++;	 
			 }
		 }
		  ptrmas2 = new int[yes];
			 for (i = 0; i < q; i++)
			 {
			 if (!(ptrmas[i] % 2 == 0))
			  {
				 ptrmas2[i] = ptrmas[i];
			  }
			 }

Безымянный.png
Получается,что-то такое.

[spectrum988]

Спасибо всем за помощь,вопрос уже неактуален.