백준 : 10845번 큐 (JAVA)

목차

JAVA 풀이 10845번 큐

https://www.acmicpc.net/problem/10845

문제

정수를 저장하는 큐를 구현한 다음, 입력으로 주어지는 명령을 처리하는 프로그램을 작성하시오.
명령은 총 여섯 가지이다.

- push X: 정수 X를 큐에 넣는 연산이다.
- pop: 큐에서 가장 앞에 있는 정수를 빼고, 그 수를 출력한다. 만약 큐에 들어있는 정수가 없는 경우에는 -1을 출력한다.
- size: 큐에 들어있는 정수의 개수를 출력한다.
- empty: 큐가 비어있으면 1, 아니면 0을 출력한다.
- front: 큐의 가장 앞에 있는 정수를 출력한다. 만약 큐에 들어있는 정수가 없는 경우에는 -1을 출력한다.
- back: 큐의 가장 뒤에 있는 정수를 출력한다. 만약 큐에 들어있는 정수가 없는 경우에는 -1을 출력한다.

입력

첫째 줄에 주어지는 명령의 수 N (1 ≤ N ≤ 10,000)이 주어진다.
둘째 줄부터 N개의 줄에는 명령이 하나씩 주어진다. 주어지는 정수는 1보다 크거나 같고, 100,000보다 작거나 같다.
문제에 나와있지 않은 명령이 주어지는 경우는 없다.

출력

출력해야하는 명령이 주어질 때마다, 한 줄에 하나씩 출력한다.

예제 입력1

15
push 1
push 2
front
back
size
empty
pop
pop
pop
size
empty
pop
push 3
empty
front

예제 출력1

1
2
2
0
1
2
-1
0
1
-1
0
3

 

---

풀이 (리팩토링 전)

import java.io.*;
import java.util.*;

public class Main {
	public static void main(String[] args) throws IOException {
		BufferedReader br = new BufferedReader(new InputStreamReader(System.in));
		StringBuffer sb = new StringBuffer();
		
		int n = Integer.parseInt(br.readLine());
		int last = 0;
		Queue<Integer> q = new LinkedList<>();
        
		while(n-- > 0) {
			String m = br.readLine();
			
			if(m.contains("push")) {
				String[] str = m.split(" ");
				last = Integer.parseInt(str[1]);
				q.offer(last);
			}else {
				if(m.equals("empty")) {
					sb.append(q.isEmpty() ? 1 : 0);
				}else if(m.equals("size")) {
					sb.append(q.size());
				}else if(q.isEmpty()) {
					sb.append(-1);
				}else if(m.equals("pop")) {
					sb.append(q.poll());
				}else if(m.equals("front")) {
					sb.append(q.peek());
				}else if(m.equals("back")) {
					sb.append(last);
				}
				sb.append("\n");
			}
		}
		System.out.println(sb);
	}
}

 

변수명도 엉망진창이네요.

(그렇지만 이해하기 쉬운 코드라 따로 주석을 달지 않았습니다.)

 

---

풀이 (리팩토링 후)

import java.io.*;
import java.util.*;

public class Main {
	private static final String PUSH_COMMAND = "push";
	private static final String POP_COMMAND = "pop";
	private static final String SIZE_COMMAND = "size";
	private static final String EMPTY_COMMAND = "empty";
	private static final String FRONT_COMMAND = "front";
	private static final String BACK_COMMAND = "back";
	
	public static void main(String[] args) throws IOException {
		BufferedReader br = new BufferedReader(new InputStreamReader(System.in));
		StringBuilder sb = new StringBuilder();
		
		// 명령의 수
		int commandCount = Integer.parseInt(br.readLine());
		// 정수를 저장할 큐
		Queue<Integer> q = new LinkedList<>();
		
		while(commandCount-- > 0) {
			String command = br.readLine();
			
			// push 명령이라면
			if(command.startsWith(PUSH_COMMAND)) {
				pushCommand(q, command);
			// 아니라면
			}else {
				sb.append(anotherCommand(q, command)).append("\n");
			}
		}
		
		// 출력
		System.out.println(sb);
	}
	
	// push 명령을 처리하는 메서드
	public static void pushCommand(Queue<Integer> q, String command) {
		int inputNumber = Integer.parseInt(command.substring(PUSH_COMMAND.length() + 1));
		q.offer(inputNumber);
	}
	
	// push 명령을 제외한 나머지 명령을 처리하고, 출력할 정수를 반환하는 메서드
	public static int anotherCommand(Queue<Integer> q, String command) {
		switch(command) {
			case EMPTY_COMMAND:
				return q.isEmpty() ? 1 : 0;
			case SIZE_COMMAND:
				return q.size();
			case POP_COMMAND:
				return q.isEmpty() ? -1 : q.poll();
			case FRONT_COMMAND:
				return q.isEmpty() ? -1 : q.peek();
			case BACK_COMMAND:
				return q.isEmpty() ? -1 : ((LinkedList<Integer>) q).peekLast();
			default:
				return -1;
		}
	}
}

 

조금 더 깔끔하게 수정할 수 있을 것 같은데 어렵습니다.

Clean Code를 읽고, 짧고 간결한 코드가 좋은 코드라는 생각을 버릴 수 있었습니다.