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

2022.04.11(월) jaewon1676 7문제의 풀이 추가, README.md 수정 #24

Merged
merged 8 commits into from
Apr 11, 2022

Conversation

jaewon1676
Copy link
Contributor

@jaewon1676 jaewon1676 commented Apr 11, 2022

기존의 파일의 풀이 추가 - 6문제

파일 생성 후 풀이 추가 - 1문제 (level2 구명보트)

level2 파일 추가에 따른 README.md 83줄의 문제수, 날짜 수정도 진행 하였습니다 !

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.

좋은 코드 잘 보았습니다! 주석도 잘 달아주시고, conflict도 없어서 많은 도움이 될 것 같습니다! 고생하셨습니다 😄

@@ -80,7 +80,7 @@
### Level 2 👨🏻‍💻(풀이 중..)

- 전체 문제 수: 64문제
- 풀이 문제 수: 44문제(2022.04.07)
- 풀이 문제 수: 45문제(2022.04.11)
Copy link
Owner

Choose a reason for hiding this comment

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

리드미까지 감사합니다!

cnt++;
}
return cnt;
} // 4주차 2번 문제와 유사함
Copy link
Owner

Choose a reason for hiding this comment

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

해당 레퍼지토리와 저희 팀에서 private으로 하는 레퍼지토리는 연관성이 없어서 해당 주석이 저희 팀이 아닌 분들에게는 혼란이 올 수 있을 것 같습니다!

// s의 길이를 2로 나눠서 cnt 값이 된다면 문자열의 개수만큼 잘 잘라졌다는 뜻입니다
return answer;
}
// 코테 1주차 5번 문제와 유사합니다. 문자열 유형을 연습하기에 좋은 문제입니다
Copy link
Owner

Choose a reason for hiding this comment

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

해당 레퍼지토리와 저희 팀에서 private으로 하는 레퍼지토리는 연관성이 없어서 해당 주석이 저희 팀이 아닌 분들에게는 혼란이 올 수 있을 것 같습니다!
문자열 유형을 연습하기에 좋은 문제라는 코멘트는 너무 좋은 것 같습니다 👍

@codeisneverodd codeisneverodd merged commit 450c8a6 into codeisneverodd:main Apr 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants