Resolved Javascript scope in multiple object instances

[Kyle123]

Hi,

My brain's hurting a bit, would anyone be kind enough to explain why this:

var myObj = function() {
	var value;
	init = function() {
		value = 10;
		function1();
	}
	function1 = function() {
		alert(value);
	}
	return {
		init : init
	}
};

var inst1 = myObj();
inst1.init();

var inst2 = myObj();
inst2.init();

Works, but this:

var myObj = function() {
	var value;
	init = function() {
		value = 10;
		function1();
	}
	function1 = function() {
		alert(value);
	}
	return {
		init : init
	}
};

var inst1 = myObj();
var inst2 = myObj();
inst1.init();
inst2.init();

Doesn't?

Thanks in advance

[Airblader]

I think you're missing the "new" keyword when initializing your instances.

[Kyle123]

Thanks for your response, but it doesn't make any difference I'm afraid

[Airblader]

You initialize your inner functions at global, which causes interference. Make them local variables, too:

var myObj = function() {
	var value;

	var init = function() {
		value = 10;
		function1();
	};

	var function1 = function() {
		alert(value);
	};

	return {
		init : init
	}
};

var inst1 = new myObj();
var inst2 = new myObj();
inst1.init();
inst2.init();

[Kyle123]

Oh yeah, that was slack

always the simple things. Thanks