-
Notifications
You must be signed in to change notification settings - Fork 72
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
[Bug]: removeLastHangulCharacter 함수에서 disassembledGroups의 길이가 4개인 경우 에러가 발생합니다. #123
Labels
bug
Something isn't working
Comments
요 pr 로 해결 되겠네요 :) |
@crucifyer 안녕하세요! 해당 부분 이슈나 PR이 없어서 작업이 진행된 줄 몰랐네요..! 😅 해당 파일에서 문제가 되는 제안하는 코드 export function removeLastHangulCharacter(words: string) {
const disassembledGroups = disassembleHangulToGroups(words);
const lastCharacter = disassembledGroups[disassembledGroups.length - 1];
if (lastCharacter == null) {
return '';
}
const withoutLastCharacter = words.substring(0, words.length - 1);
const [[first, middle, last]] = excludeLastElement(lastCharacter);
const result = middle != null ? combineHangulCharacter(first, middle, last) : first;
return [withoutLastCharacter, result].join('');
} 우선 해당 이슈는 올려주신 PR이 머지되거나 따로 메인테이너분께서 안내해주시기 전까지는 열어두겠습니다..! |
해당 PR에서 개선되어서 close 합니다..! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Bug description
removeLastHangulCharacter: 인자로 주어진 한글 문자열에서 가장 마지막 문자 하나를 제거하여 반환
현재
removeLastHangulCharacter
에서withoutLastCharacter
를 구할 때map
함수 내부에서first
,second
,last
로 구성되어 있어요. 하지만last
에서 생각대로 동작하지 못하는 부분을 발견했습니다.disassembleHangulToGroups
함수의 테스트 예시로값
을 넣었을 경우['ㄱ', 'ㅏ', 'ㅂ', 'ㅅ']
을 기대하고 있습니다.하지만 실제
combineHangulCharacter
로 다시 결합시킬 때는값
을 받고 싶으면'ㄱ', 'ㅏ', 'ㅂㅅ'
을 입력받기를 기대하고 있습니다.그래서 다음과 같이 테스트를 진행했을 때 에러가 발생해요.
안녕하세요 값이
안녕하세요 값ㅇ
안녕하세요 갑ㅇ
콘솔로 확인해보면
![스크린샷 2024-06-19 오전 1 22 10](https://private-user-images.githubusercontent.com/79239852/340755473-50db64aa-fb5c-406a-b74a-780ada38ac27.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjE5NTg0ODcsIm5iZiI6MTcyMTk1ODE4NywicGF0aCI6Ii83OTIzOTg1Mi8zNDA3NTU0NzMtNTBkYjY0YWEtZmI1Yy00MDZhLWI3NGEtNzgwYWRhMzhhYzI3LnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA3MjYlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNzI2VDAxNDMwN1omWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTk3OGRkOTA4YmVlNjg1N2ExNjQ0MGE0ZjY5ZjIzYjRmNmNhMWU1OWJlNTk2NTE3ZTQwNjFlYTkyMTJjZDI3OWMmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.S_3uP2n01xvYMfRIyZiL-BeEdGf26KuHr3oByOOmBV8)
last
에서ㅅ
이 생략되고 있습니다.정리하면 종성으로 겹받침이 오면
withoutLastCharacter
변수를 구할 때map
함수에서 생략되는 부분이 발생해요.Expected behavior
No response
To Reproduce
종성에 겹받침이 오는 문자가 마지막 문자가 아닌 경우 에러가 발생합니다.
ex)
안녕하세요 많이
,안녕하세요 값이
Possible Solution
나머지 매개변수를 활용해서 마지막은
join
으로 합쳐서last
를 구성하는 것을 생각했어요.안녕하세요 많이
로 테스트 진행할 경우안녕하세요 많ㅇ
안녕하세요 많ㅇ
etc.
라이브러리를 보면서 많이 배우고 있습니다..! 감사합니다! 🙇♂️
The text was updated successfully, but these errors were encountered: