Необходимо проверить условие, если значение равно `YES` или` NO`.И для недопустимых значений он должен выдать какое-то сообщение об ошибке

Question

Вот код, который я пробовал:

  if(!string.IsNullOrEmpty(Taskinfo.DoRollOver) &&(Taskinfo.DoRollOver != "YES" || Taskinfo.DoRollOver != "NO"))
  {
     throw new DataException("DoRollOver is not valid");
  }

Но если я пытаюсь предоставить действительные данные, такие как YES или NO, все равно возникает исключение.

Answer 1

Это условие всегда верно.Если Taskinfo.DoRollOver - «ДА», то оно не равно «НЕТ», и наоборот.Вы должны использовать логический оператор &&, а не логический оператор ||:

if (string.IsNullOrEmpty(Taskinfo.DoRollOver) ||
    (Taskinfo.DoRollOver != "YES" && // Here!
     T0askinfo.DoRollOver != "NO"))
{
    throw new DataException("DoRollOver is not valid");
}

Answer 2

Я бы сказал, что вы немного перевернули логику.

if((Taskinfo.DoRollOver == "YES")||(Taskinfo.DoRollOver == "NO"))
{
  // do something
}
else
  throw new DataException("DoRollOver is not valid");

Также, если TaskInfo.DoRollOver - строка, я бы использовал TaskInfo.DoRollOver.ToUpper () - на всякий случай.

Answer 3

Попробуйте изменить условия, подобные этим

if (string.IsNullOrEmpty(Taskinfo.DoRollOver) ||(Taskinfo.DoRollOver != "YES" && T0askinfo.DoRollOver != "NO"))
{
    throw new DataException("DoRollOver is not valid");
}

Answer 4

может быть, это может помочь вам

 if(!string.IsNullOrEmpty(Taskinfo.DoRollOver))
  {
    if(Taskinfo.DoRollOver != "YES" && Taskinfo.DoRollOver != "NO")
      {
       throw new DataException("DoRollOver is not valid");
      }
  }else
  {
    //Do Something
  }

Answer 5

Стремясь к удобочитаемости и простоте, я бы разделил условия:

if(Taskinfo.DoRollOver == "YES")
{
  // do smoething
}
else if(Taskinfo.DoRollOver == "NO")
{
  // do smoething else
}
else
  throw new DataException("DoRollOver is not valid");