Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

Why is this code throwing an InvalidOperationException?

I think that my code should make the ViewBag.test property equal to "No Match", but instead it throws an InvalidOperationException.

Why is this?

string str = "Hello1,Hello,Hello2";
string another = "Hello5";
string retVal = str.Split(",".ToCharArray(), StringSplitOptions.RemoveEmptyEntries)
                   .First(p => p.Equals(another));
if (str == another)
{
   ViewBag.test = "Match";
}
else
{
   ViewBag.test = "No Match"; //this does not happen when it should
}
like image 684
user2398766 Avatar asked May 19 '13 15:05

user2398766


1 Answers

As you can see here, the First method throws an InvalidOperationException when the sequence on which it is called is empty. Since no element of the result of the split equals Hello5, the result is an empty list. Using First on that list will throw the exception.

Consider using FirstOrDefault, instead (documented here), which, instead of throwing an exception when the sequence is empty, returns the default value for the type of the enumerable. In that case, the result of the call will be null, and you should check for that in the rest of the code.

It might be cleaner still to use the Any Linq method (documented here), which returns a bool.

string str = "Hello1,Hello,Hello2";
string another = "Hello5";
bool retVal = str.Split(",".ToCharArray(), StringSplitOptions.RemoveEmptyEntries)
                   .Any(p => p.Equals(another));
if (retVal)
{
   ViewBag.test = "Match";
}
else
{
   ViewBag.test = "No Match"; //not work
}

And now the obligatory one liner using the ternary operator:

string str = "Hello1,Hello,Hello2";
string another = "Hello5";
ViewBag.test = str.Split(",".ToCharArray(), StringSplitOptions.RemoveEmptyEntries)
                   .Any(p => p == another) ? "Match" : "No Match";

Note that I also used == here to compare strings, which is considered more idiomatic in C#.

like image 68
Ben Reich Avatar answered Oct 17 '22 23:10

Ben Reich