tryMove(..) 메서드 ArrayIndexOutOfBoundsException 제보

Question

안녕하세요 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(); } }

eternity · Answer

이진우님 안녕하세요. 개인적인 일이 있어 올려주신 질문을 확인하는게 늦었네요. 먼저 죄송하다는 말씀 드립니다.   지적하신 코드는 강의 자료에 오류가 있는 것이 맞고 말씀하신 것처럼 코드를 수정해야만 오류가 발생하지 않습니다. 확인해보니 공유드린 예제 코드는 정상적으로 구현되어 있고 강의 자료만 잘못 작성되어 있네요.   강의자료 꼼꼼하게 봐주시고 오류도 제보해주셔서 정말 감사드립니다. 우선 공유드린 pdf 강의자료는 수정해서 업데이트해 두었습니다.   보시다가 또 이상한 부분 있으면 질문 남겨주세요. 🙂 행복한 하루 보내시구요!

communityai8509 · Answer

안녕하세요, 인프런 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; 코드 라인만으로 충분히 좌표가 올바르게 업데이트될 수 있습니다. 참고 자료 유사한 논의가 있었던 기존의 질문을 참고하면 문제가 해결되는데 도움이 될 수 있습니다: 6-2 보호 로직 중복 이슈 저는 질문자님의 이해를 돕고자 지식공유자님의 기존 답변을 참고하여 유사한 답변 링크를 추천해드리고 있습니다. 현재 베타 기능으로 답변이 만족스럽지 않을 수 있는 점 양해 부탁드립니다. 🙏 추가적으로 궁금한 점이 있으시면, 이어서 질문해 주세요. 곧 지식공유자께서 답변해 주실 것입니다.