Question

我正在尝试制作一个协议（构建在TCP上），可以在端口457上将字符串从客户端发送到服务器。这是我到目前为止所拥有的：

Server.c

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <unistd.h>
#include <sys/types.h> 
#include <sys/socket.h>
#include <netinet/in.h>

void error(const char *msg)
{
    perror(msg);
    exit(1);
}

int main(int argc, char *argv[])
{
    int sockfd, newsockfd, portno;
    socklen_t clilen;
    char buffer[256];
    struct sockaddr_in serv_addr, cli_addr;
    int n;
    if (argc < 2) {
        fprintf(stderr,"ERROR, no port provided\n");
        exit(1);
    }
    sockfd = socket(AF_INET, SOCK_STREAM, 0);
    if (sockfd < 0) 
        error("ERROR opening socket");
    bzero((char *) &serv_addr, sizeof(serv_addr));
    portno = atoi(argv[1]);
    serv_addr.sin_family = AF_INET;
    serv_addr.sin_addr.s_addr = INADDR_ANY;
    serv_addr.sin_port = htons(portno);
    if (bind(sockfd, (struct sockaddr *) &serv_addr,
             sizeof(serv_addr)) < 0) 
        error("ERROR on binding");
    listen(sockfd,5);
    clilen = sizeof(cli_addr);
    newsockfd = accept(sockfd, 
                       (struct sockaddr *) &cli_addr, 
                       &clilen);
    if (newsockfd < 0) 
        error("ERROR on accept");
    bzero(buffer,256);
    n = read(newsockfd,buffer,255);
    if (n < 0) error("ERROR reading from socket");
    printf("Here is the message: %s\n",buffer);
    n = write(newsockfd,"U got your messaze",18);
    if (n < 0) error("ERROR writing to socket");
    close(newsockfd);
    close(sockfd);
    return 0; 
}

Client.c

    #include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
#include <string.h>
#include <sys/types.h>
#include <sys/socket.h>
#include <netinet/in.h>
#include <netdb.h> 

void error(const char *msg)
{
    perror(msg);
    exit(0);
}

int main(int argc, char *argv[])
{
    int sockfd, portno, n;
    struct sockaddr_in serv_addr;
    struct hostent *server;

    char buffer[256];
    if (argc < 3) {
        fprintf(stderr,"usage %s hostname port\n", argv[0]);
        exit(0);
    }
    portno = atoi(argv[2]);
    sockfd = socket(AF_INET, SOCK_STREAM, 0);
    if (sockfd < 0) 
        error("ERROR opening socket");
    server = gethostbyname(argv[1]);
    if (server == NULL) {
        fprintf(stderr,"ERROR, no such host\n");
        exit(0);
    }
    bzero((char *) &serv_addr, sizeof(serv_addr));
    serv_addr.sin_family = AF_INET;
    bcopy((char *)server->h_addr, 
          (char *)&serv_addr.sin_addr.s_addr,
          server->h_length);
    serv_addr.sin_port = htons(portno);
    if (connect(sockfd,(struct sockaddr *) &serv_addr,sizeof(serv_addr)) < 0) 
        error("ERROR connecting");
    printf("Please enter the message: ");
    bzero(buffer,256);
    fgets(buffer,255,stdin);
    n = write(sockfd,buffer,strlen(buffer));
    if (n < 0) 
        error("ERROR writing to socket");
    bzero(buffer,256);
    n = read(sockfd,buffer,255);
    if (n < 0) 
        error("ERROR reading from socket");
    printf("%s\n",buffer);
    close(sockfd);
    return 0;
}

我觉得好像我没有正确设置协议。我呢？

Answer 1

您应该首先为协议编写规范。然后，在查看它之后，您实现了规范。您基本上要求我们对您的协议进行反向工程。这是创建协议的后向方法。在开始实现之前，您应该已经明确了解程序需要做什么。

您的服务器程序充当有限种类的ECHO服务器，因为接受来自客户端的输入不超过255个字节。无论服务器能够读取什么，都会记录到控制台，并将消息U got your messaze作为响应发送给客户端。

您可能想要解决的一些问题：

虽然不太可能，但您的read()调用可能会返回的数据少于客户端发送的数据，即使客户端发送的字节少于256个字节。例如，如果客户端一次发送aaaaaaaaaa个字节，则您的服务器可能只会看到第一个a，并假设它是完整的消息。
您不会采取预防措施来写入已经关闭的连接。这可能会生成SIGPIPE，并导致您的程序意外退出。
信号通常可能会中断您的系统调用。如果发生这种情况，您应该检测到这种情况并重新启动系统调用。

Answer 2

您有一些不正确的类型：

htons期望uint16_t作为参数

htons((uint16_t)portno);

read()和write()期待ssize_t

ssize_t n;

不推荐使用

bzero，请使用

memset(buffer, 0, sizeof(buffer));

不推荐使用

bcopy，请使用

memmove(server->h_addr, &serv_addr.sin_addr.s_addr, server->h_length);

使用buffer时，NUL终止read()

/* bzero(buffer,256); Not needed */
n = read(newsockfd,buffer,255);
if (n < 0) error("ERROR reading from socket");
buffer[n]= '\0'; /* here */
printf("Here is the message: %s\n",buffer);

请注意，现代计划使用send()和recv()代替read()和write()

最后，不要使用幻数

fgets(buffer, 255, stdin);

代替：

fgets(buffer, sizeof(buffer), stdin); /* 256 */

为什么是255？ fgets（）函数应该从流中读取字节到s指向的数组，直到读取n-1个字节，所以256是正确的。

我是否正确创建了协议？

2 个答案: