hyein-kim / minesweeper Goto Github PK

void MineAllocate(Board my_Board[][MAXBOARD], int boardRows, int boardCols)

• srand(time(NULL)) 가 변수 선언보다 위에 있어 c파일에서 돌아가지 않는 현상. 호출 위치를 바꿀 필요성.

void CntMine(Board **my_Board, int row, int col, int boardRows, int boardCols)
int IsVictory(Board **my_Board, int boardRows, int boardCols)

• 인자값이 Board my_Board[][MAXBOARD]로 수정되어야 함.

FlagBlock() 에서
OPEN인 칸에 깃발을 꽂고 다시 깃발을 해제하면
칸이 다시 닫힙니다

깃발을 뺄 때
연 칸이면 연 칸으로 바꾸고
닫힌 칸이면 닫힌 칸으로 바꾸도록 수정해 주세요.

drawingfunctions.c 파일에서
printf("★");
printf("○");
printf("▶");

부분의 특수문자가 오류가 생깁니다.

Play 함수 내부에서 IsVictory 함수가 while문이 처음 돌 때 폭탄이 할당이 아직 안 된 상태에서 실행되어 쓰레기값을 검사합니다.
@HyeIn-Kim

`while (IsVictory(my_Board, boardRows, boardCols)) {
do {
printf("\nPlease Enter Rows 0~%d / Cols 0~%d: ", boardRows - 1, boardCols - 1);
scanf_s(" %d %d", &row, &col);
} while (!checkRowsinGame(row, boardRows) || !checkColsinGame(col, boardCols));

if (init == 0) {
	MineAllocate(*my_Board, boardRows, boardCols, row, col);
	DrawBoard(*my_Board, boardRows, boardCols);
	init = 1;
}

do {
	printf("\nSelect your action (1: Open Block / 2: Flag):");
	scanf_s(" %d", &action);
} while (!ActionInRange(action));

switch (action) {
	case 1: 
		if (OpenBlock(my_Board, row, col, boardRows, boardCols)) {
			return;
		}
		break;
	case 2:
		FlagBlock(row, col, my_Board);
		break;
}

DrawBoard(*my_Board, boardRows, boardCols);

}`

저희 retry할 때 play함수에서 init이 다시 초기화가 안돼서 지뢰가 재배치가 안되는 버그가 있어요. 이 사진은 지뢰 여부랑 칸이 열렸는지 ox로 표시해 논 것입니다.

해당 화면은 임시로 지뢰 여부를 표시하였습니다.
지뢰를 다 찾지 않았음에도 게임이 끝나는 오류가 발생합니다. @woozu99

현재 IsVictory 함수의 게임 지속 조건은

1. STATUS_CLOSE이거나 STATUS_FLAG인 블록 중에서
2. 지뢰가 없는 칸이 있을 경우
   입니다.

저 3*3 맵에서 (2,0)을 열면
지뢰를 제외한 모든 칸이 열리고,
남은 칸은 STATUS_CLOSE이나 지뢰가 있는 칸이라
IsVictory 함수가 게임이 끝나지 않았는데도 승리라고 판단합니다.

게임 지속 조건을
STATUS_CLOSE (지뢰 여부 상관 없이) 칸이 남아있거나,
STATUS_FLAG 중에서 지뢰가 없는 칸이 있다면
으로 STATUS마다 쪼개야 한다고 생각합니다.

IsVictory반환값이 승리면 1, 승리가 아니면 0인데 게임을 진행시키는 while문이 IsVictory가 1일 경우 돌도록 되어있습니다.(조건을 IsVictory가 0일 경우로 바꿔야 될 것 같아요.)

@HyeIn-Kim

`while (IsVictory(my_Board, boardRows, boardCols)) {
do {
printf("\nPlease Enter Rows 0~%d / Cols 0~%d: ", boardRows - 1, boardCols - 1);
scanf_s(" %d %d", &row, &col);
} while (!checkRowsinGame(row, boardRows) || !checkColsinGame(col, boardCols));

	if (init == 0) {
		MineAllocate(*my_Board, boardRows, boardCols, row, col);
		DrawBoard(*my_Board, boardRows, boardCols);
		init = 1;
	}

	do {
		printf("\nSelect your action (1: Open Block / 2: Flag):");
		scanf_s(" %d", &action);
	} while (!ActionInRange(action));

	switch (action) {
		case 1: 
			if (OpenBlock(my_Board, row, col, boardRows, boardCols)) {
				return;
			}
			break;
		case 2:
			FlagBlock(row, col, my_Board);
			break;
	}

	DrawBoard(*my_Board, boardRows, boardCols);
}`

play.c의 CntMine 함수에서 무한재귀가 일어납니다.
디버깅 결과 cnt가 0일 때 CntMine을 재귀호출하면서 그 안에서 빠져나오지 못하고 있어요. (cnt가 0이 아니라면 정상 동작합니다.) @woozu99
if (cnt == 0) { for (checkrow = row - 1; checkrow <= row + 1; checkrow++) { if (checkrow >= 0 && checkrow < boardRows) { for (checkcol = col - 1; checkcol <= col + 1; checkcol++) { if (checkcol >= 0 && checkcol < boardCols) { if (my_Board[checkrow][checkcol].statusBlock == STATUS_CLOSE) { my_Board[checkrow][checkcol].statusBlock == STATUS_OPEN; CntMine(my_Board, checkrow, checkcol, boardRows, boardCols); } } } } } }

1. utilityfunctions.c 에서 IsGameReset 함수 내부에 char 변수가 null값을 받도록 되어있는 부분이 있다.(user = NULL;)

2. scanf_s는 표준이 아님

3.IsGameReset()함수가 if, else if로 끝남.

지뢰를 제외한 모든 칸을 열었음에도 종료되지 않는 문제 발생. 승리조건 체크 함수 수정이 필요.