JavaScript coding guidelines

From MozillaZine Knowledge Base

(Difference between revisions)
Revision as of 04:48, 20 March 2005
Grimholtz (Talk | contribs)
(fixed example to match variable naming guidelines)
<-- Previous diff
Revision as of 18:56, 20 March 2005
Asqueella (Talk | contribs)
(Avoiding name collisions - that wasn't a typo)
Next diff -->
Line 59: Line 59:
<pre>var ExtensionName = { <pre>var ExtensionName = {
_privateMember: 3, _privateMember: 3,
- _anotherMember: "A string",+ publicMember: "A string",
init: function() { init: function() {

Revision as of 18:56, 20 March 2005

This page is part of the extension development documentation project.

Ask your questions in MozillaZine Forums. Also try browsing example code.

Note: development documentation is in process of being moved to Mozilla Development Center (MDC).


Javascript coding guidelines

Here is a list of conventions (discuss). Some people disagree, but some think it's a good idea to follow them.

Naming Convensions

There are six basic rules.

  1. Every variable object name's 1st char should be a lower case.
    var myName = "Torisugari";
  2. Every constant object name should be upper cases.
    const MY_NAME = "Torisugari";
  3. Every constructor's name should begin with an upper case character:
    function MozillaZine(){
      this.description="A Cool Site";"mozillaZine";
    var gMyFav = new MozillaZine();
  4. By prefix g the author means that is a global object, and unique in the window. Exception: A global function doesn't have to start with g.
  5. By prefix a the author means that is an argument.
  6. By prefix _ or m the author means that is a private member.

Bad examples, though gramatically correct (see next section for a good example):

var bar=0; /*bar should be gBar*/
function foo(){
  var gFoobar = 0; /* gFoobar is not a global value */
  return gFoobar;

function foo(bar){  /* bar should be aBar */
  var aFoobar="foo" + bar; /* aFoobar is not an argument */
  return aFoobar;

var gFoo={

var foobar=gFoo._bar; /* _bar is a public member ? */
/* In other words, you can write only "this._bar" */

Avoiding name collisions

In order to avoid name collisions, use an unique prefix (for example, name of the extension) for identifiers in overlays (this also applies to XUL ids in overlays).

To avoid prefixing each identifier, you can put them inside an object with unique name:

var ExtensionName = {
  _privateMember: 3,
  publicMember: "A string",

  init: function() {

  doSomething: function(aParam) {

Avoiding memory leaks

Even though javascript is an interpreted, garbage collected, language, it is still possible to leak memory, meaning that memory that is no longer being used will not be freed up for other applications. This is particularly true when accessing C++ components via XPCOM. Developers should read Using XPCOM in JavaScript without leaking to understand how to avoid causing memory leaks.