how to sort this code?

[smansakra]

hei, i can create a code on js, but i'm not able to make it shorter.
can anyone here help me to sort this code?

function checkingMyNewComm(opt){
var comm_profile = myNewComm('Profile','check',false);
var comm_blog = myNewComm('Blog','check',false);
var comm_photo = myNewComm('Photo','check',false);
var comm_music = myNewComm('Music','check',false);
var comm_video = myNewComm('Video','check',false);
var comm_classified = myNewComm('Classified','check',false);
var comm_guestbook = myNewComm('Guestbook','check',false);
return comm_profile+comm_blog+comm_photo+comm_music+comm_video+comm_classified+comm_guestbook;
}

thanks....

[smansakra]

may be we can sort it by using array(Profile,Blog,Photo,Video,Music,Classified,Guestbook) , but i don't know how

hum?

[codeexploiter]

You can do the same thing mentioned in your code through the following code also.

function checkingMyNewComm(opt){
	var arr = [];
	var firstparam = ['Profile','Blog','Photo','Music','Video','Classified','Guestbook'];
	for(var i = 0; i < firstparam.length; ++i){
		arr.push(myNewComm(firstparam[i],'check',false));
	}
	return arr.join(' ');
}

In the refined version I've removed the multiple occurences of 'myNewComm' using a looping construct. As well as instead of multiple local variables I've used an array to store the result. Finally I've joined the resultant array content and delimited by space(' ').

Hope this helps

[smansakra]

hum... let me think

[smansakra]

does the value of "return comm_profile+comm_blog+comm_photo+comm_music+comm_video+comm_classified+comm_guestbook;" same as "return arr.join(' ');" ?

[codeexploiter]

As I've mentioned in my post earlier, I have used arrays instead of scalar variables in my version. The step you've mentioned is similar to the one I've used the only difference is I have mentioned to insert a space between two items. Do have a look at here to learn more about the join method associated with the Array object.

[smansakra]

hum,
return comm_profile+comm_blog+comm_photo+comm_music+comm_video+comm_classified+comm_guestbook
will return 12 (2+2+1+3+4 and so on ) so the result is 12,

i think your code will result 22134

but i have not tried yet

[smansakra]

Hey, by the way thanks for explanation. You're really help

[jscheuer1]

var checkingMyNewComm = function(){
 var m = myNewComm, c = 'check', f = false;
 return m('Profile', c, f) + m('Blog', c, f) + m('Photo', c, f) + m('Music', c, f) +
 m('Video', c, f) + m('Classified', c, f) + m('Guestbook', c, f);
};

or:

var checkingMyNewComm = function(){
 var a = ['Profile', 'Blog', 'Photo', 'Music', 'Video', 'Classified', 'Guestbook'];
 for (var t = 0, i = a.length - 1; i > -1; --i)
  t += myNewComm(a[i], 'check', false);
 return t;
};

[codeexploiter]

John what do you think about the string concatenation through the + operator compared to the join method? What is the performance overhead? I mean not with this input but with a larger number of inputs..