JAVA SCRIPT/코딩테스트
프로그래머스 :: 오픈채팅방 (JS)
ㅈㅣㄴ
2023. 12. 27. 14:42
프로그래머스 :: 오픈채팅방 (JS)
문제 설명
카카오톡 오픈채팅방에서는 친구가 아닌 사람들과 대화를 할 수 있는데, 본래 닉네임이 아닌 가상의 닉네임을 사용하여 채팅방에 들어갈 수 있다.
신입사원인 김크루는 카카오톡 오픈 채팅방을 개설한 사람을 위해, 다양한 사람들이 들어오고, 나가는 것을 지켜볼 수 있는 관리자창을 만들기로 했다. 채팅방에 누군가 들어오면 다음 메시지가 출력된다.
"[닉네임]님이 들어왔습니다."
채팅방에서 누군가 나가면 다음 메시지가 출력된다.
"[닉네임]님이 나갔습니다."
채팅방에서 닉네임을 변경하는 방법은 다음과 같이 두 가지이다.
- 채팅방을 나간 후, 새로운 닉네임으로 다시 들어간다.
- 채팅방에서 닉네임을 변경한다.
닉네임을 변경할 때는 기존에 채팅방에 출력되어 있던 메시지의 닉네임도 전부 변경된다.
예를 들어, 채팅방에 "Muzi"와 "Prodo"라는 닉네임을 사용하는 사람이 순서대로 들어오면 채팅방에는 다음과 같이 메시지가 출력된다.
"Muzi님이 들어왔습니다.""Prodo님이 들어왔습니다."
채팅방에 있던 사람이 나가면 채팅방에는 다음과 같이 메시지가 남는다.
"Muzi님이 들어왔습니다.""Prodo님이 들어왔습니다.""Muzi님이 나갔습니다."
Muzi가 나간후 다시 들어올 때, Prodo 라는 닉네임으로 들어올 경우 기존에 채팅방에 남아있던 Muzi도 Prodo로 다음과 같이 변경된다.
"Prodo님이 들어왔습니다.""Prodo님이 들어왔습니다.""Prodo님이 나갔습니다.""Prodo님이 들어왔습니다."
채팅방은 중복 닉네임을 허용하기 때문에, 현재 채팅방에는 Prodo라는 닉네임을 사용하는 사람이 두 명이 있다. 이제, 채팅방에 두 번째로 들어왔던 Prodo가 Ryan으로 닉네임을 변경하면 채팅방 메시지는 다음과 같이 변경된다.
"Prodo님이 들어왔습니다."
"Ryan님이 들어왔습니다."
"Prodo님이 나갔습니다."
"Prodo님이 들어왔습니다."
채팅방에 들어오고 나가거나, 닉네임을 변경한 기록이 담긴 문자열 배열 record가 매개변수로 주어질 때, 모든 기록이 처리된 후, 최종적으로 방을 개설한 사람이 보게 되는 메시지를 문자열 배열 형태로 return 하도록 solution 함수를 완성하라.
제한사항
record는 다음과 같은 문자열이 담긴 배열이며, 길이는 1 이상 100,000 이하이다.
다음은 record에 담긴 문자열에 대한 설명이다.
- 모든 유저는 [유저 아이디]로 구분한다.
- [유저 아이디] 사용자가 [닉네임]으로 채팅방에 입장 - "Enter [유저 아이디] [닉네임]" (ex. "Enter uid1234 Muzi")
- [유저 아이디] 사용자가 채팅방에서 퇴장 - "Leave [유저 아이디]" (ex. "Leave uid1234")
- [유저 아이디] 사용자가 닉네임을 [닉네임]으로 변경 - "Change [유저 아이디] [닉네임]" (ex. "Change uid1234 Muzi")
- 첫 단어는 Enter, Leave, Change 중 하나이다.
- 각 단어는 공백으로 구분되어 있으며, 알파벳 대문자, 소문자, 숫자로만 이루어져있다.
- 유저 아이디와 닉네임은 알파벳 대문자, 소문자를 구별한다.
- 유저 아이디와 닉네임의 길이는 1 이상 10 이하이다.
- 채팅방에서 나간 유저가 닉네임을 변경하는 등 잘못 된 입력은 주어지지 않는다.
입출력 예
record result ["Enter uid1234 Muzi", "Enter uid4567 Prodo","Leave uid1234","Enter uid1234 Prodo","Change uid4567 Ryan"] ["Prodo님이 들어왔습니다.", "Ryan님이 들어왔습니다.", "Prodo님이 나갔습니다.", "Prodo님이 들어왔습니다."]
문제 풀이
이 문제는 알고리즘 자체는 어려운 문제는 아닌데, 시간초과랑 이상한 오류? 때문에 푸는데 오래 걸렸다.
function solution(record) { const uidName = {}; const result = []; for (let str of record) { recordArr = str.split(' '); switch (recordArr[0]) { case 'Enter' : result.push(recordArr[1] + "님이 들어왔습니다."); uidName[recordArr[1]] = recordArr[2]; break; case 'Leave' : result.push(recordArr[1] + "님이 나갔습니다.") break; case 'Change' : uidName[recordArr[1]] = recordArr[2]; break; } } for (let key in uidName) { for (let i = 0; i < result.length; i++) { result[i] = result[i].replace(key, uidName[key]); } } return result; }
이게 처음 풀었던 코드인데, 시간초과랑 이상한 오류(?) 때문에 20점대가 나왔다.
아무리 봐도 왜 틀렸는지 모르겠는... 느낌이었는데 for 문이 많았어서 시간초과는 무조건 나오겠다고 생각하긴 했다.
그래서 어떤 대단한 분 도움을 받아 처음부터 다시 시작했다.
function solution(record) { let uidName = {}; let result = []; for (let str of record) { recordArr = str.split(' '); switch (recordArr[0]) { case 'Enter' : uidName[recordArr[1]] = recordArr[2]; break; case 'Change' : uidName[recordArr[1]] = recordArr[2]; } } for (let i = 0; i < record.length; i++) { switch (record[i][0]) { case 'E' : result.push(`${uidName[record[i].split(' ')[1]]}님이 들어왔습니다.`); break; case 'L' : result.push(`${uidName[record[i].split(' ')[1]]}님이 나갔습니다.`); } } return result; }
이 코드가 통과한 코드...
먼저 record 를 split 해서 이름과 아이디 객체만 만들어주고,
그 후에 switch 문을 사용하여 Enter 와 Leave 에 맞게 result 에 push 해주는 방식이다.
거의 주입식 교육으로 받은 코드라서 스스로 해냈다는 느낌은 없는 그런... ㅠㅠ
시간초과를 없애기 위해서 한 번에 Enter, Leave, Change 를 다 구현하는 게 아니라
먼저 객체를 만들고 나중에 Enter, Leave 에 맞게 리턴값을 반환해주는 방식으로 코드를 짰다.
그리고 기존 코드에 있는 오류를 찾기 위해서 고군분투 해보았는데,
멘토분께서 찾아낸 게 replace 방식에 대한 것이었다.
//기존 replace for (let key in uidName) { for (let i = 0; i < result.length; i++) { result[i] = result[i].replace(key, uidName[key]); } } //고친 방식 for (let i = 0; i < result.length; i++) { const uid = result[i].split('님')[0]; if (uidName[uid]) { result[i] = result[i].replace(uid, uidName[uid]); } }
객체 순회가 필요할 것 같다고 생각해서 uidName 에 대한 for in을 돌리고 이중으로 result 에 대한 for 문을 돌렸었는데, 객체는 키값만 있으면 굳이 순회하지 않아도 되었다.
그래서 멘토님이 고쳐주신 부분이 result 에 대한 for문을 돌리고 uid 라는 변수에 아이디값을 할당해주고,
uid 를 uidName 객체 value 값으로 치환하는 방식이었다.
하지만 이 코드도 시간초과가 떴다.
그래서 다시 고쳐본 코드가 다음과 같다.
function solution(record) { const uidName = {}; const result = []; for (let str of record) { recordArr = str.split(' '); switch (recordArr[0]) { case 'Enter' : result.push(recordArr[1] + "님이 들어왔습니다."); uidName[recordArr[1]] = recordArr[2]; break; case 'Leave' : result.push(recordArr[1] + "님이 나갔습니다.") break; case 'Change' : uidName[recordArr[1]] = recordArr[2]; break; } } for (let i = 0; i < result.length; i++) { result[i] = result[i].replace(result[i].split('님')[0],uidName[result[i].split('님')[0]]); } return result; }
이중 for 문이 문제였던 것 같다.
테스트 통과!
아무튼... 골머리 아팠던 문제 ~~ 그래도 풀어내서 다행이다.
좀 더 효율적인 코드를 짤 수 있도록 다방면으로 생각해보고 노력해야겠다.
다른 사람의 풀이
function solution(record) { let ret = [] const uids = new Map() record.forEach(entry => { let [command, uid, nick] = entry.split(' ') if (command === 'Enter' || command === 'Change') uids.set(uid, nick) }) record.forEach(entry => { let [command, uid] = entry.split(' ') if (command === 'Enter') ret.push(`${uids.get(uid)}님이 들어왔습니다.`) if (command === 'Leave') ret.push(`${uids.get(uid)}님이 나갔습니다.`) }) return ret }
- new Map() 이 객체랑 비슷한 메서드라고 들었는데, 나중에 공부 해봐야겠다.
- forEach 문을 이용해서 배열을 만들고, command 가 enter 면 들어왔습니다 push leave 면 나갔습니다 push.
- 세미콜론이 없는 게 조금 불편하긴 하지만... 괜찮은 코드인 것 같다 !