코딩테스트 연습 Lv1 달리기 경주

달리기 경주
얀에서는 매년 달리기 경주가 열립니다. 해설진들은 선수들이 자기 바로 앞의 선수를 추월할 때 추월한 선수의 이름을 부릅니다. 예를 들어 1등부터 3등까지 "mumu", "soe", "poe" 선수들이 순서대로 달리고 있을 때, 해설진이 "soe"선수를 불렀다면 2등인 "soe" 선수가 1등인 "mumu" 선수를 추월했다는 것입니다. 즉 "soe" 선수가 1등, "mumu" 선수가 2등으로 바뀝니다.
선수들의 이름이 1등부터 현재 등수 순서대로 담긴 문자열 배열 players와 해설진이 부른 이름을 담은 문자열 배열 callings가 매개변수로 주어질 때, 경주가 끝났을 때 선수들의 이름을 1등부터 등수 순서대로 배열에 담아 return 하는 solution 함수를 완성해주세요.

제한사항
•5 ≤ players의 길이 ≤ 50,000
•players[i]는 i번째 선수의 이름을 의미합니다.
•players의 원소들은 알파벳 소문자로만 이루어져 있습니다.
•players에는 중복된 값이 들어가 있지 않습니다.
•3 ≤ players[i]의 길이 ≤ 10
•2 ≤ callings의 길이 ≤ 1,000,000
•callings는 players의 원소들로만 이루어져 있습니다.
•경주 진행중 1등인 선수의 이름은 불리지 않습니다.

처음 해당 문제를 봤을때는 그냥 순차탐색으로 풀어도 되겠거니 하고 순차탐색으로 접근하니...

테스트 케이스중 4개가 시간초과가 나와버렸다.
처음에는 시간초과의 원인이 인덱스 값이 너무 커지다 보니 매번 callings 배열의 요소마다 순차 탐색을 진행을 해서 해당 오류가 나왔다고 생각을 하여 그러면 callings 배열에 있던 선수를 키로 해당 선수의 players 배열에서의 인덱스를 값으로 하여 HashMap에 저장을 시키고 만약 그 뒤에 같은 선수가 나오면 순차 탐색을 진행하지 않고 HashMap에서 인덱스 값을 사용하도록 하면 해결 되겠거니하고 코드를 만들었다. 근데 이 방법은 오히려 테스트 케이스 대부분에서 런타임 오류가 나와버려서(2023.9.25)

import java.util.HashMap;
import java.util.Arrays;
class Solution {
    public String[] solution(String[] players, String[] callings) {
        String[] answer = {};
        HashMap<String, Integer> map = new HashMap<>();
        int index;
        String temp;
        answer=players;
        
        for(int i = 0 ; i < callings.length ; i++){
            if(map.containsKey(callings[i])){
                index= map.get(callings[i]);
                temp=answer[index-1];
                answer[index-1]=answer[index];
                answer[index]=temp;
                map.replace(callings[i],index-1);
                map.replace(temp,index);
            }else{
                index=Arrays.asList(answer).indexOf(callings[i]);
                temp=answer[index-1];
                answer[index-1]=answer[index];
                answer[index]=temp;
                map.put(callings[i],index-1);
                map.put(temp,index);
            }
            
        }
        return answer;
    }
}

 

(2023.10.2)

근데 위  방법으로 다시 해보니 통과가 되었다 왜 그런지는 전에 코드를 저장을 따로 안해서 모르겠네...

일단 어쨌든 위 방법은 HashMap에 callings배열의 요소가 키로 존재하는지 확인을 하고 있다면 HashMap에 있는 키를 가지고 인덱스 값을 알아내고 HashMap에 없다면 answer배열을 Arrays의 indexOf메서드를 이용해서 callings요소의 현재 인덱스 값을 알아내서 위치를 바꾼후 HashMap에 put을 하는 방식이다.

 

아래 방식은 그냥 players 배열에 있는 모든 요소들을 HashMap에 저장하고 callings 배열에서 인덱스 값이 필요로 할때는 HashMap을 이용해서 인덱스 값을 알아내도록 하니까 해결되었다. 사실 이 방식이 더 간편하고 효율적이고 좋은 코드인거 같다.

import java.util.HashMap;
class Solution {
    public String[] solution(String[] players, String[] callings) {
        String[] answer = {};
        HashMap<String, Integer> map = new HashMap<>();
        int index;
        String temp;
        answer=players;
        
        for(int i =0 ; i<players.length ; i++){
            map.put(players[i],i);
        }
        for(int i = 0 ; i < callings.length ; i++){
            index= map.get(callings[i]);
            temp=answer[index-1];
            answer[index-1]=answer[index];
            answer[index]=temp;
            map.replace(callings[i],index-1);
            map.replace(temp,index);
        }
        return answer;
    }
}

 

두개의 방식중 어떤 방식이 더 효율적이고 좋은 방식인지는 HashMap의 get, put, containsKey메서드들이 어떻게 동작하는지 몰라서 판단을 정확히 내리기가 어렵다. 하지만 나의 짧은 식견으로 일단 indexOf메서드는 위 코드에서 순차탐색으로 동작을 하고 get 메서드가 Hash를 이용해서 값을 찾는다는 전제하에서 말하자면

일단 첫번째 방식은 map에 특정 키가 없을 때는 answer배열에서 순차탐색으로 해당 키의 인덱스 값을 찾기 때문에 callings배열의 요소가 중복이 없거나 작을때는 사실상 map의 역할이 거의 없다시피 하기 때문에 최악의 상황에서는 그냥 순차탐색으로 모든 인덱스 값을 찾아서 진행하는 것과 다를바 없어 보인다. 이에 반해 두번째 방식은 무조건 인덱스 값을 map에서 찾기 때문에 처음에 HashMap에 answer 배열을 넣는 과정을 거치고 나면 이 문제의 제한 사항때문에 항상 같은 시간복잡도를 가지기 때문에 더 좋은 결과를 낼 것이다.