Capitalising each word in a sentence using an array-javascript

Question

i have a task for my homework where i have to write a function that will capitalize each word in a sentence that is written into that function. The idea i had was to convert each word into an array, make a loop targeting first letter of each item of that array, and then turning that array back into a string. The code i came up with is this

function titleCase(string) {
  var words = string.split(' ');
  for (var i = 0; i < words.length; i++) {
    const lettersUp = ((words[i])[0]).toUpperCase();
    const result = words[i].replace((words[i])[0], lettersUp);
    return result;
  }
}

The problem i have now is that it returns only the first word of an array. From troubleshooting i have been doing i have a feeling i messed up the loop but i just have no idea how. Any help would be greatly appreciated. Thanks.

You are returning from the very first iteration, so the loop is useless. — ibrahim mahrir, May 14 '17 at 20:26

score 0 · Answer 1 · answered May 14 '17 at 20:47

You are returning from the first iteration, so your code won't work.

What you are looking for is something like this:

function titleCase(string) {
    var words = string.split(" ");
    for (var i = 0; i < words.length; i++) {
        const lettersUp = ((words[i])[0]).toUpperCase();
        words[i] = words[i].replace((words[i])[0], lettersUp);
    }
    return words.join(" ");
 }

Regexes are the way to go, though. Please try and use though.

score 0 · Answer 2 · answered May 14 '17 at 20:51

Keep it as simple as possible - there's no need for additional variables such as lettersUp, you can simply manipulate the strings in the words array.

function titleCase(str) {
  var words = str.split(' ');
  for (var i = 0; i < words.length; i++) {
    words[i] = words[i].charAt(0).toUpperCase() + words[i].substring(1);     
  }
  return words.join(' '); 
}

Capitalising each word in a sentence using an array-javascript

2 Answers2