Вот мой код:
#include <stdio.h>
#include <string.h>
#include <stdlib.h>
#include <netdb.h>
#include <sys/types.h>
#include <sys/socket.h>
#include <arpa/inet.h>
int main (void) {
struct addrinfo hints;
memset (&hints, 0, sizeof hints);
hints.ai_family = AF_UNSPEC;
hints.ai_socktype = SOCK_DGRAM;
hints.ai_flags = AI_CANONNAME;
struct addrinfo *res;
getaddrinfo ("example.com", "http", &hints, &res);
printf ("Host: %s\n", "example.com");
void *ptr;
while (res != NULL) {
printf("AI Family for current addrinfo: %i\n", res->ai_family);
switch (res->ai_family) {
case AF_INET:
ptr = (struct sockaddr_in *) res->ai_addr;
struct sockaddr_in *sockAddrIn = (struct sockaddr_in *) res->ai_addr;
break;
}
res = res->ai_next;
}
return 0;
}
который компилируется нормально.
Однако, когда я комментирую эту строку:
//ptr = (struct sockaddr_in *) res->ai_addr;
Я получу:
$ gcc ex4.c
ex4.c:30:9: error: expected expression
struct sockaddr_in *sockAddrIn = (struct sockaddr_in *) res->ai_addr;
^
1 error generated.
Что мне не хватает?
case
(без окружающих фигурных скобок, как было предложено в верхнем ответе) - плохая идея, потому что тогда имя переменной будет видно позже,case
но оно будет неинициализированным (если вы не упали).Ответы:
Технически говоря, каждый case в операторе switch является меткой. По некоторым неясным и старым причинам вам не разрешается указывать объявление переменной в качестве первой строки после метки. Комментируя задание
линия
struct sockaddr_in *sockAddrIn = (struct sockaddr_in *) res->ai_addr;
становится первой строкой после метки,
AF_INET:
что, как я уже сказал, недопустимо в C.Решение состоит в том, чтобы заключить все операторы case в фигурные скобки, например:
#include <stdio.h> #include <string.h> #include <stdlib.h> #include <netdb.h> #include <sys/types.h> #include <sys/socket.h> #include <arpa/inet.h> int main (void) { struct addrinfo hints; memset (&hints, 0, sizeof hints); hints.ai_family = AF_UNSPEC; hints.ai_socktype = SOCK_DGRAM; hints.ai_flags = AI_CANONNAME; struct addrinfo *res; getaddrinfo ("example.com", "http", &hints, &res); printf ("Host: %s\n", "example.com"); void *ptr; while (res != NULL) { printf("AI Family for current addrinfo: %i\n", res->ai_family); switch (res->ai_family) { case AF_INET: { ptr = (struct sockaddr_in *) res->ai_addr; struct sockaddr_in *sockAddrIn = (struct sockaddr_in *) res->ai_addr; break; } } res = res->ai_next; } return 0; }
В любом случае, я думаю, что это лучшая практика кодирования.
источник
В дополнение к принятому ответу вы можете объявить свои переменные перед метками case.
switch(a) { int b; //can't initialize variable here case 0: ... }
Или просто используйте пустой оператор.
источник