eubinecto / politely

A hybrid politeness styler for the Korean Language / 하이브리드 존댓말 변환기
https://politely.streamlitapp.com
49 stars 0 forks source link

processors.py -> styler.py / translator.py / explainer.py 로 분리하기 #72

Closed eubinecto closed 2 years ago

eubinecto commented 2 years ago

Why?

왜 굳이 processors.py 하나에 묶어야 할까?

그냥 transformers 라이브러리처럼... project의 __init__.py 에다가 임포트를 해서 쓸 거라면...
image

processors.py에 정의를 하던, 따로 정의를 하던, 결국 사용을 할 때는

from politely import Styler,  Explainer, Translator
...

이렇게 쓰게 되는 것에는 큰 변함이 없다.

그렇다면... 일일히 밑으로 스크롤을 내려야하는 all in processors.py 구조 보다는, 빠르게 클래스를 확인해볼 수 있는 구조가 더 나을 것이다.

eubinecto commented 2 years ago

아, 다만 한가지 문제점.

그렇게 분리를 하게되면, 이렇게 알파벳 순으로 정렬했을 때 한 곳으로 모이지 않게된다
image

아, 뭐... 그런 aesthetics는 크게 신경쓰지 말자.. 어차피 너가 자바를 썼다면 신경 안 썼을 거잖아?

그리고 tweepy 처럼 쿨한 라이브러리도 많다
image

사소한 것에 신경쓰지 말자!

eubinecto commented 2 years ago
init.py에 이렇게 import를 해주면
image
이렇게 바로 import를 할 수 있다
image

즉 프로젝트 구조는 그냥 무지성으로 잡아도, init만 잘해두면 사실상 사용성에는 큰 문제는 없다는 것.

그냥 이렇게 하자!