toss / es-hangul

A modern JavaScript library for handling Hangul characters.
https://es-hangul.slash.page/
MIT License
1.34k stars 95 forks source link

[Bug]: 받침이 있는 경우에 josa 함수의 '이에/에' 붙이기 동작이 정확하지않음. #139

Closed JAAAAAEMKIM closed 4 months ago

JAAAAAEMKIM commented 4 months ago

Bug description

이에/에 판별이 정확한지 재확인이 필요합니다. 받침이 있는 경우에 josa('받침', '이에/에')와 같이 josa함수를 사용하면 받침이에라는 표현이 나옵니다. 받침에라고 정확하게 나와야합니다.

이에라는 조사가 사용되는 경우가 없음에도 잘못 들어간 상황으로 보입니다.

Expected behavior

josa('받침', '이에/에')
// expect '받침에'

To Reproduce

josa('바깥', '이에/에')

// expect: '바깥에'
// result: '바깥이에'

Possible Solution

'이에/에'라는 그룹이 있어야하는지 확인이 필요합니다. 만약 유효하지 않은 그룹이라면 제거를 통해 해결할 수 있습니다.

etc.

No response

JAAAAAEMKIM commented 4 months ago

문서 오류 같네요.

// 코드 상 Josa
  | '이/가'
  | '을/를'
  | '은/는'
  | '으로/로'
  | '와/과'
  | '이나/나'
  | '이란/란'
  | '아/야'
  | '이랑/랑'
  | '이에요/예요'
  | '으로서/로서'
  | '으로써/로써'
  | '으로부터/로부터';

// 문서 상 josa
    | '이/가'
    | '을/를'
    | '은/는'
    | '으로/로'
    | '와/과'
    | '이나/나'
    | '이에/에' **
    | '이란/란'
    | '아/야'
    | '이랑/랑'
    | '이에요/예요'
    | '으로서/로서'
    | '으로써/로써'
    | '으로부터/로부터'
okinawaa commented 4 months ago

직접 기여도 진행해주셔서 감사합니다!