Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

파이브라인스 오브 코드 - 예제 풀이 #11

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

파이브라인스 오브 코드 - 예제 풀이 #11

wants to merge 3 commits into from

Conversation

jihwooon
Copy link
Owner

@jihwooon jihwooon commented Jun 2, 2024 •
edited
Loading

요약

Pull Request 체크리스트

TODO

  • PR 올리기 전 rebase 동기화를 하셨나요?
  • 마지막 줄에 공백 처리를 하셨나요?
  • 커밋 단위를 의미 단위로 나눴나요?
    • 예시
      • 코드 가독성을 위해 메서드를 추출하라
      • if-else 문을 if 문으로 분리하라
      • 불필요한 메서드를 인라인화하라
  • 커밋 본문을 작성하셨나요?
    • 예시
      • 함수는 한 가지 일을 해야 한다는 원칙에 따라 메서드를 추출합니다.
      • if-else는 컴파일 시 처리가 되어 재컴파일 없이 수정 할 수 없습니다.
        이에 따라 코드가 실행되는 순간에 실행이 결정되는 if 문으로 수정합니다.

@jihwooon
reportPrimes 함수를 더 작은 단위의 함수로 분리하고 테스트 추가
6bd95c4 
- reportPrimes 함수를 reportLoopPrimes와 reportIfPrimes 함수로 분리
- reportLoopPrimes 함수는 n 미만의 숫자에 대해 반복문을 돌면서 reportIfPrimes 함수 호출
- reportIfPrimes 함수는 주어진 숫자가 소수인 경우 "x is prime" 형식의 문자열 반환, 아닌 경우 null 반환
- reportPrimes 함수에 대한 테스트 추가
@jihwooon jihwooon changed the title reportPrimes 함수를 더 작은 단위의 함수로 분리하고 테스트 추가 파이브라인스 오브 코드 - 예제 풀이 Jun 2, 2024
jihwooon added 2 commits June 3, 2024 01:02
@jihwooon
minimum 함수를 더 작은 단위의 함수로 분리하고 테스트 추가
c20edfd 
- minimum 함수를 calculateMinimum, calculateRow, calculateRowMin, minimumIfPositiveInfinity 함수로 분리
- 각 함수는 더 작고 명확한 기능을 수행
- minimum 함수에 대한 테스트 추가
- 가독성 및 유지보수성 향상
@jihwooon
평균 계산 함수 리팩토링 및 테스트 추가
1bed665 
- sum 함수를 재귀 함수로 변경하여 arr.length를 size 함수로 추출
- average 함수에서 sum 함수를 호출하도록 변경
- average 함수에 대한 테스트 코드 작성 및 검증
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@jihwooon