Skip to content

Conversation

jaejeong1
Copy link
Contributor

@jaejeong1 jaejeong1 commented Sep 2, 2024

답안 제출 문제

체크 리스트

  • PR을 프로젝트에 추가하고 Week를 현재 주차로 설정해주세요.
  • 바로 앞에 PR을 열어주신 분을 코드 검토자로 지정해주세요.
  • 문제를 모두 푸시면 프로젝트에서 Status를 In Review로 설정해주세요.
  • 코드 검토자 1분 이상으로부터 승인을 받으셨다면 PR을 병합해주세요.

@jaejeong1 jaejeong1 requested a review from ijlijijij September 2, 2024 13:34
@jaejeong1 jaejeong1 self-assigned this Sep 2, 2024
@github-actions github-actions bot added the java label Sep 2, 2024
@jaejeong1 jaejeong1 marked this pull request as ready for review September 3, 2024 12:09
@jaejeong1 jaejeong1 requested a review from a team as a code owner September 3, 2024 12:09
Copy link
Member

@DaleSeo DaleSeo left a comment

Choose a reason for hiding this comment

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

질문이 하나 있지만 승인하는데는 무리가 없어 보입니다. 수고하셨습니다!

Comment on lines +24 to +33
while(!Character.isLetterOrDigit(charLt) && idxLt < s.length() - 1) {
idxLt++;
charLt = s.charAt(idxLt);
}

// 영문 또는 숫자일때까지 rt--
while(!Character.isLetterOrDigit(charRt) && idxRt > 0) {
idxRt--;
charRt = s.charAt(idxRt);
}
Copy link
Contributor

Choose a reason for hiding this comment

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

기존 문자열을 수정하지 않고 이렇게도 할 수 있었군요! 많이 배웠습니다!

Copy link
Contributor

@ijlijijij ijlijijij left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!

@jaejeong1 jaejeong1 merged commit 5589fc9 into DaleStudy:main Sep 6, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
Status: Completed
Development

Successfully merging this pull request may close these issues.

4 participants