```java
public record AdditionResult(String result) {
public static AdditionResult from(double result) {
return new AdditionResult(formatResult(result));
}
private static...
효재님께서 말씀하신 명확성 부분도 중요한 고려사항이라고 생각합니다.
다만, 클린 코드 원칙에서는 else 문을 지양하는 것이 가독성을 높이는 이유로 불필요한 중첩을 줄이고 조건에 따른 동작 흐름을 더 직관적으로 만들기 위함이라고 생각합니다.
특히 `if-else` 구조에서는, `else` 문을 읽을 때 "앞의 `if` 조건이 아니라면?"을 전제...
인덴트 깊이가 너무 큽니다. 프로그래밍 요구사항에 준수하라고 한 자바 컨벤션에 따르면 최대 2단계입니다. 헬퍼함수로 분리하시거나 하셔서 나누셔야 합니다.
https://github.com/woowacourse/woowacourse-docs/blob/main/styleguide/java/README.md
https://github.com/Ju...
클래스의 역할이 너무 많습니다. 위에서 말씀드린 것처럼 view 클래스를 따로 만드시는 편이 좋을 것 같습니다. veiw -> controller -> service -> controller -> view의 흐름을 따르면 클래스의 역할도 작게 유지하고 분명해질 것입니다.
잘 봤습니다. Number 클래스가 쓰이던데 다른 버젼에서 구현하신 것이겠죠?
controller가 직접 메인 기능을 수행하지 않고 service에 넘긴 것이 인상 깊네요.
네이밍 컨벤션을 잘 지키신 부분도 있고 조금 미흡한 부분도 있지만 금방 보완하신 것이라고 믿습니다.
잘 봤습니다. Number 클래스가 쓰이던데 다른 버젼에서 구현하신 것이겠죠?
controller가 직접 메인 기능을 수행하지 않고 service에 넘긴 것이 인상 깊네요.
네이밍 컨벤션을 잘 지키신 부분도 있고 조금 미흡한 부분도 있지만 금방 보완하신 것이라고 믿습니다.
main메서드에 너무 많은 책임이 있는것 처럼 보여서(사용자 입력 처리, 문자열 분할, 구분자 추출, 덧셈 계산이 모두 main에 들어감) 컨트롤러 클래스를 하나 만들어서 분리해보는건 어떨까요?
아니면 간단한 계산기의 기능을 구현하는 과제였으므로 빠르고 간단하게 기능을 구현하는것이 더 중요하게 판단되었을까요?
정민님이 작성하신 계산기 메서드에서는 double 타입을 지원하는 계산기 로직으로 작성하셨는데,
해당 메서드에서 커스텀 구분자로 온점이 들어올경우 소수와 구분자를 제대로 구별하지 못할거 같아서 이를 어떤식으로 처리하는지 고민해보는것도 좋을것 같아요
저는 이러한 문제때문에 들어오는 수의 타입을 int타입으로 제한하여 구현했었던것 같아요. 만약 소수...