DelimiterConstants의 두 상수가 현재 DelimiterValidator와 StringParser 클래스에서 사용되고 있어, 이를 상수 패키지로 따로 관리하게 되었습니다.
저희 기준은 현재 클래스뿐만 아니라 다른 클래스에서도 사용될 여지가 있는 상수는 상수 패키지에서 관리하자는 것이었는데, 이 부분에 대해 길준님의 의견이 궁금합니다!
값이 없으면 0을 반환하는 편이 낫지 않을까요?
https://velog.io/@oyeon/%EC%95%84%EC%9D%B4%ED%85%9C55.-%EC%98%B5%EC%85%94%EB%84%90-%EB%B0%98%ED%99%98%EC%9D%80-%EC%8B%A0%EC%A4%91%ED%9E%88-%ED%95%98%EB%9D%BC
```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...