Why should I read this?
There are many different ways of writing code that achieve the same goal. However, programmers generally agree that there are some ways of writing code that are more understandable and readable than other ways.
To help write more understandable code, programmers follow a style guide. It provides guidelines for how to write code, how to structure it, common problems to avoid, etc.
Before you start
Using Prettier to format code automatically
Some of the guidelines are so common that there are automatic tools to do it for you! One of these tools is called Prettier. You should have installed this tool when you set up your VSCode extensions.
Prettier is a tool that rearranges your code (called formatting). It follows a set of rules that programmers have agreed make your code easier to read and understand.
You might have a common problem when running Prettier. If you see something like this:
Then you have written some code that is invalid and Prettier can’t format it. A common issue is a missing bracket, so try looking for that first.
Essential
From when you begin programming it is essential that you follow the guidelines below - a good habit is much easier to get in to when you start!
Indent your code
You may have noticed that programmers tend to indent their code by starting a line of code with some space.
Here’s an example in HTML:
<div>
<p>Hello world!</p>
</div>
And JavaScript:
if (isVegetarian) {
return "Macaroni and Cheese";
} else {
return "Steak and Chips";
}
Indentation helps us to read our code by visually marking how the code is going to run. Programmers rely on this extra information to quickly read and understand code.
There are some guidelines about where to add indentation. For example, lines of code are indented when they are inside a function, an if
statement or a for
loop. Additionally, if any of those structures are inside another structure, then they are indented further.
The guidelines about where to add indentation can be complex, but the good news is that Prettier adds indentation automatically for us! However it is good to pay attention to where indentation is added, as you may find yourself writing code in an environment where Prettier is not installed.
Don’t leave lots of commented out code
When you are debugging a problem, you might comment out some sections of your code.
// π Don't do this!
function addToShoppingList(item) {
// console.log("Shopping list before", shoppingList);
// console.log("Adding item", item);
shoppingList.add(item);
// console.log("Shopping list after", shoppingList);
}
In this example, there is some extra debugging information that is commented out.
Once you have solved the problem, you should remove large sections of commented out code. When you (or someone else) are reading your code, you want to know only the important pieces. Removing commented out code helps find the relevant code faster and easier.
Saving old versions with Git
You might be worried about “losing” some experimental code that you want to “save” by commenting it out. However, this can cause confusion about which bits of code still work.
If you want to save some code, remember that you can commit it using Git. It will then be saved forever. If you decide to remove the code later you can delete it and make another commit. And if you want it back again, you can always look at the deleted code in the git history.
Don’t leave unused variables
As you write code, you may make changes to the variables that you are using. You may rename some variables, make some new variables or change how the variables are used. This may leave some variables that are unused.
function orderTaxi(pickUpTime) {
let driverName = getDriverName();
let customerName = getCustomerName(); // π Don't do this!
return `${driverName} will pick you up at ${pickUpTime}`;
}
In this example, the customerName
variable isn’t used anywhere.
You should remove any variables that are unused. This is because if you (or someone else) is reading your code, it can be confusing if you see a variable and then find out later that it isn’t used. It could make you think that there’s a bug, because the variable must have been put there for a reason!
Think of good names for your variables
Making our code understandable by others is critical to being a programmer. One of the main tools in our toolbox is good naming for our variables and function names.
There are no strict rules to follow when thinking of variable names, but there are some general guidelines.
π‘ tip
When thinking about whether a variable name is good, try to imagine that you are reading the code again in the future and you have forgotten exactly how it works.
Do the variable names help explain what the code is supposed to do?
Use camelCase
When writing JavaScript, programmers tend to use a convention called camelCase
. This means starting your variable name with a lower case letter, and then every “word” after that starts with an upper case letter.
startingLocation
timeToDestination
trafficOnRoute
By doing this it makes it easier to read variable name that have multiple “words”.
Avoid short names
Very short variable names can be difficult to understand since the purpose of the variable can be unclear. They are also difficult to remember, especially if there are many similarly named variables. Try to avoid short names or abbreviations.
Here are some examples of bad names that you should avoid:
- Single letters like
x
ory
- Abbreviations like
evt
instead ofevent
- Generic names like
array
orstring
Describe what the variable is/does
A good variable name quickly explains what it represents to anyone reading the code. Try to describe what the variable is or what it does within in code.
// π Try to avoid this
let song = true;
// β
This is better
let isPlaying = true;
In this example, the variable name song
doesn’t tell us what it is used for. The isPlaying
variable name is better since it tells us whether a song is playing or not.
// π Try to avoid this
function percentage() {
// ...
}
// β
This is better
function getPercentage() {
// ...
}
In this example, the function calculates a percentage and returns it. Therefore it is good to name it using the “get” verb to show that it returns something.
// π Try to avoid this
function isOldEnough(number) {
// ...
}
// β
This is better
function isOldEnough(yearOfBirth) {
// ...
}
Finally, parameters of functions should also have names that properly represent what is going to be received into the function.
Using let
, const
and var
In JavaScript there are three ways to define a variable - let
, const
and var
. The rules on using them at CodeYourFuture are as follows
var myAge = 21; // π Do not use
let yourAge = 21; // β
Use for values that change
const secondsInMinute = 60; // β
Use for values that never change
Using var
You should never use var
. This is an outdated way of writing JavaScript code and it is always better to use let
or const
.
The reason it still exists is because lots of code already uses it and so it can’t be removed from JavaScript safely.
Using let
You should use let
for a variable that will change, for example:
let buttonClickedCount = 0;
buttonClickedCount = buttonClickedCount + 1;
Using const
For other variables that will never change you should use const
, for example
const centimetersInMeter = 100;
Avoid Long Functions
Sometimes writing long functions are unavoidable, however most of the time is better to split a long function into a few shorter ones.
This will
- Make your code easier to read
- Make your code easier to maintain
- Make your code easier to review by our volunteers
When you writing code you should also try to keep in mind that you will only ever write the code once, but you will read the code many times. Always aim to write code that is readable.
For more details you can read these useful articles
- (The Art of Writing Small and Plain Functions)[https://dmitripavlutin.com/the-art-of-writing-small-and-plain-functions/]
Advanced
These additional rules may be harder for you to understand at the start of your programming journey however they are important concepts as you develop.
Remember your audience
Remember that you are communicating with another programmer!
Don’t forget that the person reading your code probably has some understanding of the context. If your function is named getCustomer
, it’s probably fine to name variables name
, age
, etc instead of customerName
, customerAge
. You don’t need to explain everything.
Naming variables is more of an art than a science. Often you need some experience reading other people’s code before you get really good at it.
π‘ tip
Good scoping of your variables
Define your variables with the narrowest scope they can have. This is easiest to explain with an example…
function findLongestFirstName(fullNames) {
let longest;
let firstName;
for (let i = 0; i < fullNames.length; i++) {
firstName = fullNames[i].split(" ")[0];
if (!longest || firstName.length > longest.length) {
longest = firstName;
}
}
return longest;
}
It is considered better code to combine the two highlighted lines into one like this…
function findLongestFirstName(fullNames) {
let longest;
for (let i = 0; i < fullNames.length; i++) {
let firstName = fullNames[i].split(" ")[0];
if (!longest || firstName.length > longest.length) {
longest = firstName;
}
}
return longest;
}
This is important as when a function (or your code more generally) becomes more complex it will be harder to keep track of what variable is used where.