Search code examples
.netinstallationinno-setuppascalscript

'Check' function is executing multiple times in Inno Setup


I am new to Inno Setup scripting and I am trying to install .NET framework 3.5 using below code as a prerequisite. The Check function is executing multiple times. Can some one please help me understand why?

Note: All other sections (Setup, Icons, etc) in this below code are having proper contents.

[Files]
Source: "Frameworks\dotnetfx35setup.exe"; DestDir: {tmp}; Flags: deleteafterinstall; \
    BeforeInstall: Install35Framework; Check: Framework35IsNotInstalled
[Code]
function IsDotNetDetected(version: string; service: Cardinal): boolean;
begin
  Result := { ... };
end;

function Framework35IsNotInstalled: Boolean;
begin
  if IsDotNetDetected('v3.5', 1) then
  begin
    MsgBox('Framework35IsNotInstalled: FALSE ', mbConfirmation, MB_YESNO);
    Result := False;
  end else begin
    MsgBox('Framework35IsNotInstalled: TRUE ', mbConfirmation, MB_YESNO);
    Result := True;
  end;
end; 

procedure Install35Framework;
begin
  { ... }
end;

Solution

  • Quoting Check parameter documentation:

    Setup might call each check function several times, even if there's only one entry that uses the check function. If your function performs a lengthy piece of code, you can optimize it by performing the code only once and 'caching' the result in a global variable.

    So the behavior is as designed.

    And as your code is quite simple, I do not even think it needs any optimization. It's perfectly ok, if it runs few times.


    Were it not, you can optimize it like this:

    var
      Framework35IsNotInstalledCalled: Boolean; 
      Framework35IsNotInstalledResult: Boolean;
    
    function Framework35IsNotInstalled: Boolean;
    begin
      if not Framework35IsNotInstalledCalled then
      begin
        Framework35IsNotInstalledResult := IsDotNetDetected('v3.5', 1);
        Framework35IsNotInstalledCalled := True;
      end;
    
      Result := Framework35IsNotInstalledResult;
    end;