Все три case`a функционально одинаковы, во 2-м и 3-м варианте методы SendData() и RecvData() объединены в один SendRecv(). Какой код обладает наибольшей читаемостью? Мое мнение: 1. много if/else 2. много точек выхода 3. оптимизация в ущерб читаемости CmdStatus = CMD_EXEC; switch(Command) { ... case CommandSetMotoHourLimit: if(SendData() == RET_OK) { if(RecvData() == RET_OK) { if(Data.Command == CommandReceived) CmdStatus = CMD_COMPLETE; else CmdStatus = CMD_ERROR; } else CmdStatus = CMD_ERROR; } else CmdStatus = CMD_ERROR; break; case 1000: if(SendRecv() != RET_OK) { CmdStatus = CMD_ERROR; break; } if(Data.Command != CommandReceived) { CmdStatus = CMD_ERROR; break; } CmdStatus = CMD_COMPLETE; break; case 1001: if(SendRecv() == RET_OK && Data.Command == CommandReceived) CmdStatus = CMD_COMPLETE; else CmdStatus = CMD_ERROR; break; }
Ответ на:
комментарий
от bugmaker
Ответ на:
комментарий
от anonymous
Ответ на:
комментарий
от Sectoid
Вы не можете добавлять комментарии в эту тему. Тема перемещена в архив.
Похожие темы
- Форум [c++] Оптимизация кода работы с байтами (2011)
- Форум Хотелось бы узнать Ваше мнение по поводу читаемости кода (2018)
- Форум C Оптимизация алгоритма чтение данных с компорта (2011)
- Форум FOSS-средства анализа читаемости текста (2015)
- Форум как проверить читаемость файлов копированием в null? (2016)
- Форум Мы нашли очередную порцию глюков в Linux Kernel (2016)
- Форум Код (2012)
- Галерея Кодю... (2009)
- Форум Код (2010)
- Галерея Кодю :) (2006)