[Java] Level.2_Trouble Shooting

1) Trouble Shooting_remove

if (msg.equals("remove")) {
    List<Integer> stored = calc.getResults();
    if (stored.isEmpty()) {
        System.out.println("저장된 결과가 없음.");
    } else {
        double removed = stored.remove(0);
        System.out.println("삭제된 결과 : " + removed);
    }
    System.out.println();
    continue;
}

 

구현하고자 했던 부분

Calculator내부에 저장된 연산결과 리스트에서 가장 오래된 (0번 인덱스)결과 하나를 삭제하는 로직 구현.

삭제된 값이 콘솔에 출력되어야 한다.

 

문제발생

 

remove 키워드로 삭제된 부분 콘솔에 찍히나, 조회해보면 그대로 남아있음.

 

발생 원인

삭제는 되었으나, 삭제 후 리스트 내부에 반영하는 코드가 누락되어 조회 시 결과가 남아있는 현상이 발생했다.

또한 double타입으로 선언하여 타입도 부적절함을 발견하였다.

 

개선코드

if (msg.equals("remove")) {
    List<Integer> stored = calc.getResults();        // 복사본 가져오기
    if (stored.isEmpty()) {
        System.out.println("저장된 결과가 없습니다.");
    } else {
        int removed = stored.remove(0);              // 삭제된 값 받기
        calc.setResults(stored);                     // 내부 리스트에 반영
        System.out.println("삭제된 결과: " + removed);
    }
    System.out.println();
    continue;
}

 

이렇게 개선한 이유

- calc.getResults()는 Calculator 내부 리스트의 복사본을 만든다.

- 복사본에서 remove(0)를 해도 원본에는 절대 영향을 주지 않는다.

- 따라서 삭제 뒤에 calc.setResults(복사본)을 호출해야 원본 리스트가 실제로 바뀌게 된다.

 

- stored.remove(0)이 돌려주는 값은 int.

- 원래 코드에선 double removed로 받아서 타입이 맞지 않는다.

- int removed로 바꾸면 컴파일 오류 없이 정확히 값을 받으 수 있다.

 

- App 클래스가 Calculator의 내부 리스트(results)를 직접 꺼내거나 수정하면 안 된다.

- 대신 제공된 getResults()/ setResults() 메서드만 사용하여 안전하게 상태를 읽고 바꾸는 구조는 유지해야 한다.

---

 

2)Trouble Shooting_incompatible types

 

구현하고자 했던 부분

인터페이스를 도입하여 AddOperator, SubtractOperator, ... 등 연산자를 모두 Operator타입 배열에 담고 

for문으로 일괄 순회하면서 각 연산자의 operate(a,b)를 호출하여 사칙연산 및 나머지 연산을 다형성으로 처리하려고 한다.

 

 

문제발생

 

컴파일 에러 발생

incompatible types: AddOperator cannot be converted to Operator

 

발생원인

 

연산자 클래스를 선언할 때 implements Operator빠트림

따라서 AddOperator는 Operator타입으로 취급되지 못해 배열에 담을 수 없었다.

 

개선코드

// 변경 전
public class AddOperator {
    public double operate(double a, double b) { return a + b; }
}

// 변경 후: Operator 인터페이스 구현
public class AddOperator implements Operator {
    @Override
    public char getSymbol() { return '+'; }

    @Override
    public double operate(double a, double b) {
        return a + b;
    }
}

// ArithmeticCalculator 생성자 호출 예시
Operator[] ops = { new AddOperator(), new SubtractOperator(), new MultiplyOperator(),
                    new DivideOperator(), new ModOperator() };
ArithmeticCalculator ac = new ArithmeticCalculator(ops);
for (Operator op : ops) {
    // now op.operate(x, y) 호출 가능
}

 

이렇게 개선한 이유

 

• Operator 인터페이스로 묶어 관리하므로 운영 로직을 수정하지 않고 새 연산자를 추가할 수 있다.
•  각 연산자 클래스가 Operator계약을 지켜 일관된 API를 제공
• implemetns Operator를 빠뜨리면 컴파일 단계에서 즉시 오류를 찾아줄 수 있어 안정적