New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

🚀 1단계 - 문자열 덧셈 계산기 #776

Open

sumiini wants to merge 11 commits into next-step:sumiini from sumiini:step1

+210 −0

sumiini commented Jan 26, 2025

안녕하세요.
요구사항별로 커밋하려 했으나, 모든 테스트 조건에 만족시켜야 한다는 생각에 분리하여 커밋하는게 어려웠습니다.

add 내 매개변수 text는 변경 불가한 값이므로 final 선언이 맞다고 생각했습니다.
내부의 private 함수들 중 text를 매개변수로 받는 것들도 모두 final을 선언해야하는지? 아니면, 이미 add 매개변수에서 final text로 들어와서 상관없는지 궁금합니다.

hongsumin added 11 commits

January 21, 2025 21:59


          feat(racingcar):Car 생성

7a209bb


          feat(racingcar): move 기능 추가

66e2d15


          feat(racingcar): 인터페이스 생성

958d190


          docs : 요구사항 정의

d21602b


          feat : 문자열 계산기에 숫자 이외의 값 또는 음수를 전달하는 경우 RuntimeException 예외 생성

6b98e7b


          docs : 요구사항 수정

03f9aa1


          refactor : StringCalculator add 함수 변경

d8ed15c


          feat : 빈 문자열 또는 null을 입력할 경우 0을 반환 & 숫자 하나를 문자열로 입력할 경우 해당 숫자를 반환

80c2853


          feat : 숫자 두개를 쉼표(,) ,(:} 구분자로 입력할 경우 두 숫자의 합을 반환

16da34d


          feat : //와 \\n 문자 사이에 커스텀 구분자 추가

7a3eaa9


          feat : 문자열 파싱 함수화

sah3122 commented Jan 27, 2025

add 내 매개변수 text는 변경 불가한 값이므로 final 선언이 맞다고 생각했습니다.
내부의 private 함수들 중 text를 매개변수로 받는 것들도 모두 final을 선언해야하는지? 아니면, 이미 add 매개변수에서 final text로 들어와서 상관없는지 궁금합니다.

private 함수에 한정되지 않고 일반적으로 값이 불변한지 변경 가능한지만 판단 해보면 될것 같습니다.
매개 변수가 불변하다면 final 선언을 하고, 지역 변수를 선언하는 상황에서도 불변 값을 유지한다면 final 선언을 하여 불변성을 유지 하는것이 올바르다고 생각해요 😄

sah3122 requested changes

View reviewed changes

sah3122 left a comment

문자열 계산기 구현 잘해주셨습니다 👍
다음 미션을 진행하기 전 몇가지 생각거리를 남겨두었으니 확인 부탁드려요 😄

src/main/java/calculator/StringCalculator.java

    
                  private static final String DELIMITER = "//(.)\n(.*)";

                  public int add(final String text) {

                      if (text == null || text.isEmpty()) return 0;

sah3122 Jan 27, 2025

Suggested change

      
                    if (text == null || text.isEmpty()) return 0;
          
                    if (text == null || text.isEmpty()) {
          
                        return 0;
          
                    }

https://google.github.io/styleguide/javaguide.html#s4.1.1-braces-always-used

구글 스타일 가이드를 참고하여 중괄호를 생략하지 않고 작성해보세요 😄

src/main/java/calculator/StringCalculator.java

    
                      try {

                          String[] numbers = parse(text);

                          return sumOfNumbers(numbers);

                      } catch (Exception e) {

sah3122 Jan 27, 2025

최상위 Exception을 catch 한다면 어떤 문제점이 있을까요 ?

src/main/java/calculator/StringCalculator.java

    
                  }

                  private String[] parse(String text) {

                      Matcher m = Pattern.compile(DELIMITER).matcher(text);

sah3122 Jan 27, 2025

매번 Pattern.compile(DELIMITER)을 통해 객체가 생성되지 않고 재사용할수 있도록 변경해보는건 어떨까요 ?

src/main/java/calculator/StringCalculator.java

    
                              .sum();

                  }

                  private int validate(String number) {

sah3122 Jan 27, 2025

이번 과정을 진행하며 최대한 작은 단위의 클래스를 도출해보는것을 권장드립니다.
양수를 보장할수 있는 클래스를 추가해보세요 😄

src/main/java/calculator/StringCalculator.java

    
              import java.util.regex.Matcher;

              import java.util.regex.Pattern;

              public class StringCalculator {

sah3122 Jan 27, 2025

문자열 계산기 구현 잘해주셨습니다 👍
한가지 개선사항으로 현재 StringCalculator 가 너무 많은 역할을 담당하고 있다고 생각됩니다.
문자열 파싱을 담당하는 클래스, 계산을 담당하는 클래스 등으로 역할을 나누는 연습을 해보세요 !

src/main/java/calculator/StringCalculator.java

    
                  private int validate(String number) {

                      try {

                          int num = Integer.parseInt(number);

                          if (num < 0) {

sah3122 Jan 27, 2025

매직넘버는 의미를 가지는 상수로 정의하여 가독성을 높혀보세요 😄
https://www.slipp.net/questions/356

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet