inflearn logo
강의

강의

N
챌린지

챌린지

멘토링

멘토링

N
클립

클립

로드맵

로드맵

지식공유

코딩으로 학습하는 리팩토링

리팩토링 18. 변수 쪼개기

[내용오류 제보] 강의 중 Rectangle::updateGeometry 경고 관련 제보입니다.

427

jacoba
6

큰 문제가 아니지만 혹시나 강의를 보시는 분들께 참고하시길 바라며 제보합니다.

public class Rectangle {

private double perimeter;
private double area;

public void updateGeometry(double height, double width) {
double temp = 2 * (height + width);
System.out.println("Perimeter: " + temp);
perimeter = temp;

temp = height * width;
System.out.println("Area: " + temp);
area = temp;
}

public double getPerimeter() {
return perimeter;
}

public double getArea() {
return area;
}
}

위의 코드를 리팩토링 할 때 변수 area에 인텔리제이 경고가 뜨는데, 필드의 area가 아닌 로컬 변수 area를 사용하도록 double을 선언해주면 됩니다!

public class Rectangle {

private double perimeter;
private double area;

public void updateGeometry(double height, double width) {
double perimeter = 2 * (height + width);
System.out.println("Perimeter: " + perimeter);
this.perimeter = perimeter;

double area = height * width;
System.out.println("Area: " + area);
this.area = area;
}

public double getPerimeter() {
return perimeter;
}

public double getArea() {
return area;
}
}

오류제보

답변 3

0

ljh

ㅋㅋㅋ.. 개발하다가 문제 생기면 조금 풀어보다가 IDE나 외부 시스템 탓하는거는 어느 개발자나 똑같나 봅니다..

0

김우철

굿

0

Lee Chungku

ㅎㅎㅎ 역시 잘 짚어주셨네요..

이런건 누구나 다 흔히 범하는 부분이니 ㅎㅎ

화이팅입니다.

html 템플릿 오류 제보드립니다.

0

362

1