Форум программистов [Powered by Invision Power Board]

Версия для печати темы
Нажмите сюда для просмотра этой темы в оригинальном формате

Форум программистов > C/C++: Сети > Проверка состояния при создании сокета.

Автор: tstr 28.6.2009, 05:24

Здравствуйте!
Кто как делает проверку состояния при создании сокетов? Как лучше smile

Код


/* Например, создаем сокет: */

    sock = socket(PF_INET, SOCK_STREAM, IPPROTO_IP);

/* ..а теперь проверяем состояние (вариант 1): */

    if (-1 == errno)
        fprintf(stderr, "*** Error: can't create socket\n"),
        exit(EXIT_FAILURE);

/* вариант 2: */

    if (-1 == sock)
        fprintf(stderr, "*** Error: can't create socket\n"),
        exit(EXIT_FAILURE);

/* вариант 3: */

    if (0 > errno)
        fprintf(stderr, "*** Error: can't create socket\n"),
        exit(EXIT_FAILURE);

/* вариант 4: */

    if (0 > sock)
        fprintf(stderr, "*** Error: can't create socket\n"),
        exit(EXIT_FAILURE);

Кто что думает по этому поводу? Спасибо smile

Автор: Andrew121 28.6.2009, 05:35

4 вариант.

Автор: tstr 28.6.2009, 05:40

А почему именно 4?
Так надежней?

Хотел бы услышать обоснования выбора.
Я думаю так: ведь функция может вернуть и не -1, а другое отрицательное число, поэтому лучше писать (0 > x), а переменная errno может быть случайно перезаписана.

Автор: Олег2005 28.6.2009, 20:31

Цитата(tstr @ 28.6.2009, 04:24

)

sock = socket(PF_INET, SOCK_STREAM, IPPROTO_IP);

Только не надо писать IPPROTO_IP в потоковом стиле взаимодействия
Или 0, или IPPROTO_TCP

Автор: Alca 29.6.2009, 12:12

http://msdn.microsoft.com/en-us/library/ms740506(VS.85).aspx

Цитата

socket Function

The socket function creates a socket that is bound to a specific transport service provider.
...
Return Value

If no error occurs, socket returns a descriptor referencing the new socket. Otherwise, a value of INVALID_SOCKET is returned, and a specific error code can be retrieved by calling WSAGetLastError.

Добавлено через 3 минуты и 43 секунды
вариант 2

Автор: tstr 29.6.2009, 15:03

Цитата

Только не надо писать IPPROTO_IP в потоковом стиле взаимодействия
Или 0, или IPPROTO_TCP

Время познее было - очепятался.. smile

Цитата

... a specific error code can be retrieved by calling WSAGetLastError

Надо не виндовый сокет, поэтому WSAGetLastError не поможет.
Но вот что я заметил, заглянув в /usr/src/sys/sys/errno.h:

Цитата

...
#define ENFILE                        23            /* Too many open files in system */
#define EMFILE                        24            /* Too many open files */
...
#define ESOCKTNOSUPPORT    44            /* Socket type not supported */
#define EPFNOSUPPORT          46            /* Protocol family not supported */
#define EAFNOSUPPORT          47            /* Address family not supported by protocol family */
...

Получается, что переменная errno может быть установлена в значение, например, 23 и,
видимо лучшим для отлова ошибки при создании сокета будут варианты проверок 2 и 4,
а для поиска конкретной причины ошибки, сравнивать переменную errno с предопределенными
в /usr/src/sys/sys/errno.h константами с помощью какого-нить switch-case...

Что скажете?

Автор: Alca 30.6.2009, 09:43

Код


if (-1 == sock) {
        printf("Socket failure with error: %i", errno);
        exit(EXIT_FAILURE);
}

Автор: Олег2005 30.6.2009, 19:01

В профессиональной программе надо проверять АБСОЛЮТНО все возможные ошибки!
В том числе и при создании сокета.
Проще сделать обертку для каждой сетевой функции - ну а внутри switch очень даже......

Автор: tstr 1.7.2009, 13:16

Цитата

Проще сделать обертку для каждой сетевой функции - ну а внутри switch очень даже......

Так и буду делать. Спасибо smile