search

pineoc / pino-bot

for pino performance
1 stars 1 forks source link

slack 메세지에서 명령어 확인하는 부분 개선 #12

Closed pineoc closed 5 years ago

pineoc commented 6 years ago

구현 내용

문제 부분

Complexity(복잡도)가 11으로 나타나고 있음. 9 이하로 줄여보자.

https://github.com/pineoc/pino-bot/blob/c91498deeef45c2629437bbb15404c45fe703351/routes/slack.js#L44

pineoc commented 6 years ago

https://github.com/tj/commander.js 커맨드를 잘 나눠서 작성할 수 있도록 하는 라이브러리도 있었다!

pineoc commented 5 years ago

command.js로 정리해서 꺼냈다. (복잡도는 줄였으나 더 정리할 필요는 있을 것)

일단 고민하던 내용은 해결했다.