views:

1340

answers:

5

I came across a javascript puzzle asking: Write a one-line piece of JavaScript code that concatenates all strings passed into a function:


    function concatenate(/*any number of strings*/) {
      var string = /*your one line here*/
      return string;
    } 

@ meebo

Seeing that the function arguments are represented as an indexed object MAYBE an array, i thought can be done in a recursive way. However my recursive implementation is throwing an error. --"conc.arguments.shift is not a function" --


    function conc(){
        if (conc.arguments.length === 0) 
            return "";
        else 
            return conc.arguments.shift() + conc(conc.arguments);
}

it seems as though conc.arguments is not an array, but can be accessed by a number index and has a length property??? confusing -- please share opinions and other recursive implementations.

Thanks

+1  A: 

The arguments list is not a genuine array. I think you can borrow the array methods and use them on arguments with "call" or "apply."

Nosredna
+4  A: 

Mozilla on the subject:

The arguments object is not an array. It is similar to an array, but does not have any array properties except length. For example, it does not have the pop method. However it can be converted to an real array:

var args = Array.prototype.slice.call(arguments);

Therefore the solution to your problem is fairly simple:

var string = Array.prototype.slice.call(arguments).join("");

BTW: It further states:

The arguments object is a local variable available within all functions; arguments as a property of Function can no longer be used.

You should only use arguments instead of func.arguments

Georg
Why make arguments into a real array? I wonder how fast slice is..
svinto
+7  A: 

arguments is said to be an Array-like object. As you already saw you may access its elements by index, but you don't have all the Array methods at your disposal. Other examples of Array-like objects are HTML collections returned by getElementsByTagName() or getElementsByClassName(). jQuery, if you've ever used it, is also an Array-like object. After querying some DOM objects, inspect the resulting jQuery object with Firebug in the DOM tab and you'll see what I mean.

Here's my solution for the Meebo problem:

function conc(){
    if (arguments.length === 0)
        return "";
    else
        return Array.prototype.slice.call(arguments).join(" ");
}

alert(conc("a", "b", "c"));

Array.prototype.slice.call(arguments) is a nice trick to transform our arguments into a veritable Array object. In Firefox Array.slice.call(arguments) would suffice, but it won't work in IE6 (at least), so the former version is what is usually used. Also, this trick doesn't work for collection returned by DOM API methods in IE6 (at least); it will throw an Error. By the way, instead of call one could use apply.

A little explanation about Array-like objects. In JavaScript you may use pretty much anything to name the members of an object, and numbers are not an exception. So you may construct an object that looks like this, which is perfectly valid JavaScript:

var Foo = {
    bar : function() {
        alert('I am bar');
    },

    0 : function() {
        alert('I am 1');
    },

    length : 1
}

The above object is an Array-like object for two reasons:

  1. It has members which names are numbers, so they're like Array indexes
  2. It has a length property, without which you cannot transform the object into a veritable Array with the construct: Array.prototype.slice.call(Foo);

The arguments object of a Function object is pretty much like the Foo object, only that it has its special purpose.

Ionuț G. Stan
I get an Object Expected error on slice.call in IE
Josh Stodola
+1  A: 

You could do this:

function concatenate() {
    if (arguments.length > 1) {
        return arguments[0] + concatenate.apply(this, Array.prototype.splice.call(arguments, 1));
    }
    return arguments[0];
}
Gumbo
+4  A: 

This works:

function concatenate(){
    return [].join.call(arguments, "");
}
alert(concatenate("one", "two", "three"));
svinto
Why aren't you calling the function with 3 string arguments instead of 1 array argument with 3 strings? That would get rid of the commas :)I really like this solution. +1
Ionuț G. Stan
Because I'm tired and ill! Thanks for pointing out my stupidity, now it works.
svinto
It works, but it's not recursive!
Nosredna