注释掉未使用的行后,开关盒将无法编译


82

这是我的代码:

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

我想念什么?


也许这个问题的标题应该被编辑?如果同意的话,有经验的人可以这样做吗?
Koray Tugay 2015年

如果需要,您应该可以自己编辑它。但我同意,标题可能会更好。
QuestionC

@KorayTugay,我试了一下。
Paul Draper 2015年

1
在变量内部声明变量case(顶部答案中没有大括号)是一个不好的主意,因为这样变量的名称将在以后的cases中可见,但不会被初始化(除非您下拉列表)。
MM

Answers:


111

从技术上来讲,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;
}

无论如何,我认为这是更好的编码实践。


22
真好 细看“每种情况……都是贴有标签的陈述”。这就是原因:可以标记语句,但不能标记声明。
undur_gongor

4
@KorayTugay有时编译器消息没有提供足够的信息,因为我们希望它们是....有时它们提供的信息太多(咳嗽C ++ stl咳嗽)。
约翰M

5
@BlueMoon你是对的。当您可以购买简化C ++ stl错误消息的工具时,您就会知道信息密度非常低!
John M

3
或只是将声明移到切换块之前
Pavel Gatnar,2015年

3
@immibis:在C ++声明中是语句,这就是为什么在C ++中可以无限制地标记声明。在C中,声明不是语句,这就是为什么您不能标记它们。以下示例说明了C和C ++之间的这种区别:stackoverflow.com/a/19830820/187690
AnT 2015年

15

作为已接受答案的补充,您可以在案例标签之前声明变量。

switch(a) {
    int b; //can't initialize variable here
    case 0:
    ...
}

或者只是使用一个空语句。

By using our site, you acknowledge that you have read and understand our Cookie Policy and Privacy Policy.
Licensed under cc by-sa 3.0 with attribution required.