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.

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

[Feature]: 초성, 종성, 중성에 일관된 변수명을 사용하기 #131

Closed
okinawaa opened this issue Jun 24, 2024 · 2 comments · Fixed by #138
Closed

[Feature]: 초성, 종성, 중성에 일관된 변수명을 사용하기 #131

okinawaa opened this issue Jun 24, 2024 · 2 comments · Fixed by #138

Comments

@okinawaa
Copy link
Member

Description

@crucifyer 님의 코맨트를 인용합니다.

chosung이라는 것도 choseong이라는 함수로 변경하면 어떨까요? hangul은 표준이지만 chosung은 표준이 아닌것 같아서 표준대로 혹은 정해진 규칙대로하면 일관된 인터페이스를 만들 수 있을 것 같아요 좋을 것 같아요
http://roman.cs.pusan.ac.kr/result_all.aspx?input=%ec%b4%88%ec%84%b1
https://www.unicode.org/charts/PDF/U1100.pdf
unicode.org 에서도 CHOSEONG, JUNGSEONG, JONGSEONG 이네요.

Possible Solution

No response

etc.

No response

@Collection50
Copy link
Contributor

안녕하세요 @okinawaa !

제가 해당 이슈를 맡고 싶습니다 :)

@okinawaa
Copy link
Member Author

@Collection50 너무 좋아요! 먼저 말씀해주셔서 정말 감사합니다!

po4tion added a commit to po4tion/es-hangul that referenced this issue Jun 26, 2024
okinawaa added a commit that referenced this issue Aug 4, 2024
* test: 음성 표기법 테스트 코드 작성

* test: isNotUndefined 유틸 구현

* test: 단일자모 테스트 케이스 추가

* test: 단일 자모 테스트 케이스 수정

* feat: 음성 표기법 기능 구현

* feat: 음절분해 속 disassembleCompleteHangulCharacter 함수는 undefined일 수 없음

* feat: 다음 음절 변수 조건식 강화

* refactor: 16항 -1 인덱스 조건 처리

* refactor: 16항 조건문 상수화하여 축약

* refactor: 음의 동화 조건 상수화

* refactor: ㄴ,ㄹ이 덧나는 경우 조건 상수화

* refactor: 19항 조건문 상수화

* refactor: 18항 조건문 상수화

* refactor: 20항 유음화 조건문 순서 수정

* refactor: 12항 조건에 상수화 적용

* refactor: 13,14항 상수화 및 반환타입추론

* refactor: 9,10,11항 조건문 상수화

* refactor: internal 함수 사용

* docs: phoneticNotation 문서 작성

* test: covertToRoman 테스트 코드 작성

* test: convertToRoman 테스트 케이스 추가

* feat: convertToRoman 구현

* refactor: 음절분해 return type 명시

* test: 테스트 케이스 수정

* docs: romanize

* refactor: convertToRoman에서 romanize로 이름을 변경합니다.

* refactor: phoneticNotation에서standardPronunciation으로 이름을 변경합니다

* feat: type guard include 함수를 구현합니다

* test: 경음화를 적용합니다

* feat: 경음화 기능을 옵션으로 추가합니다

* refactor: 표준발음 함수에서 사용되는 상수를 상수파일로 이동

* docs: 함수 설명문 수정

* docs: 표준발음법 문서 수정

* docs: 표준 발음법 영문 문서의 주석을 변경합니다

* test: isIncludedInArray 테스트 코드를 작성합니다

* fix: 특정회사 언급 문자 제거

* fix: 보다 간결하게 isIncludedInArray > arrayIncludes로 이름 변경

* fix: standardPronunciation 상수와 관련하여 새파일을 만든다

* fix: map 메소드를 축약한다

* fix: arrayIncludes > isArrayIncludes로 이름을 변경

* fix: arrayIncludes 함수를 개선합니다

* fix: romanize 가독성을 개선합니다

* fix: romanize > romanizeHangul #121

* refactor: 경음화 로직 분리

* refactor: nextSyllabel 연산 로직 축약

* refactor: 16항 로직 분리

* refactor: 17,19항 로직을 분리합니다

* refactor: 18항 로직을 분리합니다.

* refactor: 불필요한 조건문 return 제거

* refactor: 12항 로직 분리

* refactor: 13,14 항 로직을 분리합니다

* refactor: 9,10,11항 로직 분리

* refactor: standardPronunciation 기능을 개별 폴더로 분리합니다

* refactor: ㄴ,ㄹ 동화작용 설명 추가

* refactor: standardPronunciation 조건문 중복 처리

* test: 경음화 등의 된소리를 적용하지 않는 테스트 케이스 추가

* refactor: 주요 로직 추상화

* fix: romanizeHangul에 일관된 변수명 적용 #131

* test: 한글과 영어가 혼합된 테스트 케이스를 추가합니다

* test: 테스트 케이스 예시를 변경합니다

* fix: ㄴ/ㄹ이 덧나는 경우에도 연음규칙 적용이 가능

* fix: 주석 변경

* fix: 이해하기 쉬운 용어로 주석 변경

* fix: 메서드 컨벤션을 적용합니다 #132

* fix: Received Object applyRules

* fix: SRP 원칙 적용 - 1

* refactor: replace받침ㅎ

* refactor: make pure function

* refactor: applyRules

* refactor: rule 별로 파일 분리

* refactor: rule naming 변경

* test: rules에 test case 추가

* fix: type import

* fix: update import case

* fix: isNotUndefined, defined, arrayIncludes를 internal로 이동합니다.

* test: romanize test case 수정

* Create hip-pianos-live.md

---------

Co-authored-by: 박찬혁 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants