Brett Zamir
Jan 29th

  @Robin: It wasn't hostility at all! I just didn't see your notice, so I was just stating what I said in a matter of fact way, not with any sarcasm or anything. And as you say, it was my mistake for missing your additional comment. We're really sorry if you were met with hostility earlier, but sometimes I think the lack of opportunity for clarification of intent may mistakenly give that impression; if you were met with genuine hostility, or we could have been more clear or more polite, our apologies certainly for that! And yes, we are eager to make JSLint fixes, so thank you very much for the function fixes! I'll see about applying them in Git now...

Robin
Jan 28th

  @Brett: Thanks for the feedback, but if you read the rest of my comment, you'll see I did not intend to change the function in any way, except to make it conform to the JSLint conventions, as shown by the "Open syntax issues."

"Eventually we want all code to pass or at least take into consideration most fixes suggested by JsLint, following this JsLint configuration we’ve decided on."

If the previous statement no longer stands, I suggest you remove it from your website.

This isn't the first time I've tried to contribute to this site, and been met by hostility but thankfully, this will be the last, you may commence the group sigh of relief.

Brett Zamir
Jan 26th

  @Robin: Thanks for the submission, but besides differing in a variable name, your function looks identical to what we already have...

Robin
Jan 25th

1
2
3
4
5

function ucwords(str) {
        return (str + '').replace(/^(.)|\s(.)/g, function (m) {
                return m.toUpperCase();
        });
}

Tony
Jan 11th

  Great function - and picked up #1 on google search for what I was after. Saved me a host of time. Thanks

Alessandro
31 Jul '08

  It doesn't work to me on Firefox 3, it returns the same input sentence (I've used a complete uppercase sentence)...

Kevin van Zonneveld
31 Jan '08

   @ _argos: That's good enough for me :) I'm replacing the function and leave the original one here for future reference:

1
2
3
4
56
7
8
9
10

function ucwords( str ) {
    // http://kevin.vanzonneveld.net
    // +   original by: Jonas Raoni Soares Silva (http://www.jsfromhell.com)
    // *     example 1: ucwords('kevin van zonneveld');
    // *     returns 1: 'Kevin Van Zonneveld'    
    return str.toLowerCase().replace(/\w+/g, function(s){
        return s.charAt(0).toUpperCase() + s.substr(1);
    });
}

_argos
31 Jan '08

   Hi Kevin, I'm making a benchmark of this function using RegExp an is 25%-30% more faster than your original function, this is the bechmark that I try:

1
2
3
4
56
7
8
9
1011
12
13
14
1516
17
18
19
2021
22
23
24
2526
27
28
29
3031
32
33
34
3536
37
38

// _argos's function
function ucwords_w ( str ) {
 
        return str.replace(/^(.)|\s(.)/g, function ( $1 ) { return $1.toUpperCase ( ); } );
 }
 
// Original function
function ucwords( str ) { 
    return str.toLowerCase().replace(/\w+/g, function(s){        return s.charAt(0).toUpperCase() + s.substr(1);
    });
}
 
// BENCHMARK 
var start;
 
start=new Date();
 iterations = 10000;
while ( iterations-- ) {
ucwords('Kevin van Zonneveld');
ucwords('Waldo Malqui Silva');
ucwords('Alexa Valentina Malqui Cordova');}
 
console.log('strrev = ' + ( (new Date())-start))
 
start=new Date();iterations = 10000;
while ( iterations-- ) { 
ucwords_w('Kevin van Zonneveld');
ucwords_w('Waldo Malqui Silva');
ucwords_w('Alexa Valentina Malqui Cordova');}
 
console.log( 'strrev_w = ' + ( (new Date())-start ))

Kevin van Zonneveld
19 Jan '08

  @ Lars: You're welcome ;)

Lars
19 Jan '08

  thanks!