인프런 커뮤니티 질문&답변
작성한 질문수
tryMove(..) 메서드 ArrayIndexOutOfBoundsException 제보
작성
·
24
·
수정됨
0
안녕하세요
3-2. 조합 메서드로 리팩터링하기 의 11분 08초에 예제 코드 보면서 리팩터링을 해보고 있는데 예외가 발생했습니다.
java.lang.ArrayIndexOutOfBoundsException: Index 6 out of bounds for length 6
moveNorth() 실행시 y가 -1이 입력되는데
else 부분에 y를 한번 더 더해주다보니 문제가 발생한것으로 보입니다.
private void moveNorth() {
tryMove(0, -1);
}
// before
private void tryMove(int incX, int incY) {
if(y + incY < 0
|| y + incY >= height
|| x + incX >= width
|| x + incX < 0
|| roomAt(x + incX, y + incY) == null) {
showBlocked();
} else {
this.x += incX;
this.y += y + incY; // 💩 y = -1일때 -1 + (-1)이 됨
showRoom();
}
}
// after
private void tryMove(int incX, int incY) {
if(y + incY < 0
|| y + incY >= height
|| x + incX >= width
|| x + incX < 0
|| roomAt(x + incX, y + incY) == null) {
showBlocked();
} else {
this.x += incX;
this.y += incY; // ✅
showRoom();
}
}
답변 1
0
안녕하세요, 인프런 AI 인턴입니다. 이진우님. 질문 주셔서 감사합니다.
먼저, tryMove 메서드에서 발생하는 ArrayIndexOutOfBoundsException 문제는 코드에서 y 값을 잘못 수정하여 범위를 벗어난 값이 나오는 것에서 시작됩니다. moveNorth() 메서드 실행 시 y 값이 음수가 되고, 여기에 잘못된 산술 연산이 추가되면서 예외가 발생하게 되는 상황입니다.
수정 방법
문제를 해결하기 위해서는 tryMove 메서드의 else 부분에서 y 값을 계산할 때, incY 만 더해주면 됩니다. 이렇게 하면 y 값이 정상적으로 범위 안에서 업데이트됩니다.
private void tryMove(int incX, int incY) {
if(y + incY < 0
|| y + incY >= height
|| x + incX >= width
|| x + incX < 0
|| roomAt(x + incX, y + incY) == null) {
showBlocked();
} else {
this.x += incX;
this.y += incY; // 올바른 값만 더해 줍니다.
showRoom();
}
}
이렇게 수정하면 ArrayIndexOutOfBoundsException 오류를 방지할 수 있습니다. 기존의 코드는 조건에 따라 이미 y + incY 를 검증하고 있기 때문에 추가 연산이 필요 없습니다. this.y += incY; 코드 라인만으로 충분히 좌표가 올바르게 업데이트될 수 있습니다.
참고 자료
유사한 논의가 있었던 기존의 질문을 참고하면 문제가 해결되는데 도움이 될 수 있습니다:
저는 질문자님의 이해를 돕고자 지식공유자님의 기존 답변을 참고하여 유사한 답변 링크를 추천해드리고 있습니다.
현재 베타 기능으로 답변이 만족스럽지 않을 수 있는 점 양해 부탁드립니다. 🙏
추가적으로 궁금한 점이 있으시면, 이어서 질문해 주세요. 곧 지식공유자께서 답변해 주실 것입니다.