Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Enhance check for assignment by rather checking if assigned variable is used in assignment block #1

Merged
merged 10 commits into from
Feb 17, 2018
Merged

Enhance check for assignment by rather checking if assigned variable is used in assignment block #1

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
a6285b4
prototype of detecting variable usage on LHS to avoid false positives
bergmeister Feb 13, 2018
2ea4546
simplify and reduce nesting
bergmeister Feb 13, 2018
3d33169
fix regression by continue statements
bergmeister Feb 13, 2018
f9fd379
fix last regression
bergmeister Feb 13, 2018
4f7dbe6
add simple test case for new improvemeent
bergmeister Feb 13, 2018
135be11
finish it off and adapt tests
bergmeister Feb 17, 2018
d8666b0
update docs
bergmeister Feb 17, 2018
bd72cc3
minor style update
bergmeister Feb 17, 2018
0269ac0
fix typo
bergmeister Feb 17, 2018
edd37c0
fix test due to warning level change
bergmeister Feb 17, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
minor style update
  • Loading branch information
@bergmeister
bergmeister committed Feb 17, 2018
commit bd72cc30542548c149843c5206f8ff201b3c5dd8
6 changes: 4 additions & 2 deletions Rules/PossibleIncorrectUsageOfAssignmentOperator.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -59,15 +59,17 @@ public IEnumerable<DiagnosticRecord> AnalyzeScript(Ast ast, string fileName)
if (leftVariableExpressionAstOfAssignment != null)
{
var statementBlockOfIfStatenent = clause.Item2;
var variableExpressionAstsInStatementBlockOfIfStatement = statementBlockOfIfStatenent.FindAll(testAst => testAst is VariableExpressionAst, searchNestedScriptBlocks: true);
var variableExpressionAstsInStatementBlockOfIfStatement = statementBlockOfIfStatenent.FindAll(testAst =>
testAst is VariableExpressionAst, searchNestedScriptBlocks: true);
if (variableExpressionAstsInStatementBlockOfIfStatement == null) // no variable uages at all
{
yield return PossibleIncorrectUsageOfComparisonOperatorAssignmentOperatorError(assignmentStatementAst.ErrorPosition, fileName);
}
else
{
var variableOnLHSIsBeingUsed = variableExpressionAstsInStatementBlockOfIfStatement.Where(x => x is VariableExpressionAst)
.Any(x => ((VariableExpressionAst)x).VariablePath.UserPath.Equals(leftVariableExpressionAstOfAssignment.VariablePath.UserPath, StringComparison.OrdinalIgnoreCase));
.Any(x => ((VariableExpressionAst)x).VariablePath.UserPath.Equals(
leftVariableExpressionAstOfAssignment.VariablePath.UserPath, StringComparison.OrdinalIgnoreCase));

if (!variableOnLHSIsBeingUsed)
{
Expand Down