letsplayants / Ants-Auto-Trading-Bot

6 stars 6 forks source link

설정파일에서 사용할 거래소를 지정할 수 있도록 해야함 #24

Open MuYoul opened 5 years ago

MuYoul commented 5 years ago

현재 사용하지 않는 거래소까지 설정파일을 다 읽어들이는데, 초기화 할때 경고와 에러가 발생함.

동작에는 문제는 없지만 의도하지 않는 실수로 오동작하는 것을 막기 위해서 사용할 거래소를 지정할 필요가 있음

MuYoul commented 5 years ago

이 이슈는 거래소 인스턴스를 관리해주는 서비스 모듈을 생성하는 방향으로 설계를 수정

거래소 인스턴스를 API키마다 싱글톤으로 작성해서 관리하면 여러곳에서 호출해도 하나의 인스턴스로 호출되니 코드 관리의 편리성은 높아진다고 볼 수 있음

대신 이 경우 쓰레드 환경에 노출될 수 있으므로 거래소 함수를 작성할 때 멀티쓰레드 환경에 대비해서 작성해야하는 코드의 복잡성이 증가한다. 이는 파이썬 언어가 가지는 함수형 프로그래밍의 단순화 원칙에서 벗어 나는 것으로 생각되고 이는 잘못된 방향이 아닌가 고민해본다

아무튼 당장은 지원하는 모든 거래소를 다 넣어서 사용하는 방법으로 결정하고 이 이슈는 추후 작업으로 미룬다

MuYoul commented 5 years ago

MQ 시스템을 도입하여 해당 문제를 해결할 예정