코딩테스트 - 개인정보 수집 유효기간

https://school.programmers.co.kr/learn/courses/30/lessons/150370

프로그래머스

카카오톡 채용 코딩테스트를 풀어보았습니다.

문제나 조건들은 문제에 잘 나와있으니 굳이 블로그에 옮기지 않으면 안친절해 보이니 옮기겠습니다.

문제풀이 바로가기

문제

고객의 약관 동의를 얻어서 수집된 1~n번으로 분류되는 개인정보 n개가 있습니다. 약관 종류는 여러 가지 있으며 각 약관마다 개인정보 보관 유효기간이 정해져 있습니다. 당신은 각 개인정보가 어떤 약관으로 수집됐는지 알고 있습니다. 수집된 개인정보는 유효기간 전까지만 보관 가능하며, 유효기간이 지났다면 반드시 파기해야 합니다.

예를 들어, A라는 약관의 유효기간이 12 달이고, 2021년 1월 5일에 수집된 개인정보가 A약관으로 수집되었다면 해당 개인정보는 2022년 1월 4일까지 보관 가능하며 2022년 1월 5일부터 파기해야 할 개인정보입니다.
당신은 오늘 날짜로 파기해야 할 개인정보 번호들을 구하려 합니다.

모든 달은 28일까지 있다고 가정합니다.

다음은 오늘 날짜가 2022.05.19일 때의 예시입니다.

약관 종류유효기간

A	6 달
B	12 달
C	3 달

번호개인정보 수집 일자약관 종류

1	2021.05.02	A
2	2021.07.01	B
3	2022.02.19	C
4	2022.02.20	C

• 첫 번째 개인정보는 A약관에 의해 2021년 11월 1일까지 보관 가능하며, 유효기간이 지났으므로 파기해야 할 개인정보입니다.
• 두 번째 개인정보는 B약관에 의해 2022년 6월 28일까지 보관 가능하며, 유효기간이 지나지 않았으므로 아직 보관 가능합니다.
• 세 번째 개인정보는 C약관에 의해 2022년 5월 18일까지 보관 가능하며, 유효기간이 지났으므로 파기해야 할 개인정보입니다.
• 네 번째 개인정보는 C약관에 의해 2022년 5월 19일까지 보관 가능하며, 유효기간이 지나지 않았으므로 아직 보관 가능합니다.

따라서 파기해야 할 개인정보 번호는 [1, 3]입니다.

오늘 날짜를 의미하는 문자열 today, 약관의 유효기간을 담은 1차원 문자열 배열 terms와 수집된 개인정보의 정보를 담은 1차원 문자열 배열 privacies가 매개변수로 주어집니다. 이때 파기해야 할 개인정보의 번호를 오름차순으로 1차원 정수 배열에 담아 return 하도록 solution 함수를 완성해 주세요.

제한사항

제한사항

• today는 "YYYY.MM.DD" 형태로 오늘 날짜를 나타냅니다.
• 1 ≤ terms의 길이 ≤ 20
  • terms의 원소는 "약관 종류 유효기간" 형태의 약관 종류와 유효기간을 공백 하나로 구분한 문자열입니다.
  • 약관 종류는 A~Z중 알파벳 대문자 하나이며, terms 배열에서 약관 종류는 중복되지 않습니다.
  • 유효기간은 개인정보를 보관할 수 있는 달 수를 나타내는 정수이며, 1 이상 100 이하입니다.
• 1 ≤ privacies의 길이 ≤ 100
  • privacies[i]는 i+1번 개인정보의 수집 일자와 약관 종류를 나타냅니다.
  • privacies의 원소는 "날짜 약관 종류" 형태의 날짜와 약관 종류를 공백 하나로 구분한 문자열입니다.
  • 날짜는 "YYYY.MM.DD" 형태의 개인정보가 수집된 날짜를 나타내며, today 이전의 날짜만 주어집니다.
  • privacies의 약관 종류는 항상 terms에 나타난 약관 종류만 주어집니다.
• today와 privacies에 등장하는 날짜의 YYYY는 연도, MM은 월, DD는 일을 나타내며 점(.) 하나로 구분되어 있습니다.
  • 2000 ≤ YYYY ≤ 2022
  • 1 ≤ MM ≤ 12
  • MM이 한 자릿수인 경우 앞에 0이 붙습니다.
  • 1 ≤ DD ≤ 28
  • DD가 한 자릿수인 경우 앞에 0이 붙습니다.
• 파기해야 할 개인정보가 하나 이상 존재하는 입력만 주어집니다.

 

문제풀이

처음에는 new Date()로 문제를 풀어보려고 했지만... 19번 테스트를 통과하지 못하였습니다. 대체 왜 틀리는지 이유라도 알고 싶지만... 

일단 저의 지저분한 코드를 올려보고 new Date()로 깔끔하게 푼 답변도 밑에 작성해보겠습니다.

(그리고 저는 왜 자꾸 korea time으로 자동으로 변경이 안되는지 궁금하네요)

첫 번째 풀이(19번 테스트 실패!)

function solution(today, terms, privacies) {
  let answer = [];
  let objectTerms = {}
for (let i of terms){
if(i[3]===undefined) objectTerms[i[0]] = Number(i[2])
else objectTerms[i[0]] = Number(i[2]+ i[3])
}
const Today = new Date(today)
const KoreaTime = new Date(Today.setHours(Today.getHours()+9))
for (let i =0 ; i<privacies.length; i++){
  let collectPeriod = new Date(privacies[i].slice(0,-2))
  let expiredPeriod = collectPeriod.setMonth(collectPeriod.getMonth()+ objectTerms[privacies[i][11]])
  let koreaExpiredPeriod = new Date(expiredPeriod - (15*60*60*1000))
  
  if(koreaExpiredPeriod.getDate()>28){
    koreaExpiredPeriod.setDate(28)
  }
  if(koreaExpiredPeriod<KoreaTime){
   answer.push(i+1) 
  }
}
  return answer;
}

업그레이드 코드(성공 사례)

function solution(today, terms, privacies) {
  const termsT = {};
  const arr = [];
  today = new Date(today);

  for (const v of terms) {
    const a = v.split(" ");
    termsT[a[0]] = Number(a[1]);
  }

  for (const idx in privacies) {
    const a = privacies[idx].split(" ");
    const b = new Date(a[0]);
    b.setMonth(b.getMonth() + termsT[a[1]]);
    if (b <= today) {
      arr.push(Number(idx) + 1);
    }
  }
  return arr;
}

위에 있는 성공 사례와 실패 사례의 차이점은 19번 테스트가 통과될 뿐만 아니라, 코드가 깔끔합니다.

저는 배열 분리를 할 때, slice와 뭐 if문을 잔뜩 사용하면서 나눴다면... 업그레이드 코드는 split으로 깔끔하게 나눴습니다.

 

console.log로 확인해보면 차이점은 일(date) 를 28일로 바꾼 디테일 밖에 없는데 19번 테스트를 통과하지 못하더라고요. 그래서 28일로 바꾸는 detail을 삭제하고 진행해도 안되더라고요! ㅎㅎ 그래서 업그레이드 코드를 알기 전이기 때문에 new Date() 함수를 사용하지 않고 진행했습니다.

 

두 번째 풀이

function solution(today, terms, privacies) {
  let answer = [];
  let Today = today.split('.').map((v) => Number(v));
  let Terms = {};
  terms.forEach((v) => {
    let [kind, period] = v.split(' ');
    Terms[kind] = Number(period);
  });
  privacies.forEach((v, i) => {
    let [collectDate, termskind] = v.split(' ');
    let [y, m, d] = collectDate.split('.').map((v) => Number(v));

    m += Terms[termskind];    //유효기간 더하기
    d--;					//하루 빼야 유효기간 내 보관이 가능합니다.
    if (m > 12) {
      if (m % 12 === 0) {		//12의 배수이면 년도가 변화하지 않습니다. 아니면 변화합니다.
        y += parseInt(m / 12) - 1;
        m = 12;
      } else {
        y += parseInt(m / 12);
        m = m % 12;
      }
    }

    if (y < Today[0]) {   	//년도가 현재 년도보다 작으면 파기
      answer.push(i + 1);
    } else if (y === Today[0]) {
      if (m < Today[1]) {		//년도가 같고 달이 작으면 파기
        answer.push(i + 1);
      } else if (m === Today[1]) {		//달이 같고 일수가 작으면 파기
        if (d < Today[2]) {
          answer.push(i + 1);
        }
      }
    }
  });
  return answer;
}

복잡해 보이지만 위에 있는 처음 짠 코드를 new Date() 함수 없이 작성한 코드라고 생각이 됩니다.

유효기간을 객체로 나누고

개인정보 저장 날짜와 저장 기간을 더해서 여러개의 if 문을 이용하여 조건을 만족시켰습니다.

 

아직 갈 길이 먼 것 같습니다. 조금 더 노력하여 코드를 깔끔하게 작성해보겠습니다.