인프런 커뮤니티 질문&답변

10thouseo님의 프로필 이미지
10thouseo

작성한 질문수

김영한의 실전 자바 - 기본편

문제와 풀이

ProductOrderMain3 문제 풀이 방법에 질문 있습니다

작성

·

338

·

수정됨

0

[질문 템플릿]
1. 강의 내용과 관련된 질문인가요? 예
2. 인프런의 질문 게시판과 자주 하는 질문에 없는 내용인가요? 예
3. 질문 잘하기 메뉴얼을 읽어보셨나요? 예
[질문 내용]

저는 enterProduct라는 메서드에서 객체를 생성하고 사용자의 입력까지 모두 받아 해당 객체를 return하도록 풀었습니다. 제가 풀은 방식처럼 메서드 안에서 입력까지 받는 방식이 지양해야 하는 방식인지 궁금합니다

 

package ref.ex;

import java.util.Scanner;

public class ProductOrderMain3 {
    public static void main(String[] args) {
        Scanner scanner = new Scanner(System.in);

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

        ProductOrder[] product = new ProductOrder[count];

        for (int i = 0; i < product.length; i++) {
            product[i] = enterProduct(i);
        }

        printProduct(product);

        int totalPrice = calTotalPrice(product);
        System.out.println("총 결제 금액: " + totalPrice);
    }

    static ProductOrder enterProduct(int i) {
        Scanner scanner = new Scanner(System.in);

        ProductOrder productOrder = new ProductOrder();

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

        System.out.print("상품명: ");
        productOrder.productName = scanner.nextLine();

        System.out.print("가격: ");
        productOrder.price = scanner.nextInt();

        System.out.print("수량: ");
        productOrder.quantity = scanner.nextInt();


        return productOrder;
    }

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

    static int calTotalPrice(ProductOrder[] product) {
        int totalPrice = 0;

        for (ProductOrder p : product) {
            totalPrice += p.price * p.quantity;
        }

        return totalPrice;
    }
}

답변 2

0

가독성이 너무 떨어져요

0

안녕하세요. 10thouseo님, 공식 서포터즈 OMG입니다.

enterProduct는 입력과 객체 생성을 두 가지 로직이 섞여 있어 이해하기 (상대적으로) 어렵고 코드의 가독성이 떨어집니다. 입력과 객체 생성 로직이 결합되어 있다보니 각각 분리되어 있을 때 보다 재사용성이 떨어집니다. 이러한 이유로 입력과 생성을 분리하여 개발하는 것을 지향하시면 좋을 것 같아요

감사합니다.

10thouseo님의 프로필 이미지
10thouseo

작성한 질문수

질문하기