Все три 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)
- Форум Читаемость генераторов, развесистых лямбд и прочий write-only код... (2025)
- Форум Хотелось бы узнать Ваше мнение по поводу читаемости кода (2018)
- Форум C Оптимизация алгоритма чтение данных с компорта (2011)
- Форум FOSS-средства анализа читаемости текста (2015)
- Форум как проверить читаемость файлов копированием в null? (2016)
- Форум Мы нашли очередную порцию глюков в Linux Kernel (2016)
- Форум Код (2012)
- Форум Код (2010)
- Галерея Кодю... (2009)