object.property = new function(){

[emailscotta]

Some of the object properties in the Dojo Toolkit are set to objects
but they are using syntax like this:

object.property = new function() {
this.property = someValue;
this.property = someFunction;
}

Is the property set to a new object and if so what is the "new
function()" statment doing?

Thanks,

Scott

[Lasse Reichstein Nielsen]

em*********@gma il.com writes:

object.property = new function() {
this.property = someValue;
this.property = someFunction;
}

Is the property set to a new object and if so what is the "new
function()" statment doing?

The "function ..." is simply a function expression, so the above
is equivalent to:
---
var Temp = function() {
this.property = someValue;
this.property = someFunction;
}
object.property = new Temp
---
where "new Temp" is the same as "new Temp()".

So, yes, object.property is assigned a new object as value,
and that object is initialized using the following function.
If the anonymous initializer function is as simple as in this
example, it would be simpler to use an object literal, i.e.,

object.property = { property:someVa lue, property2:someF unction };

Even for more elaborate computations, you can do them first and
then create an object literal from the resulting values.

Compared to using "new" on an anonymous function, the object literal
approach does not create a prototype object referring to the anonymous
function, so it would be more memory efficient, but probably not
as readable

/L
--
Lasse Reichstein Nielsen - lr*@hotpop.com
DHTML Death Colors: <URL:http://www.infimum.dk/HTML/rasterTriangleD OM.html>
'Faith without judgement merely degrades the spirit divine.'

[Richard Cornford]

em*********@gma il.com wrote:

Some of the object properties in the Dojo Toolkit are set
to objects but they are using syntax like this:

object.property = new function() {
this.property = someValue;
this.property = someFunction;
}

Is the property set to a new object and if so what is
the "new function()" statment doing?

The (right side) operand of the - new - operator is expected to evaluate
as a reference to a function object, and that function object is used to
construct a new object that is the result of the - new - expression (an
arguments list is optional with the - new - operator and omitted here).

The the right of the new operator is a function _Expression_, which is
evaluated when executed and results in a function object being created,
with the value of the function expression being a reference to that new
(anonymous, in this case) function object.

The two together are a completely legal way of creating a single unique
object, but completely pointless (particularly in terms of the needless
complexity of the expression). When a function is used as a constructor
its use allows for the inheritance of methods/properties from a
prototype, but here no such inheritance is possible because there is no
method of accessing the prototype of the anonymous function object (it
goes out of scope as soon as the assignment expression is finished).

A simpler alternative would be:-

object.property = {
property:someVa lue,
property2:someF unction
};

- and the effect would be the same (but probably much more efficiently).

I would question the depth of understanding of javascript possessed by
someone who wrote your original version in preference to the above
(Which doesn't bode well for the Dojo Toolkit). However, there may be
closure-based code within the real (anonymous) constructor functions
which may justify their use.

Richard.

[emailscotta]

Richard Cornford wrote:

em*********@gma il.com wrote:

Some of the object properties in the Dojo Toolkit are set
to objects but they are using syntax like this:

object.property = new function() {
this.property = someValue;
this.property = someFunction;
}

Is the property set to a new object and if so what is
the "new function()" statment doing?

The (right side) operand of the - new - operator is expected to evaluate
as a reference to a function object, and that function object is used to
construct a new object that is the result of the - new - expression (an
arguments list is optional with the - new - operator and omitted here).

The the right of the new operator is a function _Expression_, which is
evaluated when executed and results in a function object being created,
with the value of the function expression being a reference to that new
(anonymous, in this case) function object.

The two together are a completely legal way of creating a single unique
object, but completely pointless (particularly in terms of the needless
complexity of the expression). When a function is used as a constructor
its use allows for the inheritance of methods/properties from a
prototype, but here no such inheritance is possible because there is no
method of accessing the prototype of the anonymous function object (it
goes out of scope as soon as the assignment expression is finished).

A simpler alternative would be:-

object.property = {
property:someVa lue,
property2:someF unction
};

- and the effect would be the same (but probably much more efficiently).

I would question the depth of understanding of javascript possessed by
someone who wrote your original version in preference to the above
(Which doesn't bode well for the Dojo Toolkit). However, there may be
closure-based code within the real (anonymous) constructor functions
which may justify their use.

Richard.

Here is the actual code from the toolkit (which I hope is not a problem
since it's open source). So if I understand correctly sampleTransport
becomes a new object but instead of using an object literal they used
the "new function()" statement. Is there a benefit to doing this?

/*
dojo.io.sampleT ranport = new function(){
this.canHandle = function(kwArgs ){
// canHandle just tells dojo.io.bind() if this is a good transport to
// use for the particular type of request.
if(
(
(kwArgs["mimetype"] == "text/plain") ||
(kwArgs["mimetype"] == "text/html") ||
(kwArgs["mimetype"] == "text/javascript")
)&&(
(kwArgs["method"] == "get") ||
( (kwArgs["method"] == "post") && (!kwArgs["formNode"]) )
)
){
return true;
}

return false;
}

this.bind = function(kwArgs ){
var hdlrObj = {};

// set up a handler object
for(var x=0; x<dojo.io.hdlrF uncNames.length ; x++){
var fn = dojo.io.hdlrFun cNames[x];
if(typeof kwArgs.handler == "object"){
if(typeof kwArgs.handler[fn] == "function") {
hdlrObj[fn] = kwArgs.handler[fn]||kwArgs.handle r["handle"];
}
}else if(typeof kwArgs[fn] == "function") {
hdlrObj[fn] = kwArgs[fn];
}else{
hdlrObj[fn] = kwArgs["handle"]||function(){};
}
}

// build a handler function that calls back to the handler obj
var hdlrFunc = function(evt){
if(evt.type == "onload"){
hdlrObj.load("l oad", evt.data, evt);
}else if(evt.type == "onerr"){
var errObj = new dojo.io.Error(" sampleTransport Error: "+evt.msg);
hdlrObj.error(" error", errObj);
}
}

// the sample transport would attach the hdlrFunc() when sending the
// request down the pipe at this point
var tgtURL = kwArgs.url+"?"+ dojo.io.argsFro mMap(kwArgs.con tent);
// sampleTransport .sendRequest(tg tURL, hdlrFunc);
}

dojo.io.transpo rts.addTranspor t("sampleTranpo rt");
}
*/

[emailscotta]

Lasse Reichstein Nielsen wrote:

em*********@gma il.com writes:

object.property = new function() {
this.property = someValue;
this.property = someFunction;
}

Is the property set to a new object and if so what is the "new
function()" statment doing?

The "function ..." is simply a function expression, so the above
is equivalent to:
---
var Temp = function() {
this.property = someValue;
this.property = someFunction;
}
object.property = new Temp
---
where "new Temp" is the same as "new Temp()".

So, yes, object.property is assigned a new object as value,
and that object is initialized using the following function.
If the anonymous initializer function is as simple as in this
example, it would be simpler to use an object literal, i.e.,

object.property = { property:someVa lue, property2:someF unction };

Even for more elaborate computations, you can do them first and
then create an object literal from the resulting values.

Compared to using "new" on an anonymous function, the object literal
approach does not create a prototype object referring to the anonymous
function, so it would be more memory efficient, but probably not
as readable

/L
--
Lasse Reichstein Nielsen - lr*@hotpop.com
DHTML Death Colors: <URL:http://www.infimum.dk/HTML/rasterTriangleD OM.html>
'Faith without judgement merely degrades the spirit divine.'

So is this essense an anonymous constructor function?

[Lasse Reichstein Nielsen]

"Richard Cornford" <Ri*****@litote s.demon.co.ukwr ites:

[ object.property = new function(){...} ]

When a function is used as a constructor its use allows for the
inheritance of methods/properties from a prototype, but here no such
inheritance is possible because there is no method of accessing the
prototype of the anonymous function object (it goes out of scope as
soon as the assignment expression is finished).

Actually, and probably unintended, it is available, as:
object.property .constructor.pr ototype

That leaves both the function and its prototype hanging around for
no good reason.

/L
--
Lasse Reichstein Nielsen - lr*@hotpop.com
DHTML Death Colors: <URL:http://www.infimum.dk/HTML/rasterTriangleD OM.html>
'Faith without judgement merely degrades the spirit divine.'

[Michael Winter]

On 11/08/2006 02:01, em*********@gma il.com wrote:

Richard Cornford wrote:

[snip]

>I would question the depth of understanding of javascript possessed
by someone who wrote your original version in preference to the
above (Which doesn't bode well for the Dojo Toolkit). However,
there may be closure-based code within the real (anonymous)
constructor functions which may justify their use.

So if I understand correctly sampleTransport becomes a new object but
instead of using an object literal they used the "new function()"
statement.

That's correct.

Is there a benefit to doing this?

As Richard noted, the latter approach is useful in closure-based code
where the execution context formed by the function expression becomes
the basis for that closure. However, this is not the case for the code
you posted: the canHandle and bind properties could have been added
within an object literal, and the addTransport method call could have
been made at any point after the object was assigned to the
sampleTranport [sic] property.

There are two other issues, both within the bind method:

}else{
hdlrObj[fn] = kwArgs["handle"]||function(){};

This assignment statement is executed within a loop. Whenever the
function expression here is evaluated (if the handle property of kwArgs,
type-converted to Boolean, evaluates to false), a new function object
will be created with its [[scope]] property containing the entire
execution context stack. Needless to say, this is quite pointless for a
function that, when called, does nothing. It would have been better to
define a dummy function elsewhere and assign a reference to that, thus
adding less overhead.

var hdlrFunc = function(evt){

The second issue is much simpler: why the above isn't simply a function
declaration, I don't know. Unless the bind method errors out, the above
will always be evaluated, and the time of evaluation is irrelevant in
this instance. I think the authors were getting a little carried away,
using function expressions just for the hell of it.

Mike
By the way, please don't post code containing tabs: they wrap far too
easily. Use a couple of spaces per level instead.

[Lasse Reichstein Nielsen]

em*********@gma il.com writes:

Here is the actual code from the toolkit (which I hope is not a problem
since it's open source). So if I understand correctly sampleTransport
becomes a new object but instead of using an object literal they used
the "new function()" statement. Is there a benefit to doing this?

In this particular case, none.

Generally, probably none too. If you do not need the extra layer of a
prototype for the constructed object, there is no reason to create
it.

/L
--
Lasse Reichstein Nielsen - lr*@hotpop.com
DHTML Death Colors: <URL:http://www.infimum.dk/HTML/rasterTriangleD OM.html>
'Faith without judgement merely degrades the spirit divine.'

[Richard Cornford]

em*********@gma il.com wrote:

Richard Cornford wrote:

<snip>

>I would question the depth of understanding of javascript
possessed by someone who wrote your original version in
preference to the above (Which doesn't bode well for the
Dojo Toolkit). However, there may be closure-based code
within the real (anonymous) constructor functions which
may justify their use.

Here is the actual code from the toolkit (which I hope is not
a problem since it's open source). So if I understand
correctly sampleTransport becomes a new object but instead
of using an object literal they used the "new function()"
statement. Is there a benefit to doing this?

/*
dojo.io.sampleT ranport = new function(){
this.canHandle = function(kwArgs ){

<snip>

}
this.bind = function(kwArgs ){

<snip>

}

dojo.io.transpo rts.addTranspor t("sampleTranpo rt");

This is the only line of code that actually gets executed as a
consequence of calling the anonymous function object as a constructor.

}
*/

There does not appear to be any reason in this code for not using an
object literal:-

dojo.io.sampleT ranport = {
canHandle:funct ion(kwArgs){
...
},
bind:function(k wArgs){
...
}
};
dojo.io.transpo rts.addTranspor t("sampleTranpo rt");

- as the closures formed at the constructor level in the original are
not (and so represent another avoidable overhead).

Richard.

[Richard Cornford]

Lasse Reichstein Nielsen wrote:

"Richard Cornford" <Ri*****@litote s.demon.co.ukwr ites:

[ object.property = new function(){...} ]

>When a function is used as a constructor its use allows for
the inheritance of methods/properties from a prototype, but
here no such inheritance is possible because there is no
method of accessing the prototype of the anonymous function
object (it goes out of scope as soon as the assignment
expression is finished).

Actually, and probably unintended, it is available, as:
object.property .constructor.pr ototype

That leaves both the function and its prototype hanging
around for no good reason.

That must be the second or third time that construct has slipped my
mind, probably because it is far too convoluted to ever consider using
it.

Richard.