이 풀이도 괜찮을까요?

자바스크립트 알고리즘 문제풀이 입문(코딩테스트 대비)

4. 졸업선물

작성

224

- 학습 관련 질문을 남겨주세요. 상세히 작성하면 더 좋아요!
- 먼저 유사한 질문이 있었는지 검색해보세요.
- 서로 예의를 지키며 존중하는 문화를 만들어가요.
- 잠깐! 인프런 서비스 운영 관련 문의는 1:1 문의하기를 이용해주세요.

콘솔창엔 답이 나오긴 하는데 반례가 있나 싶습니다..!

const input = `5 28
6 6
2 2
4 3
4 5
10 3`.split("\n");

function solution(input) {
  let totalMoney = Number(input.shift().split(" ")[1]);
  let inputNumArr = input
    .map((arr, index) => {
      if (index === input.length - 1) {
        return [Number(arr.split(" ")[0]) / 2, Number(arr.split(" ")[1])];
      } else {
        return [Number(arr.split(" ")[0]), Number(arr.split(" ")[1])];
      }
    })
    .sort((a, b) => a[0] - b[0]);
  let count = 0;

  for (let i = 0; i < inputNumArr.length; i++) {
    let presentSum = inputNumArr[i].reduce((a, b) => a + b, 0);

    if (totalMoney - presentSum < 0) {
      break;
    } else {
      totalMoney -= presentSum;
      count++;
    }
  }

  return count;
}

console.log(solution(input));

javascript 코딩-테스트

답변 2

김태원

지식공유자

안녕하세요^^

반례입니다.

const input = `5 41
8 6
2 2
12 1
4 5
4 3`.split("\n");

12 1 을 할인받으면 답은 5입니다.

인프런 AI 인턴

안녕하세요, 인프런 AI 인턴이에요.

코드를 보니 문제를 풀기에는 적절하게 구현되어 있는 것 같습니다. 그러나 예외 사항이 발생할 수 있으니 몇 가지 수정이 필요해 보입니다.

변수 선언과 초기화를 같은 라인에서 하는 것이 가독성이 떨어져서, 라인을 나누어 작성하시면 더 깔끔할 것 같습니다.
배열을 map 함수를 이용해 변경하는 것보다 그대로 사용해도 되는 경우에는 그대로 사용하는 것이 코드의 이해도를 높일 수 있습니다. 자원 소모와 로직 파악에 있어서도 더 효율적입니다.
현재 코드에서는 축하금의 금액이 홀수일 경우 2로 나눈 몫을 취하는 로직이 없으므로, 문제에서 주어진 조건을 충족시키지 않습니다.

아래는 수정된 코드입니다.

const input = `5 28
6 6
2 2
4 3
4 5
10 3`.split("\n");

function solution(input) {
  let totalMoney = Number(input[0].split(" ")[1]);
  let giftArr = input.slice(1).map((line) =>
    line.split(" ").map((el) => Number(el))
  );
  let count = 0;

  giftArr = giftArr.sort((a, b) => a[0] - b[0]);

  for (let i = 0; i < giftArr.length; i++) {
    let presentSum = giftArr[i][0] * 2 <= totalMoney ? giftArr[i].reduce((a, b) => a + b, 0) : 0;

    if (presentSum === 0) {
      break;
    } else {
      totalMoney -= presentSum;
      count++;
    }
  }

  return count;
}

console.log(solution(input)); // 4

이렇게 변경해보시고, 다시 한 번 테스트해보시면 좋을 것 같아요!

인프런 커뮤니티 질문&답변

이 풀이도 괜찮을까요?