Javascript For Loop: Assignment to character not working

Question

I was writing a test function to capitalize each word in a sentence. I ended up solving it; however, one of my first attempts to solve the problem didn't work when I thought it would.

function capSentence(str) { 
  var strArray = str.split(" ");
  var answer = '';
  var temp = '';
  for(var i = 0; i < strArray.length; i++){
    strArray[i][0] = strArray[i][0].toUpperCase();
    answer += strArray[i];
    if(i !== strArray.length-1){
      answer += ' ';
    }
  }
  return answer; 

}

capSentence("this is a test");

I thought the above code would output "This Is A Test", but instead it outputs "this is a test".

strArray[i][0] = strArray[i][0].toUpperCase();

doesn't seem to have any affect. Why is that?

One line answer. Strings are immutable in JavaScript

thefourtheye
– thefourtheye

2014-05-20 13:39:10 +00:00
Commented May 20, 2014 at 13:39 — thefourtheye
– thefourtheye, Commented May 20, 2014 at 13:39
stackoverflow.com/questions/4878756/…

Klemen Tusar
– Klemen Tusar

2014-05-20 13:41:06 +00:00
Commented May 20, 2014 at 13:41 — Klemen Tusar
– Klemen Tusar, Commented May 20, 2014 at 13:41

J.Wells · Accepted Answer · 2014-05-20 13:54:30Z

1

@thefourtheye's comment is correct. You need to build a new string.

function capSentence(str) { 
  var strArray = str.split(" ");
  var answer = '';
  var temp = '';

  for(var i = 0; i < strArray.length; i++){
      answer += strArray[i][0].toUpperCase();
      answer += strArray[i].slice(1,strArray[i].length);

      if(i !== strArray.length-1){
        answer += ' ';
      }
   }

   return answer;

}

answered May 20, 2014 at 13:54

J.Wells

1,75912 silver badges13 bronze badges

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

Comments

mohamedrias · Accepted Answer · 2014-05-20 13:49:24Z

0

Strings are immutable in Javascript. That's why you are not able to change the value of the strArray.

But you can do in this way:

function capSentence(str) { 
  strArray = str.split(" ");
  console.log(strArray);
  var answer = '';
  var temp = '';
  for(var i = 0; i < strArray.length; i++){

    for (var j = 0; j< strArray[i].length; j++) {
       if(j==0) {
          temp = strArray[i][j].toUpperCase();
       }  else {
         temp+=strArray[i][j];
       }
    }

    answer += temp;
    if(i !== strArray.length-1){
      answer += ' ';
    }
  }
  return answer; 

}

This will retrun "This Is A Test"

answered May 20, 2014 at 13:49

mohamedrias

18.6k3 gold badges41 silver badges47 bronze badges

Comments

Surendheran · Accepted Answer · 2014-05-20 13:52:30Z

0

Try this simple snippet,

function capSentence(str) {
 var strArray = str.split(" ");
 var answer = '';
 var temp = '';
 for(var i = 0; i < strArray.length; i++){
  answer += (strArray[i].substring(0,1)).toUpperCase()+strArray[i].substring(1);
  if(i !== strArray.length-1){
   answer += ' ';
  }
 }
 return answer;
}

answered May 20, 2014 at 13:52

Surendheran

1971 silver badge18 bronze badges

Collectives™ on Stack Overflow

Javascript For Loop: Assignment to character not working

3 Answers 3

Comments

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related