[정기적 풀이 추가] 2022.04.26

[prove-ability]

[level - 1]
Add 음양-더하기.js
Add 약수의-개수와-덧셈.js
Add 정수-제곱근-판별.js
Add 소수-찾기.js
Add 문자열-내-p와-y의-개수.js

아.. commit 정리하고자 git reset 을 사용도 해봤는데 마음처럼 안되네요.. 😭

[codeisneverodd]

PR에 몇 문제를 풀이 하셨는지 정도는 적어주시면 좋을 것 같습니다!!

[codeisneverodd]

이 부분은 이전 코드가 올바른 것 같은데, conflict을 확인하고 수정하신 것 이실까요? @prove-ability 확인 부탁드립니다!

[prove-ability]

제가 컨플릭 수정할 때 커밋 히스토리를 보고 수정했는데 확인이 미흡했습니다! 수정하겠습니다!

[prove-ability]

PR에 몇 문제를 풀이 하셨는지 정도는 적어주시면 좋을 것 같습니다!!

@codeisneverodd
제가 아무것도 적지않아서 오해가 있었던 거 같습니다!
PR draft(초안) 을 사용하고 나중에 4/26일까지 푼 문제 갯수를 더해서 작성하고자 했고
관련해서 description 작성하겠습니다!

[codeisneverodd]

좋은 코드 잘 보았습니다 😁
갈 수록 코드가 더 깔끔해지시는 것 같아요!! 보고 많이 배웁니다
오늘은 코드에 대해 리뷰 남길 것은 없고, solution 과 별도의 함수를 만드실 때 그 위치에 대해 질문이 있습니다.
예를 들어, isPrime함수를 만드실 때 solution 함수보다 상위에 위치시키신 명확한 이유가 있으신지 궁금합니다!!

[prove-ability]

@codeisneverodd

저 부분은 절차적으로 코딩하고 싶었습니다!!
solution 이나 isPrime 어디든 선언하고 호출해 사용하는 함수지만 저 파일만 봤을 때 저 두가지 함수만 존재하고 solution 에서 isPrime 을 사용할 때 "이미 만들어진? 함수를 가져와 사용한다" 정도로 코드를 읽히게 하고 싶었던 거 같습니다!!

level-1/소수-찾기.js