72

If an array is empty, it looks like you can't check it's length using ".length". What's the best way to check if an array is empty?

13 Answers 13

175

You can absolutely check an empty array's length. However, if you try to do that on a null reference you'll get an exception. I suspect that's what you're running into. You can cope with both though:

if (array == null || array.Length == 0)

If that isn't the cause, please give a short but complete program demonstrating the problem. If that was the cause, it's worth taking a moment to make sure you understand null references vs "empty" collections/strings/whatever.

15
  • 47
    The important thing to note about the code above is that in csharp, the or operator (||) is short circuiting. It sees that the array is null (statement is true) and does NOT attempt to evaluate the second statement. If it did, you'd still get a NullReferenceException and would have to nest your if statements. Not all languages are like this, VB.Net being one example. The default Or operator is NOT short-circuiting; they have OrElse for that. support.microsoft.com/kb/817250
    – Tim Coker
    Jul 9, 2010 at 14:16
  • 4
    Jon Skeet has spoken!
    – RedAces
    Jan 22, 2014 at 15:59
  • 1
    @amit: No, you'd want to use ?? there as well: array?.Length ?? 0 == 0
    – Jon Skeet
    Dec 20, 2017 at 13:56
  • 1
    @amitjha: It's not equivalent to the condition in my question. If array is null, then your if condition won't be met because null isn't equal to 0, whereas mine will.
    – Jon Skeet
    Dec 20, 2017 at 14:22
  • 2
    @amitjha: Well I don't know what your unit test looks like, but they're not the same. (Actually you need brackets round array?.Length ?? 0 due to precedence, but...) See dotnetfiddle.net/uRM74Y
    – Jon Skeet
    Dec 20, 2017 at 14:28
15

Yeah, for safety I would probably do:

if(array == null || array.Length == 0)
14

You can use

if (array == null || array.Length == 0)

OR

if (!(array != null && array.Length != 0))

NOTE!!!!! To insure that c# will implement the short circuit correctly; you have to compare that the object with NULL before you go to the children compare of the object.

C# 7.0 and above

if(!(array?.Length != 0))
2
  • 1
    the expression must change to if( ! (array?.Length > 0 ) ) since without the parenthesis, the not operator tries to evaluate on the expression array?.length which is of int type and complains. and the check should be for length > 0 when using the not operator.
    – supi
    Aug 29, 2020 at 5:04
  • if( !(array?.Length > 0 ) ) works better thanks Jan 28, 2021 at 19:47
5

Since .Net >= 5.0 the best way is to use Any:

if(!array.Any()) {
   //now you sure it's empty
}

For nullable arrays:

if(!(array?.Any() == true)) {
   //now you sure it's null or empty
}
4

You can use .Length == 0 if the length is empty and the array exists, but are you sure it's not null?

3

As other have already suggested it is likely you are getting a NullReferenceException which can be avoided by first checking to see if the reference is null. However, you need to ask yourself whether that check is actually warranted. Would you be doing it because the reference really might be null and it being null has a special meaning in your code? Or would you be doing it to cover up a bug? The nature of the question leads me to believe it would be the latter. In which case you really need to examine the code in depth and figure out why that reference did not get initialized properly in the first place.

3

If array is null, trying to derefrence array.Length will throw a NullReferenceException. If your code considers null to be an invalid value for array, you should reject it and blame the caller. One such pattern is to throw ArgumentNullException:

void MyMethod(string[] array)
{
    if (array is null) throw new ArgumentNullException(nameof(array));

    if (array.Length > 0)
    {
        // Do something with array…
    }
}

If you want to accept a null array as an indication to not do something or as an optional parameter, you may simply not access it if it is null:

void MyMethod(string[] array)
{
    if (array is not null)
    {
        // Do something with array here…
    }
}

If you want to avoid touching array when it is either null or has zero length, then you can check for both at the same time with C#-6’s null coalescing operator.

void MyMethod(string[] array)
{
    if (array?.Length > 0)
    {
        // Do something with array…
    }
}

Superfluous Length Check

It seems strange that you are treating the empty array as a special case. In many cases, if you, e.g., would just loop over the array anyway, there’s no need to treat the empty array as a special case. foreach (var elem in array) {«body»} will simply never execute «body» when array.Length is 0. If you are treating array == null || array.Length == 0 specially to, e.g., improve performance, you might consider leaving a comment for posterity. Otherwise, the check for Length == 0 appears superfluous.

Superfluous code makes understanding a program harder because people reading the code likely assume that each line is necessary to solve some problem or achieve correctness. If you include unnecessary code, the readers are going to spend forever trying to figure out why that line is or was necessary before deleting it ;-).

2

This is the best way. Please note Array is an object in NET so you need to check for null before.

2

Jon Skeet answered correctly. Just remember that the order of the test in the "IF" is important. Check for the null before the length. I also prefer to put the null on the left side of the equal which is a habit I got from Java that made the code more efficient and fast… I don't think it's important in a lot of application today, but it's a good practice!

if (null == array || array.Length == 0)
1

Your suggested test is fine, so long as the array is intialised...

Martin.

1

check if the array is null first so you would avoid a null pointer exception

logic in any language: if array is null or is empty :do ....

0

do you mean empty or null, two different things,

if the array is instantiated but empty, then length is correct, if it has not been instantiated then test vs null

0

// there is another method to check if the array contains items or not

if ( array.Count == 0) return;

1
  • Welcome to Stack Overflow, thanks for your first answer! Unfortunately I believe your answer has the same problem as the original question, which is that if the array is null it will cause a Null Reference Exception
    – Kevin
    Apr 6, 2021 at 22:20

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service, privacy policy and cookie policy

Not the answer you're looking for? Browse other questions tagged or ask your own question.