Особенности WinSock2

[Flenov]

Вопрос первый:

Как правильней отправлять текст?

Вариант первый (Код на Delphi):
function SendText(Sock: TSocket; S: String): Integer;
begin
Result:=Send(Sock, PChar(S)^, Length(S), 0);
end;




Вариант второй (Код на Delphi):
function SendText(Sock: TSocket; S: String): Integer;
var
MainBuff: Array[0..1023] of Char;
iSize: Integer;
begin
Result:=0;
While (Length(S)>0) do
begin
iSize:=Length(S);
if (iSize>SizeOf(MainBuff)) then iSize:=SizeOf(MainBuff);
FillChar(MainBuff, SizeOf(MainBuff), 0);
CopyMemory(@MainBuff, PChar(S), iSize);
Delete(S, 1, iSize);
Result:=Result+Send(Sock, MainBuff, iSize, 0);
end;
end;



Вопрос второй:

Правильно ли я перевожу в неблокирующий режим?

Фрагмент кока (код на Delphi):

var
Sock: TSocket;
iMode: Cardinal;
....
begin
....

Sock:=Socket(AF_INET, SOCK_STREAM, 0);
if (Sock=INVALID_SOCKET) then Raise Exception.Create('Ошибка создания сокета');
if (Connect(Sock, @SockAddr, Sizeof(SockAddr))=0) then
begin
if NoneBlock then
begin
iMode:=1;
IOCtlSocket(Sock, FIONBIO, iMode);
end;
...

[slesh]

Вопрос 1:
Вариант Result:=Send(Sock, PChar(S)^, Length(S), 0); приемлем для строк относительно не большого размера. Чтобы люде не говорили, что данные не смогт сразу послаться итд итп. но это уже более низкий уровень будет думать. А от тебя данные уйдут в ядро и вернется значение сколько данных послан. Именно послано, а не дошло.

Второй вариант - принципиально не правильный. Зачем копировать данные во временный буфер? Юзай указатели. т.е. это можно оформить примерно так:


function SendText(Sock: TSocket; S: String): Integer;
var
SendLen: Integer;
BufStart : pchar;
BufLen : integer;
begin
BufLen := Length(s); // сразу вычислим размер, чтобы потом постоянно не считать
BufStart := @s[1]; // установим указатель на начало строки

While ( BufLen > 0) do // если есть что отправлять
begin
if BufLen < 1024 then SendLen := BufLen else BufLen := 1024; // вычислим сколько байт отправлять. Но не более 1024 (1024- это так к примеру)

SendLen := Send(Sock, BufStart, SendLen, 0); // отправим
if SendLen <= 0 then break; // если ошибка была то выйти из цикла отправки
BufStart := pchar(dword(BufStart) + SendLen); // переместим указатель дальше
BufLen := BufLen - SendLen; // уменьшим кол-во оставшихся данных
end;
end;


т.е. примерно так должно идти.

Второй вопрос. В принципе правильно.

[Chrome~]

function SendText(Sock: TSocket; S: String): Integer;
var
SendLen: Integer;
BufStart : pchar;
BufLen : integer;
begin
BufLen := Length(s); // сразу вычислим размер, чтобы потом постоянно не считать
BufStart := @s[1]; // установим указатель на начало строки

While ( BufLen > 0) do // если есть что отправлять
begin
if BufLen < 1024 then SendLen := BufLen else BufLen := 1024; // вычислим сколько байт отправлять. Но не более 1024 (1024- это так к примеру)

SendLen := Send(Sock, BufStart, SendLen, 0); // отправим
if SendLen <= 0 then break; // если ошибка была то выйти из цикла отправки
BufStart := pchar(dword(BufStart) + SendLen); // переместим указатель дальше
BufLen := BufLen - SendLen; // уменьшим кол-во оставшихся данных
end;
end;


Отлично, спасибо за код! Я не знал о таком способе, не очень люблю работать с указателями.

[Flenov]

Вопрос 1:
Второй вариант - принципиально не правильный. Зачем копировать данные во временный буфер? Юзай указатели. т.е. это можно оформить примерно так:


function SendText(Sock: TSocket; S: String): Integer;
var
SendLen: Integer;
BufStart : pchar;
BufLen : integer;
begin
BufLen := Length(s); // сразу вычислим размер, чтобы потом постоянно не считать
BufStart := @s[1]; // установим указатель на начало строки

While ( BufLen > 0) do // если есть что отправлять
begin
if BufLen < 1024 then SendLen := BufLen else BufLen := 1024; // вычислим сколько байт отправлять. Но не более 1024 (1024- это так к примеру)

SendLen := Send(Sock, BufStart, SendLen, 0); // отправим
if SendLen <= 0 then break; // если ошибка была то выйти из цикла отправки
BufStart := pchar(dword(BufStart) + SendLen); // переместим указатель дальше
BufLen := BufLen - SendLen; // уменьшим кол-во оставшихся данных
end;
end;


т.е. примерно так должно идти.

Второй вопрос. В принципе правильно.

Принципиально неправильный - из-за нерациональности с памятью?

Проверил твою идею.
Да указатели действительно облегчили жизнь.

ЗЫ походу не BufLen а SendLen и ещё походу BufStart ты тут в конце указатель забыл :)

[Flenov]

А приёмку я осуществляю верно?

function ReceiveText(Sock: TSocket; var S: String): Integer;
var
MainBuff: Array[0..1023] of Char;
iSize: Integer;
begin
Result:=0;
While True do
begin
FillChar(MainBuff, SizeOf(MainBuff), 0);
iSize:=Recv(Sock, MainBuff, SizeOf(MainBuff), 0);
if (iSize<1) then Break;
S:=S+String(MainBuff);
Result:=Result+iSize;
end;
end;

[Chrome~]

function SendText(Sock: TSocket; S: String): Integer;
var
MainBuff: Array[0..1023] of Char;
iSize: Integer;
begin
Result:=0;
While (Length(S)>0) do
begin
iSize:=Length(S);
if (iSize>SizeOf(MainBuff)) then iSize:=SizeOf(MainBuff);
FillChar(MainBuff, SizeOf(MainBuff), 0);
CopyMemory(@MainBuff, PChar(S), iSize);
Delete(S, 1, iSize);
Result:=Result+Send(Sock, MainBuff, iSize, 0);
end;
end;
function ReceiveText(Sock: TSocket; var S: String): Integer;
var
MainBuff: Array[0..1023] of Char;
iSize: Integer;
begin
Result:=0;
While True do
begin
FillChar(MainBuff, SizeOf(MainBuff), 0);
iSize:=Recv(Sock, MainBuff, SizeOf(MainBuff), 0);
if (iSize<1) then Break;
S:=S+String(MainBuff);
Result:=Result+iSize;
end;
end;
Оба твои коды правильные. Они не слишком хороши в реализации. Также, долго могут выполняться следующие процедуры: FillChar(); CopyMemory();

[Flenov]

А приёмку как лучше и быстрее можно выполнить?

Кстати, я тут на днях от нехера делать DownLoader написал на чистых сокетах.
Если я статеечку напишу с исходниками - это кому-то нужно или опять всё будут говорить Flenov, ты хернёй маешься?

[Chrome~]

А приёмку как лучше и быстрее можно выполнить?

Кстати, я тут на днях от нехера делать DownLoader написал на чистых сокетах.
Если я статеечку напишу с исходниками - это кому-то нужно или опять всё будут говорить Flenov, ты хернёй маешься?
Для новичков будет полезно, особенно если работа с Winsock.