Skip to main content
Apr 27, 2011

Common Javascript and jQuery Mistakes

Javascript and jQuery can be tricky. Here is some advice to avoid common pitfalls.

Missing Semicolons
Javascript will automatically insert semicolons at the end of any line, so not including them isn't necessarily 'wrong'. However, for code maintainability and debugging they should always be included. Also, you cannot minify Javascript without including semicolons, which is desirable for reducing file size and code obfuscation.

Braces on new lines
In K&R style coding, braces begin on a new line, such as:

function foo(param)

However, this is bad practice in Javascript, such as in the following example:

'firstname' : 'James',
'score' : 150

This looks like the function is returning JSON, but Javascript's auto-insertion of semicolons means that it isn't returning anything! The 'return' command terminates immediately, and the JSON is ignored. For this reason, it's good practice to put the opening brace on the same line as the preceding code, in every situation:

return {
'firstname' : 'James',
'score' : 150

Hiding Elements
CSS has two values you can use for hiding elements, 'visibility' and 'display'. The difference is that 'display' can be used to completely hide an element and the space it contains, while 'visibility' keeps the space but appears invisible. When most people hide elements, they mean completely removing the space, which can be accomplished dynamically using jQuery:

$('#id_name').css('display', 'none');

It's easier to use a built in jQuery function for hiding and showing elements:


Toggle can be used to flip the visibility state of the object if you don't care about being precise:


Variable Scope
It's quite easy to get confused by the 'this' keyword, which references the current scope. Without delving too deeply, using 'this' can cause a wide variety of unintended consequences if the current scope is misunderstood. If you want to read further, read up on the 'apply' Javascript function, which lets you manually set what scope 'this' refers to.

If you want a quick fix, manually set a variable to 'this', and you can be confident that it's referencing the correct scope. For example:

this.Name = 'James';
this.Score = 150;
personThis = this;

this.getScore = function() {
this.Score = 200;
return personThis.Score; // returns 150

If we had returned this.Score in the getScore() function, 200 would have been returned, which would be wrong if we want James' score. We set 'this' to a specific variable so we don't get confused as to what scope 'this' references.

That's it for now, next week we will discuss more mistakes and focus on common AJAX errors.


More from the Blog
Mar 13, 2018

Latest Revelations Confirm Arxan’s Suspicions of Source of Apple Source Code Leak Issue

In early February news broke
Read more
Jan 11, 2017

Apperian Moving Up and To the Right as Part of Arxan Technologies

A letter from Mark Lorion, President & General Manager of Apperian. Dear Apperian Customers and Partners,
Read more
Dec 27, 2016

Predictions for 2017 - Where is Enterprise Mobility Headed?

The new year is around the corner and as we look back at 2016 it was an exciting year for the enterprise mobility market, ful ...
Read more