[계산기] 권민관 미션 1차 수정본 입니다! by mlngwan · Pull Request #25 · next-step/java-calculator-unit-playground

mlngwan · 2024-05-11T13:53:19Z

초간단 계산기와 문자열 계산기 구현해보았습니다.
테스트 코드를 처음 공부해보는데 덕분에 좋은 공부했습니다. 감사합니다!

sosow0212

수고하셨습니다!
미션 요구사항이 지켜지지 않은 것 같은데 한 번 확인 부탁드릴게요
정렬 습관화 들이면 더 좋을 것 같습니다!

sosow0212 · 2024-05-12T03:54:11Z

src/main/java/simpleCalculator/controller/CalculatorController.java

+import simpleCalculator.view.CalculatorView;
+import java.util.List;
+
+public class CalculatorController {


해당 클래스 외에도 첫줄 개행 추가해주세요

네 수정하겠습니다!

sosow0212 · 2024-05-12T03:54:54Z

src/main/java/simpleCalculator/controller/CalculatorController.java

+    public void doAdd() {
+        int addResult = calculator.addNumbers();
+        System.out.println("덧셈 결과 : " + addResult);
+    }
+
+    public void doSub(){
+        int subResult = calculator.subNumbers();
+        System.out.println("뺄셈 결과 : " + subResult);
+    }
+
+    public void doDivide(){
+        int divideResult = calculator.divideNumbers();
+        System.out.println("나눗셈 결과 : " + divideResult);
+    }
+
+    public void doMultiple(){
+        int multipleResult = calculator.multipleNumbers();
+        System.out.println("곱셈 결과 : " + multipleResult);
+    }


사용자의 입력에 따라서 곱셈인지 뺄셈인지 등등 나뉘어야할 것 같아요
지금은 모든 결과를 출력하는데 다른 방식으로 해야할 것 같아요

따로 결과를 출력하는 OutputView로 분리하여 출력하도록 수정하겠습니다.

sosow0212 · 2024-05-12T03:55:27Z

src/main/java/simpleCalculator/model/Calculator.java

+    private final int num1;
+    private final int num2;
+
+    public Calculator(int num1, int num2){


음수일 때 예외가 없습니다 요구사항 지켜주세요~

음수일 때 예외처리 하도록 수정했습니다.

sosow0212 · 2024-05-12T03:55:42Z

src/main/java/simpleCalculator/model/Calculator.java

+        return num1 - num2;
+    }
+
+    public int divideNumbers(){


정렬이 안 되어있습니다

나누는 수가 0일 때 예외가 없습니다

모든 코드들 정렬했습니다!

나누는 수가 0일 때 예외 처리 진행했습니다!

sosow0212 · 2024-05-12T03:56:12Z

src/main/java/simpleCalculator/model/Calculator.java

+    public int multipleNumbers(){
+        return num1 * num2;
+    }
+}


EOF가 발생하네요 검색하셔서 해결해주세요~

EOF에 대해 맞게 이해 했는지와 궁금한 점이 생겼습니다.
입력의 개수가 정해지지 않은 경우에만 더 이상 입력이 없게 하기 위해 사용하는 것으로 이해했습니다.
scanner.close()와 차이점이 무엇인가요??

scanner.close()는 resource를 풀어주는 거라 다른 결이긴 합니다!
해당 링크 참고해보시면 좋을 것 같아요!

sosow0212 · 2024-05-12T03:59:34Z

src/main/java/stringCalcculator/model/StringCalculator.java

+        return number;
+    }
+
+    public int sum(int[] inputList){


sum이 여기도 있는 이유가 있을까요?

메서드 네이밍이 헷갈려서 그런가요??
아니면 필요 없는 메서드라고 생각하셔서 그러신 걸까요??
해당 메서드는 배열 안의 요소를 합하는 메서드라 필요하다고 생각했습니다!
혹시 메서드 명이 문제라면 수정하도록 하겠습니다.

sosow0212 · 2024-05-12T04:00:28Z

src/test/java/calculator/CalculatorTest.java

+        int testNum1 = 2;
+        int testNum2 = 1;
+        Calculator calculator = new Calculator(testNum1, testNum2);


위 아래로 계행 하나씩 추가해주세요

넵 수정했습니다!

sosow0212 · 2024-05-12T04:00:44Z

src/test/java/calculator/CalculatorTest.java

+        void addTest(){
+            //given
+            int expectValue = 3;
+            int testAdd = calculator.addNumbers();


이건 when으로 가야겠네요~

넵 확인하고 수정 완료 했습니다!

sosow0212 · 2024-05-12T04:01:31Z

src/test/java/calculator/CalculatorTest.java

+        int testNum1 = 2;
+        int testNum2 = 1;
+        Calculator calculator = new Calculator(testNum1, testNum2);
+        @DisplayName("더하기 테스트 1")


예외를 검증하는 거면 네이밍을 바꾸는게 좋을 것 같습니다

예외 테스트 네이밍 명시적으로 수정 완료했습니다.

sosow0212 · 2024-05-12T04:02:10Z

src/main/java/stringCalcculator/model/StringCalculator.java

@@ -0,0 +1,57 @@
+package stringCalcculator.model;


패키지 명 오타났네요!
그리고 다른 패키지에 둔 이유가 있을까요?

다른 패키지에 둘 이유가 없다고 생각해서 기존의 model 패키지에 합병했습니다!

mlngwan · 2024-05-16T11:31:47Z

코드 전체적으로 Refactoring 진행하였습니다.
진행 내역은 다음과 같습니다.

코드 전반적으로 내부에서 사용하는 메서드는 public이 아닌 private를 사용하도록 접근제어자 수정했습니다.
EOF를 처리하도록 수정했습니다.
의미없는 static을 사용하지 않도록 수정했습니다.
Code Convention
- Class 첫 줄에는 개행이 있도록 추가했습니다.
- 메서드 명은 동사로 시작하도록 수정했습니다.
- 메서드 명시 다음의 띄어쓰기 하도록 수정했습니다.
- 자주 사용하는 값들은 상수로 사용하게 수정했습니다.
- 상수 네이밍은 좀 더 명시적으로 수정했습니다.
Model Refactoring
- Calculator가 요구 사항을 지키도록 수정했습니다. (0으로 나누기가 불가, 음수 예외 추가)
- 추가적으로 입력 인자가 2개인지 검사하도록 수정했습니다.
- 기존의 Model에서 View에서 처리할 부분이 있었습니다. View에서 처리하도록 수정했습니다.
- StringCalculator 구분하지 않도록 수정했습니다.
View Refactoring
- View를 InputView와 OutputView로 분리하였습니다.
- 기존의 View에서 Model이 처리할 부분이 있었습니다. Model에서 처리하도록 수정했습니다.
Controller Refactoring
- Model과 View 수정에 맞춰 Controller도 잘 작동하도록 수정했습니다.
- Controller에서 InputView에서 받아온 값들을 Model의 로직으로 처리하고 결과를 OutputView로 반환하도록 수정했습니다.
Test Refactoring
- 기존의 코드들의 수정에 따라 Test가 잘 작동하게 수정했습니다.
- 보기 편하도록 Test하는 방법이 비슷하면 같은 분류로 묶었습니다.

sosow0212

이번 단계는 여기서 마무리하겠습니다!
변수명과 기본적인 자바에 대해서 리뷰를 했는데, 리뷰 내용 확인만 해주시고 다음 미션부터 적용해주세요!

수고하셨습니다 👍

sosow0212 · 2024-05-21T04:12:24Z

src/main/java/simpleCalculator/model/Calculator.java

+    public int multipleNumbers(){
+        return num1 * num2;
+    }
+}


scanner.close()는 resource를 풀어주는 거라 다른 결이긴 합니다!
해당 링크 참고해보시면 좋을 것 같아요!

sosow0212 · 2024-05-21T04:12:40Z

src/main/java/simpleCalculator/model/StringCalculator.java

+        }
+        return sum(listToInt(splitInput(input)));
+    }
+}


여기도 설정해서 수정해주세요~

보내주신 내용 확인해서 파일마다 끝에 한 줄 비워두는 방식으로 수정했습니다!

sosow0212 · 2024-05-21T04:13:08Z

src/main/java/simpleCalculator/view/InputView.java

+
+public class InputView {
+
+    private Scanner scanner;


private,
private final,
private static final,
private static

각각 차이는 무엇일까요?

private는 해당 클래스 내부에서만 접근 가능하게 만들어주는 접근 제어자 입니다.
private final은 한 번 값이 설정되면 그 값이 고정되며 인스턴스 별로 각기 다른 값을 가질 수 있습니다.
그리고 private에 의해 해당 클래스 내부에서만 접근 가능합니다.
private static final은 모든 인스턴스가 사용하는 고정된 값을 할당할 때 사용합니다. 주로 상수 설정에 사용합니다.
private static은 객체 생성 없이 사용 가능한 필드나 메서드들을 생성하고 싶을 때 사용합니다.
클래스의 모든 인스턴스가 접근 가능하지만 private에 의해 해당 클래스 내부에서만 접근 가능합니다.

위와 같이 이해하고 있는데 잘못된 부분이 있을까요?

sosow0212 · 2024-05-21T04:13:34Z

src/main/java/simpleCalculator/view/InputView.java

+        while (scanner.hasNextLine()) {
+            return scanner.nextLine();
+        }
+        return null;


return scanner.next(); 이렇게 해도 될 것 같아요!

EOF를 잘못 이해하고 나름대로 resource를 풀어주려고 작성했습니다.
보내주신 내용 확인하고 수정 완료했습니다!

sosow0212 · 2024-05-21T04:13:42Z

src/test/java/calculator/CalculatorTest.java

+    private static final int EXPECT_VALUE = 3;
+    private Calculator calculator;


Suggested change

private static final int EXPECT_VALUE = 3;

private Calculator calculator;

private static final int EXPECT_VALUE = 3;

private Calculator calculator;

수정 완료했습니다!

sosow0212 · 2024-05-21T04:14:52Z

src/test/java/stringCalculator/StringCalculatorTestAssertJ.java

+            assertThat(EXPECT_VALUE_ZERO).isEqualTo(calculator.add(null));
+            assertThat(EXPECT_VALUE_ZERO).isEqualTo(calculator.add(""));


지금은 위에 테스트 메서드가 실패하면 아래는 작동하지 않아서 여러 값을 테스트한다면 softly로 적용하는 것이 좋을 것 같습니다!

확인하고 수정하였습니다!

mlngwan · 2024-05-22T15:44:44Z

2차 Code Refactoring 진행하였습니다.

EOF 관련
- 기존의 코드에서는 더 이상 받을 resource가 없다는 의미의 코드로 작성했습니다.
- 각 파일 끝에 한 줄을 비워두는 방식으로 해결했습니다.
Test 관련
- 기존의 코드에서는 만약 테스트가 실패한다면 실패한 지점에서 테스트가 멈춰버리게 되는 문제점이 있었습니다.
- 따라서 실패 여부와 상관없이 모든 테스트를 실행하도록 문제점을 해결하였습니다.

Refactoring 과정에서 배운 내용

EOF (End OF File)
- 과거 IEEE에서 책정한 POSIX에 의해 파일 끝에는 newline 문자가 존재해야 합니다.
- 한 행씩 읽는 과정에서 newline 문자가 없으면 끝나지 않은 행으로 간주하기 때문입니다.
- 요즘에는 위 과정이 없어도 잘 작동하지만 혹시 모를 오류나 관례(?)로 사용합니다.
- 참고 : https://hjuu.tistory.com/8, https://velog.io/@doondoony/posix-eol
Test - Softly
- assertThat(...)은 테스트에 실패하게 되면 실패를 알리고 테스트를 중지합니다.
- 테스트에 실패하는 경우에도 밑의 테스트까지 시도하기 위해 SoftAssertions.assertSoftly(...)을 사용해야 합니다.
- 테스트마다 SoftAssertions 객체를 생성해줘야 하는데 이를 @InjectSoftAssertions을 통해 간결하게 할 수 있습니다.
- 참고 : https://jiwondev.tistory.com/186

mlngwan and others added 19 commits May 8, 2024 01:02

docs: 기능 명세서 작성

74f7544

feat: 기초적인 계산기 초기 단계 구현

903f21d

Docs: TODO 세분화 후 README 수정

2823002

Remove: OutputView 삭제

b0c6ed1

Rename: 파일명 수정

3bfb2cf

feat: MVC 구조를 위한 Controller 추가

ed9a151

Refactor: Controller 추가로 인한 Caculator 수정

a082226

feat: Calculator Test 초안 작성

5352af0

feat: Calculator 계산 테스트 구현

2b11bab

feat: 입력내용 저장 테스트 구현

cf79840

refactor: Test를 위한 CalculatorView 코드 수정

ec61541

docs: step3 진행을 위한 문서 작성

b8751fc

Rename: step3 진행을 위한 문서 이동

ae57faf

Feat: StringCalculator 구현

6b1fde1

Feat: StringCalculator DEFAULT 값 추가 및 숫자가 아닌 다른 형식의 경우 오류값 출력 구현

f7900c1

Test: 테스트명 직관적으로 수정

b1398c6

Test: 문자열 계산기 Test 구현

31642a6

Test: 문자열 계산기 Test 쓸모없는 Import 삭제

63e45f6

Test: AssertJ를 활용한 Test 작성

d97a6c6

sosow0212 suggested changes May 12, 2024

View reviewed changes

mlngwan added 10 commits May 13, 2024 11:25

Refactor: AssertJ 사용을 위한 dependencies 추가

cfe56c3

Rename: Test Code 위치 변경 및 파일 이름 수정

eb5a102

Docs: step4를 위한 문서 작성

fc96402

Refactor: 코드 컨벤션 - 개행 및 정렬 수정

c175799

Refactor: Test Code 네이밍 수정

cab781b

Refactor: 상수값 네이밍 재설정 및 새로운 상수값 설정

127590a

Remove: InputView와 OutputView로 분리로 인한 삭제

a2f073b

Feat: OutView 구현

6efad2c

Refactor: 불필요한 개행 삭제

b14e0b1

feat: view 분리 - InputView 구현

b01efb9

mlngwan added 8 commits May 15, 2024 03:06

Refactor: 요구사항 준수, 기존의 view에서 담당하던 로직을 model에서 처리하기, 예외처리 추가

a333ec8

Refactor: View와 Model 리팩토링으로 인해 수정, 기존의 outputView가 담당해야 하는 부분 수정

3e57660

Refactor: Test 결과 예외처리 부분 수정

76e6def

Test: 리팩토링 후 새로운 Test 코드 작성

78c9949

Test: 테스트코드 상수 설정 및 정렬

d6e7d1c

Test: 테스트코드 상수 설정, 및 개행 수정, 중첩 테스트 클래스 설정

40c8548

Test: 테스트코드 상수 설정, 및 개행 수정, 중첩 테스트 클래스 설정

b115121

Refactor: 사용자의 입력을 받는 inputView EOF처리

4fb6e5c

mlngwan changed the title ~~[계산기] 권민관 미션 제출입니다!~~ [계산기] 권민관 미션 1차 수정본 입니다! May 16, 2024

sosow0212 approved these changes May 21, 2024

View reviewed changes

mlngwan added 3 commits May 23, 2024 00:46

Refactor: 파일의 끝에 개행을 통해 EOF 처리

8de5ddc

Test: 컨벤션 지켜지지 않은 부분 수정

4601969

Refactoring: 불필요한 부분 제거로 코드 간결화

f185842

		private static final int EXPECT_VALUE = 3;
		private Calculator calculator;

		assertThat(EXPECT_VALUE_ZERO).isEqualTo(calculator.add(null));
		assertThat(EXPECT_VALUE_ZERO).isEqualTo(calculator.add(""));

Conversation

mlngwan commented May 11, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

sosow0212 left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

mlngwan May 15, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

mlngwan commented May 16, 2024

코드 전체적으로 Refactoring 진행하였습니다. 진행 내역은 다음과 같습니다.

Uh oh!

sosow0212 left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

mlngwan commented May 22, 2024

mlngwan commented May 11, 2024 •

edited

Loading

mlngwan May 15, 2024 •

edited

Loading

코드 전체적으로 Refactoring 진행하였습니다.
진행 내역은 다음과 같습니다.