23 asked

2

안녕하세요. 옙버디님, 공식 서포터즈 OMG입니다.

배열의 데이터를 선언과 동시에 초기화하지 않고 첫번째 데이터(60)와 고정으로 증가하는 값(10)을 기반으로 리펙토링하려는 것 같습니다 😀

그러한 의도라면 올리신 코드처럼 작성하여도 무방하지만 리펙토링은 강의에서 영한님이 설명해주시지만 기존의 코드 기능은 유지하면서 내부 구조를 개선하여 가독성을 높여야 하지만

출력 시 배열의 시작 인덱스가 0이 아닌 1로 시작하는 점(=코드 기능은 유지X)

인덱스의 범위가 배열의 길이로의 조건이 중복되는 점(=내부 구조를 개선 X)

Condition 'i < students.length' is always 'true' when reached

조건 'i < students.length'는 도달했을 때 항상 '참(true)'이다.

for문의 조건식 부분에 이미 인덱스는 배열의 길이보다 작을 때의 조건이 포함되어 있는데, if문에서 한번 더 동일한 조건을 추가하여 내부 구조를 개선했다고 보기는 힘들 것 같습니다

참고로 인텔리제이에서 코드라인에 노랗게 표기되는 것은 오류는 아니지만 코드를 인지하여 문제가 발생되는 것을 알려주거나, 코드의 개선이 필요한 내용을 알려주니 이 부분도 참고해주세요 ^^

설명드린 내용을 바탕으로 다시한번 고민하여 댓글 혹은 글을 새로 남겨주시면 같이 학습하시는 분들에게도 많은 도움이 될 것 같습니다 😃

감사합니다.

0

개인적으로 int i의 초기값을 0으로 둬야한다는 강박만 벗어나도 훨씬 코드가 간결해질 것이라고 생각합니다. (int를 1로만 잡아도 if문과 그 안에 들어있는 내용이 대폭 줄어들꺼라 생각합니다.)

그래도 덕분에 새로운 생각을 많이 하게 되어 즐겁습니다. 제가 짠 코드 중 핵심 부분만 붙여놓고 가겠습니다.

...
System.out.println("students["+0+"] = " + students[0]);
for (int i = 1; i < students.length; i++) {
    students[i] = students[i-1] + score;
    System.out.println("students["+i+"] = " + students[i]);
}
...