DEV Community

9 Tricks To Write Less JavaScript.

Shoaib Sayyed on May 07, 2020

Hi Dev 👋, Thanks for opening my blog. I hope you are doing well and ready to learn some Tricks To Write Less JavaScript 😎. So, let's start! 1. De...

Read full post

James Thomson • May 7 '20

All good tips for newbies, but personally I wouldn't recommend the 1st. I think it's important to be explicit when defining variables so others can quickly scan the code.

PatricNox • May 7 '20

If the vars are of the same value then I would only see it as a benefit as someone who reviews the code.

James Thomson • May 7 '20

Why would the vars all be the same value? Unless you had them all undefined: let x,y,z.

If you're assuming that let x, y, z = "post" assigns "post" to all, it doesn't, and so this reinforces my point.

PatricNox • May 7 '20

That would then be a matter of code inexperience rather than writing clean code.

function example(backgrounds) {
    let background;
    let array;
    let color = getUserBgColor();

    backgrounds.forEach(color => {
        if (color === 'red') {
            array.push(color);
        }
    })
    background = array[Math.floor(Math.random() * array.length)];
    return background;
}

compared to this

function example(backgrounds) {
    let background, array, color = getUserBgColor();
    backgrounds.forEach(color => {
        if (color !== 'red') {
            array.push(color);
        }
    }) 

    background = array[Math.floor(Math.random() * array.length)];
    return background;
}

It would also reduce amount of lines which could be a great refactor for a big file.
(In addition, the foreach loop would not need to have a space before it if the variable declares are set within the same line)

JP Antunes • May 7 '20 • Edited

I'm not following your train of thought here PatricNox... or did you change your mind between comments?
"If the vars are of the same value...."
"let background, array, color = getUserBgColor();"

Also, If the purpose of refactoring is to reduce the cognitive load and error surface area, then your example would probably be best written as:

function example(backgrounds, userBGcolour = 'red') {
  const array = backgrounds.filter(e => e === userBGcolour);
  const background = array[Math.floor(Math.random() * array.length)];

  return background
}

This assuming that I correctly guessed the behaviour and purpose for the "color" variable you define outside the forEach loop, and the other one inside it, as well as the magic string 'red'... which your colleagues should never have to do.

James Thomson • May 7 '20

When you work with and review thousands of lines of code per day you want clean legible code. Scanning code vertically is more efficient. Writing a couple extra lines greatly helps the reviewers and maintainers.

JP Antunes • May 7 '20

Agreed, but I also want to point out that making extra lines of code that helps you reason better doesn't make the code more readable at all. In fact, it can make your code thousands of lines long when a a dozen might do the job with less bugs.

James Thomson • May 7 '20

We’re only talking variables here. Besides, with destructuring and using modules your files should never be that big.

JP Antunes • May 7 '20

I think we are talking about different things. I replied to PatricNox's comment with an example that takes less LOCs and is imho less error prone. What were you replying to?

James Thomson • May 7 '20

This

Agreed, but I also want to point out that making extra lines of code that helps you reason better doesn't make the code more readable at all. In fact, it can make your code thousands of lines long when a a dozen might do the job with less bugs.

JP Antunes • May 8 '20

Oh, that's not about variables at all, it's really about what happens inside a function and what should be outside it.

Both examples of uninitialised variable declarations let a, b, c = 1 or let a; let b; let c =1 or even let [a,b,c] = [,,1] are code smells imho, it just looks messy and error prone.

James Thomson • May 8 '20

Ah I see, then I think we are talking about different things. The max thread levels makes this confusing.

And I agree about undefined variable declarations.

PatricNox • May 7 '20 • Edited

The object Destructuring (8) one was new to me, nice!

(4) can be tidy in small projects, but bigger ones where any coding standard should be followed then it would not be applicable. There's benefits to always use brackets, even though there's only one action within.

Joaquin Wojcik • May 8 '20

Just a thing about arrow functions:

An arrow function expression is a syntactically compact alternative to a regular function expression, although without its own bindings to the this, arguments, super, or new.target keywords. Arrow function expressions are ill suited as methods, and they cannot be used as constructors.

Source: developer.mozilla.org/en-US/docs/W...

matoval • May 7 '20

Object Destructuring is my favorite!

Ngan B. Nguyen • May 7 '20

Some are just the new syntax in ES6. We can't call it "tricks" in my opinion (Template Literals, Arrow Function).

ruslK • May 7 '20

Nice, short, and useful!

Ben Calder • May 7 '20

Beginners beware: this is yet another post that mixes useful information with some bad practice..
Avoid #1 and don't ommit { }

Madza • May 7 '20

Love all the syntax features from ES6 and above. Tho there is still room for improvement, like more concise syntax for targeting DOM elements (like jquery does)

James Thomson • May 8 '20

That's a browser API consideration. ECMAScript (i.e. ES6) has no concept of a browser.

Besides that, have you used querySelector? You can do pretty much any similar jQuery selection with it.

Vasyl Boroviak • May 8 '20

Tip 10

npm i lodash