Martin Krause

Posted on Oct 25, 2021 • Edited on Nov 11, 2021

1 line of code: How to check if an Array is empty

#javascript #webdev #productivity #codequality

const isEmptyArray = ({ length }) => length === 0;

Assumes that the given argument is an Array and checks if it has entries.

Returns:

a boolean false if the array is not empty
a boolean true if the array is empty

The repository & npm package

You can find the all the utility functions from this series at github.com/martinkr/onelinecode
The library is also published to npm as @onelinecode for your convenience.

The code and the npm package will be updated every time I publish a new article.

Follow me on Twitter: @martinkr and consider to buy me a coffee

Photo by zoo_monkey on Unsplash

Top comments (15)

Aleksandr Hovhannisyan • Oct 25 '21 • Edited

I would prefer something like this:

const isEmptyArray = (arr) => {
  return Array.isArray(arr) && !arr.length;
}

No need for a ternary since Array.isArray and the negation operator already return booleans.

Imo, it doesn't make sense to throw (edit: return) an error in this case since the function is just a predicate. So it should return true or false. Otherwise, the user has to wrap it in a try-catch.

I also don't recommend using braceless arrow functions; I've often found myself refactoring them so I can use a debugger/console log inside the function later on. It seems tempting to omit the braces at first, but there's no practical benefit in doing so.

Martin Krause • Oct 25 '21

Thank you for your extensive contribution.

I can understand you comments and see where ou are coming from. Let me explain why I use this code.

I was using the version you posted for some time but ran into the problem that it returns false if the argument is not an array. This lead to problems in the code relying on the return value.

I'm deliberately not throwing the error,so it's not necessary to wrap it in an try ... catch - you can decide how to handle the error by yourself.

Cheers!

𒎏Wii 🏳️‍⚧️ • Oct 26 '21

It seems tempting to omit the braces at first, but there's no practical benefit in doing so

Readability; there's also no real downside (if you want to add a console.log, there's better ways to do it).

Aleksandr Hovhannisyan • Oct 26 '21

I disagree. I've run into this trap so many times myself and then still had to go back and add a console statement or debugger. Same issue with early-return statements that don't have braces, especially if you have multiple ones. It's useful to know when/if you've landed in a particular scope at all.

Adam Crockett 🌀 • Jan 19 '22

Because 0 is falsey, it's very important to remember that.

I would try this slightly shorter question, "is length of 0 not not falsey?"

const isEmptyArray = ({ length }) => !!length

But here you can see the problem with the entire concept, if length is already falsey or truthy and length can be infinitly truthy or infinitly falsey, why even cast to a Boolean?

I would even go as far as to say that a Boolean is not actually useful in most cases as information is lost 🙂, know your trutheys from your falseys and you won't have to overerly defend your code. Typescript helps too 😂

Erik Waters • Oct 26 '21

IMO the utility of this function depends on how you interpret an array being "empty": if you pass an array of empty items the result may not be what the user expects

let arr = Array(2) // [ <2 empty items> ]
isEmptyArray(arr) // false

lizy0329 • Oct 27 '21

lodash not be user expects too

const arr = Array(2)
_.isEmpty(arr);
=>
false

Martin Krause • Oct 26 '21

Thank you all for your contributions.

I closely followed the discussions and took some valuable insights for this and the upcoming code snippets.

For all the functions in this series we will stick to certain assumptions:

as long as it's javascript, we assume the user provides valid input. No type checking or other verifications.
E.g.: No more checking if it's actually an array
keep the functions as simple as possible and reduce the functionality to a core. E.g.: empty items are still items, if necessary clean the array before passing it to the function.

I adjusted the code to the function LUKESHIRU suggested.

It seems to be the simplest form and passes all my test cases (given the assumptions).

Thanks you again and looking forward to your contributions to the upcoming articles.

Cheers!

Nicolas Hervé • Oct 25 '21

Returning an error is not a best practice, your function should thow an error instead with a message when possible.

Martin Krause • Oct 25 '21

Thank you for your comment.

This being part of a more "universal library" approach, I decided not to throw an error with an error message.
The purpose of this one liner is to check if the given argument is an emtpy array, How you handle an error is out of scope. Throwing an error would force a try ... catch block, returning the error let's you decide how to handle (or ignore) the issue.
The same goes for the error message. I could be a status code or a text message, there are different preferences which should fit into your complete application - so again, check for the error and proceed to your liking.

I hope you can see my rationale behind the code.

Cheers!

Nicolas Hervé • Oct 26 '21

A "universal library" should respect language best practices. There is a standard way to deal with errors in javascript and you just introduce a new solution that is really questionable in my opinion.

You say throwing an error would force the use of a try catch block but your solution introduce the need to check the result too.
One big problem is that Error is a truthy value so that is error prone for the caller of the function :

if(isEmptyArray({})) {
  // oups its not an empty array
}

It is a bad practice to have a function that can return different types in js (boolean | Error) and having the caller to check the return type. If you really need to, you could use a tuple or an object that the caller can deconstruct.

Kuch311 • Oct 27 '21

Following

João Lucas Silva • Oct 25 '21

Why use the not operator (!) when you can simply invert the ternary? For one use this is negligible, but it can sum up to hurt performance, and also makes the code less readable