MozillaZine

JavaScript coding guidelines

From MozillaZine Knowledge Base

(Difference between revisions)
Revision as of 01:26, 19 December 2004
Asqueella (Talk | contribs)
(reformat, add {{extdev}} and Category:Development)
<-- Previous diff
Revision as of 04:34, 19 December 2004
Asqueella (Talk | contribs)

Next diff -->
Line 72: Line 72:
===Miscellaneous=== ===Miscellaneous===
-[[Dev : Javascript variables]]+*[[Dev : Javascript variables | Variables in Javascript]]
-[[Category:Development]]+[[Category:Development]] [[Category:Javascript]]

Revision as of 04:34, 19 December 2004

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).

Contents

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";
      this.name="mozillaZine";
      this.url="http://www.mozillazine.org/";
    }
    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 _ 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={
  _bar:0
}

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,
  anotherMember: "A string",

  init: function() {
    this.doSomething(this.anotherMember);
  },

  doSomething: function(aParam) {
    alert(aParam);
  }
};

Miscellaneous