When not to use Blank Lines

Tags:
Categories:

Description

Do not use blank lines:

  • at the beginning or end of any functions (after begin and before end)
  • inside multiline expression
  • after blank lines

Example 1

Bad code

procedure MATRIX_OnDrillDown(MATRIX_ColumnOrdinal: Integer);
begin

    SetupDrillDownCol(MATRIX_ColumnOrdinal);
    DrillDown(false, ValueType);

end;

Good code

procedure MATRIX_OnDrillDown(MATRIX_ColumnOrdinal: Integer);
begin
    SetupDrillDownCol(MATRIX_ColumnOrdinal);
    DrillDown(false, ValueType);
end;

Example 2

Bad code

if NameIsValid and

    Name2IsValid
then

Good code

if NameIsValid and
    Name2IsValid
then

Example 3

Bad code

var
    GLSetup: Record "General Ledger Setup";
    GLSetupRead: Boolean;


local procedure GetGLSetup()
begin
    if not GLSetupRead then
        GLSetup.Get();


    GLSetupRead := true;


    OnAfterGetGLSetup(GLSetup);
end;


[IntegrationEvent(false, false)]
local procedure OnAfterGetGLSetup(var GLSetup: Record "General Ledger Setup")
begin
end;

Good code

var
    GLSetup: Record "General Ledger Setup";
    GLSetupRead: Boolean;

local procedure GetGLSetup()
begin
    if not GLSetupRead then
        GLSetup.Get();

    GLSetupRead := true;

    OnAfterGetGLSetup(GLSetup);
end;

[IntegrationEvent(false, false)]
local procedure OnAfterGetGLSetup(var GLSetup: Record "General Ledger Setup")
begin
end;

Tips

The AZ AL Dev Tools/AL Code Outline extension adds two new commands to Visual Studio Code to remove empty duplicate lines.

  • Remove Empty Lines from the Active Editor : removes empty duplicate lines from the current editor
  • Remove Empty Lines from the Active Project : removes empty duplicate lines from the current project
Last modified October 9, 2023: Merge pull request #235 from microsoft/dependabot/npm_and_yarn/postcss-8.4.31 (8c58628) by Jeremy Vyska