使用拼接Java脚本在数组中增量添加元素

Question

I'm trying to add "-" before capital letters. eg helloWorld becomes hello-world.

My code, however, is placing the "-" in the array at the wrong places. ie thisIsSpinalTap becomes this-I-sSpin-alTap

What's wrong with my code?

function spinalCase(str){
str = str.replace(/ /g,'-');
strArr = str.split("");
for(i=1;i<str.length;i++){
    if(str.charAt(i)==str.charAt(i).toUpperCase()&&str.charAt(i)!=="-"){
        console.log(i);
        strArr.splice(i,0,"-");
    }
} 
return strArr.join("");
}

spinalCase('thisIsSpinalTap'); // returns this-I-sSpin-alTap 

Answer 1

What's wrong is that every time you do a splice, strArr drifts to the left; so ideally you should keep another counter that starts at 0 and increases whenever you do another splice, eg:

var k = 0;
// ...
    strArr.splice(i + k, 0, '-');
    ++k;

Assuming you're not just doing this to exercise, this is a much easier way:

var s = 'thisIsSpinalTap';

var res = s.replace(/([a-z])([A-Z])/g, '$1-$2'); // "this-Is-Spinal-Tap"

The expression matches a lowercase letter followed by an uppercase letter and then replaces it with a dash in between.

Answer 2

The problem here is you are modifying the array in the loop, but you are checking the characters using the string which is not updated based on the new character insertions that happens in the array, so when a - is inserted the indexes in the string will be 1 less that that of the array that is why the - are inserted 1 position less than what is required in the second instance. The same pattern will continue for each insertion so the 3rd insertion will happen at postion-2

 function spinalCase(str) { str = str.replace(/ /g, '-'); var strArr = str.split(""), i, code; for (i = 1; i < str.length; i++) { //since we are modifying the array the indexes in the original array can't be used code = strArr[i].charCodeAt(0); if (code >= 65 && code <= 90) { strArr.splice(i, 0, "-"); //since we are inserting a new element before current element we need to skip an extra element i++; } } return strArr.join(""); } var result = spinalCase('thisIsSpinalTap'); snippet.log(result) 

 <!-- Provides the `snippet` object, see http://meta.stackexchange.com/a/242144/134069 --> <script src="http://tjcrowder.github.io/simple-snippets-console/snippet.js"></script>