🤔 이런 고민을 했습니다.

변수명과 함수명

변수에는 어떤 데이터가 담겨있는지와 함수는 무슨 동작을 하는지 쉽게 파악할 수 있는 네이밍을 짓기위해 노력하였습니다.
예시) isExceptionSituationOccurred 예외상황이 발생했는지 여부를 반환하는 함수

배열 메소드

배열 메소드를 사용했을때 코드 길이와 가독성 측면에서 for문을 사용했을때 보다 이점이 있다고 판단했습니다.
따라서 로직을 짤때 최대한 배열메소드를 사용하려 노력하였습니다.

for 문 사용

const eachDigitAllMultiply = (bookPage) =>{
  cosnt bookPageArray = String(bookPage).split("")
  let sumOfMultiplication = 1
  for (let i=0; i<bookPageArray.length; i+=1){
    sumOfMultiplication*= Number(curr)
  }

  return sumOfMultiplication
}

배열 메소드 사용

const eachDigitAllMultiply = (bookPage) =>
  String(bookPage)
    .split("")
    .reduce((prev, curr) => prev * Number(curr), 1);

📖 이런점을 배웠습니다.

Set 자료구조

7번 문제를 풀면서 이미 친구인 사람의 목록을 배열로 만들었습니다. 이제 그 배열에 includes 메소드를 사용하여 아는 사람의 목록을 만들었고, 다른 방법은 없을까 생각해보다가 Set 과 Map 자료구조가 접근하는 시간복잡도가 더 빠르다라는 것이 생각났습니다.

var arr = Array.from({ length: 1000000 }, (_, i) => i);
var set = new Set(arr);
arr.includes(987654);
set.has(987654)

테스트 사이트에서 시간을 측정한 결과 유의미한 차이가 났기때문에 기존 7번 문항에서 includes 메소드를 사용하는 방식에서 set 을 사용하는 방식으로 로직을 변경하였습니다.

Pipe 함수

처음 제 코드는 다음과 같았습니다.

function problem7(user, friends, visitors) {
  const friendSet = makeFriendSet(user, friends);
  let acquaintanceSet = makeAcquaintanceSet(friends, friendSet);
  acquaintanceSet = differenceOfSet(acquaintanceSet, friendSet);
  const pointMap = giveFriendAlgorithmPoint(acquaintanceSet);
  const copyPointMap = giveVisitorPoint(visitors, pointMap, friendSet);
  const pointArray = convertPointMapToSortedPointArray(copyPointMap);
  return selectTopFive(test);
}

저는 함수명을 보고 어떤 값이 return이 될지 파악할 수 있다고 판단하였고, 변수에 값을 할당하는 과정을 생략해보기로 하였습니다.
검색한 결과 함수 여러개를 이어주는 파이프 함수라는 방법을 찾을 수 있었고 이를 한번 시도해보았습니다.

function problem7(user, friends, visitors) {
  const solve = pipe(
    makeFriendSet,
    makeAcquaintanceSet,
    differenceOfSet,
    giveFriendAlgorithmPoint,
    giveVisitorPoint,
    convertPointMapToSortedPointArray,
    selectTopFive
  )({ user, friends, visitors });

  return solve;
}

처음 시도해보아서 코드 가독성은 어떤지 커뮤니티가 열리면 참여하신분들에게 여쭤보고싶습니다.

Git rebase

과제를 이해했을때 각 문제들이 구현기능이라고 파악을 하였습니다. 하지만 슬랙 커뮤니티를 살펴보니 각 문제별로 기능목록을 작성하여 구현하셨다는 분이 계셨고, 저도 그렇게 바꾸는게 낫다고 생각했습니다.
하지만 이미 fork 한 레포에 commit이 push 가 되어있는 상태였기때문에 commit history를 변경하기 위해 rebase 를 사용하여 새로운 commit을 추가했습니다. 이번과제를 통해 익숙하지 않던 rebase 명령어를 사용할 수 있어 좋았습니다.

저작자표시 (새창열림)