oss2024hnu / coursegraph-py

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

validation 기능은 yaml 형식을 검사하는 schema_checker에 집중화되어 있어야지 다른 모듈에 분산하는 것은 좋지 않습니다. #557

Open kyagrd opened 2 weeks ago

kyagrd commented 2 weeks ago

애초에 yaml 파일을 읽어들일 때 제대로 된 형식인지 한곳에 모든 검사 로직을 모아놓아 일종의 실행가능한 스펙으로 활용하자고 yaml 스미카 관련 모듈이 있는건데

예를 들면 show_dot 에도 무슨 validate 이런 거 따로 작성하는 것은 코드 중복입니다.

모두 제거하고 만일 추가 기능이나 미비한 검사가 있어 개선할 필요가 있다면 yaml 스키마 모듈에 관련 기능을 정리해야 합니다.

이곳저곳에 분산된 validation 관련 내용은 모두 불필요하고 여러 곳에서 서로 다른 검사를 하면 혹시라도 기능 개선이나 변경시 sync가 되지 않아 오히려 코드 관리가 어려워지고 오류의 원인이 될 수 있습니다.

그래서 schema_checker.py 외의 다른 곳에서 여기서 처리해야 할 기능을 중구난방으로 부분 부분 반복하고 있는 코드들은 모두 제거하고 정리해야 합니다.