Упростите, если проверка обоих имеет значения и оба равны

Я пытаюсь упростить это выражение if, чтобы оно было более читабельным, логика проверяет, имеют ли и x, и y значение, отличное от Undefined, и возвращает false, если они не равны.

bool IsBothMatch(MyClassX x, MyClassY y)
{
    if (y.MyEnumVal != MyEnum.Undefined &&
        x.MyEnumVal != MyEnum.Undefined)
    {
        if (x.MyEnumVal != y.MyEnumVal)
            return false;
    }

    return true;
} 

кажется, вы должны опубликовать это на review.stackexchange.com

MakePeaceGreatAgain 22.05.2019 11:45

If((x.val == y.val) && ((x.val != undefined) && (y.val != undefined)) вернуть true;иначе вернуть false;

Vuk Uskokovic 22.05.2019 11:46

@HimBromBeere К сожалению, review.stackexchange.com не активен как ТАК

mshwf 22.05.2019 11:52

@mshwf Что не меняет того, что вопросы о рабочем коде должны быть здесь.

MakePeaceGreatAgain 22.05.2019 11:54

это codereview.stackexchange.com/questions кстати

mshwf 22.05.2019 11:59
"читабельнее" Это довольно читабельно. Ты хотел сказать короче?
Cid 22.05.2019 12:05
3 метода стилизации элементов HTML
3 метода стилизации элементов HTML
Когда дело доходит до применения какого-либо стиля к нашему HTML, существует три подхода: встроенный, внутренний и внешний. Предпочтительным обычно...
Формы c голосовым вводом в React с помощью Speechly
Формы c голосовым вводом в React с помощью Speechly
Пытались ли вы когда-нибудь заполнить веб-форму в области электронной коммерции, которая требует много кликов и выбора? Вас попросят заполнить дату,...
Стилизация и валидация html-формы без использования JavaScript (только HTML/CSS)
Стилизация и валидация html-формы без использования JavaScript (только HTML/CSS)
Будучи разработчиком веб-приложений, легко впасть в заблуждение, считая, что приложение без JavaScript не имеет права на жизнь. Нам становится удобно...
Flatpickr: простой модуль календаря для вашего приложения на React
Flatpickr: простой модуль календаря для вашего приложения на React
Если вы ищете пакет для быстрой интеграции календаря с выбором даты в ваше приложения, то библиотека Flatpickr отлично справится с этой задачей....
В чем разница между Promise и Observable?
В чем разница между Promise и Observable?
Разберитесь в этом вопросе, и вы значительно повысите уровень своей компетенции.
Что такое cURL в PHP? Встроенные функции и пример GET запроса
Что такое cURL в PHP? Встроенные функции и пример GET запроса
Клиент для URL-адресов, cURL, позволяет взаимодействовать с множеством различных серверов по множеству различных протоколов с синтаксисом URL.
2
6
56
2
Перейти к ответу Данный вопрос помечен как решенный

Ответы 2

Ответ принят как подходящий

Вы можете сжать это в одно утверждение:

bool IsBothMatch(MyClassX x, MyClassY y) {
  return x.MyEnumVal == MyEnum.Undefined
      || y.MyEnumVal == MyEnum.Undefined
      || x.MyEnumVal == y.MyEnumVal;
}

Привет, @Igor, спасибо, но мне нужно, чтобы код вернул true, если y.MyEnumVal равно MyEnum.Undefined

Shahar Shokrani 22.05.2019 12:02

@ShaharShokrani - готово. Просто инвертируйте и используйте логические or (||)

Igor 22.05.2019 12:03

Если вы сравниваете два похожих класса, вы можете наследовать IEquatable<T>

Вы наследуете класс

public class MyClassX : IEquatable<MyClassX>
{

}

Затем вы можете сравнить два

MyClassX.Equals(MyClassY);

Другие вопросы по теме