[2주차] 객체지향 코드 연습(choi wonbin) #7

ChoiTheCreator · 2024-04-01T07:32:37Z

추가 Q & A

🤔 잘 모르겠거나, 더 궁금한 내용이 있었나요?

집중적으로 리뷰 받고싶은 것이 있나요?

KoSeonJe

메소드(=함수)의 길이는 20줄이상을 넘어가면 코드의 가독성이 떨어지게 됩니다. 그 이상 길이의 코드를 작성하게 된다면 메소드를 분리해주세요

코드 컨벤션을 지켜야 할 것 같습니다. 아래 링크를 참고해주세요
https://sihyung92.oopy.io/af26a1f6-b327-45a6-a72b-c6fcb754e219

이 코드가 실행되는 Main메소드가 구현이 되지 않고 있는 것 같습니다. 추가적으로 구현 해주시면 좋을 것 같아요

KoSeonJe · 2024-04-02T03:36:36Z

bank/src/main/java/org/example/Account/Account.java

+import java.math.BigDecimal;
+
+public class Account {
+    //필요한 정보 : 일반 계좌 클래스의 속성은 종류 2가지 ( 1. N 예금계좌 , 2. S 적금계좌)


코드 내에 많은 주석을 만들어 놓는 것은 자신이 작성한 코드만 보고 이해할 수 없다는 것을 명시하고 있는 것입니다. 꼭 필요한 주석만 남겨주세요

KoSeonJe · 2024-04-02T03:37:52Z

bank/src/main/java/org/example/Account/Account.java

+    //두번째 생성자에는 계좌번호와 소유자, 잔액을 표기한다.
+    public Account(String accountNum, String owner, BigDecimal balance) {
+        //생성자 체이닝을 활용하여, 위의 account()의 내용도 같이 선언해준다.
+        this();


생성자를 한번 더 호출하는 것이 아닌, 이 생성자 내부에서 같이 초기화 하는 것이 더 좋은 것 같습니다.

KoSeonJe · 2024-04-02T03:38:26Z

bank/src/main/java/org/example/Account/Account.java

+
+
+
+


필요없는 줄은 모두 삭제해주세요

KoSeonJe · 2024-04-02T03:39:53Z

bank/src/main/java/org/example/Account/Account.java

+        //계좌의 잔액이 출금하고자 하는 금액보다 낮다면 ( 에러 던지기)
+        if(this.balance.compareTo(amount) < 0){
+            throw new WithdrawException("잔액이 모자랍니다.");
+        }else{


else 문은 해당 메소드가 어떤 행위를 하고 있는지 더 많은 고민을 하게 만들기에 사용하는 것을 지양합니다.

early return 패턴을 사용해주세요

KoSeonJe · 2024-04-02T03:41:04Z

bank/src/main/java/org/example/Account/Account.java

+    // 뱅크 클래스에서 호출할 출금, 입금 기본 메서드를 생성합니다.
+
+    //출금 관련 메서드 withdraw
+    public BigDecimal withdraw(BigDecimal amount) throws WithdrawException {


메소드의 책임을 줄여주는 것이 좋습니다.
출금 뿐만아니라 해당 잔고가 출금하려는 잔액보다 많은지의 여부를 판단하는 기능을 따로 메소드로 분리해주는 것이 좋을 것 같습니다

KoSeonJe · 2024-04-02T04:14:40Z

bank/src/main/java/org/example/Bank/CentralBank.java

+
+    // getInstance 함수
+    public static CentralBank getInstance(){
+        if(instance == null)


코드 컨벤션을 지켜주세요
if문 내의 코드가 한줄이어도 괄호를 사용하여 감싸주는 것이 좋습니다

KoSeonJe · 2024-04-02T04:15:24Z

bank/src/main/java/org/example/Bank/SavingBank.java

+            System.out.printf("%s님 계좌가 발급되었습니다.", owner);
+            return account;
+        }catch (InputMismatchException ime){
+            if(seq > 0) seq--;


어떤 일을 수행하고 있나요?

KoSeonJe · 2024-04-02T04:16:03Z

bank/src/main/java/org/example/Bank/SavingInterestCalculator.java

+
+        // 적금 계좌의 경우 잔액이 100만원 이상은 이자율이 50%, 그 외에는 1% 입니다.
+        if(balance.compareTo(BigDecimal.valueOf(1000000)) >= 0)
+            interest = BigDecimal.valueOf(0.5);


0.5가 어떤 것을 의미하나요? 변수를 선언하고 구체적인 변수명을 지어주세요

bank/src/main/java/org/example/Main.java

KoSeonJe · 2024-04-02T05:04:30Z

bank/src/main/java/org/example/Constant/ErrorCode.java

+    E104("E104", "적금 계좌는 잔액이 목표 금액(%s원) 이상이어야 출금 가능합니다."),
+
+    // IO Error
+    E201("E201", "입력값 오류 발생");


위 숫자들은 어떤 의미를 가지고 있나요? 더 의미있는 이름을 사용해보도록 합시다

Suggested change

E201("E201", "입력값 오류 발생");

INPUT_ERROR("E201", "입력값 오류 발생");

처럼 해당 필드의 제목은 떠오르기 쉽게 관리하는 것이 더 나을 것 같습니다!

Gopistol

전체적으로 구현에 초점을 맞춘 코드들이 있었습니다. 객체지향의 핵심인 역할, 책임, 협력에 대하여 한 번 찾아보시는 것을 추천합니다.

리뷰에서 역할, 책임 등의 단어가 많이 나왔습니다. 이렇게 추상적인 단어를 대입해 고민하는 이유는
추후 변경하기 용이한, 유연한 코드를 만들기 위함입니다. 또한 예상치 못한 에러 발생 시 더 빠르고 쉽게 대처할 수 있습니다.

남은 시간 동안 위에서 말씀드린 역할과 책임의 관점에서 구현한 코드에 대해 고민해보시면 좋을 것 같습니다!

Gopistol · 2024-04-02T07:37:06Z

bank/src/main/java/org/example/Constant/ErrorCode.java

+    E104("E104", "적금 계좌는 잔액이 목표 금액(%s원) 이상이어야 출금 가능합니다."),
+
+    // IO Error
+    E201("E201", "입력값 오류 발생");


Suggested change

E201("E201", "입력값 오류 발생");

INPUT_ERROR("E201", "입력값 오류 발생");

처럼 해당 필드의 제목은 떠오르기 쉽게 관리하는 것이 더 나을 것 같습니다!

Gopistol · 2024-04-02T07:55:29Z

bank/src/main/java/org/example/Account/savingAccount.java

+//적금 계좌 클래스는 일반 예금 계좌 클래스에서 상속을 받고 목표 금액 속성이 추가 됩니다.
+
+
+public class savingAccount extends Account {


클래스 네이밍 컨벤션을 지켜주세요!

Gopistol · 2024-04-02T08:38:05Z

bank/src/main/java/org/example/Bank/Bank.java

+            return (T)input;
+        }
+    }
+}


askInput()이라는 메소드의 이름으로 봤을 때, 실제 이 메소드가 입력을 요청하는 행동을 할 것으로 생각됩니다. 하지만 실제 메소드에서는 입력이라는 역할보다 더 많은 역할을 가지고 있는 것 같습니다.

오로지 입력만을 위한 클래스를 따로 만든 후, 입력을 받는 과정은 입출력 용도로 사용하는 클래스를 사용할 수 있습니다.

Erichong7

전반적으로 자신이 어떤 코드를 작성했는지와 자신의 코드 흐름의 이해가 부족한 거 같습니다. 기능 개발을 무작정하는 것보다는 자신의 코드를 리펙토링하는 과정이 더 중요해보입니다. 좋은 예시들을 많이 접해보고 적용해봅시다. 코드 컨벤션 공부 중요합니다!

Erichong7 · 2024-04-03T09:38:54Z

bank/src/main/java/org/example/Account/savingAccount.java

+    public savingAccount(){
+        super();
+        this.category = "Saving";
+    }


Category를 자바 enum을 사용한다면 조금 더 좋은 코드가 될 거 같습니다

Erichong7 · 2024-04-03T09:40:59Z

bank/src/main/java/org/example/Bank/Bank.java

+ */
+
+public class Bank {
+    protected static Scanner scanner = new Scanner(System.in);


입출력에 대한 책임은 Bank에 있나요? Scanner 또한 static으로 설정한 이유가 궁금합니다. static이 붙은 변수, Sanncer 둘다 가비지 컬렉션 대상에서 제외됩니다. 입출력에 대한 책임 분리에 대하여 고민해봅시다.

Erichong7 · 2024-04-03T09:42:37Z

bank/src/main/java/org/example/Bank/Bank.java

+                //이자 계산
+                BigDecimal interest = interestCalculators.get(account.getCategory()).getInterest(account.getBalance());
+                result = account.withdraw(amount);
+                System.out.printf("\n출금액 %s원과 이자 %s원이 정상적으로 출금되었습니다.\n", df.format(result), df.format(interest));


매직넘버에 대해 알아보고 적용해봅시다

Erichong7 · 2024-04-03T09:45:34Z

bank/src/main/java/org/example/Account/Account.java

+        this.category = category;
+    }
+
+    //필요한 계좌정보를 불러내는 메서드


모든 기능에 대하여 일일이 주석을 다는 것은 좋아보이지 않습니다. 주석없이 이해가 되는 코드가 좋은 코드입니다. 만약 정말로 주석을 달고싶다면 JSdoc에 대하여 알아보고 적용하는 것도 좋아보이네요

Erichong7 · 2024-04-03T09:47:23Z

bank/src/main/java/org/example/Account/Account.java

+
+    }
+
+    //getter & setter part - 접근 지정자가 protected 이기 때문에


setter를 남발하는 것은 좋지 않습니다. 자바의 캡슐화에 대하여 고민해봅시다.

Erichong7 · 2024-04-03T09:52:51Z

bank/src/main/java/org/example/Bank/Bank.java

+            //signum() 함수는 0,-1, 1 셋 중 하나를 반환하며 음수인지 양수인지 0인지를 파악하는 함수이다.
+
+            // 즉 input type = bigdecimal인데, 소수점이 존재하거나 음수일 경우
+            if (input instanceof BigDecimal && (((BigDecimal) input).scale() > 0 || ((BigDecimal) input).signum() < 0)) {


instanceof는 SOLID원칙을 해치기 때문에 지양해야합니다. instanceof를 사용하지 않는 방식은 정말정말 많습니다.
https://tecoble.techcourse.co.kr/post/2021-04-26-instanceof/

Erichong7 · 2024-04-03T09:56:39Z

bank/src/main/java/org/example/Bank/Bank.java

+            }catch(WithdrawException we) {
+                throw we;
+            }catch (Exception e){


개별적으로 커스텀한 Exception을 만들어 두셨음에도 에러 메세지를 생성자로 보내고 있지 않는데 이유가 무엇인가요?

KoSeonJe

고쳐야할 점 말씀 드리겠습니다.

사용하지 않는 메소드, 클래스 삭제하기
MVC패턴을 사용하기 위해서 MVC패턴이 어떤 것인지 정확히 파악하기. 저도 10번이상 검색하면서 많이 찾아봤습니다. 그만큼 이해가 잘안됐습니다.
메소드 20줄이상 넘기지 않기
switch문 enum으로 바꾸기
메소드명, 변수명 신경써서 짓기
접근 제어자, final키워드, static키워드 신경써서 적용하기
입출력을 객체로부터 분리하기. 객체 내에서 입출력 제발 하지 말기
코드 컨벤션 지키기... 중요!!

화이팅입니다! 스프링이 중요한게 아닙니다. 이것부터 제대로 알고 가야하는 것이 중요해요

KoSeonJe · 2024-05-08T14:37:01Z

bank/src/main/java/org/example/Main.java

+
+//TIP To <b>Run</b> code, press <shortcut actionId="Run"/> or
+// click the <icon src="AllIcons.Actions.Execute"/> icon in the gutter.
+public class Main {


Main이라는 이름보다 더 의미있는 이름으로 변경해주세요
예를 들어 BankingApplication, BankingSystem

KoSeonJe · 2024-05-08T14:38:57Z

bank/src/main/java/org/example/Main.java

+public class Main {
+    public static void main(String[] args) {
+        Bank bank = new Bank("국민은행");
+        bank.menuBank();


menuBank() 명사+명사 보다 동사+명사로 메소드명을 짓는게 의미를 전달하기 더욱 좋습니다

KoSeonJe · 2024-05-08T14:40:10Z

bank/src/main/java/org/example/account/Account.java

+
+

Suggested change

KoSeonJe · 2024-05-08T14:41:10Z

bank/src/main/java/org/example/account/Account.java

+    public String id;
+    public String pw;


Suggested change

public String id;

public String pw;

User에도 id와 pw가 있는데, 사용되지 않는다면 지워주시길 바랍니다

KoSeonJe · 2024-05-08T14:42:37Z

bank/src/main/java/org/example/account/Account.java

+
+public class Account {
+
+    public static int howmanyaccout = 0;


Suggested change

public static int howmanyaccout = 0;

private static int howmanyaccout = 0;

접근제어자를 private로 하여 캡슐화를 지키도록 합시다.
Account를 관리하고 있는 객체가 있을텐데, 그 객체를 이용해 계좌 개수를 알아내도 괜찮을 것 같습니다
위 변수는 필요가 없을 것 같습니다

KoSeonJe · 2024-05-08T14:59:36Z

bank/src/main/java/org/example/bank/Bank.java

+
+    public void displayinfo_Bank(){
+        System.out.println("현재 사용하고 계신 은행 : "+ nameofBank);
+        for(int i=0; i<accountList.size(); i++){


stream에 대해 공부해보시고 적용하시키세요

KoSeonJe · 2024-05-08T15:00:02Z

bank/src/main/java/org/example/bank/Bank.java

+    }
+
+
+    public void displayinfo_Bank(){


스네이크 표기법이 아닌 카멜케이스 표기법을 사용해주세요

KoSeonJe · 2024-05-08T15:00:32Z

bank/src/main/java/org/example/bank/Bank.java

+            AccountView accountView = new AccountView(accountList.get(accountcnt));
+            accountView.showmenu();
+        }
+        else{


else는 사용하지 않는 것이 가급적 좋습니다.
early return 패턴을 사용해보세요

KoSeonJe · 2024-05-08T15:01:10Z

bank/src/main/java/org/example/bank/Bank.java

+        delete(accountList.get(accountcnt).id,pw);
+    }
+
+    public void delete(String id, String pw){


Suggested change

public void delete(String id, String pw){

private void delete(String id, String pw){

KoSeonJe · 2024-05-08T15:01:40Z

bank/src/main/java/org/example/bank/Bank.java

+
+
+
+
+


Suggested change

뱅크 시스템 커밋 0401

b4fdf18

ChoiTheCreator requested review from Hoya324, KoSeonJe, Gopistol and TaetaetaE01 as code owners April 1, 2024 07:32

KoSeonJe reviewed Apr 2, 2024

View reviewed changes

Gopistol reviewed Apr 2, 2024

View reviewed changes

Erichong7 reviewed Apr 3, 2024

View reviewed changes

ChoiTheCreator added 4 commits April 7, 2024 19:58

뱅크 시스템 커밋 0407

74aa8da

뱅크 시스템 커밋 0407

19a1555

뱅크 시스템 커밋 0409 my new version

85cdc96

뱅크 시스템 커밋 0508 my new version

691937e

KoSeonJe reviewed May 8, 2024

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[2주차] 객체지향 코드 연습(choi wonbin) #7

[2주차] 객체지향 코드 연습(choi wonbin) #7

ChoiTheCreator commented Apr 1, 2024

KoSeonJe left a comment •

edited

Loading

KoSeonJe Apr 2, 2024

KoSeonJe Apr 2, 2024

KoSeonJe Apr 2, 2024

KoSeonJe Apr 2, 2024

KoSeonJe Apr 2, 2024

KoSeonJe Apr 2, 2024

KoSeonJe Apr 2, 2024

KoSeonJe Apr 2, 2024

KoSeonJe Apr 2, 2024

Gopistol Apr 2, 2024

Gopistol left a comment

Gopistol Apr 2, 2024

Gopistol Apr 2, 2024

Gopistol Apr 2, 2024

Erichong7 left a comment

Erichong7 Apr 3, 2024

Erichong7 Apr 3, 2024

Erichong7 Apr 3, 2024

Erichong7 Apr 3, 2024

Erichong7 Apr 3, 2024

Erichong7 Apr 3, 2024

Erichong7 Apr 3, 2024

KoSeonJe left a comment

KoSeonJe May 8, 2024

KoSeonJe May 8, 2024

KoSeonJe May 8, 2024

KoSeonJe May 8, 2024

KoSeonJe May 8, 2024

KoSeonJe May 8, 2024

KoSeonJe May 8, 2024

KoSeonJe May 8, 2024

KoSeonJe May 8, 2024

KoSeonJe May 8, 2024

	E201("E201", "입력값 오류 발생");
	INPUT_ERROR("E201", "입력값 오류 발생");

		//적금 계좌 클래스는 일반 예금 계좌 클래스에서 상속을 받고 목표 금액 속성이 추가 됩니다.


		public class savingAccount extends Account {


		}

		//getter & setter part - 접근 지정자가 protected 이기 때문에

	public static int howmanyaccout = 0;
	private static int howmanyaccout = 0;

	public void delete(String id, String pw){
	private void delete(String id, String pw){

[2주차] 객체지향 코드 연습(choi wonbin) #7

Are you sure you want to change the base?

[2주차] 객체지향 코드 연습(choi wonbin) #7

Conversation

ChoiTheCreator commented Apr 1, 2024

추가 Q & A

집중적으로 리뷰 받고싶은 것이 있나요?

KoSeonJe left a comment • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Gopistol left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Erichong7 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

KoSeonJe left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

KoSeonJe left a comment •

edited

Loading