Closed jungwoo3490 closed 1 month ago
Latest commit: 4a615ae137b1450d6151935d42db5f9bd05ef90a
The changes in this PR will be included in the next version bump.
Not sure what this means? Click here to learn what changesets are.
Click here if you're a maintainer who wants to add another changeset to this PR
The latest updates on your projects. Learn more about Vercel for Git ↗︎
Name | Status | Preview | Comments | Updated (UTC) |
---|---|---|---|---|
es-hangul | ✅ Ready (Inspect) | Visit Preview | 💬 Add feedback | Jul 12, 2024 3:06pm |
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 100.00%. Comparing base (
8f7cc9a
) to head (d5da627
). Report is 1 commits behind head on main.
@okinawaa 리뷰 반영 완료했습니다!! 확인 부탁드려요 :)
Overview
resolve #167
Issue에도 설명되었듯이, amountToHangul이 선행 0이 있는 문자열을 처리할 경우 의도치 않은 동작을 하는 것을 확인했어요.
이를 해결하기 위해 다음과 같이 integerPart의 선행 0을 모두 제거하는 로직을 추가했어요.
여기서 '0'인지 검사하는 이유는, tempDecimalPart가 존재하지 않는 상황에서 integerPart가 위 정규표현식에 의해 제거되어 빈 문자열이 되게 되면
이 조건식이 false가 되어 결국 '0'일 경우 "영"이 아닌 빈 문자열로 변환을 하게 되는 문제가 발생하는 것을 확인했어요.
그래서 '0'이 입력으로 들어왔을 경우만 선행 0을 제거하는 정규표현식이 동작하지 않도록 해주었습니다.
PR Checklist