Clear/Reset of local scoped variables

[Arthurvdv]

Started from this post.

On local scoped variable, it's redundant to do a Clear(Customer) and/or Customer.Reset().

procedure MyProcedure()
var
    Customer: Record Customer;
begin
    Customer.Reset(); // No need to reset a local variable

    ...
end;

[fvet]

Would that only apply to the very first occurence (before any find operations)?
Other subsequent calls might be valid cases.

procedure MyProcedure()
var
    Customer: Record Customer;
begin
    Customer.Reset(); // No need to reset a local variable
    Customer.SetRange(Field1, ...)
    ...
    // Do Some stuff
    ...
    Customer.Reset(); // Allowed in case you re-use the same local variable for looping a different set of records
    Customer.SetRange(Field2, ...)
    ...
    // Do Some stuff
    ...

end;

[Arthurvdv]

I was thinking of applying the KISS principle™ for this rule and for each local record variable only look at the very first action that is applied to that variable. This will maybe not cover all possible scenario's, but will prevent false positives like your example.

[lv-janpieter]

I would not prefer to have this, I would probably disable such rule.

If you often have to read the code the reset is convenient knowing you don't have to inspect the procedure if anything else is happening to the variable prior to the code line you are looking at. It is valuable to have this insight.

In my opinion readability outweighs reduction of code lines. The reset (most likely) doesn't have any performance overhead.

Also be aware the KISS principle might cause false positives in repeat loops.