Javascript string conversion to 'Title Case' returns no output

Question

I can't pinpoint where the error in the code is, it produces no output:

function capitalize (x) {
        var arr = x.split(' ');
        var output = [];

        for (var i = 0; i < arr.length; i++) {

            output.push(arr[x].charAt(0).toUpperCase()+arr[x].slice(1));
        };

        return output.join(' ');
    }

    alert(capitalize("the quick brown fox"));

timolawl · Accepted Answer · 2016-04-18 20:38:42Z

4

I believe arr[x] should be arr[i].

function capitalize (x) {
        var arr = x.split(' ');
        var output = [];

        for (var i = 0; i < arr.length; i++) {

            output.push(arr[i].charAt(0).toUpperCase()+arr[i].slice(1));
        };

        return output.join(' ');
    }

    alert(capitalize("the quick brown fox"));

edited Apr 18, 2016 at 20:38

answered Apr 18, 2016 at 20:33

timolawl

5,56415 silver badges31 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

James Barr Over a year ago

Thank you, I was so used to using [x] as the placeholder haha.

RomanPerekhrest · Accepted Answer · 2016-04-18 20:48:50Z

1

Short one-line solution:

var str = "the quick brown fox",
    capitalized = str.split(" ").map(function(w){ return w[0].toUpperCase() + w.slice(1); }).join(" ");

console.log(capitalized);   // "The Quick Brown Fox"

answered Apr 18, 2016 at 20:48

RomanPerekhrest

93.1k4 gold badges75 silver badges112 bronze badges

1 Comment

James Barr Over a year ago

That's awesome, I wouldn't of thought of the program being done like that.

Collectives™ on Stack Overflow

Javascript string conversion to 'Title Case' returns no output

2 Answers 2

1 Comment

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related