Эти странные потоки

[crossmark]

Доброго времени!
Поставил цель побороться с WinSock. Но столкнулся с проблемой с потоками. Не получается в поток передать параметры. Подскажите что не так. Может из-за того что класс и память получается разная?
Это лишь кусочек кода.

unit socket_u;

interface

uses windows,winsock,messages;

type TParamThread=record
      hServSocket:integer;
end;
PParamThread=^TParamThread;


type TServerSock=class
private
  wsData : WSADATA;
  SocketAdrr : sockaddr_in;
  ServType : TServType;
  function CreateSocket:integer; //шаг 2 создание сокета
  function SetServSocket:integer; // шаг 3  настройка сокета
  function SyncServ:integer;
  function SockConnect: integer; //соединение
  function SockAccept(accept_cocket:TSocket; addr: PSOCKADDR; addrlen: PInteger ): TSocket;
public
  ServSocket : integer;
  errCreate : boolean;
  constructor create;    //шаг 1 инициализация сокета
  destructor destroy;
  function ServerConnect:boolean;
  function GetNameHost:String;
  function GetIPAdress(name:PAnsiChar):string;
  procedure NetMsg(var M:TMessage); message WM_MESS;
  function ServerRecv(accept_cocket:TSocket; buf:PAnsiChar;len,flags:integer):integer; // recv получение данных
  function ServerSend(accept_cocket:TSocket; buf:PAnsiChar;len,flags:integer):integer; // send отправка данных
  procedure ServerCreateThread;
  function SockWait(s:pointer):DWORD;
end;


var vServType:TServType;
    vClientType:TClientType;
    Param:PParamThread;
implementation


procedure TServerSock.ServerCreateThread;
var //thread:array[1..100] of cardinal;
    hThread:THandle;
    ThreadID:DWORD;
    p:pointer;
begin
    hThread:=BeginThread(nil, 0, Addr(TServerSock.SockWait), param, 0, threadID);

end;

function TServerSock.SockWait(s:pointer):DWORD;
var csocket:TSocket;
    cs:integer;
    buf:array[1..100]of pansichar;
    i:integer;
    p:TParamThread;
begin
   p:=PParamThread(s)^;
   Finalize(PParamThread(S)^);
 //   FreeMem(S);
  while True do
  begin
      sleep(1);
      //csocket:=SockAccept(ServSocket,Pointer(@SocketAdrr),Pointer(@SocketAdrr));
      i:=sizeof(SocketAdrr);
      csocket:=accept(Result,Pointer(@SocketAdrr),pointer(i));
      if csocket<>-1 then
      begin
       serverrecv(csocket,buf[1],sizeof(buf),0);
       messagebox(0,PWideChar(buf[1]),PCHAR('Recv'),mb_ok);
      end;
      if csocket<> INVALID_SOCKET then
         begin
             closesocket(csocket); // закрываем сокет
         end;
  end;
end;

Т.е. hThread:=BeginThread(nil, 0, Addr(TServerSock.SockWait), param, 0, threadID); Тут param получает значение номер сокета, а в функции function TServerSock.SockWait(sointer):DWO RD;
s получается пустой. точнее какой то мусор наверное. тут p:=PParamThread(s)^; очень большое значение.
Что не так, уже сутки думаю....
Спасибо

[veniside]

прототип входа в поток описан так:

TThreadFunc = function(Parameter: Pointer): Integer;

и "Addr(TServerSock.SockWait)" не прокатит, т.к. метод TServerSock.SockWait() ожидает первым неявым параметром self, а потом уже s. Естественно, никакого self ему не передается, а в s попадает мусор.

[crossmark]

процедуру вынес за пределы класса и все успешно передается... а почему тогда в классе приходит мусор? и что нет решения?

[veniside]

> а почему тогда в классе приходит мусор?

ожидает первым неявым параметром self, а потом уже s. Естественно, никакого self ему не передается, а в s попадает мусор.

> и что нет решения?

процедуру вынес за пределы класса и все успешно передается

[crossmark]

спасибо!
и все же появился вопрос... ПОЧЕМУ ТАК передается... где можно подробно это прочитать... ? уж больно интересно

[veniside]

почему self ожидается первым параметром? ну, так устроены все методы классов. Можно считать, что любой метод класса (кроме конструктора), объявлен примерно так:

type
  myClass = class
    procedure some(self: myClass; ...остальные параметры, если есть...)
  end;

Виндовс, естественно об этом ничего не знает, и считает, что процедура входа в поток ожидает ровно один параметр: тот, значение которого задается в CreateThread(). Этим можно воспользоваться, чтобы вернуться в метод класса:

type
  myThreadClass = class
    procedure onThreadStart(); // выполняется в новом потоке    
  end;

// --  --
function threadEntry(param: myThreadClass): DWORD; stdcall;
begin
  param.onThreadStart();
  //
  result := 0;
end;

  ....
  // создаем и запускаем новый поток
  thread := myThreadClass.create();
  CreateThread(nil, 0, @threadEntry, thread, 0, id);

почему так не сделали в BeginThread(), не знаю, видимо, посчитали, что класса TThread будет достаточно.

[xoodoo]

почему так не сделали в BeginThread(), не знаю

А чего там "не сделали" ?
Прототип BeginThread() за исключением соглашения о вызове один-в-один совпадает с CreateThread, так что ничто не мешает и в BeginThread() передать параметром свой объект, так же как в CreateThread ..
Ну и получить при вызове BeginThread() еще и "бонусы" : как минимум сэкономить на реализации собственной предобработки исключений и автоматически известить встроенный дельфийский менеджер памяти о потенциальных мультипоточных к нему обращениях.

посчитали, что класса TThread будет достаточно

Скорее так - посчитали что BeginThread будет недостаточно и представили дополнительно "более удобный" интерфейс для организации многопоточности в VCL-приложении.

[crossmark]

Сообщение от veniside

почему self ожидается первым параметром? ну, так устроены все методы классов. Можно считать, что любой метод класса (кроме конструктора), объявлен примерно так:

type
  myClass = class
    procedure some(self: myClass; ...остальные параметры, если есть...)
  end;

Виндовс, естественно об этом ничего не знает, и считает, что процедура входа в поток ожидает ровно один параметр: тот, значение которого задается в CreateThread(). Этим можно воспользоваться, чтобы вернуться в метод класса:

type
  myThreadClass = class
    procedure onThreadStart(); // выполняется в новом потоке    
  end;

// --  --
function threadEntry(param: myThreadClass): DWORD; stdcall;
begin
  param.onThreadStart();
  //
  result := 0;
end;

  ....
  // создаем и запускаем новый поток
  thread := myThreadClass.create();
  CreateThread(nil, 0, @threadEntry, thread, 0, id);

почему так не сделали в BeginThread(), не знаю, видимо, посчитали, что класса TThread будет достаточно.

обалдеть.... я даже и не думал что так можно. кинуть весь класс в параметры. я такого и не видел ни где.
спасибо! я обязательно это запомню. и при первой же возможности применю.
огромное спасибо!

[veniside]

> А чего там "не сделали" ?

ну могли бы TObject расширить ещё одним виртуальным методом, который бы вызывался из BeginThread(). Было бы красиво.


> не думал что так можно. кинуть весь класс в параметры

экземпляр класса -- это указатель, и параметр в CreateThread() тоже имеет размер указателя, так что проблем не возникает.

[crossmark]

не сделал потому что... потому что сервер не хочет видеть клиента. хотя через TCPView вроде они подключаются.
вот когда они увидят друг друга, вот тогда попробую все это применить