5 JavaScript "tips" that might bite you back.

snigo

Igor Snitkin

Posted on January 1, 2021

5 JavaScript "tips" that might bite you back.

How many times have you seen articles saying "Do not do that", "This is wrong" or "These tips will make you a pro developer" etc. πŸ˜‹ I don't know about you, but I've seen enough of those. Don't get me wrong, many of the tips are actually useful and quite valuable, it's not a problem with the implementation itself, rather it's a problem with imitation aka copy/pasting.

Tip meme

Let's see and discuss a couple of those tips. Before we start though, let's determine types of bites, as they would differ by impact:

  • Readability Bite: will not affect you directly, it will rather bite your teammate who reviews your code.
  • Type Bite: will bite with using certain types
  • Syntax Bite: will bite using certain syntactic expression

Alright! Without further ado.

1. Convert to number

This is my favorite, and I have to admit I myself use it all the time. The trick is quite simple, by attaching Unary Plus(+) operator to any value you will force it to be coerced to number:


const strNum = '3645';
const strNaN = 'hi, i am not a number';

typeof +strNum; // "number"
typeof +strNaN; // "number"

+strNum; // 3645
+strNaN; // NaN

Enter fullscreen mode Exit fullscreen mode

This tip is quite light on errors and works pretty much all the time, it's a suggested conversion method by many teams.

Readability Bite:

I'm pretty sure, you saw it coming πŸ™‚ It's no-brainer that any developer who doesn't know how unary plus operator works will WTF following code:


function sum(a, b) {
  return +a + +b;
}

Enter fullscreen mode Exit fullscreen mode

Not mentioning the fact that we're all fans of functional programming and this πŸ‘† doesn't align very well with its principles.

Type Bite:

Unfortunately this will not work with BigInt, a new numeric data type introduced in 2019.


const veryBigInt = 45n;
+veryBigInt; // TypeError: Cannot convert a BigInt value to a number

Enter fullscreen mode Exit fullscreen mode

Before you start complaining in the comments below, I'm pretty aware that your app will never deal with this type, but we all can agree that functionality that makes no presumptions is more stable.

Solution:

One solution that improves readability, is functional and accounts for BigInt:


const veryBigInt = 45n;
const strNum = '3645';
const strNaN = 'hi, i am not a number';

Number(veryBigInt); // 45
Number(strNum); // 3645
Number(strNaN); // NaN

Enter fullscreen mode Exit fullscreen mode

I'm not including conversion to a string here, as from readability point of view it bites the same way:


const ugly = 42 + '';
const good = String(42);
const goodToo = `${42}`;

Enter fullscreen mode Exit fullscreen mode

2. Concatenate arrays

Another extremely popular tip β€” concatenate arrays using Spread Operator:


const a = [1, 2, 3];
const b = [4, 5, 6];

[...a, ...b]; // [1, 2, 3, 4, 5, 6]

Enter fullscreen mode Exit fullscreen mode

How on Earth this might bite? Well let's say I kinda like this functionality and I want to extract it into function (because functional programming, you know πŸ€—).

Type Bite:

So here's our union function:


function union(a, b) {
  return [...a, ...b];
}

Enter fullscreen mode Exit fullscreen mode

I have one issue right from the start - I want union of any number of arrays, not just two. Any ideas how to refactor it still using the spread operator?

The second issue is that it will include empty cells which depending on situation might not be desirable:


const a = [1, 2, 3];
const b = Array(3);
b.push(4);
union(a, b); // [1, 2, 3, undefined, undefined, undefined, 4]

Enter fullscreen mode Exit fullscreen mode

Finally, we would need to be really really careful with what we're passing as arguments to union:


const a = [1, 2, 3];
const b = null;
const c = 42;
const d = 'hello';

union(a, b); // TypeError: b is not iterable
union(a, c); // TypeError: c is not iterable
union(a, d); // [1, 2, 3, "h", "e", "l", "l", "o"] :/

Enter fullscreen mode Exit fullscreen mode

Putting union aside, this approach forces you to always presume values are arrays, which is pretty bold assumption.

Solution:

Let's rewrite our function, so it accounts for all the issues above:


function union(...args) {
  return args.flat();
}

const a = [1, 2, 3];
const b = null;
const c = 42;
const d = 'hello';
const e = Array(3);
e.push(99);

union(a, b, c, d, e); // [1, 2, 3, null, 42, "hello", 99]

Enter fullscreen mode Exit fullscreen mode

I think I hear CS maniacs screaming at me now "Flat iiiss sloooow!" Ok. If your program operates with arrays over 10000 items and you worry about performance, then use .concat():


function union(...args) {
  return [].concat(...args);
}

Enter fullscreen mode Exit fullscreen mode

A bit more performant way, but grabs empty cells. Chances that you will deal with empty cells are super tiny anyway πŸ‘

I guess my message here is that the .concat() method is not obsolete and you shall not treat it this way. Furthermore using functions over operators will make your program just a little bit more stable.

3. Round number using bitwise operators.

Low-level nature of bitwise operators makes them VERY fast and on top of that you have to admit they are quite nerdy and I see how many people can be attracted to them πŸ€“. Of course any bitwise operator will cause Readability Bite, we won't even discuss it.

Let's get back to "rounding". You might notice that different people will do it with different operators, popular ones are bitwise OR | and double bitwise NOT ~~. In fact you can use all of them:


const third = 33.33;
/* Bitwise AND */
third & -1; // 33

/* Bitwise NOT */
~~third; // 33

/* Bitwise OR */
third | 0; // 33

/* Bitwise XOR */
third ^ 0; // 33

/* Left shift */
third << 0; // 33

/* Right shift */
third >> 0; // 33

/* Zero fill right shift (positive numbers only) */
third >>> 0; // 33

Enter fullscreen mode Exit fullscreen mode

What's going on?!! Too good to be true, isn't it? Well, yes. You are not "rounding" anything you just using bitwise operators to return the same number here and given the fact that bitwise operators can only operate on 32-bit integers this effectively truncates float numbers, because they are not in 32-bit range. Which brings us...

Syntax Bite

32-bit integers are integers ranging from -2,147,483,648 to +2,147,483,647. That might sound like a lot, but in fact it's probably the average video count of Justin Bieber on YouTube. As you might guess this won't work outside the range:


const averageBieberViewsCount = 2147483648.475;
averageBieberViewsCount | 0; // -2147483648 πŸ₯²
~~averageBieberViewsCount; // -2147483648 πŸ₯²

Enter fullscreen mode Exit fullscreen mode

On top of this, it's not rounding in the first place, rather truncating off the fractional part of the number:


const almostOne = 0.9999999;
almostOne | 0; // 0 :/

Enter fullscreen mode Exit fullscreen mode

And finally, this approach has strange relationship with NaN which can cause pretty nasty bugs:


~~NaN; // 0

Enter fullscreen mode Exit fullscreen mode

Solution

Just use function built for this:


const third = 33.33;
const averageBieberViewsCount = 2147483648.475;
const almostOne = 0.9999999;

Math.round(third); // 33
Math.round(averageBieberViewsCount); // 2147483648
Math.round(almostOne); // 1
Math.round(NaN); // NaN

Enter fullscreen mode Exit fullscreen mode

4. Rounding with Number.toFixed

While we're on the topic of rounding, let's see one more that is quite popular, especially when dealing with any sort of currency-related numbers:


const number = 100 / 3;
const amount = number.toFixed(2); // "33.33"

Enter fullscreen mode Exit fullscreen mode

Floating numbers in any programming language is a problem, unfortunately it is true for JavaScript and .toFixed() is no exception.

Syntax Bite

The problem occurs in the rounding edge case when last digit to be rounded is 5. By rounding rules such case should be rounded up, so:


(1.5).toFixed(0); // 2 πŸ‘
(1.25).toFixed(1); // 1.3 πŸ‘
(1.725).toFixed(2); // 1.73 πŸ‘
/* and so on */

Enter fullscreen mode Exit fullscreen mode

Unfortunately it's not always a case:


(0.15).toFixed(1); // 0.1 πŸ‘Ž
(6.55).toFixed(1); // 6.5 πŸ‘Ž
(1.605).toFixed(2); // 1.60 πŸ‘Ž

Enter fullscreen mode Exit fullscreen mode

As you can see we're not talking about rounding to extreme precisions here, rounding to one or two decimal places is normal everyday routine.

Solution

One of the solutions is to use third-party rounding to precision function, like _.round() or similar. Or just write your own such function, it's not a rocket science πŸš€:


function round(number, precision = 0) {
  const factor = 10 ** precision;
  const product = Math.round(number * factor * 10) / 10;
  return Math.round(product) / factor;
}

round(0.15, 1); // 0.2 πŸ‘
round(6.55, 1); // 6.6 πŸ‘
round(1.605, 2); // 1.61 πŸ‘

Enter fullscreen mode Exit fullscreen mode

Cool by-product of such function is that you have negative precision rounding aka number of trailing zeros right off the bat:


round(12345, -3); // 12000
round(12345, -2); // 12300
round(12345, -1); // 12350
round(-2025, -1); // -2020

Enter fullscreen mode Exit fullscreen mode

5. Higher-order methods "shortcuts"

Another very popular trick is to use pre-built functions as arguments to higher-order methods (methods that expect function as an argument), it works exceptionally good with .map() and .filter():


const randomStuff = [5, null, false, -3, '65'];

/* Convert to string */
randomStuff.map(String); // ["5", "null", "false", "-3", "65"]

/* Convert to number */
randomStuff.map(Number); // [5, 0, 0, -3, 65]

/* Filter out falsy values */
randomStuff.filter(Boolean); // [5, -3, "65"]

/* Falsy check */
!randomStuff.every(Boolean); // true

Enter fullscreen mode Exit fullscreen mode

You get the point... Super hacky, super cool 😎

Syntax Bite

Let's say I need to parse some CSS margin value, pretty reasonable task:


const margin = '12px 15px';
const parsedMargin = margin.split(/\s+/).map(parseInt);

console.log(parsedMargin); // [12, NaN] :/

Enter fullscreen mode Exit fullscreen mode

Every high-order method will invoke a given function passing 3 arguments: element, index, reference to original array. What's happening is on every iteration of method parseInt function is given at least two arguments, and that's exactly how many arguments parseInt expects: string to parse and radix, so we're ending up passing index of the element as radix:


/* Iteration #1 */
parseInt('12px', 0); // Radix 0 is ignored and we get 12

/* Iteration #2 */
parseInt('15px', 1); // Radix 1 doesn't exists and we get NaN

Enter fullscreen mode Exit fullscreen mode

Solution

You can always check how many arguments the function you want to use expects using .length, if it's more than 1 then it is probably unsafe to pass this function as an argument and instead we'll need to wrap it:


parseInt.length; // 2

const parsedMargin = margin
  .split(/\s+/)
  .map((margin) => parseInt(margin));

console.log(parsedMargin); // [12, 15] πŸŽ‰πŸŽ‰πŸŽ‰

Enter fullscreen mode Exit fullscreen mode

Conclusion

Ask yourself a hmmm..

Don't just blindly follow whatever is written online, question yourself, research, test, then double research and double test. "It just works" should never be an excuse! If you don't know why it works, then presume it doesn't.

I actually prepared 10 tips for this article, but it appeared to be too long and code heavy for one post, so I might do a follow up soon unless you completely destroy me in the comments. Speaking of comments, feel free to discuss and let me know if you experienced any tips and tricks that have bitten you in the past.

Happy New 2021!

πŸ’– πŸ’ͺ πŸ™… 🚩
snigo
Igor Snitkin

Posted on January 1, 2021

Join Our Newsletter. No Spam, Only the good stuff.

Sign up to receive the latest update from our blog.

Related