TcpClient Read/Write

[demaker]

Скажите приемлемо ли написаны функции:
запись запроса(writeReq) и чтения ответа(readAnswer(char *data, int size))

Код

C++ (Qt)
TcpClient::writeReq(char *data, int size)
{	
	len = size;
	while(len){
		int cnt = socket->write(data,len);	//пишем, cnt - количество записсаных байт
		data += cnt;				//передвигаем указатель на cnt  		l
                len -= cnt;				        //уменьшаем длину на cnt
	}
}
 
TcpClient::readAnswer(char *data, int size)
{
	len = size;
	while(len){
		int cnt = socket->read(data,len);	//читаем, cnt - количество считанных байт 
		data += cnt;				//передвигаем указатель на cnt 
		len -= cnt;				       //уменьшаем длину на cnt		
	}
}
 
 
 

[sergek]

Код:

    length = QTcpSocket::write(buf, length);

Код:

    timeoutRead = (async)? 500 : 10000;

    while(QTcpSocket::bytesAvailable() < length){
        if(!QTcpSocket::waitForReadyRead(timeoutRead)){
            return _ERROR;
        }
    }

    length = QTcpSocket::read(buf, length);
    return length;

[demaker]

Код:

    length = QTcpSocket::write(buf, length);

Код:

    timeoutRead = (async)? 500 : 10000;

    while(QTcpSocket::bytesAvailable() < length){
        if(!QTcpSocket::waitForReadyRead(timeoutRead)){
            return _ERROR;
        }
    }

    length = QTcpSocket::read(buf, length);
    return length;

Код:

    length = QTcpSocket::write(buf, length);

Здесь вы хотите сказать, что в любом случае весь буфер запишется.

Код:

    timeoutRead = (async)? 500 : 10000;

    while(QTcpSocket::bytesAvailable() < length){
        if(!QTcpSocket::waitForReadyRead(timeoutRead)){
            return _ERROR;
        }
    }

    length = QTcpSocket::read(buf, length);
    return length;

2.А здесь мы будем ожидать до тех пор пока в буфере драйвера не накопится нужное количество
байт и затем считаем.

Я правильно Вас понял?

[sergek]

Здесь вы хотите сказать, что в любом случае весь буфер запишется.

Да, но может, не сразу - сокет буферизуется. Если не ошибаюсь, вы можете добавить вызов waitForBytesWritten, чтобы убедиться, что данные ушли.

2.А здесь мы будем ожидать до тех пор пока в буфере драйвера не накопится нужное количество
байт и затем считаем.

Да.

[demaker]

Здесь вы хотите сказать, что в любом случае весь буфер запишется.

Да, но может, не сразу - сокет буферизуется. Если не ошибаюсь, вы можете добавить вызов waitForBytesWritten, чтобы убедиться, что данные ушли.

2.А здесь мы будем ожидать до тех пор пока в буфере драйвера не накопится нужное количество
байт и затем считаем.

Да.

Тогда вопрос.
А что у меня было изначально некорректно?

[qate]

А что у меня было изначально некорректно?

лишние телодвижения по записи в сокет и неполные проверки кода возврата если уж хочется так делать
waitForBytesWritten не написано что гарантирует уход данных в канал, но только в сокет и смысл её использовать нет

при чтении также не нужно ждать в цикле - есть же события, по приходу данных

[demaker]

А что у меня было изначально некорректно?

лишние телодвижения по записи в сокет и неполные проверки кода возврата если уж хочется так делать
waitForBytesWritten не написано что гарантирует уход данных в канал, но только в сокет и смысл её использовать нет

при чтении также не нужно ждать в цикле - есть же события, по приходу данных

Логично. Спасибо.