inflearn logo
강의

Course

Instructor

Kim Younghan's Practical Java - Basics

섹션2 문제와풀이 ProductOrderMain3 문제 이렇게 풀어보았습니다.

162

wodlr12070547

13 asked

0

package ref.ex;

import java.util.Scanner;

public class ProductOrderMain3 {

    static Scanner scanner = new Scanner(System.in);

    public static void main(String[] args) {

        System.out.print("입력할 주문의 개수를 입력하세요: ");
        int arrayLength = scanner.nextInt();

        ProductOrder[] orders = createOrder(arrayLength);
        printOrders(orders);

        System.out.println("총 결제 금액: " + getTotalAmount(orders));
    }

    static ProductOrder[] createOrder(int arrayLength) {

        ProductOrder[] orders = new ProductOrder[arrayLength];

        for (int i = 0; i < arrayLength; i++) {
            orders[i] = new ProductOrder();

            scanner.nextLine();
            System.out.println((i + 1) + "번째 주문 정보를 입력하세요.");

            System.out.print("상품명: ");
            orders[i].productName = scanner.nextLine();

            System.out.print("가격: ");
            orders[i].price = scanner.nextInt();

            System.out.print("수량: ");
            orders[i].quantity = scanner.nextInt();
        }
        return orders;
    }

    static void printOrders(ProductOrder[] orders) {
        for (ProductOrder order : orders) {
            System.out.println("상품명: " + order.productName + ", 가격: " + order.price + ", 수량: " + order.quantity);
        }
    }

    static int getTotalAmount(ProductOrder[] orders) {
        int totalAmount = 0;
        for (ProductOrder order : orders) {
            totalAmount += order.price * order.quantity;
        }
        return totalAmount;
    }
}

다 풀고 선생님 풀이 보니까 훨씬 간결하게 느껴지네요 ㅠㅠ

 

메인 메소드를 줄이려고 배열을 생성하고 사용자가 값을 넣는 부분을 메인 메소드 밖에 만드려고 시도하였습니다.

 

그러다보니 스캐너를 클래스범위에서 생성해야했었고 createOrder메소드의 타입과 매개변수도 바꾸는 등 대대적으로 다 바꾸는 고생을 해야 했습니다

 

하면서 사용자가 값을 입력하는 부분과 그걸 받아서 배열에 집어넣는 부분을 구분하여 메소드를 하나 더 만드는게 좋다고 생각했으나 하다보니 산으로 가버려서 createOrder메소드가 지나치게 많은 역할을 하고 커져버린 것 같습니다.

 

피드백 부탁드립니다!

 

java 객체지향

Answer 1

0

codetrain999

안녕하세요. 정재익님, 공식 서포터즈 코즈위버입니다.

여러곳에서 사용하기 위해 Scanner 플 필드로 옮긴 점, 역할에 따른 메서드 분리가 적절한것 같습니다.

잘하셨습니다 🙂

더 복잡한 문제일 경우 코드의 역할에 따라 메서드로 분리하는 것을 넘어 클래스를 분리하여 역할을 나누는 것도 고려할 수도 있습니다.

감사합니다.

질문있습니다

0

25

1

1번 문제 질문입니다.

0

31

1

음악플레이어 문제 중 코드질문

0

26

1

9장 상속 문제와 풀이 질문

0

38

1

강의 자료에 사소한 오타가 있습니다.

0

48

2

매서드 참조값 반환??

0

66

1

접근제어자 - 쇼핑카트 문제에서 상품출력 부분 메서드

0

69

1

자바 기본편 - .(dot)에 관한 질문입니다!

0

85

1

공부방법

0

74

2

상속관계에서 멤버 변수는 오버라이딩 되는 개념이 아닌가요?

0

82

1

static method 질문

0

66

1

캡슐화 문제풀이 ShoppingCart 요구사항에 문제가 있어보입니다. 피드백 주세요

0

95

1

Method1에서 Student 객체

0

73

1

3강 18. null 질문

0

67

1

this 와 super의 호출 순서는 부모-자식관계 떄문만인가요?

0

75

1

팩토리 메서드 패턴과 일반 생성자 사용의 장단점

0

97

1

문의

0

104

1

9장 상속 문제와 풀이 부분 궁금한게 있어서 질문드립니다

0

89

1

상속과 그에 따른 메모리 구조 질문

0

62

1

call메소드에서 멤버에 접근하는 방식을 이해 못 했습니다

0

74

1

수강기한

0

135

1

기본편 객체지향 프로그래밍에 대한 정의

0

86

1

섹션10 상속 메모리 구조 및 문제 관련 질문

0

87

1

추가 지식 학습

0

152

2