Question

96

Trying to capitalize the first character in array of strings, why this is not working?

rated 0 times [ 97] [ 1] / answers: 1 / hits: 59773 / 13 Years ago, wed, october 12, 2011, 12:00:00

I'm trying to write a function that converts for example list-style-image to listStyleImage.

I came up with a function but it seems not working. Can anybody point me to the problem here ?

var myStr = "list-style-image";



function camelize(str){

    var newStr = "";    

    var newArr = [];

    if(str.indexOf("-") != -1){

        newArr = str.split("-");

        for(var i = 1 ; i < newArr.length ; i++){

            newArr[i].charAt(0).toUpperCase();

        }       

        newStr = newArr.join("");

    }

    return newStr;

}



console.log(camelize(myStr));

Answers

Only authorized users can answer the question. Please sign in first, or register a free account.

anyssaarielles

Add To Favorites

Follow

Total Points: 415

Total Questions: 107

Total Answers: 92

Location: Greenland

Member since Fri, Jul 31, 2020

4 Years ago

anyssaarielles questions

1 How can I resolve the Chrome warning: The deferred DOM Node could not be resolved to a valid node

Sun, Jun 19, 22, 00:00, 2 Years ago

1 Need to set up yup validation which is not required and works on only one condition

Thu, Dec 9, 21, 00:00, 3 Years ago

1 Upload Large file (Video) to nodejs server and aws s3 using ReactJs

Sun, May 2, 21, 00:00, 3 Years ago

1 Unable to download file in laravel ajax on button click

Mon, Nov 9, 20, 00:00, 4 Years ago

1 Convert Blob to image inside Cell in React

Thu, Jun 4, 20, 00:00, 4 Years ago

View All

answered 13 Years ago uriahw · Accepted Answer

You have to actually re-assign the array element:

    for(var i = 1 ; i < newArr.length ; i++){

        newArr[i] = newArr[i].charAt(0).toUpperCase();

    }

The toUpperCase() function returns the new string but does not modify the original.

You might want to check to make sure that newArr[i] is the empty string first, in case you get an input string with two consecutive dashes.

edit — noted SO contributor @lonesomeday correctly points out that you also need to glue the rest of each string back on:

         newArr[i] = newArr[i].charAt(0).toUpperCase() + newArr[i].substr(1);