1. ホーム
  2. c

[解決済み] Switch-caseが未使用行をコメントアウトした後にコンパイルされない

2023-06-26 05:25:35

質問

以下は私のコードです。

#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.

何が足りないのでしょうか?

どのように解決するのですか?

switch文の各ケースは、技術的に言えばラベルです。いくつかの 曖昧で古い理由 のため、ラベルの後の最初の行に変数宣言を記述することはできません。代入をコメントアウトすることで

ptr = (struct sockaddr_in *) res->ai_addr;

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;
}

とにかく、この方が良いコーディングのやり方だと思います。