Unless one will reuse the index in other places, I like to recommend includes instead of indexOf. I think it's more readable.
indexOf
if (arr.includes('foo')) { ... }
Lack of IE support is a huge bummer since I still have to support IE11 for all my work projects, but that's definitely more readable
Indeed, Brian. I have to support IE11 too. I believe the polyfill solution that others have mentioned here would help.
Cheers,
Are you sure you want to hide this comment? It will become hidden in your post, but will still be visible via the comment's permalink.
Hide child comments as well
Confirm
For further actions, you may consider blocking this person and/or reporting abuse
We're a place where coders share, stay up-to-date and grow their careers.
Unless one will reuse the index in other places, I like to recommend includes instead of
indexOf
. I think it's more readable.if (arr.includes('foo')) { ... }
Lack of IE support is a huge bummer since I still have to support IE11 for all my work projects, but that's definitely more readable
Indeed, Brian. I have to support IE11 too. I believe the polyfill solution that others have mentioned here would help.
Cheers,