Question

我正在尝试实现客户端与客户端之间的服务器通信。服务器的功能是当客户端假设客户端A向服务器发送消息时，服务器应该将该消息转发给其他客户端客户端B.同样，当客户端B向服务器发送消息时，应该转发该消息客户A.该计划仅涉及两个客户。我执行代码时得到的错误是：

Socket Operation on Non-socket

当收到来自客户A的消息转发给客户B时，我收到此错误。我认为问题是由于将客户B的收到地址存储到客户A的地址。我不确定那。

到目前为止我的服务器代码。

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <netinet/in.h>
#include <sys/socket.h>
#include <sys/types.h>
#include <arpa/inet.h>
#include <signal.h>
#include <unistd.h>
#include <errno.h>
#include <sys/time.h>
#define SERVER_PORT 5009


int main(){
unsigned int sockfd, c,c1,c2, clientlen, clientfd;
struct sockaddr_in server;
struct sockaddr_in client1;
int clientsocks[2];
char rmsg1[100], msg1[100],rmsg2[100], msg2[100];
char w_msg[] = "Connection to server established";

fd_set readfds; // For temp file descriptor list.

clientsocks[0] = 0 ;
clientsocks[1] = 0 ;
//Socket Creation Process.
sockfd = socket(AF_INET, SOCK_STREAM, 0);
if( sockfd < 0){
    perror("Socket cannot be created");
}

//For reusing the socket.
int on = 1;
if (setsockopt(sockfd, SOL_SOCKET, SO_REUSEADDR, &on, sizeof(on)) < 0)
//Socket address
bzero((char *) &server, sizeof(server));
server.sin_family = AF_INET; // IPv4 internet Protocols
inet_aton("127.0.0.1", &server.sin_addr);
server.sin_port = htons(SERVER_PORT);

//Binding socket to address.
if (bind( sockfd, (struct sockaddr*)&server, sizeof (server) ) < 0 ){
    perror("Bind Error");
    exit(EXIT_FAILURE);
}

//Listen to accept connection.
if( listen(sockfd, SOMAXCONN) < 0 ){
    perror("Error in Listen");
    exit(EXIT_FAILURE);
}

unsigned int new_sock;
clientlen =sizeof(client1);
int activity;
while(1){

    //Clear socket set.
    FD_ZERO(&readfds);

    //Adding main sockfd to the socket set.
    FD_SET(sockfd, &readfds);
    unsigned int max_sd = sockfd;

    //Add child sockets to set.
    for(int i=0 ; i<2; i++){
           c = clientsocks[i];
        if(c > 0)
            FD_SET(c, &readfds);
        if(c > max_sd)
            max_sd = c;
    }

    activity = select(max_sd + 1, &readfds, NULL, NULL, NULL);
    if(activity < 0){
        perror("Error in select()");
        exit(EXIT_FAILURE);
    }

    //Incoming connection when something happens on sockfd.
    if( FD_ISSET(sockfd, &readfds)){
        new_sock = accept(sockfd, (struct sockaddr *) &client1, &clientlen);
        if(new_sock > 0){
            for(int i=0; i<2; i++){
                if(clientsocks[i] == 0){
                    clientsocks[i] = new_sock;
                    break;
                }
            }
        }
        if(  new_sock < 0){
            perror("Error Accepting");
            exit(EXIT_FAILURE);
        }
        if( send(new_sock, w_msg, strlen(w_msg), 0) != strlen(w_msg)){
            perror("Welcome message");
            exit(EXIT_FAILURE);
        }
    c1 = clientsocks[0];
    c2 = clientsocks[1];
    FD_SET(c1, &readfds);
    FD_SET(c2, &readfds);
    }

    //Else if its not a new incoming connection.
    if(FD_ISSET(c1, &readfds)){
        if(recv(c1, rmsg1, 100, 0) < 0){
            perror("Receive 1");
            exit(EXIT_FAILURE);
        }
        printf("Client1 >> %s\n", rmsg1);
        //Forwarding to Client B.
        if( send(c2, rmsg1, 100, 0) < 0){
            perror("Error forwarding to 2");
            exit(EXIT_FAILURE);
        }
    }
    if(FD_ISSET(c2, &readfds)){
        if(recv(c2, rmsg2, 100, 0) < 0){
            perror("Receive 2");
            exit(EXIT_FAILURE);
        }
        printf("Client2 >> %s\n", rmsg2);
        if( send(c1, rmsg2, 100, 0) < 0 ){
            perror("Error Forwarding to 1");
            exit(EXIT_FAILURE);
        }
    }
}
close(sockfd);
return 1;
}

我的问题只涉及两个客户。如果你能指出其他一些改进，我将非常感激。

Answer 1

你遇到的一些结构性问题并不是StackOverflow成员要解决的问题。

确定每个客户端是否要连接到同一个套接字或它们自己的唯一套接字，请相应地进行监听。
如果你有一个共享连接，它似乎你要去...并且通过共享连接我的意思是你正在监听一个端口并进行多次接受，然后在给定的接受后你不能假设谁连接直到你从端口读取一些数据。您应该让每个客户端发送一些信息，以便您知道A是否已连接，然后是B或B是否已连接，然后是A.
你的setsockopt周围的代码似乎忘记了{}
保存clientsocks时，您可以使用客户端的信息来确定要使用的阵列槽。也许你有100个客户端，他们每个都连接，然后发送一个32位字（其中包含他们的客户端编号，1-100之间）。
一旦您可以阅读来自不同客户端的消息（可能希望暂时将其打印出来，以便您可以看到正在发生的事情），您可以构建各种消息。目标是客户A能够向服务器询问客户端B的联系信息，以便A可以直接连接到B并向B发送消息。
OR，而不是＃5，客户端26应该能够向服务器发送消息，指示它是针对客户端45的，并且服务器应该能够检查已签入的客户端阵列然后发送消息到阵列插槽45中的客户端。
客户端需要具有唯一的ID /编号，以便服务器可以将客户端ID映射到clientarray套接字。
直接处理其中一些错误比导致服务器退出更好。也许您可以关闭与套接字索引关联的套接字导致错误。您的错误消息应指示哪个客户端/套接字索引导致错误。通常，您需要更多调试消息和消息中的更多信息。

使用c中的select（）函数进行客户端到客户端的通信

1 个答案: