Merge pull request #159 from christianbraeunlich/hotfix/indentation-in-best-practice
Added indentation for best practices
This commit is contained in:
commit
6dc0ae6f2d
6 changed files with 22 additions and 24 deletions
|
|
@ -15,7 +15,7 @@ When `begin` follows `then`, `else`, `do`, it should be on the same line, preced
|
|||
```al
|
||||
if ICPartnerRefType = ICPartnerRefType::"Common Item No." then
|
||||
begin
|
||||
...
|
||||
...
|
||||
end;
|
||||
```
|
||||
|
||||
|
|
@ -23,7 +23,6 @@ When `begin` follows `then`, `else`, `do`, it should be on the same line, preced
|
|||
|
||||
```al
|
||||
if ICPartnerRefType = ICPartnerRefType::"Common Item No." then begin
|
||||
...
|
||||
...
|
||||
end;
|
||||
```
|
||||
|
||||
|
|
|
|||
|
|
@ -13,7 +13,7 @@ Only use begin..end to enclose [compound statements](https://docs.microsoft.com/
|
|||
```AL
|
||||
if FindSet() then begin
|
||||
repeat
|
||||
...
|
||||
...
|
||||
until next() = 0;
|
||||
end;
|
||||
```
|
||||
|
|
@ -23,7 +23,7 @@ end;
|
|||
```AL
|
||||
if FindSet() then
|
||||
repeat
|
||||
...
|
||||
...
|
||||
until next() = 0;
|
||||
```
|
||||
|
||||
|
|
@ -52,4 +52,3 @@ if X then begin
|
|||
end else
|
||||
(not X)
|
||||
```
|
||||
|
||||
|
|
|
|||
|
|
@ -14,14 +14,14 @@ Do not start a line with a binary operator.
|
|||
|
||||
```AL
|
||||
"Quantity to Ship" :=
|
||||
Quantity
|
||||
- "Quantity Shipped"
|
||||
Quantity
|
||||
- "Quantity Shipped"
|
||||
```
|
||||
|
||||
## Good code
|
||||
|
||||
```AL
|
||||
"Quantity to Ship" :=
|
||||
Quantity -
|
||||
"Quantity Shipped"
|
||||
Quantity -
|
||||
"Quantity Shipped"
|
||||
```
|
||||
|
|
|
|||
|
|
@ -15,11 +15,11 @@ The `end else` pair should always appear on the same line.
|
|||
```al
|
||||
if OppEntry.Find('-') then
|
||||
if SalesCycleStage.Find('-') then begin
|
||||
...
|
||||
...
|
||||
end
|
||||
else
|
||||
begin
|
||||
...
|
||||
...
|
||||
end;
|
||||
```
|
||||
|
||||
|
|
@ -28,8 +28,8 @@ The `end else` pair should always appear on the same line.
|
|||
```al
|
||||
if OppEntry.Find('-') then
|
||||
if SalesCycleStage.Find('-') then begin
|
||||
...
|
||||
...
|
||||
end else begin
|
||||
...
|
||||
...
|
||||
end;
|
||||
```
|
||||
|
|
|
|||
|
|
@ -13,15 +13,15 @@ _Created by Microsoft, Described by waldo_
|
|||
## Bad code
|
||||
|
||||
```al
|
||||
if Atom = '\>' then HasLogicalOperator := true else begin
|
||||
...
|
||||
if Atom = '>' then HasLogicalOperator := true else begin
|
||||
...
|
||||
end;
|
||||
```
|
||||
|
||||
## Good code
|
||||
|
||||
```al
|
||||
if Atom = '\>' then
|
||||
if Atom = '>' then
|
||||
HasLogicalOperator := true
|
||||
else begin
|
||||
...
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue