Skip to content

Conversation

@irhdab
Copy link

@irhdab irhdab commented Nov 19, 2025

요약:

  • 에러 상세화

변경 사항:

  1. 새로운 에러 타입 정의:

    • src/error.rs 파일 추가
    • BraillifyError 열거형 정의:
      • InvalidCharacter: 일반적인 문자 오류
      • InvalidKoreanCharacter: 한글 문자 오류
      • InvalidKoreanChoseong: 초성 오류
      • InvalidKoreanJungseong: 중성 오류
      • InvalidKoreanJongseong: 종성 오류
      • InvalidKoreanPart: 한글 분리형 문자 오류
      • InvalidEnglishCharacter: 영문자 오류
      • InvalidNumberCharacter: 숫자 오류
      • InvalidSymbolCharacter: 기호 오류
      • InvalidMathSymbolCharacter: 수학 기호 오류
      • InvalidShortcutCharacter: 단축 문자 오류
      • InvalidFractionPart: 분수 부분 오류
      • InputTooLong: 입력 길이 초과 오류
      • FractionParseError: 분수 파싱 오류
      • Other: 기타 오류
  2. 모든 관련 함수들의 리턴 타입 변경:

    • Result<T, String> → Result<T, BraillifyError>로 변경
    • lib.rs, char_struct.rs, english.rs, number.rs, jauem/choseong.rs,
      jauem/jongseong.rs, moeum/jungsong.rs, symbol_shortcut.rs, korean_part.rs,
      math_symbol_shortcut.rs, split.rs, korean_char.rs, rule.rs, fraction.rs,
      char_shortcut.rs 파일 수정
  3. 상세 에러 메시지 구현:

    • 각 에러 발생 시 구체적인 문자, 위치 정보 포함
    • 예: “Invalid Korean character ‘x’ at position 5”와 같은 형태
  4. 모듈 구조 개선:

    • lib.rs에 mod error; 추가
    • 각 모듈에서 BraillifyError를 사용하기 위한 use 문 추가

@owjs3901
Copy link
Contributor

CICD를 통과해야 합니다, anyhow를 도입할까 생각하고 있습니다

@irhdab irhdab closed this Nov 21, 2025
@owjs3901
Copy link
Contributor

@irhdab CICD만 통과하면 병합이 가능할 것 같습니다, 아이디어와 코드가 둘 다 좋다고 생각합니다

@irhdab irhdab reopened this Nov 23, 2025
Corrected a minor formatting issue in the README. (bolding issue)
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 this pull request may close these issues.

2 participants