[Javascript] 오픈채팅방 (42888)

 

문제링크

코딩테스트 연습 - 오픈채팅방

 

접근방법

"name님이 입장하셨습니다." , "name님이 퇴장하셨습니다."의 name은 업데이트 된 내용을 항상 반영해야한다.

여기서 name대신 userId를 사용함으로써 같은 userId의 메시지는
"userId님이 입장하셨습니다." , "userId님이 퇴장하셨습니다."로 표현된다.

그리고 userIdName 객체에 userId의 value 값으로 마지막에 업데이트된 name을 할당한다.

끝으로 answer 배열에 userIdLog 배열의 userId를 name으로 변경한 문자열을 삽입한다.

헤맨부분

record.forEach(e => {
    const [state, userId, name] = e.split(" ");
    if(state === "Enter"){
        userIdLog.push([userId, "님이 들어왔습니다."]);
    }else if(state === "Leave"){
        userIdLog.push([userId, "님이 나갔습니다."]);
    }  
    userIdName[userId] = name;
})​

 
위 코드가 계속해서 통과를 하지 못해서 곰곰히 생각해보니 state==='Leave' 부분에서 문제가 있었다.

record.forEach(e => {
    const [state, userId, name] = e.split(" ");
    if(state === "Enter"){
        userIdLog.push([userId, "님이 들어왔습니다."]);
    }
    if(state === "Leave"){
        userIdLog.push([userId, "님이 나갔습니다."]);
    } 
    console.log(state, userId, name);
        
    userIdName[userId] = name;
})

console.log로 state==='Leave'일때를 찍어보면
`Leave uid1234 undefined` 이처럼 name = undefined가 입력된다.

때문에 state === "Leave" 일때 return을 통해 forEach문의 다음 요소로 넘어가야한다.

여담으로 if-else if가 필요한 코드가 아니여서 if ,if로 바꾸니 코드 속도가 더 빨라졌다.

if-else if일때

if , if일때

 

 

function solution(record) {
    var userIdLog = [];
    var answer = [];
    const userIdName = {};

    record.forEach(e => {
        const [state, userId, name] = e.split(" ");
        if(state === "Enter"){
            userIdLog.push([userId, "님이 들어왔습니다."]);
        }
        if(state === "Leave"){
            userIdLog.push([userId, "님이 나갔습니다."]);
            return;
        }  
        userIdName[userId] = name;
    })


    userIdLog.map((e) => answer.push(userIdName[e[0]] + e[1]))
    return answer
}