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.

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

solution level2 - 숫자 블록 #79

Merged
merged 2 commits into from
Aug 3, 2022

Conversation

iHoHyeon
Copy link
Contributor

level2 - 숫자 블록 풀이입니다.

  • 기존에 풀이가 없던 문제라서 파일을 추가했습니다.
  • 설명은 주석으로 추가했습니다.

Copy link
Owner

@codeisneverodd codeisneverodd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

풀이가 정말 깔끔 그 자체네요.. ✨✨
주석도 잘되어있어 이해하는데 많은 도움이 되었습니다! 또 다시 기여해주셔서 감사합니다 👍👍

@codeisneverodd codeisneverodd changed the base branch from main to week-19 August 1, 2022 06:59
Co-authored-by: codeisneverodd <codeisneverodd@gmail.com>
@codeisneverodd codeisneverodd merged commit 35c30f2 into codeisneverodd:week-19 Aug 3, 2022
codeisneverodd added a commit that referenced this pull request Aug 17, 2022
* solution: level 2 / 하노이 탑 풀이 추가

* solution: 모음사전

* solution 체육복

* solution 모의고사

* solution 위장

* solution 튜플

* 브랜치 흐름 정리 (#71)

* docs: 커밋 컨벤션 수정  및 contributor 추가

* Automatic Update README.md

* fix: 오타 수정

* docs: 커밋 컨벤션 변경 및 리드미 디자인 수정

* docs: 풀이 완료시점 수정

* Automatic Update README.md

* docs: PR 템플릿 추가

Co-authored-by: github-actions <github-actions@github.com>

* fix: base url 핫픽스 (#74)

* docs: 커밋 컨벤션 수정  및 contributor 추가

* Automatic Update README.md

* fix: 오타 수정

* docs: 커밋 컨벤션 변경 및 리드미 디자인 수정

* docs: 풀이 완료시점 수정

* Automatic Update README.md

* docs: PR 템플릿 추가

* fix: 링크 base url 수정 (#73)

* Automatic Update README.md

Co-authored-by: github-actions <github-actions@github.com>

* [손재영] level2 배달 풀이 및 주석

* solution 구명보트

* fix: 최소힙 클래스 바깥 분리&파일 컨벤션 적용

* solution level2 - 숫자 블록 (#79)

* solution level2 - 숫자 블록

* Update level-2/숫자-블록.js

Co-authored-by: codeisneverodd <codeisneverodd@gmail.com>

Co-authored-by: codeisneverodd <codeisneverodd@gmail.com>

* solution: level2 - 피로도 문제 풀이

* refactor: level2 - 피로도(가독성 개선)

1. 최대한 순수함수에 가깝도록, 함수 내부에서 외부 변수를 참조하거나
조작하지 않도록 수정하였습니다.

2. 변수와 함수 이름을 수정하여 의미를 명확히 하였습니다.

3. 로직의 목적을 쉽게 파악할 수 있도록 일반 for문 대신 배열 메서드를 적극 활용하였습니다.

* feat: PR 템플릿 위치 변경

* feat: husky 및 pre-commit hook 설정, prettierignore 추가

* fix: 잘못 업로드된 해설 삭제

Co-authored-by: chaerin-dev <chaerin.dev@gmail.com>
Co-authored-by: github-actions <github-actions@github.com>
Co-authored-by: Jaeyeong son <nardi0302@gmail.com>
Co-authored-by: iHoHyeon <hyunee169@naver.com>
Co-authored-by: iHoHyeon <74816327+iHoHyeon@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants