Как я могу остановить время? - PullRequest
0 голосов
/ 08 марта 2020

У меня есть TCP-сервер-клиент, где я ввожу число с количеством раз, которое я хочу отправить слово на сервер. Поэтому я отправляю правильно, потому что я печатаю информацию на al oop (в клиенте), но на сервере отображается бесконечно, потому что я ставлю while (1), но если я не использую, просто напечатайте его один раз. Я не хочу отправлять номер на сервер.

Если я введу число 4, я хочу напечатать слово "привет" четыре раза. Это сервер, теперь он печатает бесконечно, очевидно.

#define MAXPENDING 5    /* Maximum number of simultaneous connections */
#define BUFFSIZE 255    /* Size of message to be received */

void err_sys(char *mess) { perror(mess); exit(1); }

void handle_client(int sock) {
    char buffer[BUFFSIZE];
    int received = -1;
    while (1) {
        /* Read from socket */
        read(sock, &buffer[0], BUFFSIZE);
        printf("Message from client: %s\n", buffer);

        /* Write to socket */
        write(sock, buffer, strlen(buffer) + 1);

        /* Close socket */
        close(sock);
    }
}
int main(int argc, char *argv[]) {
    struct sockaddr_in echoserver, echoclient;
    int serversock, clientsock;
    int result;

    /* Check input arguments */
    if (argc != 2) {
        fprintf(stderr, "Usage: %s <port>\n", argv[0]);
        exit(1);
    }

    /* Create TCP socket */
    serversock = socket(PF_INET, SOCK_STREAM, IPPROTO_TCP);
    if (serversock < 0) {
        err_sys("Error socket");
    }

    /* Set information for sockaddr_in structure */
    memset(&echoserver, 0, sizeof(echoserver));       /* we reset memory */
    echoserver.sin_family = AF_INET;                  /* Internet/IP */
    echoserver.sin_addr.s_addr = htonl(INADDR_ANY);   /* ANY address */
    echoserver.sin_port = htons(atoi(argv[1]));       /* server port */

    /* Bind socket */
    result = bind(serversock, (struct sockaddr *) &echoserver, sizeof(echoserver));
    if (result < 0) {
        err_sys("Error bind");
    }

    /* Listen socket */
    result = listen(serversock, MAXPENDING);
    if (result < 0) {
        err_sys("Error listen");
    }
    while (1) {
        unsigned int clientlen = sizeof(echoclient);

        /* Wait for a connection from a client */
        clientsock = accept(serversock, (struct sockaddr *) &echoclient, &clientlen);
        if (clientsock < 0) {
            err_sys("Error accept");
        }
        fprintf(stdout, "Client: %s\n", inet_ntoa(echoclient.sin_addr));

        /* Call function to handle socket */
        handle_client(clientsock);
    }
}

Это клиент

  printf("Enter a number between 0 to 9[0 to exit]: ");
fgets(number, 100, stdin);
while(strtol(number,&pEnd,10)!=0){

    while (!((strtol(number,&pEnd,10) <= 9) && (strtol(number,&pEnd,10) >= 0))) {
        printf("[ERROR] The number you entered is out of range\n");
        printf("Enter a number between 0 to 9[0 to exit]: ");
        fgets(number, 100, stdin);
    }
    if(strtol(number,&pEnd,10)!=0){
        printf("Enter a word: ");
        fgets(buffer, 100, stdin);
        for(int i=0;i<strtol(number,&pEnd,10);i++){
            write(sock, buffer, strlen(buffer) + 1);
            fprintf(stdout, " sent \n");

            read(sock, buffer, BUFFSIZE);
            fprintf(stdout, " %s ...done \n", buffer);
        }

    }
    printf(" Enter a number between 0 to 9[0 to exit]: ");
    fgets(number, 100, stdin);
}

Ответы [ 2 ]

3 голосов
/ 08 марта 2020

В вашем коде есть несколько проблем:

  • вы не проверяете, успешно ли read(). Вы должны оторваться от l oop, если он потерпит неудачу.
  • вы безоговорочно закроете сокет в теле l oop: следующий read потерпит неудачу и немедленно вернет -1, вы напечатаете фиктивное сообщение и итерация в тошноте.
  • Вы вызываете strlen() и printf() с буфером, который не может быть завершен нулем, потенциально вызывая неопределенное поведение.

Вот модифицированная версия:

void handle_client(int sock) {
    char buffer[BUFFSIZE];
    int received = -1;
    if (sock < 0)
        return;
    for (;;) {
        /* Read from socket */
        received = read(sock, buffer, sizeof(buffer) - 1);
        if (received <= 0)
            break;
        buffer[received] = '\0';
        printf("Message from client: %s\n", buffer);

        /* Write to socket */
        write(sock, buffer, received);
    }
    /* Close socket */
    close(sock);
}
0 голосов
/ 08 марта 2020

Обычно у вас в руках бесконечный l oop, что не рекомендуется, но если вам нужно сделать это, добавьте оператор if, такой:

while(1){
if(clientsock < 0){/*statement that is met so that your code stops running the if */
  break;
 }
}
...