Функция не работает, не понимаю почему, как можно исправить?

Question

[Nexpie]

Функция преобразует массив так, чтобы числа во введенном диапазоне были первыми, а затем уже остальные, но код выдает какую то ошибку и я не могу понять в чем проблема, я пробовал запустить отдельно и без этой функции все работает, что я сделал не так, подскажите пожалуйста

void result(double* a, int n, int x, int y) {
	double* b;
	b = new double[n];
	for (int i = 0; i < n; i++) {
		b[i] = a[i];
	}
	int j = 0;
	for (int i = 0; i < n; i++) {
		if (b[i] >= x || b[i] <= y) {
			a[j] = b[i];
			j++;
		}
	}
	for (int i = 0; i < n; i++) {
		if (b[i] < x || b[i] > y) {
			a[j] = b[i];
			j++;
		}
	}
	delete[]b;
}

Comments

[VoidVolker]

Nexpie https://qna.habr.com/help/rules#3.8

Answer 1

[jcmvbkbc]

что я сделал не так, подскажите пожалуйста

if (b[i] >= x || b[i] <= y) {
  a[j] = b[i];
  j++;
}
...
if (b[i] < x || b[i] > y) {
  a[j] = b[i];
  j++;
}

Условия в операторах if не являются взаимоисключающими, а значит j может стать больше чем n и запись в a[j] вылезет за границу массива.
Я не уверен в правильности первого условия, но особо не задумываясь можно второе условие переписать как прямое отрицание первого: if (!(b[i] >= x || b[i] <= y)) и это гарантирует, что индекс не выйдет за пределы массива, вне зависимости от правильности первого условия.

Answer 2

[res2001]

Если x и y - это min и max, то у вас ошибка:
if (b[i] >= x || b[i] <= y)
надо
if (b[i] >= x && b[i] <= y)

Копировать элементы можно с помощью std::memcpy, а не в цикле поэлементно.