oss2024hnu / coursegraph-py

BSD 2-Clause "Simplified" License
5 stars 69 forks source link

schema_checker.py 불필요 코드 개선 #474

Closed seonghun0825 closed 3 months ago

seonghun0825 commented 3 months ago

466 에서

# 과목명,트랙,마이크로디그리,선수과목에는 문자만 사용되었는지 확인하는 함수
def validate_string_or_sequence(value : str) -> str:
    """
    과목명, 트랙, 마이크로디그리, 선수 과목에 문자만 사용되었는지 확인하는 함수입니다.

    parameter:
    value(str) : 과목 하나당 문자열

    return:
    값이 유효하다면 value를 반환학고, 그렇지 않다면 ValueError 를 발생시킵니다.
    """
    if value is None or value ==  None:
        value
        return value

불필요한 조건문과 리턴 값 제거, validate_string_or_sequence 함수의 유효성 검사 로직을 단순화 하였습니다. 추가로 사용하지 않는 sys 모듈을 import하지 않게 수정하고 오류 메시지를 명확하게 작성할 수 있게 수정하였습니다.

그리고 전체적인 오류메시지를 개선하고 코드의 가독성과 유지보수성을 높였습니다. 최종 수정본은 bcbbac49a691a27a6e655b4c61720cce2736115a 입니다.

kyagrd commented 3 months ago

충돌이 나고 이씁니다

seonghun0825 commented 3 months ago

다시 수정 했습니다. b0f3c56