Почему в Debug всё работает, а в Release баг?

Question

[Михаил]

Пишу небольшую прогу на Microsoft Visual Studio 2022 с использованием Blackmagic ATEM Switchers SDK 9.2.1, которая вытягивает параметр камеры из ATEM по сети и отправляет его по UDP.
Суть в том что в программе цикл перебирает 20 камер, вытягивая параметр каждой и отправляет его по сети если он изменился.
В Dubug всё работает, а в Release зацикливается переменная первой камеры, хотя она не меняется.
Собственно вопрос в том что я делаю не так и как это поправить?

Кусок кода

spoiler

short wb[20]{ 0 };
	short oldWb[20]{ 0 };

while (1) {
	Sleep(10);
	for (int cam = 1; cam <= 20; cam++) {
		//WHITE BALANCE
		result = cameraControl->GetInt16s(cam, 1, 2, &forCount[cam], &wb[cam]); // White Balance
		if (wb[cam] != oldWb[cam]) {
			tempString = "camera/";
			tempString.append(std::to_string(cam));
			tempString.append("/wb/");
			tempString.append(std::to_string(wb[cam]));
			sendto(sock, tempString.c_str(), tempString.length(), 0, (sockaddr*)&sendAddr, sizeof(sendAddr));
			Sleep(10);
			std::cout << tempString << std::endl;
			oldWb[cam] = wb[cam];
		}
	}
}

DEBUG

spoiler

RELEASE

spoiler

Comments

[Adamos]

short wb[20]{ 0 };
short oldWb[20]{ 0 };
for (int cam = 1; cam <= 20; cam++) {

Вы раньше на Паскале не писали?
При cam = 20 происходит выход за границу массива.
Индексы для [20] - от 0 до 19.

[Михаил]

Adamos, Согласен, глупость сделал, но что-то не могу понять до конца ошибку, мне надо вынимать 20 камер, их индексы от 1 до 20 в первом параметре вот этой функции
GetInt16s(cam, 1, 2, &forCount[cam], &wb[cam]);
, при

spoiler

while (1) {
	Sleep(10);
	for (int cam = 0; cam < 19; cam++) {
		//WHITE BALANCE
		result = cameraControl->GetInt16s(cam+1, 1, 2, &forCount[cam], &wb[cam]); // White Balance
		if (wb[cam] != oldWb[cam]) {
			std::cout << "Переменная cam= " << cam << std::endl;
			tempString = "camera/";
			tempString.append(std::to_string(cam+1));
			tempString.append("/wb/");
			tempString.append(std::to_string(wb[cam]));
			sendto(sock, tempString.c_str(), tempString.length(), 0, (sockaddr*)&sendAddr, sizeof(sendAddr));
			Sleep(10);
			std::cout << tempString << std::endl;
			oldWb[cam] = wb[cam];
		}
	}
}

я вытаскиваю от 1 до 19 камеры и всё работает нормально, но только для этих камер, без двадцатой,

вот

при

spoiler

while (1) {
	Sleep(10);
	for (int cam = 0; cam <= 19; cam++) {
		//WHITE BALANCE
		result = cameraControl->GetInt16s(cam+1, 1, 2, &forCount[cam], &wb[cam]); // White Balance
		if (wb[cam] != oldWb[cam]) {
			std::cout << "Переменная cam= " << cam << std::endl;
			tempString = "camera/";
			tempString.append(std::to_string(cam+1));
			tempString.append("/wb/");
			tempString.append(std::to_string(wb[cam]));
			sendto(sock, tempString.c_str(), tempString.length(), 0, (sockaddr*)&sendAddr, sizeof(sendAddr));
			Sleep(10);
			std::cout << tempString << std::endl;
			oldWb[cam] = wb[cam];
		}
	}
}

или

spoiler

for (int cam = 0; cam < 20; cam++)

не важно.
Вытаскивается 20я камера и далее начинает бесконечно вытаскиваться первая камера

вот

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

[Adamos]

Михаил, при выходе за границу массива портится следующая переменная - вполне возможно, та самая переменная цикла. Поэтому он никогда не заканчивается.

[Михаил]

Adamos, Это я понял и поправил этот момент, и массив за границу 19 индекса не выходит на данный момент, но переменная с индексом 0 всё равно видимо портится, не могу понять почему.

[Adamos]

Михаил, в функцию передаются аргументы по ссылке. Возможно, она ожидает long вместо short и опять-таки пишет мимо. Сделайте вывод всего состояния, оно небольшое - увидите, что куда записалось.

[Михаил]

Вот ввырезка функции из документации

spoiler

Подогнал код точно под переменные функции, в итоге вытягивает не все камеры,

вот код

uint32_t forCount[20];
	int16_t wb[20]{ 0 };
	int16_t oldWb[20]{ 0 };

while (1) {
	Sleep(10);
	for (uint32_t cam = 0; cam < 20; cam++) {
		//WHITE BALANCE
		result = cameraControl->GetInt16s(cam+1, 1, 2, &forCount[cam], &wb[cam]); // White Balance
		if (wb[cam] != oldWb[cam]) {
			std::cout << "Переменная cam= " << cam << std::endl;
			tempString = "camera/";
			tempString.append(std::to_string(cam+1));
			tempString.append("/wb/");
			tempString.append(std::to_string(wb[cam]));
			sendto(sock, tempString.c_str(), tempString.length(), 0, (sockaddr*)&sendAddr, sizeof(sendAddr));
			Sleep(10);
			std::cout << tempString << std::endl;
			oldWb[cam] = wb[cam];
		}
	}
}

вот результат

[Михаил]

В общем и целом, это видимо чтото с SKD или с моим пониманием SDK, так как изначально с вытягиванием параметра White Balance были проблемы, после того как я поправил вытягивание других параметров по вашим советам, с ними всё (естественно) нормализовалось, а white balance так и продолжает глючить. Буду разбираться с ним, но это уже совсем другая история. Благодарю за помощь!

[Adamos]

Михаил, в коде, кстати, не нужны два массива, достаточно одного (который сейчас oldWb).
Другой можно заменить локальной переменной внутри цикла.
Или совершенно отдельным буфером - я что-то не вижу, чтобы вы проверяли, что именно возвращается в forCount...

[Михаил]

Adamos, forCount мне эта переменная не нужна и в том что я вытягиваю не задействуется, это переменная количества элементов в значении и нужна для RGB значений, когда вытягиваются тень, гамма и свет

вот

[Adamos]

Михаил, когда все правильно, но что-то идет не так - стоит убедиться, что возвращается именно то, что ожидается. И если forCount нигде не используется - под нее тоже не нужен массив.

[Михаил]

Adamos, у forCount массив убрал, дальнейшее буду потом оптимизировать, для параметра White Balance вот такой вариант полностью рабочий, вы очень помогли

код

uint32_t forCount;
	int16_t wb[20]{ 0 };
	int16_t oldWb[20]{ 0 };

while (1) {
	Sleep(1);
	for (uint32_t cam = 0; cam <= 19; cam++) {
		//WHITE BALANCE
		result = cameraControl->GetInt16s(cam + 1, 1, 2, &forCount, &wb[cam]); // White Balance
		if (wb[cam] != oldWb[cam]) {
			tempString = "camera/";
			tempString.append(std::to_string(cam + 1));
			tempString.append("/wb/");
			tempString.append(std::to_string(wb[cam]));
			sendto(sock, tempString.c_str(), tempString.length(), 0, (sockaddr*)&sendAddr, sizeof(sendAddr));
			Sleep(10);
			std::cout << tempString << std::endl;
			oldWb[cam] = wb[cam];
		}
	}
}

Далее у меня возникает вопрос с тем что именно изначально я вытягиваю несколько параметров ZOOM, FOCUS, IRIS, SHUTTER, GAIN и все они вытаскиваются без проблем по всем камерам(после поправки кода с вашей подсказкой). А WHITE BALANCE, как и до этого, так и сейчас просто останавливает этот процесс, при этом сам процесс не виснет и не выходит из цикла, но вытягивать какие-либо параметры отказывается. А отдельно от всех прекрасно работает, но это я уже буду скорее всего дяльше бороться с SDK.