[4기 - 소재훈] 1~2주차 과제: 계산기 구현 미션 제출합니다. by jay-so · Pull Request #157 · prgrms-be-devcourse/java-calculator

jay-so · 2023-06-10T11:27:28Z

구현한 내용

콘솔을 통해 사칙 연산(더하기, 빼기, 곱하기, 나누기, 우선 순위(사칙연산), 맵으로 계산 이력,정규식을 구현했습니다.

🔎 요구사항

콘솔로 구현입니다.(스윙으로 구현하시는 분들 계실까봐)
객체지향적인 코드로 계산기 구현하기
- 더하기
- 빼기
- 곱하기
- 나누기
- 우선순위(사칙연산)
테스트 코드 구현하기
계산 이력을 맵으로 데이터 저장기능 만들기
- 애플리케이션이 동작하는 동안 데이터베이스 외에 데이터를 저장할 수 있는 방법을 고안해보세요.
정규식 사용

✅ 구현할 때, 궁금했던 점 + 주로 봐주셨으면 하는 점

테스트 코드는 작성하지 못했으나, 본 프로그램 상에 피드백을 반영하여 코드를 수정하였습니다!
피드백을 반영하여, 매직넘버를 enum으로 분리하여 사용하였습니다!
본 코드에서 입출력 검증 부분에서 어려움을 겪어 간단하게만 구현했습니다

Hunkik

재훈님 코드 잘 봤습니다.

Map의 키 값으로 사용하도록 순서대로 하셨다고 했는데 나중에 구현된걸 보고 이야기해야할것같아요!
Number타입으로 했다고 하셨는데 그러면 소숫점이 없을땐 int,long 아닐땐 double, float을 쓰시는건가요??

Hunkik · 2023-06-10T12:16:35Z

src/main/java/Calculator.java

+    public Number calculator() {
+
+        //1. 연산자 우선순위를 계산함
+        Stack<Object> stack = new Stack<>();


타입과 네이밍을 의미있게 해주세요! 어떤 스택인지 알 수 있으면 좋아보입니다. 그리고 이름 지으실때 자료구조는 끝에 안넣으셔도 됩니다. (ex: objectStack과 같은식은 쓰지않기)

Hunkik · 2023-06-10T12:17:36Z

src/main/java/Calculator.java

+        result = operand1 + operand2;
+        return result;
+    }
+


이건 Tip인데 안쓰는 공백이나 import문은 지우는 습관이 필요합니다.

Hunkik · 2023-06-10T12:19:43Z

src/main/java/Calculator.java

+    int operand1 = scanner.nextInt(); //피연산자1
+    String operator = scanner.nextLine(); //연산자
+    int operand2 = scanner.nextInt(); //피연산자2


위의 operand1,operator,operand2같은 경우는 전역변수로 하면 해당 인스턴스를 재사용할 때 상태가 유지돼서 예상치 못한 버그를 발생시킬 수 있습니다. 전역변수보단 지역변수가 더 어울려보입니다. 입력같은경우도 입력책임을 갖는 클래스나 메서드를 이용하면 더 좋아보입니다.

그리고 전역변수로 할땐 접근제어자, static, final키워드도 같이 고려해보시면 좋을것같습니다.

Hunkik · 2023-06-10T12:33:15Z

src/main/java/Console.java

+public class Console {
+
+    public void start() {
+        Scanner scanner = new Scanner(System.in);


Scanner같은 경우는 런타임에서 재사용성이 많기 때문에 전역변수로 써도 될것같구, private, static, final과 같은 키워드도 같이 보려해보시면 좋을것같습니다.

Hunkik · 2023-06-10T12:36:12Z

src/main/java/Console.java

+            int selectMenu = scanner.nextInt();
+
+            //입력받은 값이 1이라면
+            if (selectMenu == 1) {


매직넘버를 지양해주세요. 만약 1을 Enum으로 쓴다면 더 좋아보입니다! 코드상으로 1이라는 메뉴가 무엇인지 모르기 때문에 고려해주세요.

Hunkik · 2023-06-10T12:37:21Z

src/main/java/Memorizer.java

+    //계산한 결과를 저장하는 ArrayList 클래스 - 계산 결과를 순번으로 저장함
+    ArrayList<Number> result = new ArrayList<>();


Map의 Value값으로 들어가는 result인가요? 이름도 의미있게 바꿔주시면 좋을듯합니다.

Hunkik · 2023-06-10T12:38:49Z

src/main/java/Operator.java

+    public int priority(String operator){
+        if(operator.equals('+')||operator.equals('-'))
+            return 1;
+        else if (operator.equals('*')||operator.equals('-')) {
+            return 2;
+        }
+        return -1;


매직넘버인 '+'를 쓰는것보단 ENUM을 통해서 쓰는게 더 좋아보입니다. Enum안에 우선순위를 넣으면 더 좋구요!

Hunkik

재훈님 고생하셨습니다! 2차 피드백남겼습니다

Hunkik · 2023-06-15T13:30:04Z

src/main/java/Console.java

+public class Console implements Input, Output {
+


Input과 Output을 하나로 한 이유가 있을까요? 추후 Input만 달라진 구현체가 필요하다고 하면 애매해 보입니다.

Hunkik · 2023-06-15T13:30:40Z

src/main/java/calculator/Application.java

+public class Application {
+    public static void main(String[] args) {
+        Console console = new Console();
+        Memorizer repository = new Memorizer();
+        InitCalculator initCalculator = new InitCalculator();
+        PostfixCalculator postfixCalculator = new PostfixCalculator();
+        new Calculator(console,console,repository,initCalculator, postfixCalculator).run();
+    }


Application이 두갠데 어디가 진짜 Application인가요??

Hunkik · 2023-06-15T13:31:18Z

src/main/java/calculator/Calculator.java

+    private final PostfixCalculator postfixCalculator;
+    public Calculator(Input input, Output output, Memorizer repository, InitCalculator initCalculator, PostfixCalculator postfixCalculator) {
+        this.input = input;
+        this.output = output;
+        this.repository = repository;
+        this.initCalculator = initCalculator;
+        this.postfixCalculator = postfixCalculator;
+    }
+    public void run() {


공백으로 변수, 생성자 메서드 등을 구분하면 가독성이 더 좋아일것같습니다

Hunkik · 2023-06-15T13:31:35Z

src/main/java/calculator/Calculator.java

+                case 1:
+                    output.MemoryCalculator(repository.getMemoroizer());
+                    break;
+                case 2:


여긴 아직 매직넘버네요..!

Hunkik · 2023-06-15T13:33:59Z

src/main/java/calculator/Console.java

+
+public class Console implements Input, Output {
+    private final Scanner scanner = new Scanner(System.in);
+    private static final Validation validation = new Validation();


콘솔메뉴쪽에서 Validation하는 코드만 있는것같은데, Enum에서 처리해주셔도 될 것 같습니다.

또 public static 메서드 같은 경우는 인스턴스 없이 사용 가능합니다.

Hunkik · 2023-06-15T13:37:05Z

src/main/java/calculator/Console.java

+    public void printCalculator(int result) {
+        System.out.println(result);
+    }
+}


마지막 빈줄을 통일해주셨으면 좋겠습니다!

Hunkik · 2023-06-15T13:38:02Z

src/main/java/calculator/engine/model/InitCalculator.java

+            List<String> postfixExpression = new ArrayList<>();
+
+            for (String ch : infixCalculator.split(" ")) {
+                if(Pattern.matches("[0-9]+",ch)){


Pattern.Compile로 컴파일해두고 쓰시면 비용적으로 재사용시 더 좋습니다

Hunkik · 2023-06-15T13:38:40Z

src/main/java/calculator/engine/model/InitCalculator.java

+                    }
+                    stack.pop();
+                } else {
+                    while (!stack.isEmpty() && getPrecedence(ch) <= getPrecedence(stack.peek())) {


Validation이 너무 길다면 편의 메서드도 고려하면 좋습니다

Hunkik · 2023-06-15T13:39:35Z

src/main/java/calculator/engine/model/InitCalculator.java

+import java.util.regex.Pattern;
+
+public class InitCalculator {
+    public static List<String> calculate(String infixCalculator) {


static 메서드인 이유가 있을까요?

Hunkik · 2023-06-15T13:40:16Z

src/main/java/calculator/engine/model/PostfixCalculator.java

+                switch (operand) {
+                    case "+":
+                        result = operand1 + operand2;
+                        break;
+                    case "-":
+                        result = operand1 - operand2;
+                        break;
+                    case "*":
+                        result = operand1 * operand2;
+                        break;


ENUM으로 case 해주시면 좋을것같아여 만드셔가지구

joseph-100 · 2023-06-16T08:10:47Z

�과제하느라 고생하셨어요.
남이 잘 읽을 수 있는 코드를 짜려고 노력하는게 중요합니다. 거기엔 미래의 자기 자신도 포함 됩니다.
훈기님 리뷰�에 덧붙이기만 할게요.

��PR을 주실때 돌아가는 코드를 올려주세요.
Calculator 35번줄에서 쓰셨듯이 InitCalculator static 메소드는 객체를 생성할 필요가 없는데 Application 에서 InitCalculator를 new 하시고 Calculator에 주입하신 이유가 있을까요?

jay-so added 6 commits June 7, 2023 00:39

setting:프로젝트 초기 세팅

a6d9389

setting:프로젝트 초기 세팅

36af495

rename:프로젝트 클래스명 변경

08c5860

rename: 프로젝트의 클래스명 재변경

2de006b

feat:실행 클래스와 콘솔 클래스 분리

4ca6c43

feat: 우선순위 클래스 구현

ea35d5e

jay-so added 💧 순한맛 🔥 매운맛 Team-Joseph labels Jun 10, 2023

jay-so self-assigned this Jun 10, 2023

Hunkik reviewed Jun 10, 2023

View reviewed changes

jay-so added 19 commits June 11, 2023 21:19

Conf 🔧: 프로젝트 .gitinore 설정 수정

aea03c4

feat:입/출력 I/O 분리

f00a925

fix:매직넘버를 enum으로 분리

708f96d

Move:Enums 패키지 경로 변경

a6a5bb5

feat: 연산 기능을 수행하는 Model 패키지 & 클래스 생성

2ccdfb1

feat: Console 로직 구현

4ed7169

rename: enum 요소 이름 수정

18b15de

fix: ConsoleMenu 추가

6a15272

feat:Parser 구현

9f10e57

fix: Application 수정

26d9a53

fix: Operator 구현

f843bb8

Remove: Parser 클래스 삭제

979514d

fix: enum 클래스 수정

1172864

fix: enum 클래스 수정

36c7c8e

refactor: 입출력 I/O 수정

540f883

feat: 후위 표기법 & 정규화 구현

fbef879

feat: Validation 구현

165784a

refactor: Memorizer 리팩토링

32d29a9

refactor: Application 리팩토링

987a6d1

jay-so added 3 commits June 15, 2023 03:31

refactor: Console 리팩토링

fb442a7

feat: Calculator 클래스 구현

63b48e8

remove : Operator 클래스 삭제

7729ce5

jay-so changed the title ~~4기 - 소재훈] 1~2주차 과제: 계산기 구현 미션 제출합니다.~~ [4기 - 소재훈] 1~2주차 과제: 계산기 구현 미션 제출합니다. Jun 15, 2023

jay-so requested a review from Hunkik June 15, 2023 09:57

Hunkik reviewed Jun 15, 2023

View reviewed changes

remove :파일경로 변경으로 인한 중복된 Application,Console 삭제

70de99f

jay-so force-pushed the main branch from c678115 to 70de99f Compare June 18, 2023 16:49

Update README.md

e97a848

		//계산한 결과를 저장하는 ArrayList 클래스 - 계산 결과를 순번으로 저장함
		ArrayList<Number> result = new ArrayList<>();

Conversation

jay-so commented Jun 10, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

구현한 내용

🔎 요구사항

✅ 구현할 때, 궁금했던 점 + 주로 봐주셨으면 하는 점

Uh oh!

Hunkik 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!

Hunkik 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!

joseph-100 commented Jun 16, 2023

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

jay-so commented Jun 10, 2023 •

edited

Loading