If ((someTask?.IsCompleted ?? true) == true) Это говнокод или терпимо?

А лол, реально, сенкс, а то я автоматом всегда пишу == false или == true явно для более удобного чтения кода, в этом случае можно было и без этого обойтись, невнимательность

0

09.11.2020

Vova Lantsov 🇺🇦

Владислав
А лол, реально, сенкс, а то я автоматом всегда пиш...

== falsе явно ещё вижу смысл, == true абсолютно бессмысленно)

0

09.11.2020

Владислав Автор вопроса

Vova Lantsov 🇺🇦
== falsе явно ещё вижу смысл, == true абсолютно бе...

Не знаю, лично мое мнение, if(method() == true) намного приятнее смотреть и понимать, чем if (method()), жаль что такое не практикуется. Просто в 1 варианте, мне кажется что можно понять суть логического выражения быстрее, чем во 2. Да, 2 варик меньше по коду, но не в том дело, повторяю, лично мое мнение, лично моя субъективная реальность.

0

09.11.2020

Vladyslav T.

Владислав
Не знаю, лично мое мнение, if(method() == true) н...

если метод возвращает булевое значение, то обычно его называют начиная с Is или Has, например IsEnabled() или HasFlag(), поэтому когда пишешь if (IsEnabled()) то сразу все понятно и не надо ничего уточнять с == true

0

09.11.2020

Aleksey

Владислав
Не знаю, лично мое мнение, if(method() == true) н...

if (method() - думаю тут важно правильно назвать метод

0

09.11.2020

Aleksey

Vladyslav T.
если метод возвращает булевое значение, то обычно ...

+

0

09.11.2020

Vova Lantsov 🇺🇦

Vladyslav T.
если метод возвращает булевое значение, то обычно ...

Согласен

0

10.11.2020

Vabka

Сделай екстеншен что-то типа public static T FallbackOn(this T? value, T fallback) => value ?? fallback; и тогда будет if(someTask?.IsCompleted.FallbackOn(true))

0

10.11.2020

Vabka

Владислав
Не знаю, лично мое мнение, if(method() == true) н...

просто не надо называть это "method" Вот если if(IsSomething), то никакой ==true не надо

0

10.11.2020

Илон Маск

в случае, когда IsCompleted имеет значение отлично от true, код не валидный, например, вызвался CallbackToken, и IsCompleted кто-то поставил в false, идентифицируя таким таким образом fail state

0

10.11.2020

Илон Маск

лучше так if (someTask?.IsCompleted.HasValue && someTask?.IsCompleted.Value == true)

0

10.11.2020

Oleksii Zaichenko

Илон Маск
лучше так if (someTask?.IsCompleted.HasValue && s...

Не скомпилится

0

10.11.2020

Илон Маск