доп.кейсы для проверки и исключение лишних условий

Author: artbear

src/main/java/com/github/_1c_syntax/bsl/languageserver/diagnostics/RewriteMethodParameterDiagnostic.java

@@ -86,6 +86,7 @@ private static Stream<Pair<MethodSymbol, ParameterDefinition>> getParametersByVa
 
   private static Stream<VariableSymbol> getVariableByParameter(MethodSymbol method, ParameterDefinition parameterDefinition) {
     return method.getChildren().stream()
+      // в будущем могут появиться и другие символы, подчиненные методам
       .filter(sourceDefinedSymbol -> sourceDefinedSymbol.getSymbolKind() == SymbolKind.Variable)
       .filter(variable -> parameterDefinition.getRange().getStart().equals(variable.getSelectionRange().getStart()))
       .filter(VariableSymbol.class::isInstance)
@@ -132,13 +133,13 @@ private Optional<Reference> isOverwrited(List<Reference> references) {
           return Optional.of(firstDefIntoAssign);
         }
         var refContextInsideDefAssign = getRefContextInsideDefAssign(firstDefIntoAssign, references.get(1));
-        if (refContextInsideDefAssign.isEmpty()){
+        if (refContextInsideDefAssign.isEmpty()) {
           return Optional.of(firstDefIntoAssign);
         }
         var isSelfAssign = Boolean.TRUE.equals(refContextInsideDefAssign
           .map(RewriteMethodParameterDiagnostic::isVarNameOnlyIntoExpression)
           .orElseThrow());
-        if (isSelfAssign && references.size() > COUNT_OF_PAIR_FOR_SELF_ASSIGN){
+        if (isSelfAssign && references.size() > COUNT_OF_PAIR_FOR_SELF_ASSIGN) {
           return isOverwrited(references.subList(COUNT_OF_PAIR_FOR_SELF_ASSIGN, references.size()));
         }
       }
@@ -155,11 +156,9 @@ private Optional<RuleNode> getRefContextInsideDefAssign(Reference defRef, Refere
       .map(RuleNode::getParent)
       .filter(BSLParser.AssignmentContext.class::isInstance)
       .map(BSLParser.AssignmentContext.class::cast);
-    if (assignment.isEmpty()) {
-      return Optional.empty();
-    }
 
-    return Trees.findNodeContainsPosition(assignment.get(), nextRef.getSelectionRange().getStart())
+    return assignment.flatMap(assignContext ->
+        Trees.findNodeContainsPosition(assignContext, nextRef.getSelectionRange().getStart()))
       .map(TerminalNode::getParent);
   }
 
@@ -169,7 +168,6 @@ private static boolean isVarNameOnlyIntoExpression(RuleNode refContext) {
       .map(BSLParser.ComplexIdentifierContext.class::cast)
       .filter(node -> node.getChildCount() == 1)
       .map(RuleNode::getParent)
-      .filter(memberContext -> memberContext.getChildCount() == 1)
       .filter(BSLParser.MemberContext.class::isInstance)
       .map(RuleNode::getParent)
       .filter(expression -> expression.getChildCount() == 1)

src/test/resources/diagnostics/RewriteMethodParameterDiagnostic.bsl

@@ -55,3 +55,17 @@
     Парам131 = 1 + Парам131; // не ошибка
     Возврат Парам131;
 КонецПроцедуры
+
+Процедура Тест14(Знач Парам141)
+    Парам141.Значение = 10; // не ошибка
+КонецПроцедуры
+
+Процедура Тест15(Знач Парам151)
+    Парам151.Значение1 = Парам151.Значение2; // не ошибка
+    Парам151 = 10; // не ошибка
+КонецПроцедуры
+
+Процедура Тест16(Знач Парам161)
+    Парам161.Значение = Парам161; // не ошибка
+    Парам161 = 10; // не ошибка
+КонецПроцедуры