binchoo / selenium-kakao-vaccine-reservation

[PC / GUI /실행파일] 카카오 백신 예약. UX 개선.
GNU General Public License v3.0
13 stars 4 forks source link

[개선] `view.updateButtons`가 쿠키/지역/매크로 수행 인터벌 값을 알아야 하는 것은 이상하다. #27

Closed binchoo closed 3 years ago

binchoo commented 3 years ago

현재의 불편한 상황

view.updateButtons의 시그니쳐가 cookie, region, running을 요구한다. 뷰는 자신이 어떻게 보여질 지만 알면 된다. 서비스에 사용될 변수들을 알 필요가 없다. https://github.com/binchoo/selenium-kakao-vaccine-reservation/blob/59c643ab5ef20cda96d74b76dd21db67d70915c7/view/main.py#L63-L78

개선 내용 및 방향

view.updateButtons(button1_enabled, button2_enabled2)으로 변경할 것.

이런 식으로

macro_runnable = controller.isMacroRunnable()
if macro_runnable and not running:
  view.updateButtons(True, False) # 시작 버튼만 활성화하기

참조

binchoo commented 3 years ago

13 수정과 함께 시그니쳐가 수정된 모양이다.

To-Be대로 바뀌진 않았다. 나중에 이게 맘에 들지 않으면 이슈 리오픈 합시다.