Add useless ternary operator diagnostic by EvgSychev · Pull Request #1820 · 1c-syntax/bsl-language-server

EvgSychev · 2021-09-16T17:36:48Z

Описание

Связанные задачи

Closes #203

Чеклист

Общие

Ветка PR обновлена из develop
Отладочные, закомментированные и прочие, не имеющие смысла участки кода удалены
Изменения покрыты тестами
Обязательные действия перед коммитом выполнены (запускал команду gradlew precommit)

Для диагностик

Описание диагностики заполнено для обоих языков (присутствуют файлы для обоих языков, для русского заполнено все подробно, перевод на английский можно опустить)

Дополнительно

EvgSychev · 2021-09-16T17:37:58Z

Забыл досаду приатачить, как исправить?

nixel2007 · 2021-09-17T10:38:12Z

@EvgSychev отредактировать первый пост. но вроде @otymko уже поправил

...a/com/github/_1c_syntax/bsl/languageserver/diagnostics/UselessTernaryOperatorDiagnostic.java

artbear

похоже на ошибку

...a/com/github/_1c_syntax/bsl/languageserver/diagnostics/UselessTernaryOperatorDiagnostic.java

EvgSychev · 2021-09-24T16:48:48Z

3 варианта потому, что это не один и тот же optinal, два преобразуются и каждый по своему, третий на тот случае если ничего не найдено, тесты код прошёл

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

nixel2007 · 2021-10-06T21:30:58Z

...a/com/github/_1c_syntax/bsl/languageserver/diagnostics/UselessTernaryOperatorDiagnostic.java

+  }
+
+  private String getAdaptedText(String text) {
+    if(documentContext.getServerContext().getConfiguration().getDefaultLanguage() == MDLanguage.ENGLISH) {


лучше заменить на вызов documentContext.getScriptVariantLocale()

ну и вообще бы это перевесить на работу с Resources#getResourceString

nixel2007 · 2021-10-06T21:36:08Z

...a/com/github/_1c_syntax/bsl/languageserver/diagnostics/UselessTernaryOperatorDiagnostic.java

+      .filter(BSLParser.ConstValueContext.class::isInstance)
+      .map(BSLParser.ConstValueContext.class::cast);
+
+    return tmpCtx


тэкс, а зачем все это ниже, если можно просто tmpCtx.filter(ctx -> ctx.TRUE() || ctx.FALSE).isPresent() и сменить возвращаемое значение с int на boolean?

Дальше по коду нужно знать в какой ветке у нас Истина, в какой ложь

nixel2007 · 2021-10-06T21:38:07Z

...a/com/github/_1c_syntax/bsl/languageserver/diagnostics/UselessTernaryOperatorDiagnostic.java

+
+    var tmpCtx = Optional.of(expCtx)
+      .filter(ctx -> ctx.children.size() == 1)
+      .map(ctx -> (BSLParser.MemberContext) ctx.getChild(0))


просто .map(ctx -> ctx.member()) или сразу .map(BSLParser.ExpressionContext::member)), нет нужды работать с детьми в грязную

nixel2007 · 2021-10-06T21:39:26Z

...a/com/github/_1c_syntax/bsl/languageserver/diagnostics/UselessTernaryOperatorDiagnostic.java

+    var tmpCtx = Optional.of(expCtx)
+      .filter(ctx -> ctx.children.size() == 1)
+      .map(ctx -> (BSLParser.MemberContext) ctx.getChild(0))
+      .map(ctx -> ctx.getChild(0))


вместо этого вызова и двух вызовов ниже можно взять .map(ctx -> ctx.constValue(). если constValue в этом expression нет, то метод вернет null и Optional станет empty

nixel2007 · 2021-10-06T21:42:30Z

...a/com/github/_1c_syntax/bsl/languageserver/diagnostics/UselessTernaryOperatorDiagnostic.java

+    var exp = ctx.expression();
+    var condition = getBooleanToken(exp.get(0));
+    var trueBranch = getBooleanToken(exp.get(1));
+    var falseBranch = getBooleanToken(exp.get(2));


я бы на всякий случай перед всеми этими дерганиями по индексу проверил, что exp.size() == 3. опасаюсь IndexOutOfBoundException при наборе текста

Если не 3 падаем?

...a/com/github/_1c_syntax/bsl/languageserver/diagnostics/UselessTernaryOperatorDiagnostic.java

…tics/UselessTernaryOperatorDiagnostic.java

nixel2007 · 2021-10-08T17:40:15Z

/rebase

nixel2007 · 2021-10-26T10:12:56Z

@EvgSychev мерж поломал addDiagnostic

EvgSychev · 2021-10-26T17:03:00Z

@nixel2007 ошибочно сделал pull, будет исправлено

nixel2007 · 2025-12-02T13:55:32Z

Superseded by #3629

Add useless ternary operator diagnostic

732b5ac

EvgSychev closed this Sep 17, 2021

nixel2007 reopened this Sep 17, 2021

precommit done

f103c75

artbear suggested changes Sep 18, 2021

View reviewed changes

suggested changes have been made

cd8ba4d

artbear reviewed Sep 22, 2021

View reviewed changes

...a/com/github/_1c_syntax/bsl/languageserver/diagnostics/UselessTernaryOperatorDiagnostic.java Outdated Show resolved Hide resolved

Update UselessTernaryOperatorDiagnostic.java

541b137

artbear suggested changes Sep 24, 2021

View reviewed changes

...a/com/github/_1c_syntax/bsl/languageserver/diagnostics/UselessTernaryOperatorDiagnostic.java Show resolved Hide resolved

Update UselessTernaryOperatorDiagnostic.java

811023a

nixel2007 reviewed Oct 6, 2021

View reviewed changes

src/main/java/com/github/_1c_syntax/bsl/languageserver/diagnostics/DiagnosticStorage.java Outdated Show resolved Hide resolved

nixel2007 reviewed Oct 6, 2021

View reviewed changes

src/main/java/com/github/_1c_syntax/bsl/languageserver/diagnostics/DiagnosticStorage.java Outdated Show resolved Hide resolved

nixel2007 reviewed Oct 6, 2021

View reviewed changes

src/main/java/com/github/_1c_syntax/bsl/languageserver/diagnostics/DiagnosticStorage.java Outdated Show resolved Hide resolved

Apply suggestions from code review

0c78fb2

nixel2007 reviewed Oct 6, 2021

View reviewed changes

...a/com/github/_1c_syntax/bsl/languageserver/diagnostics/UselessTernaryOperatorDiagnostic.java Outdated Show resolved Hide resolved

Update src/main/java/com/github/_1c_syntax/bsl/languageserver/diagnos…

bac62f7

…tics/UselessTernaryOperatorDiagnostic.java

nixel2007 added the pr/waits-for-feedback label Oct 8, 2021

Merge branch 'develop' into feature/useless-ternary-operator

44e3d16

theshadowco mentioned this pull request Dec 1, 2025

Диагностика бессмысленного тернарного оператора #3629

Merged

5 tasks

nixel2007 closed this Dec 2, 2025

Conversation

EvgSychev commented Sep 16, 2021 • edited by otymko Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Описание

Связанные задачи

Чеклист

Общие

Для диагностик

Дополнительно

Uh oh!

EvgSychev commented Sep 16, 2021

Uh oh!

nixel2007 commented Sep 17, 2021 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

artbear left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

EvgSychev commented Sep 24, 2021

Uh oh!

Uh oh!

Uh oh!

Uh oh!

nixel2007 Oct 6, 2021

Choose a reason for hiding this comment

Uh oh!

nixel2007 Oct 6, 2021

Choose a reason for hiding this comment

Uh oh!

nixel2007 Oct 6, 2021

Choose a reason for hiding this comment

Uh oh!

EvgSychev Oct 31, 2021

Choose a reason for hiding this comment

Uh oh!

nixel2007 Oct 6, 2021

Choose a reason for hiding this comment

Uh oh!

nixel2007 Oct 6, 2021

Choose a reason for hiding this comment

Uh oh!

nixel2007 Oct 6, 2021 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

EvgSychev Oct 26, 2021

Choose a reason for hiding this comment

Uh oh!

Uh oh!

nixel2007 commented Oct 8, 2021

Uh oh!

nixel2007 commented Oct 26, 2021

Uh oh!

EvgSychev commented Oct 26, 2021

Uh oh!

nixel2007 commented Dec 2, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

EvgSychev commented Sep 16, 2021 •

edited by otymko

Loading

nixel2007 commented Sep 17, 2021 •

edited

Loading

nixel2007 Oct 6, 2021 •

edited

Loading