🐞랜덤지뢰 생성 개수 버그

[yeram522]

🐞랜덤지뢰 생성 개수 버그

[Yeram522]

💡 예상했던 대로 중복이 문제였다.
디버깅 전에 대충 "중복"문제라고 예상해서, 일부로 함수 따로 구현해서 했던건데,,;;
중간에 난 왜 눈치를 못챘지.. 함수 로직 다시 살펴봐야겠다.

void Screen::Set_Filed(const int _size, const int _spot, Field* _fields)//매개변수: 랜덤 지뢰개수.
{
	int placed_count= _spot;
	int* seeds = new int[_size]; //랜덤값 중복 방지를 위해 값 저장.
	memset(seeds, -1, sizeof(seeds));
	while (placed_count > 0)
	{
		int i = (rand() % (_size-1) + 1);// 1 ~ size 까지

		if ((i >= 11 && i % 11 == 0) || Check_OverLap_Seed(seeds, i)) continue;
		seeds[_spot - placed_count] = i;
		_fields[i - 1].Changestate(STATE_SPOT);
		placed_count--;
	}
	
	delete[] seeds;
}

//randseed 중복 체크 함수: 검사할 값을 넣고 중복이 없으면 false를 반환한다.
bool Screen::Check_OverLap_Seed(int* _seeds,const int _seed)
{
	for (int i = 0; i < sizeof(_seeds); i++)
	{
		if (_seeds[i] == _seed) return true;
	}

	return false;
}

[Yeram522]

✅ 디버거로 값 넣으면서 돌린 결과, seeds의 배열이 제대로 초기화 안되는 점을 발견했다.
그래서 sizeof의 값을 찍어보니...크기가 4가 나오는 것이다.
sizeof 함수의 쓰임을 제대로 알지 못하고 사용한 내 실수였다.ㅠㅠㅠ
sizeof가 아닌 배열 사이즈를 넣어주니, 제대로 초기화가 -1로 되었고, 중복 검사 함수도 불필요한 탐색을 줄이기 위해서
while문을 사용하여 바꾸었다.

//randseed 중복 체크 함수: 검사할 값을 넣고 중복이 없으면 false를 반환한다.
bool Screen::Check_OverLap_Seed(const int* _seeds,const int _seed)
{
	int i = 0;
	while (_seeds[i] != -1)
	{
		if (_seeds[i] == _seed) return true;
		i++;
	}

	return false;
}