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

[중간 퀴즈] lmhyuck #35

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

[중간 퀴즈] lmhyuck #35

wants to merge 3 commits into from

Conversation

lmhyuck
Copy link
Contributor

@lmhyuck lmhyuck commented Jan 27, 2022

Pull Request

5주차입니다.

진행 중 구현을 못한 사항

단어의 사전순이 같고 단어의 길이마저 같은 경우에 대한 예외처리를 못했습니다.
나머지는 어느정도 구현 한것 같습니다

Ki-HS
Ki-HS reviewed Feb 12, 2022
View reviewed changes
Copy link

@Ki-HS Ki-HS left a comment •
edited
Loading

Choose a reason for hiding this comment

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

image

한 개에 단어만 저장됩니다...!

47번째 줄에 있는 while문의 내부 statement를 알맞게 수정하면 입력 메서드는 제대로 완성할 수 있을 것 같아요

@Ki-HS Ki-HS changed the title 퀴즈/lmhyuck [중간 퀴즈] lmhyuck Feb 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Ki-HS Ki-HS Ki-HS left review comments

@jiione jiione Awaiting requested review from jiione jiione is a code owner

@JinSeoHan JinSeoHan Awaiting requested review from JinSeoHan JinSeoHan is a code owner

@kwYoohae kwYoohae Awaiting requested review from kwYoohae kwYoohae is a code owner

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
퀴즈
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@lmhyuck @Ki-HS @kwYoohae