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

Бесполезный тернарник / Странный тернарник #203

Open
asosnoviy opened this issue Jun 11, 2019 · 12 comments · May be fixed by #1820
Open

Бесполезный тернарник / Странный тернарник #203

asosnoviy opened this issue Jun 11, 2019 · 12 comments · May be fixed by #1820
Assignees
@EvgSychev
Labels
component/diagnostics Доработка / создание диагностики

Comments

@asosnoviy
Copy link
Member 

Если ?(Стр.Emp_emptype = Null, 0, Стр.Emp_emptype) = 0 Тогда
КонецЕсли;
@nixel2007
Copy link
Member

типа присваивание того же значения, на которое проверяется?

у тебя часто такое стреляет?

@asosnoviy
Copy link
Member Author

Только что из ревью вынул.
Имхо Равносильно

Если Стр.Emp_emptype = Null Тогда

@asosnoviy asosnoviy added the component/diagnostics Доработка / создание диагностики label Jun 16, 2019
@qtLex
Copy link
Contributor

qtLex commented Jul 21, 2019

На мой взгляд

?(Стр.Emp_emptype = Null, 0, Стр.Emp_emptype) = 0

не тоже, что и

Стр.Emp_emptype = Null

Оно равносильно

Стр.Emp_emptype = Null ИЛИ Стр.Emp_emptype = 0

Сильно ли третья запись лучше первой уже не так очевидно, как вторая лучше первой.

@asosnoviy
Copy link
Member Author

asosnoviy commented Jul 21, 2019 •
edited
Loading

Согласен, я неправильно его прочел при беглом осмотре. Значит его точно не должно быть =)

@qtLex
Copy link
Contributor

qtLex commented Jul 21, 2019

Тут согласен ))

@asosnoviy asosnoviy changed the title Бесполезный тернарник Бесполезный тернарник / Странный тернарник Jul 21, 2019
@asosnoviy
Copy link
Member Author

Не следует использовать Тернарный оператор в условии?

@qtLex
Copy link
Contributor

qtLex commented Jul 21, 2019

Думаю, да. В #13 реализована проверка на вложенные тернарные операторы. Тут, похоже, та же история.

@qtLex qtLex mentioned this issue Jul 22, 2019
Merged
@EvgSychev
Copy link
Contributor

@asosnoviy по этому ишью нужна доработка?

@EvgSychev
Copy link
Contributor

назначьте досаду на меня, пожалуйста

@asosnoviy
Copy link
Member Author

Тернарник возвращающий булево. Тернарник возвращающий два одинаковых значения.

@asosnoviy
Copy link
Member Author

И на тернарник в Если тоже можно =)

@asosnoviy
Copy link
Member Author

И на тернарник в Если тоже можно =)

А, на это уже есть

@otymko otymko linked a pull request Sep 17, 2021 that will close this issue
Open
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@EvgSychev EvgSychev

Labels
component/diagnostics Доработка / создание диагностики
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add useless ternary operator diagnostic EvgSychev/bsl-language-server
4 participants
@nixel2007 @qtLex @asosnoviy @EvgSychev