Как можно архитектурно зарефакторить такой код? switch incomeEvent.Action { case models.CALL:

Question

Как можно архитектурно зарефакторить такой код? switch incomeEvent.Action { case models.CALL:

switch incomeEvent.Method {
case models.GET:
getRequestHandler(incomeEvent)
case models.POST:
postRequestHandler(incomeEvent)
}
case models.SUBSCRIBE:
switch incomeEvent.Method {
case models.POST:
postSubscribeHandler(incomeEvent)
}
}
P.S. Внутри этих методов еще switch.
Полиморфизм? Регистрация колбеков? Мб еще варианты есть? В golang новичок. Такая вложенность, еще и раскиданная по файлам мне не нравится

#backend #go #programming #russian

0

02.12.2017

2 ответов

14 просмотров

Александр Парамонов Автор вопроса

Попробовал сделать. Мне не нравится, много действий нужно сделать. Но сам switch стал хотя бы читаемый type IncomingHandlerPool struct { messages MessagesHandler } type MessagesHandler interface { GetMessages(event models.IncomeEvent) GetChatsCount(event models.IncomeEvent) SendMessage(event models.IncomeEvent) MarkRead(event models.IncomeEvent) NotifyTyping(event models.IncomeEvent) } const ( requestMessages = "request_messages" requestChatsCount = "request_chats_count" sendMessage = "send_message" markRead = "mark_read" notifyTyping = "notify_typing" ) var ( handlerKeys = map[string]string{ requestMessages: handleKey(models.CALL, models.GET, models.TYPE_MESSAGE), requestChatsCount: handleKey(models.CALL, models.GET, models.TYPE_COUNTS), sendMessage: handleKey(models.CALL, models.POST, models.TYPE_MESSAGE), markRead: handleKey(models.CALL, models.POST, models.TYPE_READ), notifyTyping: handleKey(models.CALL, models.POST, models.TYPE_TYPING), } ) // ... switch handleKey(incomeEvent.Action, incomeEvent.Method, incomeEvent.Type) { case requestMessages: handlers.messages.GetMessages(incomeEvent) case requestChatsCount: handlers.messages.GetChatsCount(incomeEvent) case sendMessage: handlers.messages.SendMessage(incomeEvent) case markRead: handlers.messages.MarkRead(incomeEvent) case notifyTyping: handlers.messages.NotifyTyping(incomeEvent) } // ... func handleKey(a int, m string, t string) string { return fmt.Sprintf("%s-%s-%s", a, m, t) }

0

02.12.2017

Пользователь 6176c · Accepted Answer

Пользователь 6176c

выглядит как шаблон observer https://gist.github.com/sayden/d3ab61726057b7a37963

0

02.12.2017

170 похожих чатов

Как можно архитектурно зарефакторить такой код? switch incomeEvent.Action { case models.CALL:

2 ответов

Похожие вопросы