[ 프로그래머스 ] 프린터 (C++)

스택/큐를 이용해서 풀어야했던 문제다.

나는 그중에서 큐를 이용해서 풀었고 , 배열을 통해 관리했다.

내용 자체는 어렵지 않게 풀었고 다른 사람의 풀이를 보다가 새로운 함수를 발견해서 적어보려 한다.

 

< 문제 >

일반적인 프린터는 인쇄 요청이 들어온 순서대로 인쇄합니다.

그렇기 때문에 중요한 문서가 나중에 인쇄될 수 있습니다.

이런 문제를 보완하기 위해 중요도가 높은 문서를 먼저 인쇄하는 프린터를 개발했습니다.

이 새롭게 개발한 프린터는 아래와 같은 방식으로 인쇄 작업을 수행합니다.

 

1. 인쇄 대기목록의 가장 앞에 있는 문서(J)를 대기목록에서 꺼냅니다.

2. 나머지 인쇄 대기목록에서 J보다 중요도가 높은 문서가 한 개라도 존재하면 J를 대기목록의 가장 마지막에 넣습니다. 3. 그렇지 않으면 J를 인쇄합니다.

예를 들어, 4개의 문서(A, B, C, D)가 순서대로 인쇄 대기목록에 있고 중요도가 2 1 3 2 라면 C D A B 순으로 인쇄하게 됩니다.

내가 인쇄를 요청한 문서가 몇 번째로 인쇄되는지 알고 싶습니다.

위의 예에서 C는 1번째로, A는 3번째로 인쇄됩니다.

현재 대기목록에 있는 문서의 중요도가 순서대로 담긴 배열 priorities와 내가 인쇄를 요청한 문서가 현재 대기목록의 어떤 위치에 있는지를 알려주는 location이 매개변수로 주어질 때, 내가 인쇄를 요청한 문서가 몇 번째로 인쇄되는지 return 하도록 solution 함수를 작성해주세요.

 

제한사항

• 현재 대기목록에는 1개 이상 100개 이하의 문서가 있습니다.
• 인쇄 작업의 중요도는 1~9로 표현하며 숫자가 클수록 중요하다는 뜻입니다.
• location은 0 이상 (현재 대기목록에 있는 작업 수 - 1) 이하의 값을 가지며 대기목록의 가장 앞에 있으면 0, 두 번째에 있으면 1로 표현합니다.

 

 

< 내 풀이 >

#include <vector>
#include <queue>

using namespace std;

int solution(vector<int> priorities, int location) {
    int answer = 0;
    queue<int> q;
    int d[10] = {0,};
    
    for(int i=0; i<priorities.size(); i++){
        q.push(i);
        d[priorities[i]]++;
    }
    
    bool complete = true;
    while(!q.empty()){
        complete = true;
        int n = q.front();
        q.pop();
        
        for(int i=priorities[n]+1; i<10; i++){
            if(d[i]!=0){
                complete = false;
                break;
            }
        }
        
        if(!complete){
            q.push(n);
        }
        else{
            answer++;
            d[priorities[n]]--;
            if(n == location) return answer;
        }
    }
}

단순히 인쇄하려는 문서의 중요도보다 중요도가 높은 문서가 하나라도 있는 지 알면 되기 때문에

중요도를 배열의 인덱스에 맞춰 개수를 넣었다.

for문을 돌면서 해당 문서의 중요도보다 하나 큰 숫자부터 배열의 인덱스를 증가시켜가면서

0값이 아닌 원소가 하나라도 있다면, 해당 문서는 인쇄를 미뤄야하기 때문에 다시 push하여 대기목록의 맨 뒤로 가도록 구현했다. 

 

 

 

< 다른 사람의 풀이 >

#include <vector>
#include <queue>
#include <algorithm>

using namespace std;

int solution(vector<int> priorities, int location) {
    queue<int> printer;                         //queue에 index 삽입.
    vector<int> sorted;                         //정렬된 결과 저장용
   
   for(int i=0; i<priorities.size(); i++) {
        printer.push(i);
    }
    
    while(!printer.empty()) {
        int now_index = printer.front();
        printer.pop();
        if(priorities[now_index] != *max_element(priorities.begin(),priorities.end())) {
            //아닌경우 push
            printer.push(now_index);
        } 
        else {
            //맞는경우
            sorted.push_back(now_index);
            priorities[now_index] = 0;
        }
    }
    
    for(int i=0; i<sorted.size(); i++) {
        if(sorted[i] == location) return i+1;
    }
}

while문 안에 나처럼 for문을 돌릴 필요 없이 max_element() 함수를 사용했다.

max() 함수는 두 값을 비교하여 최대값을 리턴해주는 함수로 알고 있었지만, 저 함수는 처음봤다.

max_element() 함수에 관해서는 새로 글을 작성했으니 링크를 확인하자.

2020/12/27 - [C++] - [ C++ ] max_element , min_element로 정해진 구간의 원소들 중 최대/최소값 구하기

 

위와 같은 간편한 함수를 사용하여 정해진 구간 중 최대값을 구함으로써

중요도가 더 높은 문서가 있는지 확인할 수 있다.

 

 

 

<리뷰 - 수정한 더 효율적인 코드>

그런데, 위 풀이처럼 굳이 벡터를 하나 더 추가하여 만들 이유 없이

내 풀이에서처럼 location의 값과 같은 인덱스가 인쇄될 경우에 그냥 몇번째로 인쇄되는지 리턴하고 끝내면 될 것 같다.

그리고, 매번 if문에서 max_element() 함수를 실행하는 것 보다 while문 전에 한번, else문에서 실행하는게 더 효율적인 코드가 될 것 같다. 이를 수정해보고 더 좋은 코드를 만들어보았다.

#include <vector>
#include <queue>
#include <algorithm>

using namespace std;

int solution(vector<int> priorities, int location) {
    queue<int> printer;
    int answer = 0;
    
    for(int i=0; i<priorities.size(); i++)
        printer.push(i);
    
    int maxValue = *max_element(priorities.begin(),priorities.end());
    while(!printer.empty()) {
        int index = printer.front();
        printer.pop();
        
        if(maxValue != priorities[index])
            printer.push(index);
        else {
            answer++;
            if(index == location) return answer;
            priorities[index] = 0;
            maxValue = *max_element(priorities.begin(),priorities.end());
        }
    }
}