[프로그래머스] 42888 - 오픈채팅방

문제

카카오톡 오픈채팅방에서는 친구가 아닌 사람들과 대화를 할 수 있는데, 본래 닉네임이 아닌 가상의 닉네임을 사용하여 채팅방에 들어갈 수 있다.

신입사원인 김크루는 카카오톡 오픈 채팅방을 개설한 사람을 위해, 다양한 사람들이 들어오고, 나가는 것을 지켜볼 수 있는 관리자창을 만들기로 했다. 채팅방에 누군가 들어오면 다음 메시지가 출력된다.

"[닉네임]님이 들어왔습니다."

채팅방에서 누군가 나가면 다음 메시지가 출력된다.

"[닉네임]님이 나갔습니다."

채팅방에서 닉네임을 변경하는 방법은 다음과 같이 두 가지이다.

채팅방을 나간 후, 새로운 닉네임으로 다시 들어간다.
채팅방에서 닉네임을 변경한다.
닉네임을 변경할 때는 기존에 채팅방에 출력되어 있던 메시지의 닉네임도 전부 변경된다.

예를 들어, 채팅방에 "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 이하이다.
채팅방에서 나간 유저가 닉네임을 변경하는 등 잘못된 입력은 주어지지 않는다.

 

풀이

처음에는 reduceRight를 이용하여 함축적으로 코드를 짜려고 시도했었는데, 아니나 다를까 시간 초과가 뜬다...

그래서 질문하기 게시판을 눈팅해 봤는데, reduce 말고 forEach를 쓰면 된다는 글이 있는 거시어따.

그래서 고차함수의 성능 차이를 검색해 봤는데, 이상하게 이 그래프 상으로는 reduce가 forEach보다 성능이 좋다는 것이었다...?

고차함수 성능 비교

 

그래서 더 검색을 해보았다!

 

그래서 직접 간단하게 console.time()을 이용하여 실행시간 측정을 해보았다.

console.log(`for 문`)
console.time()
for(let i=0; i<1000000; i++){
  const tmp = i * i * i
}
console.timeEnd()

console.log(`map 고차함수`)
console.time()
Array(1000000).fill(0).map((e, i)=>{
  const tmp = i * i * i
});
console.timeEnd()

console.log(`forEach 고차함수`)
console.time()
Array(1000000).fill(0).forEach((e, i)=>{
  const tmp = i * i * i
})
console.timeEnd()

console.log(`reduce 고차함수`)
console.time()
Array(1000000).fill(0).reduce((e, i)=>{
  const tmp = i * i * i
});
console.timeEnd()

console.log(`filter 고차함수`)
console.time()
Array(1000000).fill(0).filter((e, i)=>{
  const tmp = i * i * i
})
console.timeEnd()

 

결과는!!! 두구두구

결과찡 by 크롬

 

빠른 순서대로 for >>>>>> reduce >== filter >== forEach >> map 이라는 결과를 얻을 수 있었다.

 

지금까지 겉멋에 취해 for보다는 forEach를 선택하는 추태를 부렸는데, 무조건 고차함수를 남발해서 코드를 짧게 짠다고 좋은 게 아니라는 사실을 깨달았다!

 

그리고 같은 for문 이라도 근소하게 성능 차이가 발생한다.

for문 성능 비교

 

 

그리고 브라우저마다 고차함수의 성능도 다 다르다는 사실도 알게 되었다.

참고로 저 위에 있는 성능 측정은 크롬 카나리아 버전에서 측정한 결과인데, 엣지 브라우저로 동일한 코드를 돌려봤더니 forEach와 filter 함수가 무진장 느려진다는 결과를 얻을 수 있었다!

결과찡 by 엣지

 

근데 결국 왜 이 문제에서 reduce를 쓰면 시간초과가 뜨는 지는 아직도 알아내지 못했따!

브라우저마다 고차함수의 퍼포먼스가 다르듯, 프로그래머스 서버에서는 forEach가 map보다 빠른 것일까?

 

암튼... 정답 코드는 다음과 같다.

 

- reduce를 사용하여 시간 초과가 뜬 코드

function solution(record) {
    const users = {}
    const enterMsg = id => () => `${users[id]}님이 들어왔습니다.`
    const leaverMsg = id => () => `${users[id]}님이 나갔습니다.`
    const msgs = record.reduce((a, b) => {
        const [ type, id, nickname ] = b.split(' ')
        if (nickname) users[id] = nickname
        if (type === 'Enter') return [...a, enterMsg(id)]
        if (type === 'Leave') return [...a, leaverMsg(id)]
        return a
    }, [])
    return msgs.map(e => e())
}

 

- forEach를 사용하여 수정한 최종 코드

function solution(record) {
    const users = {}
    const enterMsg = id => () => `${users[id]}님이 들어왔습니다.`
    const leaverMsg = id => () => `${users[id]}님이 나갔습니다.`
    const msgs = []
    record.forEach(e => {
        const [ type, id, nickname ] = e.split(' ')
        if (nickname) users[id] = nickname
        if (type === 'Enter') msgs.push(enterMsg(id))
        if (type === 'Leave') msgs.push(leaverMsg(id))
    })
    return msgs.map(e => e())
}