epoll函数中设置 events[i].data.fd = -1的作用是什么?

时间:2022-05-30 19:13:02
int main()
{
  int i, maxi, listenfd, new_fd, sockfd,epfd,nfds;
  ssize_t n;
  char line[MAXLINE];
  socklen_t clilen;
  struct epoll_event ev,events[20];//ev用于注册事件,数组用于回传要处理的事件
  struct sockaddr_in clientaddr, serveraddr;
  listenfd = socket(AF_INET, SOCK_STREAM, 0);//生成socket文件描述符
  setnonblocking(listenfd);//把socket设置为非阻塞方式
   
  epfd=epoll_create(256);//生成用于处理accept的epoll专用的文件描述符
  ev.data.fd=listenfd;//设置与要处理的事件相关的文件描述符
  ev.events=EPOLLIN|EPOLLET;//设置要处理的事件类型
  epoll_ctl(epfd,EPOLL_CTL_ADD,listenfd,&ev);//注册epoll事件
    //设置服务器端地址信息
  bzero(&serveraddr, sizeof(serveraddr));
  serveraddr.sin_family = AF_INET;
  char *local_addr= LOCAL_ADDR;
  inet_aton(local_addr,&(serveraddr.sin_addr));
  serveraddr.sin_port=htons(SERV_PORT);
  bind(listenfd,(sockaddr *)&serveraddr, sizeof(serveraddr));//绑定socket连接
  listen(listenfd, LISTENQ);//监听
  maxi = 0;
  for ( ; ; )
      {
         /* epoll_wait:等待epoll事件的发生,并将发生的sokct fd和事件类型放入到events数组中;
          * nfds:为发生的事件的个数。
          * 注:
         */
      nfds=epoll_wait(epfd,events,20,500);
      //处理所发生的所有事件
      for(i=0;i<nfds;++i)
      {
          if(events[i].data.fd==listenfd)//事件发生在listenfd上
          {
               /* 获取发生事件端口信息,存于clientaddr中;
               *new_fd:返回的新的socket描述符,用它来对该事件进行recv/send操作*/
              new_fd = accept(listenfd,(struct sockaddr *)&clientaddr, &clilen);
              if(connfd<0)
                   {
                  perror("connfd<0");
                  exit(1);
              }
              setnonblocking(connfd);
              char *str = inet_ntoa(clientaddr.sin_addr);
              ev.data.fd=connfd;//设置用于读操作的文件描述符
              ev.events=EPOLLIN|EPOLLET;//设置用于注测的读操作事件
              epoll_ctl(epfd,,,&ev);//注册ev
          }
          else if(events[i].events&EPOLLIN)
          {
              if ( (sockfd = events[i].data.fd) < 0)
                       continue;
              if ( (n = read(sockfd, line, MAXLINE)) < 0)
                   {
                  if (errno == ECONNRESET)
                      {
                      close(sockfd);
                      events[i].data.fd = -1;
                  }
                      else
                      std::cout<<"readline error"<<std::endl;
              }
                  else if (n == 0)
                  {
                  close(sockfd);
                  events[i].data.fd = -1;
             }
             ev.data.fd=sockfd;//设置用于写操作的文件描述符
             ev.events=EPOLLOUT|EPOLLET;//设置用于注测的写操作事件
             epoll_ctl(epfd,,sockfd,&ev);//修改sockfd上要处理的事件为EPOLLOUT
        }
       else if(events[i].events&EPOLLOUT)
       {
           sockfd = events[i].data.fd;
            write(sockfd, line, n);
  
            ev.data.fd=sockfd;//设置用于读操作的文件描述符
            ev.events=EPOLLIN|EPOLLET;//设置用于注测的读操作事件
            epoll_ctl(epfd,,sockfd,&ev);//修改sockfd上要处理的事件为EPOLIN
       }
   }
 }
}

这个代码中,检查下面的是否为小于0,和设置 events[i].data.fd = -1;,有什么作用!请大神解答一下
 if ( (sockfd = events[i].data.fd) < 0)
                       continue;
  if (errno == ECONNRESET)
                      {
                      close(sockfd);
                      events[i].data.fd = -1;
                  }
                      else
                      std::cout<<"readline error"<<std::endl;
              }
                  else if (n == 0)
                  {
                  close(sockfd);
                  events[i].data.fd = -1;
             }

2 个解决方案

#1


就是把已经关闭的fd在events中设成-1而已,这样在下次判断events[i].data.fd时就不会用到已经关闭的fd了。
比如 if(events[i].data.fd==listenfd)和if ( (sockfd = events[i].data.fd) < 0)

#2


引用 1 楼 wangzuxi 的回复:
就是把已经关闭的fd在events中设成-1而已,这样在下次判断events[i].data.fd时就不会用到已经关闭的fd了。
比如 if(events[i].data.fd==listenfd)和if ( (sockfd = events[i].data.fd) < 0)

为什么我觉得这个完全是没有必要的,因为每次epoll之后,返回的events都是就绪的,其中的fd不可能是-1.,同时epoll返回就绪的events数组个数nfds。这nfds个events元素都是就绪的,其event[I].fd可能是-1呢?

不知道我的理解哪里错误了

#1


就是把已经关闭的fd在events中设成-1而已,这样在下次判断events[i].data.fd时就不会用到已经关闭的fd了。
比如 if(events[i].data.fd==listenfd)和if ( (sockfd = events[i].data.fd) < 0)

#2


引用 1 楼 wangzuxi 的回复:
就是把已经关闭的fd在events中设成-1而已,这样在下次判断events[i].data.fd时就不会用到已经关闭的fd了。
比如 if(events[i].data.fd==listenfd)和if ( (sockfd = events[i].data.fd) < 0)

为什么我觉得这个完全是没有必要的,因为每次epoll之后,返回的events都是就绪的,其中的fd不可能是-1.,同时epoll返回就绪的events数组个数nfds。这nfds个events元素都是就绪的,其event[I].fd可能是-1呢?

不知道我的理解哪里错误了