Skip to content

Latest commit

 

History

History
68 lines (53 loc) · 1.6 KB

PossibleIncorrectUsageOfAssignmentOperator.md

File metadata and controls

68 lines (53 loc) · 1.6 KB
description ms.custom ms.date ms.topic title
Equal sign is not an assignment operator. Did you mean the equality operator \'-eq\'?
PSSA v1.21.0
06/28/2023
reference
PossibleIncorrectUsageOfAssignmentOperator

PossibleIncorrectUsageOfAssignmentOperator

Severity Level: Information

Description

In many programming languages, the equality operator is denoted as == or =, but PowerShell uses -eq. Therefore, it can easily happen that the wrong operator is used unintentionally. This rule catches a few special cases where the likelihood of that is quite high.

The rule looks for usages of == and = operators inside if, else if, while and do-while statements but it does not warn if any kind of command or expression is used at the right hand side as this is probably by design.

Example

Wrong

if ($a = $b)
{
    ...
}
if ($a == $b)
{

}

Correct

if ($a -eq $b) # Compare $a with $b
{
    ...
}
if ($a = Get-Something) # Only execute action if command returns something and assign result to variable
{
    Do-SomethingWith $a
}

Implicit suppresion using Clang style

There are some rare cases where assignment of variable inside an if statement is by design. Instead of suppressing the rule, one can also signal that assignment was intentional by wrapping the expression in extra parenthesis. An exception for this is when $null is used on the LHS because there is no use case for this.

if (($shortVariableName = $SuperLongVariableName['SpecialItem']['AnotherItem']))
{
    ...
}