Всем привет. Есть вопросик, по фабрике. Суть такая - из

Question

Всем привет. Есть вопросик, по фабрике. Суть такая - из

сокета приходит массив байтов(udp data). 7-ой байт - команда, на основании команды я могу определить наследника для конструирования. Другое дело, что наследников будет 35-40...Switch на 40 строк плохая идея по-моему, что посоветуете?
public static PacketBase FromByteArray(byte[] dataBuffer)
{
Command cmd = (Command) dataBuffer[6];
//here we must create instance based on cmd
}

#backend #dot #programming #russian

0

01.06.2017

3 ответов

7 просмотров

Ilya Chernoudov

лучше 100 строк понятного кода чем 50 строк фабрик и непонятных преобразований, когда switch будет недостаточно - тогда и заменишь на что нить получше

0

01.06.2017

Aleksander Khanteev

Альтернативный вариант - объявить у команды абстрактный метод IsApplicableTo(CommandType command): bool. В фабрике пройтись по списку команд, и взять применимую. Но с этим довольно сложно будет потом разбираться, словарь с enum'ами проще)

0

01.06.2017

Ilya Chernoudov · Accepted Answer

Ilya Chernoudov

юзай switch

0

01.06.2017

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

Всем привет. Есть вопросик, по фабрике. Суть такая - из

3 ответов

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