PGM : K번째수

👉 문제링크

🔹 문제 🔹

배열 array의 i번째 숫자부터 j번째 숫자까지 자르고 정렬했을 때, k번째에 있는 수를 구하려 합니다.
예를 들어 array가 [1, 5, 2, 6, 3, 7, 4], i = 2, j = 5, k = 3이라면

1. array의 2번째부터 5번째까지 자르면 [5, 2, 6, 3]입니다.
2. 1에서 나온 배열을 정렬하면 [2, 3, 5, 6]입니다.
3. 2에서 나온 배열의 3번째 숫자는 5입니다.

배열 array, [i, j, k]를 원소로 가진 2차원 배열 commands가 매개변수로 주어질 때, commands의 모든 원소에 대해 앞서 설명한 연산을 적용했을 때 나온 결과를 배열에 담아 return 하도록 solution 함수를 작성해주세요.

🔹 제한사항 🔹

- array의 길이는 1 이상 100 이하입니다.
- array의 각 원소는 1 이상 100 이하입니다.
- commands의 길이는 1 이상 50 이하입니다.
- commands의 각 원소는 길이가 3입니다.

🔹 입출력 🔹

예시

array	commands	return
[1, 5, 2, 6, 3, 7, 4]	[[2, 5, 3], [4, 4, 1], [1, 7, 3]]	[5, 6, 3]

설명
[1, 5, 2, 6, 3, 7, 4]를 2번째부터 5번째까지 자른 후 정렬합니다. [2, 3, 5, 6]의 세 번째 숫자는 5입니다.
[1, 5, 2, 6, 3, 7, 4]를 4번째부터 4번째까지 자른 후 정렬합니다. [6]의 첫 번째 숫자는 6입니다.
[1, 5, 2, 6, 3, 7, 4]를 1번째부터 7번째까지 자릅니다. [1, 2, 3, 4, 5, 6, 7]의 세 번째 숫자는 3입니다.

---

🔸 문제 분석 🔸

배열을 자르고, 정렬하고, 특정 인덱스의 원소를 가져오는 문제입니다.

배열은 vector<int> 를 사용했습니다.

🔸 코드 🔸

//#include<iostream>
#include <vector>
#include<algorithm>

using namespace std;

vector<int> solution(vector<int> array, vector<vector<int>> commands) {
    vector<int> answer;
    
    for (vector<int> cm : commands) {
        vector<int> temp(cm[1]-cm[0]+1);
        
        copy(array.begin()+cm[0]-1,array.begin()+cm[1],temp.begin());
        
        sort(temp.begin(),temp.end());
        
        answer.push_back(temp[cm[2]-1]);
    }
    return answer;
}

🔸 코드 해석 🔸

copy() 함수를 통해 배열을 슬라이싱 했습니다.

---

🔸 code review 🔸

for (vector<int> cm : commands) {

- A : cm이 복사 생성되고 있습니다. vector& 또는 auto&로 바꿔주세요.

 

- 나 : K번째수 문제에서, vector의 특정 부분을 복사하는데 temp를 사용했습니다. 이때 temp를 선언하고 reserve로 공간만 할당하면 복사가 안되더라구요..? 그래서 for문안에 vector를 계속 반복하는데 좀 낭비인 것 같아 더 좋은 방법이 없을까 합니다.

B : reserve 함수를 이용하여 capacity를 부여한 경우에는, 공간을 예약은 했지만, 실제로 할당이 되어있지 않기 때문에, temp.begin() 함수에서는 주소가 없다고 판단을 하는 것입니다. 
따라서 back_inserter 함수를 이용하여 temp에 할당하며 삽입을 해야합니다! 
해당 함수는 push_back 함수가 내장되어있어서 오버로딩이 자동으로 된다고 하네요. 
copy 함수는 = 연산을 한다고 하네요. 
=> 이거 때문에 실제 size가 없는 걸 = 연산하려니 문제가 생기는 것 같아요. 
아닐수도 있습니다.. ㅎㅎ;

A : copy는 capacity가 아니라 size가 확보되어 있어야 합니다.
A : 필요한 부분만 복사하되 가장 간결한 코드를 만드려면 벡터의 생성자 중에 초기 데이터를 순회하기 위한 반복자 쌍을 인자로 받는 버전이 있는데 그걸 통해서 temp 벡터를 생성해주는 것입니다.
가장 효율적인 방법으로는 temp 벡터를 for문 밖으로 빼내서 매 반복마다 vector가 소멸하지 않도록 해주고, for문 진입 전에 array.size()만큼의 capacity를 확보한 후 for문 내에서는 assign을 통해 필요한 항목열이 복사되도록 하는 것입니다.
이렇게 하면 벡터의 항목열이 저장될 메모리의 주소는 변하지 않으면서 필요한 만큼만 동적으로 resize되며 불필요한 중복 초기화도 발생하지 않습니다.

 

🔸 feedback code 🔸

#include <vector>
#include<algorithm>

using namespace std;

vector<int> solution(vector<int> array, vector<vector<int>> commands) {
    vector<int> answer;
    vector<int> temp(array.size());

    for (vector<int>& cm : commands) {
        temp.assign(array.begin() + cm[0] - 1, array.begin() + cm[1]);

        sort(temp.begin(), temp.end());

        answer.push_back(temp[cm[2] - 1]);
    }
    return answer;
}

---

🔸 end 🔸

vector의 복사가 아직 익숙하지 않아서 copy를 쓸데 없이 사용한 것 같다.

vector의 capacity 할당과 값 할당을 구분할 수 있어야 겠다.