feat: 한글 자모음을 인자로 입력받아 한글 문자로 합성하는 함수 추가 #53

evan-moon · 2024-04-16T13:09:52Z

Overview

#18 에서 이슈레이징된 aseemble 함수를 구현하기 위해 한글 문자를 합성하는 함수를 추가합니다.

한글 문자를 합성하는 combineHangulCharacter, combineVowels 함수 추가
combineHangulCharacter 함수의 커링된 버전인 curriedCombineHangulCharacter 함수 추가
canBeChosung, canBeJungsung, canBeJongsung 함수에 타입 가드 추가

combineHangulCharacter

초성, 중성, 종성을 인자로 받아 하나의 한글 문자를 생성합니다. 만약 잘못된 값을 입력받았다면 "Invalid hangul character" 에러를 throw 합니다.

combineHangulCharacter('ㄱ', 'ㅏ', 'ㅂㅅ') // '값'

curriedCombineHangulCharacter

combineHangulCharacter 함수의 커링된 버전입니다. #18 에서 이슈레이징된, 조건에 따라 순차적으로 문자를 합성해나가야하는 함수를 구현할 때 필요합니다.

const combineMiddleHangulCharacter = curriedCombineHangulCharacter('ㄱ')
const combineLastHangulCharacter = combineMiddleHangulCharacter('ㅏ')

combineLastHangulCharacter('ㄱ') // '각'

combineVowels

두 개의 단모음을 인자로 받아 겹모음을 생성합니다. 만약 올바르지 않은 단모음을 입력받았다면 Join하여 반환합니다.

combineVowels('ㅗ', 'ㅏ') // 'ㅘ'

PR Checklist

I read and included theses actions below

I have read the Contributing Guide
I have written documents and tests, if needed.

changeset-bot · 2024-04-16T13:09:56Z

🦋 Changeset detected

Latest commit: 1e2d292

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

netlify · 2024-04-16T13:10:09Z

✅ Deploy Preview for es-hangul ready!

Name	Link
🔨 Latest commit	`1e2d292`
🔍 Latest deploy log	https://app.netlify.com/sites/es-hangul/deploys/661f27697973e40008f3362d
😎 Deploy Preview	https://deploy-preview-53--es-hangul.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

.changeset/sweet-weeks-tie.md

Co-authored-by: Jonghyeon Ko <jonghyeon.ko@wesang.com>

src/combineHangulCharacter.spec.ts

youngjae99 · 2024-04-16T13:45:46Z

src/combineHangulCharacter.ts

+  const 중성개수 = HANGUL_CHARACTERS_BY_MIDDLE_INDEX.length;
+  const 종성개수 = HANGUL_CHARACTERS_BY_LAST_INDEX.length;
+
+  const 초성인덱스 = HANGUL_CHARACTERS_BY_FIRST_INDEX.indexOf(firstCharacter);
+  const 중성인덱스 = HANGUL_CHARACTERS_BY_MIDDLE_INDEX.indexOf(middleCharacter);
+  const 종성인덱스 = HANGUL_CHARACTERS_BY_LAST_INDEX.indexOf(lastCharacter);


마이너하긴 한데, 충분히 영어로 표현 가능한 변수명 같아보입니다. 기본적으로 변수명은 영어로 컨벤션을 가져가는 것은 어떨까요?

e.g.

중성개수 -> numMiddleCharacter

초성인덱스 -> firstCharacterIndex

좋습니당 컨벤션은 하나로 맞추는게 좋을 것 같네요!

1e2d292 에서 반영하였습니다! 하단의 코멘트에 질문해주신 유니코드에 대한 이론도 firstIndexOfTargetConsonant, firstIndexOfTargetVowel이라는 변수명으로 표현해봤어요 🙏

youngjae99 · 2024-04-16T13:48:04Z

src/combineHangulCharacter.ts

+  const 종성인덱스 = HANGUL_CHARACTERS_BY_LAST_INDEX.indexOf(lastCharacter);
+
+  const unicode =
+    COMPLETE_HANGUL_START_CHARCODE + 초성인덱스 * 중성개수 * 종성개수 + 중성인덱스 * 종성개수 + 종성인덱스;


잘 몰라서 그러는데, 혹시 unicode 계산식이 있을까요? 단순하게 생각했을 때

초성인덱스 * 초성개수 * 중성개수 + 중성인덱스 * 종성개수 + 종성인덱스

가 되어야하는게 아닌가 의문이 들어서요.

넵 요것은 유니코드표에 한글이 등재되어있는 순서가 조금 특이해서 나타나는 현상인데요.

유니코드 표 상에서 한글문자는 "가, 나, 다" 순이 아닌 "가, 각, 갂, 갃..." 등의 순서로 되어있습니다.

즉, 유니코드에 한글이 등재된 순서는 어떤 자음 문자 하나가 나타났을 때, 이 자음으로 조합할 수 있는 모든 문자를 나열한 이후 다음 자음으로 넘어간다고 볼 수 있습니다. 게다가 자음이나 모음의 나열 순서도 일반적으로 우리가 한글을 공부할 때 사용하는 "ㄱ, ㄴ, ㄷ, ㄹ..."이나 "ㅏ, ㅑ, ㅓ, ㅕ..." 순서가 아닌 "ㄱ, ㄲ, ㄴ, ㄷ, ㄹ, ㄸ..."와 "ㅏ, ㅐ, ㅑ, ㅒ..." 순서로 되어있어요.

그래서 만약 초성으로 ㄴ, 중성으로 ㅐ, 종성으로 ㄱ이 들어온 상황이라면

ㄴ의 초성 인덱스 3 * 중성 개수 21 * 종성 개수 28 = ㄱ, ㄲ으로 조합할 수 있는 모든 문자를 건너뛰어야 ㄴ으로 시작하는 첫 문자인 나까지 갈 수 있음

중성 인덱스 1 * 종성 개수 28 = 1번까지 수행했다면 나부터 세야하니, ㄴ+ㅏ+종성으로 조합할 수 있는 모든 문자를 건너뛰어야 ㄴ+ㅐ로 시작하는 첫 문자인 "내"까지 갈 수 있음.

종성 인덱스 1= 여기서부터 내, 낵, 낶, 낷... 순서로 나타남. 그러니 종성 인덱스만큼만 앞으로 가면 원하는 문자에 도달.

와 같은 원리로 계산을 하여 원하는 문자를 찾아간다고 봐주시면 됩니다.

말씀해주신 "초성인덱스 * 초성개수"라는 식은 자음이 "ㄱ, ㄴ, ㄷ, ㄹ..." 순서로 나타난다는 상황을 가정하고 있지만 유니코드의 한글 문자 등재 순서가 "특정 자음 선택 + 해당 자음으로 조합할 수 있는 모든 문자를 나열"이기 때문에 계산식이 조금 특이합니다 😅

@evan-moon 혹시나 오타일까 확인 차 여쭤봤는데 올바른 구현이었군요. 상세한 설명 넘 감사드립니다 👍

Co-authored-by: Youngjae Jang <youngjae.j99@gmail.com>

src/combineHangulCharacter.ts

okinawaa

긴 작업 수고하셨습니다! 😄

okinawaa · 2024-04-17T08:18:14Z

src/combineHangulCharacter.spec.ts

+    assert.throws(() => combineHangulCharacter('ㄱ', 'ㄴ', 'ㅃ'), Error, 'Invalid hangul Characters: ㄱ, ㄴ, ㅃ');
+  });
+
+  it('종성이 될 수 없는 문자가 종성으로 입력되면 에러를 반환한다. (ㄱ, ㅏ, ㅃ)', () => {


요런거 테스트 케이스 너무 좋네요👍

evan-moon added 2 commits April 16, 2024 21:56

feat: 초성, 중성, 종성을 합쳐 하나의 한글 문자를 생성하는 combineHangulCharacter 함수 추가

3b17f4c

chore: jsDoc 추가

0057cff

evan-moon self-assigned this Apr 16, 2024

evan-moon added the enhancement New feature or request label Apr 16, 2024

evan-moon requested review from raon0211 and okinawaa April 16, 2024 13:11

Create sweet-weeks-tie.md

305f3ac

manudeli reviewed Apr 16, 2024

View reviewed changes

.changeset/sweet-weeks-tie.md Outdated Show resolved Hide resolved

Update sweet-weeks-tie.md

3dc68da

Co-authored-by: Jonghyeon Ko <jonghyeon.ko@wesang.com>

youngjae99 reviewed Apr 16, 2024

View reviewed changes

src/combineHangulCharacter.spec.ts Outdated Show resolved Hide resolved

youngjae99 reviewed Apr 16, 2024

View reviewed changes

src/combineHangulCharacter.spec.ts Outdated Show resolved Hide resolved

youngjae99 reviewed Apr 16, 2024

View reviewed changes

evan-moon and others added 2 commits April 16, 2024 23:03

Update combineHangulCharacter.spec.ts

240ac69

Co-authored-by: Youngjae Jang <youngjae.j99@gmail.com>

Update combineHangulCharacter.spec.ts

9faf4c0

Co-authored-by: Youngjae Jang <youngjae.j99@gmail.com>

ssi02014 reviewed Apr 16, 2024

View reviewed changes

src/combineHangulCharacter.ts Show resolved Hide resolved

ssi02014 reviewed Apr 16, 2024

View reviewed changes

src/combineHangulCharacter.ts Show resolved Hide resolved

refactor: 변수명을 영어로 변경

1e2d292

evan-moon requested review from youngjae99, manudeli and ssi02014 April 17, 2024 05:03

okinawaa approved these changes Apr 17, 2024

View reviewed changes

ssi02014 approved these changes Apr 17, 2024

View reviewed changes

evan-moon merged commit 314f0a9 into toss:main Apr 17, 2024
4 checks passed

github-actions bot mentioned this pull request Apr 17, 2024

chore: version packages #55

Merged

evan-moon mentioned this pull request Apr 18, 2024

[Docs]: 공식 문서 최신화 #58

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: 한글 자모음을 인자로 입력받아 한글 문자로 합성하는 함수 추가 #53

feat: 한글 자모음을 인자로 입력받아 한글 문자로 합성하는 함수 추가 #53

evan-moon commented Apr 16, 2024 •

edited

Loading

changeset-bot bot commented Apr 16, 2024 •

edited

Loading

netlify bot commented Apr 16, 2024 •

edited

Loading

youngjae99 Apr 16, 2024

evan-moon Apr 17, 2024

evan-moon Apr 17, 2024

youngjae99 Apr 16, 2024

evan-moon Apr 17, 2024

youngjae99 Apr 18, 2024

okinawaa left a comment

okinawaa Apr 17, 2024

feat: 한글 자모음을 인자로 입력받아 한글 문자로 합성하는 함수 추가 #53

feat: 한글 자모음을 인자로 입력받아 한글 문자로 합성하는 함수 추가 #53

Conversation

evan-moon commented Apr 16, 2024 • edited Loading

Overview

combineHangulCharacter

curriedCombineHangulCharacter

combineVowels

PR Checklist

changeset-bot bot commented Apr 16, 2024 • edited Loading

🦋 Changeset detected

netlify bot commented Apr 16, 2024 • edited Loading

✅ Deploy Preview for es-hangul ready!

youngjae99 Apr 16, 2024

Choose a reason for hiding this comment

evan-moon Apr 17, 2024

Choose a reason for hiding this comment

evan-moon Apr 17, 2024

Choose a reason for hiding this comment

youngjae99 Apr 16, 2024

Choose a reason for hiding this comment

evan-moon Apr 17, 2024

Choose a reason for hiding this comment

youngjae99 Apr 18, 2024

Choose a reason for hiding this comment

okinawaa left a comment

Choose a reason for hiding this comment

okinawaa Apr 17, 2024

Choose a reason for hiding this comment

evan-moon commented Apr 16, 2024 •

edited

Loading

changeset-bot bot commented Apr 16, 2024 •

edited

Loading

netlify bot commented Apr 16, 2024 •

edited

Loading