Оптимизация операторов "else if"

Друзиа, "обращаюсь не корысти ради, а токмо воли пославшей мя жоны" (с)
В наследство достался блок кода, состоящий из пары сотен операторов "switch" и "else if"
Что-то типа такого:

Код (Text):

1. void detectError(int* errCode)
2. {
3.          char * error;
4.     *errCode = WSAGetLastError();
5.  
6.     if ( *errCode == WSANOTINITIALISED )
7.         // error = '"";
8.     else if ( *errCode == WSAENETDOWN )
9.         // error = '"";
10.     else if ( *errCode == WSAEAFNOSUPPORT )
11.         // error = '"";
12. ..........
13.         много else if

Как можно избавиться от этого ужоснаха, оптимизировать код, увеличить скорость исполнения штатными средствами, без использования сторонних либ типа Boost'a?
Буду очень признателен любым дельным предложениям и пинкам в нужную сторону.

 

gringoz0id
Если оптимизировать по скорости, то этот код лучше вообще не трогать. Или ошибок так много, что их обработка самое узкое место программы?!

 

Black_mirror
Ошибок немного на самом деле. Эта функция является частью системы логирования и автор, видимо, хотел предусмотреть все возможные варианты.
Mikl___

Это Boost За ссылку спасибо, изучаю

 

gringoz0id

Ну и зачем ее тогда оптимизировать, если и ошибки возникают редко и само логирование наверняка пожирает намного больше времени, чем все эти ифы? Если эта функция должна обрабатывать не только ошибки, но и штатные отказы типа WSAEWOULDBLOCK и т.п., то их можно проверять в первую очередь с выходом по return

 

Юзать мапу, ключ - код ошибки, значение - указатель на функцию обработчик, лучше лямбду.

 

Booster
Спасибо за подсказку
Mikl___

www.boost.org

 

Самый быстрый - завернуть каждую ветку ELSEIF в функцию, сделать таблицу указателей на функции, где errCode - порядковый номер элемента в таблице, потом читать указатель из таблицы и вызывать по нему функцию. Если эти коды сильно разреженные, хреначим таблицу соответствий "код ошибки - указатель на функцию", сортируем и ищем в ней бинарным поиском.

 

А разве switch/case в большинстве компиляторов не сделает подобную же таблицу переходов? В условиях ведь сравнение на равенство константам, вполне ведь можно заменить на switch/case?

 

gringoz0id

1. Расскажи, в чем именно тут заключается ужоснах?
2. Зачем оптимизировать этот код на скорость? Ты выполняешь его 50`000`000 раз в секунду?
3. Ты не поверишь, но самый разумный способ, это таки воспользоваться бустом.

 

_DEN_
Ты под кайфом или как? Я думаю человек сам разберётся зачем ему оптимизировать и что использовать.

 

Booster

Нет, все кончилось

Вопрос ТС аналогичен вопросу "Посоны, у меня на первой передаче после 100 км/ч мотор чото сильно греется. Не знаете почему?".

 

Раунд № 1. По углам как обычно Ден и Бустер
По теме: покажите, пожалуйста, типичный для данной программы вызов этой ф-ии. + учитывая то, что она работает с ошибками сети, то сколько должно быть этого много else if, чтобы они стали заметны по времени выполнения, даже если detectError вызывается после каждого вызовы сетефой ф-ии.

 

В первую очередь надо подумать в сторону уже готового hashmap...

Ну если первое почему-то не помогло, то

Код (Text):

1. char** textTable;
2. size_t textTableSize;
3.  
4. textTableSize = MAX_ERROR;
5. textTable = malloc(MAX_ERROR*sizeof(char*));
6. for(size_t i=0; i<MAX_ERROR; ++i)
7. {
8.    textTable[i] = NULL;
9. }
10. textTable[WSANOTINITIALISED] = "text1";
11. textTable[WSAENETDOWN] = "text2";
12.  
13. ...
14. ...
15. ...
16.  
17. *errCode = WSAGetLastError();
18. if (*errCode < textTableSize)
19. {
20.    error = textTable[*errCode];
21.    if (NULL == error)
22.    {
23.       error = "Unknown error";
24.    }
25. }
26. else
27. {
28.    error = "Unknown error";
29. }

Если вдруг размер массива не устраивает, то можно попробовать разбить на 2 или более массивов, либо взять errCode по модулю, выбрав его таким образом, чтобы не было коллизий.

 

Booster
100gold
Благодарствую еще раз, буду ковырять в указанную сторону

 

gringoz0id

Зацени:

Код (Text):

1. #include <windows.h>
2. #include <iostream>
3. #include <boost/system/error_code.hpp>
4.  
5. int main()
6. {
7.     CreateWindowA(0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0);
8.     DWORD le = GetLastError();
9.     std::cout << boost::system::error_code(le, boost::system::get_system_category()).message() << std::endl;
10.     return 0;
11. }

 

_DEN_
Я тебя ща разочарую - в этом проекте по требованию заказчика не допускается использование сторонних библиотек. Вообще никаких. Даже MFC считается external dependency. Я бы рад использовать библиотеки типа буста, но нельзя.
Посему мне жутко любопытно почему моя фраза: "без использования сторонних либ типа Boost'a" воспринялась тобой как "Ты не поверишь, но самый разумный способ, это таки воспользоваться бустом"?
Что в данной фразе для тебя было непонятно?
Если не понял, повторюсь - без использования сторонних библиотек, это все-таки, согласно семантике русского языка, означает "без использования". Как здесь можно было найти потайной смысл, хз.
P.S. Крайне нехотя согласились на использование std::map, посмотрим, что получится, хотя видимо придется остановится на варианте, который предложил 100gold

 

Dmitry_Milk

А вот хрен его знает - стандарт оптимизацию не гарантирует, а всё, что не в стандарте, может быть реализовано "как фишка ляжет".

 

gringoz0id

Тогда и я тебя разочарую - Boost.System вошла в STL три года назад. Или ты все еще пишешь на стандарте 10-ти летней давности?

Лол, ну конечно, MFС - это самая ближайшая библиотека к C++.

Библиотек типа буста нет.

См. выше, C++2009.