Переписать готовую функцию более элегантно

@kquick · Регистрация: 15.05.2014

Студворк — интернет-сервис помощи студентам

Мое понимание элегантной функции: записано наиболее коротко, понятно, ничего лишнего, работает наиболее оптимальным образом не в ущерб читаемости и пониманию.

Например есть функция, которая возвращает результат сложения двух чисел:

C++
1
2
3
4
5
6
int foo(int a,int b) {
  int c;
  c=a;
  c=c+b;
  return c;
}

Данная функция работает, но записана слишком не элегантно. Та же функция, переписанная более элегантно:

C++
1
2
3
int sum(int a, int b) {
  return a+b;
}

Я написал функцию, которая проверяет наличие хоста в сети, но, как мне кажется, сделал это не элегантно(много лишних return и манипуляций с памятью). Помогите переписать более элегантно:

Кликните здесь для просмотра всего текста

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
int pingHOST(std::string HostString) {
    // Declare and initialize variables
    WSADATA wsaData;
    struct hostent *remoteHost;
 
    HANDLE hIcmpFile;
    char SendData[] = "ping";
    DWORD ReplySize = sizeof(ICMP_ECHO_REPLY) + sizeof(SendData);
    char *ReplyBuffer = new char[ReplySize];
    unsigned long ipaddr;
 
    // Initialize Winsock
    if(WSAStartup(MAKEWORD(2, 2), &wsaData)) {
        delete [] ReplyBuffer;
        return 0; // WSAStartup failed
    }
 
    if( (remoteHost = gethostbyname(HostString.c_str())) == NULL) {
        delete [] ReplyBuffer;
        return 0; // gethostbyname Error
    };
    
    if(remoteHost->h_addrtype != AF_INET) {
        // ! delete remotehost ?
        delete [] ReplyBuffer;
        return 0;
    }
 
    if(remoteHost->h_addr_list[0] != 0) {
        ipaddr = *(u_long *) remoteHost->h_addr_list[0];
 
        // ping start
        if ((hIcmpFile = IcmpCreateFile()) != INVALID_HANDLE_VALUE) {
 
            if (ipaddr != INADDR_NONE) {
                if (IcmpSendEcho(hIcmpFile, ipaddr, SendData, sizeof(SendData), NULL, ReplyBuffer, ReplySize, 1000) != 0) {
                    PICMP_ECHO_REPLY pEchoReply = (PICMP_ECHO_REPLY)ReplyBuffer;
                    if( pEchoReply->Status == IP_SUCCESS ) {
                        // ! delete remotehost ?
                        delete [] ReplyBuffer;
                        IcmpCloseHandle(hIcmpFile);
                        return 1;
                    } else {
                        // ! delete remotehost ?
                        delete [] ReplyBuffer;
                        IcmpCloseHandle(hIcmpFile);
                        return 0;
                    }
                } else {
                    // ! delete remotehost ?
                    delete [] ReplyBuffer;
                    IcmpCloseHandle(hIcmpFile);
                    return 0;
                }
            } else {
                // ! delete remotehost ?
                delete [] ReplyBuffer;
                IcmpCloseHandle(hIcmpFile);
                return 0;
            }
 
        } else {
            // ! delete remotehost ?
            delete [] ReplyBuffer;
            return 0;
        }
        // ping end
 
    } else {
        // ! delete remotehost ?
        delete [] ReplyBuffer;
        return 0;
    }
}

@schdub · 02.02.2015, 18:59

kquick, для начала, можно не выделять динамически ReplyBuffer

C++
1
    CHAR  ReplyBuffer[sizeof(ICMP_ECHO_REPLY) + sizeof(SendData)];

и вынести вызов WSAStartup() из функции - достаточно вызвать единожды.

@kquick · 02.02.2015, 19:11 **[ТС]**

Получилось так:

Кликните здесь для просмотра всего текста

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
int pingHOST(std::string HostString) {
    // Declare and initialize variables
    struct hostent *remoteHost;
 
    HANDLE hIcmpFile;
    char SendData[] = "ping";
    DWORD ReplySize = sizeof(ICMP_ECHO_REPLY) + sizeof(SendData);
    char ReplyBuffer[sizeof(ICMP_ECHO_REPLY) + sizeof(SendData)];
    unsigned long ipaddr;
 
 
    if( (remoteHost = gethostbyname(HostString.c_str())) == NULL) {
        return 0; // gethostbyname Error
    };
    
    if(remoteHost->h_addrtype != AF_INET) {
        return 0;
    }
 
    if(remoteHost->h_addr_list[0] != 0) {
        ipaddr = *(u_long *) remoteHost->h_addr_list[0];
        if ((hIcmpFile = IcmpCreateFile()) != INVALID_HANDLE_VALUE) {
            if (ipaddr != INADDR_NONE) {
                if (IcmpSendEcho(hIcmpFile, ipaddr, SendData, sizeof(SendData), NULL, ReplyBuffer, ReplySize, 1000) != 0) {
                    PICMP_ECHO_REPLY pEchoReply = (PICMP_ECHO_REPLY)ReplyBuffer;
                    if( pEchoReply->Status == IP_SUCCESS ) {
                        IcmpCloseHandle(hIcmpFile);
                        return 1;
                    }
                }
            } 
            IcmpCloseHandle(hIcmpFile);
        }
    }
 
    return 0;
}

Еще элегантнее ?

@schdub · 02.02.2015, 19:17

kquick,

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
BOOL pingHOST(const std::string & HostString) {
 
    // Initialize Winsock
    WSADATA wsaData;
    if (WSAStartup(MAKEWORD(2, 2), &wsaData)) {
        return FALSE; // WSAStartup failed
    }
 
    struct hostent *remoteHost = gethostbyname(HostString.c_str());
    if (remoteHost == NULL ||
        remoteHost->h_addrtype != AF_INET ||
        remoteHost->h_addr_list[0] == NULL) {
        return FALSE;
    }
    IPAddr ipaddr = *(u_long *) remoteHost->h_addr_list[0];
 
    // ping start
    HANDLE hIcmpFile = IcmpCreateFile();
    if (hIcmpFile == INVALID_HANDLE_VALUE) {
        return FALSE;
    }
 
    static CHAR SendData[] = "ping";
    DWORD ReplySize = sizeof(ICMP_ECHO_REPLY) + sizeof(SendData);
    CHAR  ReplyBuffer[sizeof(ICMP_ECHO_REPLY) + sizeof(SendData)];
    BOOL ok = IcmpSendEcho(hIcmpFile, ipaddr, SendData, sizeof(SendData), NULL,
                           ReplyBuffer, ReplySize, 1000) != 0;
    if (ok) {
        PICMP_ECHO_REPLY pEchoReply = (PICMP_ECHO_REPLY)ReplyBuffer;
        ok = pEchoReply->Status == IP_SUCCESS;
    }
 
    IcmpCloseHandle(hIcmpFile);
 
    return ok;
}

@kquick · 02.02.2015, 19:18 **[ТС]**

Еще можно вложенные if'ы объединить в принципе через &&.

@schdub · 02.02.2015, 19:30

kquick,

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
BOOL pingHOST(const std::string & HostString) {
    struct hostent *remoteHost = gethostbyname(HostString.c_str());
    if (remoteHost == NULL ||
        remoteHost->h_addrtype != AF_INET ||
        remoteHost->h_addr_list[0] == NULL) {
        return FALSE;
    }
 
    HANDLE hIcmpFile = IcmpCreateFile();
    if (hIcmpFile == INVALID_HANDLE_VALUE) {
        return FALSE;
    }
 
    static CHAR SendData[] = "ping";
    CHAR ReplyBuffer[sizeof(ICMP_ECHO_REPLY) + sizeof(SendData)];
    PICMP_ECHO_REPLY pEchoReply = (PICMP_ECHO_REPLY)ReplyBuffer;
    IPAddr ipaddr = *(u_long *) remoteHost->h_addr_list[0];
    BOOL ok = IcmpSendEcho(hIcmpFile, ipaddr,
        SendData, sizeof(SendData), NULL,
        ReplyBuffer, sizeof(ReplyBuffer), 1000) != 0;
    IcmpCloseHandle(hIcmpFile);
 
    return ok == TRUE && pEchoReply->Status == IP_SUCCESS;
}

Новые блоги и статьи Все статьи Все блоги /
Новый ноутбук volvo 07.12.2025 Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro	Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .	От async/await к виртуальным потокам в Python IndentationError 23.11.2025 Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .	Поиск "дружественных имён" СОМ портов Argus19 22.11.2025 Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .	Сколько Государство потратило денег на меня, обеспечивая инсулином. Programma_Boinc 20.11.2025 Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .
Ломающие изменения в C#.NStar Alpha Etyuhibosecyu 20.11.2025 Уже можно не только тестировать, но и пользоваться C#. NStar - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .	Мысли в слух kumehtar 18.11.2025 Кстати, совсем недавно имел разговор на тему медитаций с людьми. И обнаружил, что они вообще не понимают что такое медитация и зачем она нужна. Самые базовые вещи. Для них это - когда просто люди. . .	Создание Single Page Application на фреймах krapotkin 16.11.2025 Статья исключительно для начинающих. Подходы оригинальностью не блещут. В век Веб все очень привыкли к дизайну Single-Page-Application . Быстренько разберем подход "на фреймах". Мы делаем одну. . .	Фото: Daniel Greenwood kumehtar 13.11.2025	Расскажи мне о Мире, бродяга kumehtar 12.11.2025 — Расскажи мне о Мире, бродяга, Ты же видел моря и метели. Как сменялись короны и стяги, Как эпохи стрелою летели. - Этот мир — это крылья и горы, Снег и пламя, любовь и тревоги, И бескрайние. . .

Переписать готовую функцию более элегантно

Решение

Решение

@kquick 6 / 6 / 5 Регистрация: 15.05.2014 Сообщений: 104
	02.02.2015, 19:18 [ТС]
	Еще можно вложенные if'ы объединить в принципе через &&. 0