VPF::Вызов select грузит процессор - Форум программистов

paradox91

Дата 26.5.2011, 10:39 (ссылка)

(голосов:1)

Загрузка ...

Новичок

Профиль
Группа: Участник
Сообщений: 3
Регистрация: 16.5.2011

Репутация: нет
Всего: нет

Может уже были такие темы, но не смог найти. Написал TCP клиент для курсовой, задача клиента дать пользователю вводить с терминала и отправлять по соединения, а также принимать из соединения и отображать на терминале текстовую информацию.
Клиентская программа устанавливает соединение и входит в режим одновременного ожидания данных с терминала и со стороны через сокет.
Проблема в том, что когда устанавливается соединение - загрузка процессора примерно на планке 60%. Подскажите пожалуйста - как исправить..А то это неправильно как то)

Код программы:

Код


#include <sys/types.h>
#include <sys/socket.h>
#include <sys/time.h>
#include <netinet/in.h>
#include <unistd.h>
#include <fcntl.h>
#include <arpa/inet.h>
#include <stdio.h>
#include <string.h>
#include <stdlib.h>
#define PORT 12345

int main(int argc, char *argv[])
{
if(argc<2)
{
    puts("Arguments weren't given ./my_tcp_client<remote_ip><remote_port>");
}
int sock;
struct sockaddr_in addr,saddr;
    sock = socket(AF_INET, SOCK_STREAM, 0);
    saddr.sin_family = AF_INET;
    saddr.sin_addr.s_addr =  htonl(atoi(argv[1]));
    saddr.sin_port = htons(atoi(argv[2]));
if(connect(sock, (struct sockaddr *)&saddr, sizeof(saddr)) < 0)
{
        perror("connect");
        exit(2);
}
char buf[1024]; char buf1[1024];
int count=0;
int buffull=0;
int buf1full=0;
fd_set readset, writeset;
while(1)
{
    //fd_set readset,writeset;
    FD_ZERO(&readset);
    FD_SET(sock, &readset);
    FD_SET(0, &readset);
    FD_ZERO(&writeset);
    FD_SET(sock, &writeset);
    FD_SET(1, &writeset);
    struct timeval timeout;
    timeout.tv_sec = 1;
    timeout.tv_usec = 0;
    if(select (sock+1, &readset, &writeset, NULL, &timeout) < 0)
    {
         perror("select");
         exit(3);
    }
    if (FD_ISSET(sock, &readset) && !buffull)
    {
        count = read (sock, buf, 1024);
        if(count <= 0)
         {
            puts("disconection");
            return 1;
        }
        printf("count: %d\n", count);
        write(1,buf,count);
        buffull=1;
    }
    if(FD_ISSET(0, &readset) && !buf1full)
    {
        count = read (0, buf1, 1024);
        buf1full=1;
        printf("count: %d\n", count);
    }
    if(FD_ISSET(1, &writeset) && buffull)
    {
        write(1, buf, count);
        buffull=0;
    }
    if(FD_ISSET(sock, &writeset) && buf1full)
    {
        write(sock, buf1, count);
        buf1full=0;
    }
}
return EXIT_SUCCESS;
}

Это сообщение отредактировал(а) paradox91 - 26.5.2011, 13:20

boostcoder

Дата 26.5.2011, 10:45 (ссылка)

(голосов:1)

Загрузка ...

pattern`щик

Профиль
Группа: Завсегдатай
Сообщений: 5458
Регистрация: 1.4.2010

Репутация: 16
Всего: 110

отформатируйте код. глаз жалко.

--------------------

компилятор онлайн с поддержкой C++11
сборки компиляторов MinGW

GrayCardinal

Дата 26.5.2011, 15:55 (ссылка)

(нет голосов)

Загрузка ...

Фигасе

Профиль
Группа: Завсегдатай
Сообщений: 3039
Регистрация: 9.11.2003

Репутация: 8
Всего: 58

paradox91,
Попробуйте struct timeval вынести вверх, за while

--------------------

Go go go !

boostcoder

Дата 26.5.2011, 16:03 (ссылка)

(нет голосов)

Загрузка ...

pattern`щик

Профиль
Группа: Завсегдатай
Сообщений: 5458
Регистрация: 1.4.2010

Репутация: 16
Всего: 110

я не вижу никаких причин чему-либо тормозить в этом коде..

--------------------

компилятор онлайн с поддержкой C++11
сборки компиляторов MinGW

paradox91

Дата 26.5.2011, 21:28 (ссылка)

(нет голосов)

Загрузка ...

Новичок

Профиль
Группа: Участник
Сообщений: 3
Регистрация: 16.5.2011

Репутация: нет
Всего: нет

А есть вероятность того, что это сервер виноват? Для проверки всегда соединял с Эхо-сервером вот отсюда: http://www.rsdn.ru/article/unix/sockets.xml

svlary

Дата 27.5.2011, 05:17 (ссылка)

(голосов:2)

Загрузка ...

Бывалый

Профиль
Группа: Участник
Сообщений: 207
Регистрация: 8.9.2009

Репутация: 4
Всего: 4

В строке 47 у Вас стоит вызов select, у которого задан параметр &writeset.
В этом параметре Вы установили проверку 1-го дескриптора - stdout (строки 42-43) Вашего кода.
В момент запуска вашего приложения, stdout готов к записи.

Поэтому функция select сразу "слетает" с ожидания и программа продолжает исполнение.
Так как переменные buffull и buf1full равны нулю, то ни один из операторов записи не срабатывает.
Поток управления достигает конца цикла и возвращается в начало.
Управление опять передается на select
Функция select опять информирует Вас о том, что stdout готов к записи
.....

GrayCardinal

Дата 27.5.2011, 06:51 (ссылка)

(нет голосов)

Загрузка ...

Фигасе

Профиль
Группа: Завсегдатай
Сообщений: 3039
Регистрация: 9.11.2003

Репутация: 8
Всего: 58

svlary,
Однако.
+1 smile

--------------------

Go go go !

0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема \| C/C++: Программирование под Unix/Linux \| Следующая тема »