Question

我正在尝试使用AsyncCalls在整个C类子网上执行Netbios查找。理想情况下，我希望它能同时执行10次以上的查找，但目前它一次只进行1次查找。我在这里做错了什么？

我的表单包含1个按钮和1个备忘录。

unit main;

interface

uses
  Windows,
  Messages,
  SysUtils,
  Classes,
  Forms,
  StdCtrls,
  AsyncCalls,
  IdGlobal,
  IdUDPClient,
  Controls;

type
  PWMUCommand = ^TWMUCommand;

  TWMUCommand = record
    host: string;
    ip: string;
    bOnline: boolean;
  end;

type
  PNetbiosTask = ^TNetbiosTask;

  TNetbiosTask = record
    hMainForm: THandle;
    sAddress: string;
    sHostname: string;
    bOnline: boolean;
    iTimeout: Integer;
  end;

const
  WM_THRD_SITE_MSG  = WM_USER + 5;
  WM_POSTED_MSG     = WM_USER + 8;

type
  TForm2 = class(TForm)
    Button1: TButton;
    Memo1: TMemo;
    procedure Button1Click(Sender: TObject);
  private
    procedure ThreadMessage(var Msg: TMessage); message WM_POSTED_MSG;
    { Private declarations }
  public
    { Public declarations }
  end;

var
  Form2             : TForm2;

implementation

{$R *.dfm}

function NetBiosLookup(Data: TNetbiosTask): boolean;
const
  NB_REQUEST        = #$A2#$48#$00#$00#$00#$01#$00#$00 +
    #$00#$00#$00#$00#$20#$43#$4B#$41 +
    #$41#$41#$41#$41#$41#$41#$41#$41 +
    #$41#$41#$41#$41#$41#$41#$41#$41 +
    #$41#$41#$41#$41#$41#$41#$41#$41 +
    #$41#$41#$41#$41#$41#$00#$00#$21 +
    #$00#$01;

  NB_PORT           = 137;
  NB_BUFSIZE        = 8192;
var
  Buffer            : TIdBytes;
  I                 : Integer;
  RepName           : string;
  UDPClient         : TIdUDPClient;
  msg_prm           : PWMUCommand;
begin
  RepName := '';
  Result := False;
  UDPClient := nil;

  UDPClient := TIdUDPClient.Create(nil);
  try
    try
      with UDPClient do
      begin
        Host := Trim(Data.sAddress);
        Port := NB_PORT;

        Send(NB_REQUEST);
      end;

      SetLength(Buffer, NB_BUFSIZE);
      if (0 < UDPClient.ReceiveBuffer(Buffer, Data.iTimeout)) then
      begin

        for I := 1 to 15 do
          RepName := RepName + Chr(Buffer[56 + I]);

        RepName := Trim(RepName);
        Data.sHostname := RepName;

        Result := True;
      end;

    except
      Result := False;
    end;
  finally
    if Assigned(UDPClient) then
      FreeAndNil(UDPClient);
  end;

  New(msg_prm);
  msg_prm.host := RepName;
  msg_prm.ip := Data.sAddress;
  msg_prm.bOnline := Length(RepName) > 0;

  PostMessage(Data.hMainForm, WM_POSTED_MSG, WM_THRD_SITE_MSG, integer(msg_prm));

end;

procedure TForm2.Button1Click(Sender: TObject);
var
  i                 : integer;
  ArrNetbiosTasks   : array of TNetbiosTask;
  sIp               : string;
begin
  //

  SetMaxAsyncCallThreads(50);

  SetLength(ArrNetbiosTasks, 255);
  sIp := '192.168.1.';
  for i := 1 to 255 do
  begin

    ArrNetbiosTasks[i - 1].hMainForm := Self.Handle;
    ArrNetbiosTasks[i - 1].sAddress := Concat(sIp, IntToStr(i));
    ArrNetbiosTasks[i - 1].iTimeout := 5000;

    AsyncCallEx(@NetBiosLookup, ArrNetbiosTasks[i - 1]);
    Application.ProcessMessages;
  end;
end;

procedure TForm2.ThreadMessage(var Msg: TMessage);
var
  msg_prm           : PWMUCommand;
begin
  //
  case Msg.WParam of
    WM_THRD_SITE_MSG:
      begin
        msg_prm := PWMUCommand(Msg.LParam);
        try
          Memo1.Lines.Add(msg_prm.ip + ' = ' + msg_prm.host + ' --- Online? ' + BoolToStr(msg_prm.bOnline));
        finally
          Dispose(msg_prm);
        end;
      end;
  end;

end;

end.

Answer 1

棘手的东西。我做了一些调试（好吧，相当一些调试），发现代码在1296行的AsyncCallsEx中阻塞：

Result := TAsyncCallArgRecord.Create(Proc, @Arg).ExecuteAsync;

进一步挖掘显示它在System.pas（_IntfCopy）中的接口副本中阻塞

CALL    DWORD PTR [EAX] + VMTOFFSET IInterface._Release

查看相同代码的pascal版本，似乎此行释放先前存储在destination参数中的引用计数。但是，目标是调用者（您的代码）中未使用的结果。

现在是棘手的部分。

AsyncCallEx返回一个接口（在你的情况下）调用者抛出的接口。所以理论上编译的代码（伪形式）应该看起来像这样

loop
  tmp := AsyncCallEx(...)
  tmp._Release
until

然而，编译器将其优化为

loop
   tmp := AsyncCallEx(...)
until
tmp._Release

为什么呢？因为它知道分配接口会自动释放存储在tmp变量中的接口的引用计数（在_IntfCopy中调用_Release）。所以没有必要明确地调用_Release。

释放IAsyncCall会导致代码在线程完成时等待。所以基本上你每次调用AsyncCallEx时都等待上一个线程完成......

我不知道如何使用AsyncCalls很好地解决这个问题。我尝试了这种方法，但不知何故它没有按预期完全工作（ping大约50个地址后的程序块）。

type
  TNetbiosTask = record
    //... as before ...
    thread: IAsyncCall;
  end;

  for i := 1 to 255 do
  begin

    ArrNetbiosTasks[i - 1].hMainForm := Self.Handle;
    ArrNetbiosTasks[i - 1].sAddress := Concat(sIp, IntToStr(i));
    ArrNetbiosTasks[i - 1].iTimeout := 5000;

    ArrNetbiosTasks[i - 1].thread := AsyncCallEx(@NetBiosLookup, ArrNetbiosTasks[i - 1]);
    Application.ProcessMessages;
  end;
  for i := 1 to 255 do // wait on all threads
    ArrNetbiosTasks[i - 1].thread := nil;

Answer 2

如果您调用AsyncCallEx()或任何其他AsyncCalls函数，则会返回IAsyncCall接口指针。如果其引用计数器达到0，则底层对象将被销毁，这将等待工作线程代码完成。您在循环中调用AsyncCallEx()，因此每次将返回的接口指针分配给相同（隐藏）变量时，递减引用计数器，从而同步释放先前的异步调用对象。

要解决此问题，只需将IAsyncCall的私有数组添加到表单类中，如下所示：

private
  fASyncCalls: array[byte] of IAsyncCall;

并将返回的接口指针分配给数组元素：

fASyncCalls[i] := AsyncCallEx(@NetBiosLookup, ArrNetbiosTasks[i - 1]);

这将使接口保持活动状态并启用并行执行。

请注意，这只是一般的想法，您应该在调用返回时添加代码以重置相应的数组元素，并在释放表单之前等待所有调用完成。

是否可以使用AsyncCalls单元创建线程池？

2 个答案: