[MOD] UsingFindElementByString - замечания на ПоискПоНомеру с параметром-литералом

docs/diagnostics/UsingFindElementByString.md

@@ -1,20 +1,30 @@
-# Использование методов "НайтиПоНаименованию" и "НайтиПоКоду" (UsingFindElementByString)
+# Использование методов "НайтиПоНаименованию", "НайтиПоКоду" и "НайтиПоНомеру" (UsingFindElementByString)
 
 <!-- Блоки выше заполняются автоматически, не трогать -->
 ## Описание диагностики
 
-Запрещено использовать методы поиска элементов "НайтиПоНаименованию" или "НайтиПоКоду".
+Правило находит использование методов `НайтиПоНаименованию`, `НайтиПоКоду` или `НайтиПоНомеру` с указанием конкретных и
+уникальных номеров, кодов и названий элементов или документов.
+Скорее всего, подобный код не будет работать в других ИБ, что приведет к ошибкам выполнения.
+Возможно, в финальный код попал тестовый код, что также не рекомендовано.
+
+Константные значения данных из ИБ рекомендуется указывать в константах ИБ, предопределенных элементах метаданных.
 
 ## Примеры
 
+Неправильно:
 ```bsl
 Должность = Справочники.Должности.НайтиПоНаименованию("Ведущий бухгалтер");
 ```
+или
+```bsl
+Должность = Справочники.Должности.НайтиПоКоду("00-0000001");
+```
 
 или
 
 ```bsl
-Должность = Справочники.Должности.НайтиПоКоду("00-0000001");
+ОбъектНазначения = Документы.ПередачаТоваровМеждуОрганизациями.НайтиПоНомеру("0000-000001", ТекущаяДата());
 ```
 
 Допустимо использование:
@@ -24,3 +34,7 @@
 ```bsl
 Справочники.КлассификаторБанков.НайтиПоКоду(СведенияОБанке.БИК);
 ```
+
+```bsl
+Документы.Реализация.НайтиПоНомеру(НомерДокумента);
+```

docs/en/diagnostics/UsingFindElementByString.md

@@ -1,4 +1,4 @@
-# Using FindByName and FindByCode (UsingFindElementByString)
+# Using FindByName, FindByCode and FindByNumber (UsingFindElementByString)
 
 <!-- Блоки выше заполняются автоматически, не трогать -->
 ## Description

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

@@ -30,7 +30,6 @@
 import com.github._1c_syntax.utils.CaseInsensitivePattern;
 import org.antlr.v4.runtime.tree.ParseTree;
 
-import java.util.regex.Matcher;
 import java.util.regex.Pattern;
 
 @DiagnosticMetadata(
@@ -48,13 +47,13 @@
 public class UsingFindElementByStringDiagnostic extends AbstractVisitorDiagnostic {
 
  private final Pattern pattern = CaseInsensitivePattern.compile(
- "(НайтиПоНаименованию|FindByDescription|НайтиПоКоду|FindByCode)"
+ "(НайтиПоНаименованию|FindByDescription|НайтиПоКоду|FindByCode|НайтиПоНомеру|FindByNumber)"
  );
 
  @Override
  public ParseTree visitMethodCall(BSLParser.MethodCallContext ctx) {
- Matcher matcher = pattern.matcher(ctx.methodName().getText());
- if (matcher.find()) {
+ var matcher = pattern.matcher(ctx.methodName().getText());
+ if (matcher.matches()) {
  BSLParser.CallParamContext param = ctx.doCall().callParamList().callParam().get(0);
  if (param.children == null ||
  param.getStart().getType() == BSLParser.STRING ||

src/main/resources/com/github/_1c_syntax/bsl/languageserver/configuration/parameters-schema.json

@@ -1947,13 +1947,13 @@
  "$id": "#/definitions/UsingExternalCodeTools"
  },
  "UsingFindElementByString": {
-  "description": "Using FindByName and FindByCode",
+ "description": "Using FindByName, FindByCode and FindByNumber",
  "default": true,
  "type": [
  "boolean",
  "object"
  ],
-  "title": "Using FindByName and FindByCode",
+ "title": "Using FindByName, FindByCode and FindByNumber",
  "$id": "#/definitions/UsingFindElementByString"
  },
  "UsingGoto": {

src/main/resources/com/github/_1c_syntax/bsl/languageserver/diagnostics/UsingFindElementByStringDiagnostic_en.properties

@@ -1,2 +1,2 @@
 diagnosticMessage=Don't use method "%s" and finding by string.
-diagnosticName=Using FindByName and FindByCode
+diagnosticName=Using FindByName, FindByCode and FindByNumber

src/main/resources/com/github/_1c_syntax/bsl/languageserver/diagnostics/UsingFindElementByStringDiagnostic_ru.properties

@@ -1,2 +1,2 @@
 diagnosticMessage=Не следует использовать метод "%s" и поиск по строке
-diagnosticName=Использование методов "НайтиПоНаименованию" и "НайтиПоКоду"
+diagnosticName=Использование методов "НайтиПоНаименованию", "НайтиПоКоду" и "НайтиПоНомеру"

src/test/java/com/github/_1c_syntax/bsl/languageserver/diagnostics/UsingFindElementByStringDiagnosticTest.java

@@ -42,16 +42,17 @@ void runTest() {
  List<Diagnostic> diagnostics = getDiagnostics();
 
  // then
- assertThat(diagnostics).hasSize(6);
+ assertThat(diagnostics).hasSize(9);
  assertThat(diagnostics, true)
  .hasRange(7, 38, 7, 78)
  .hasRange(9, 40, 9, 61)
  .hasRange(13, 27, 13, 59)
  .hasRange(24, 35, 24, 53)
  .hasRange(27, 35, 27, 51)
  .hasRange(30, 27, 30, 53)
+ .hasRange(39, 67, 110)
+ .hasRange(41, 35, 53)
+ .hasRange(44, 29, 49)
  ;
-
  }
-
 }

src/test/resources/diagnostics/UsingFindElementByStringDiagnostic.bsl

@@ -30,4 +30,18 @@
  А = Справочники.Валюты.Функция(
  Справочники.Валюты.НайтиПоНаименованию("777") // сработает
  );
-КонецПроцедуры
+КонецПроцедуры
+
+Процедура Тест3()
+
+ Наименование = "333"; // Пока не сработает
+ Значение = Документы.Реализация.НайтиПоНомеру(Наименование);
+
+ ОбъектНазначения = Документы.ПередачаТоваровМеждуОрганизациями.НайтиПоНомеру("0000-000001", ТекущаяДата()); // замечание
+
+ Значение3 = БизнесПроцессы.БП1.НайтиПоНомеру(333); // замечание
+
+ А = Документы.Реализация.Функция(
+ Документы.Реализация.НайтиПоНомеру("333") // замечание
+ );
+КонецПроцедуры