refactor(diagnostics): улучшить безопасность и читаемость кода в AbstractMagicValueDiagnostic

Добавлены методы для безопасного получения контекстов вызова и параметров, что упрощает логику проверки и уменьшает количество проверок на null. Обновлены существующие методы для использования новых вспомогательных функций, улучшая читаемость и поддерживаемость кода.

Author: johnnyshut

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

@@ -128,16 +128,23 @@ private boolean checkInsideStructureInsertOrAdd(BSLParser.ExpressionContext expr
    * @return true, если это второй параметр метода Вставить структуры
    */
   private boolean isSecondParameterOfStructureInsert(BSLParser.CallParamContext callParam) {
-    var callParamList = (BSLParser.CallParamListContext) callParam.getParent();
+    var callParamListOpt = getCallParamList(callParam);
+    if (callParamListOpt.isEmpty()) {
+      return false;
+    }
+    var callParamList = callParamListOpt.get();
     var callParams = callParamList.callParam();
     var paramIndex = callParams.indexOf(callParam);
 
     if (paramIndex != 1) {
       return false;
     }
 
-    var doCall = (BSLParser.DoCallContext) callParamList.getParent();
-    var methodCall = isInsertMethod(doCall);
+    var doCallOpt = getDoCall(callParamList);
+    if (doCallOpt.isEmpty()) {
+      return false;
+    }
+    var methodCall = isInsertMethod(doCallOpt.get());
     if (methodCall == null) {
       return false;
     }
@@ -181,6 +188,48 @@ private static BSLParser.CallStatementContext findCallStatement(BSLParser.Method
     return null;
   }
 
+  /**
+   * Безопасно получает CallParamListContext из CallParamContext.
+   *
+   * @param callParam контекст параметра вызова
+   * @return Optional с CallParamListContext или empty, если родитель не является CallParamListContext
+   */
+  private static Optional<BSLParser.CallParamListContext> getCallParamList(BSLParser.CallParamContext callParam) {
+    var parent = callParam.getParent();
+    if (parent instanceof BSLParser.CallParamListContext callParamList) {
+      return Optional.of(callParamList);
+    }
+    return Optional.empty();
+  }
+
+  /**
+   * Безопасно получает DoCallContext из CallParamListContext.
+   *
+   * @param callParamList контекст списка параметров вызова
+   * @return Optional с DoCallContext или empty, если родитель не является DoCallContext
+   */
+  private static Optional<BSLParser.DoCallContext> getDoCall(BSLParser.CallParamListContext callParamList) {
+    var parent = callParamList.getParent();
+    if (parent instanceof BSLParser.DoCallContext doCall) {
+      return Optional.of(doCall);
+    }
+    return Optional.empty();
+  }
+
+  /**
+   * Безопасно получает CallParamContext из ExpressionContext.
+   *
+   * @param expression контекст выражения
+   * @return Optional с CallParamContext или empty, если родитель не является CallParamContext
+   */
+  private static Optional<BSLParser.CallParamContext> getCallParam(BSLParser.ExpressionContext expression) {
+    var parent = expression.getParent();
+    if (parent instanceof BSLParser.CallParamContext callParam) {
+      return Optional.of(callParam);
+    }
+    return Optional.empty();
+  }
+
   /**
    * Проверяет, является ли метод методом "Вставить".
    *
@@ -207,7 +256,7 @@ private static BSLParser.MethodCallContext isInsertMethod(BSLParser.DoCallContex
    * @param methodCall контекст вызова метода
    * @return Optional с типом переменной или empty, если не удалось определить
    */
-  private Optional<Boolean> getVariableTypeFromCallStatement(BSLParser.MethodCallContext methodCall) {
+  private static Optional<Boolean> getVariableTypeFromCallStatement(BSLParser.MethodCallContext methodCall) {
     var callStatement = findCallStatement(methodCall);
     if (callStatement == null) {
       return Optional.empty();
@@ -223,14 +272,19 @@ private Optional<Boolean> getVariableTypeFromCallStatement(BSLParser.MethodCallC
    * @return массив из трёх элементов: [CallParamContext, Integer индекс, CallParamListContext] или пустой массив, если не найден
    */
   private static Object[] getCallParamInfo(BSLParser.ExpressionContext expression) {
-    var callParam = expression.getParent();
-    if (callParam instanceof BSLParser.CallParamContext callParamContext) {
-      var callParamList = (BSLParser.CallParamListContext) callParamContext.getParent();
-      var callParams = callParamList.callParam();
-      var paramIndex = callParams.indexOf(callParamContext);
-      return new Object[]{callParamContext, paramIndex, callParamList};
+    var callParamOpt = getCallParam(expression);
+    if (callParamOpt.isEmpty()) {
+      return new Object[0];
+    }
+    var callParamContext = callParamOpt.get();
+    var callParamListOpt = getCallParamList(callParamContext);
+    if (callParamListOpt.isEmpty()) {
+      return new Object[0];
     }
-    return new Object[0];
+    var callParamList = callParamListOpt.get();
+    var callParams = callParamList.callParam();
+    var paramIndex = callParams.indexOf(callParamContext);
+    return new Object[]{callParamContext, paramIndex, callParamList};
   }
 
   /**
@@ -240,7 +294,7 @@ private static Object[] getCallParamInfo(BSLParser.ExpressionContext expression)
    * @param callStatement контекст вызова метода с идентификатором переменной
    * @return Optional с типом: true - соответствие, false - структура, empty - неопределено
    */
-  private Optional<Boolean> getVariableTypeFromAST(BSLParser.CallStatementContext callStatement) {
+  private static Optional<Boolean> getVariableTypeFromAST(BSLParser.CallStatementContext callStatement) {
     var identifier = callStatement.IDENTIFIER();
     var variableName = identifier.getText();
     var assignment = findVariableAssignment(callStatement, variableName);
@@ -461,9 +515,15 @@ private static BSLParser.CallParamContext findCallParamContext(BSLParser.Express
    * @return typeName структуры или null, если не является конструктором структуры
    */
   private static BSLParser.TypeNameContext getStructureTypeName(BSLParser.CallParamContext callParamContext) {
-    var callParamList = (BSLParser.CallParamListContext) callParamContext.getParent();
-    var doCall = (BSLParser.DoCallContext) callParamList.getParent();
-    var newExpression = doCall.getParent();
+    var callParamListOpt = getCallParamList(callParamContext);
+    if (callParamListOpt.isEmpty()) {
+      return null;
+    }
+    var doCallOpt = getDoCall(callParamListOpt.get());
+    if (doCallOpt.isEmpty()) {
+      return null;
+    }
+    var newExpression = doCallOpt.get().getParent();
     if (!(newExpression instanceof BSLParser.NewExpressionContext)) {
       return null;
     }
@@ -571,8 +631,11 @@ private boolean checkInsideCorrespondenceInsert(BSLParser.ExpressionContext expr
     }
 
     var callParamList = (BSLParser.CallParamListContext) callParamInfo[2];
-    var doCall = (BSLParser.DoCallContext) callParamList.getParent();
-    var methodCall = isInsertMethod(doCall);
+    var doCallOpt = getDoCall(callParamList);
+    if (doCallOpt.isEmpty()) {
+      return false;
+    }
+    var methodCall = isInsertMethod(doCallOpt.get());
     if (methodCall == null) {
       return false;
     }
@@ -597,10 +660,19 @@ private boolean checkInsideCorrespondenceInsertFirstParam(BSLParser.ExpressionCo
       return false;
     }
 
-    var callParam = (BSLParser.CallParamContext) expression.getParent();
-    var callParamList = (BSLParser.CallParamListContext) callParam.getParent();
-    var doCall = (BSLParser.DoCallContext) callParamList.getParent();
-    var methodCall = isInsertMethod(doCall);
+    var callParamOpt = getCallParam(expression);
+    if (callParamOpt.isEmpty()) {
+      return false;
+    }
+    var callParamListOpt = getCallParamList(callParamOpt.get());
+    if (callParamListOpt.isEmpty()) {
+      return false;
+    }
+    var doCallOpt = getDoCall(callParamListOpt.get());
+    if (doCallOpt.isEmpty()) {
+      return false;
+    }
+    var methodCall = isInsertMethod(doCallOpt.get());
     if (methodCall == null) {
       return false;
     }
@@ -626,8 +698,11 @@ private static boolean isFirstParameterOfInsertMethod(BSLParser.ExpressionContex
     }
 
     var callParamList = (BSLParser.CallParamListContext) callParamInfo[2];
-    var doCall = (BSLParser.DoCallContext) callParamList.getParent();
-    return isInsertMethod(doCall) != null;
+    var doCallOpt = getDoCall(callParamList);
+    if (doCallOpt.isEmpty()) {
+      return false;
+    }
+    return isInsertMethod(doCallOpt.get()) != null;
   }