2017-08-06 16 views
1

で文字を置き換えます)しかし、私は上記の何が間違っているかを見ることができません。どんな助けも素晴らしいだろう。Javascriptをこれはシンプルでなければなりませんが、それは動作していない理由を私はわからないループ

+0

例を、入力および出力のため? – Microsmsm

+3

@basementが答えを得ました。 str.length not leght –

+0

あなたのコードをデバッグするとどうなりますか?(例えば、デバッガの行ごとにステップ実行します)?あなたは何を見ますか? –

答えて

3

「長さ」は間違っています。(オンライン4)

スペルの修正後に動作します。

function kebabToSnake (str){ 
 
    var string = ""; 
 
    var chart = ""; 
 
    for(i=0; i < str.length; i++){ //fixed spelling from 'str.lenght' 
 
     if (str.charAt(i) == "-") { 
 
      chart = "_"; 
 
      string = string + chart; 
 
     } 
 
     else { 
 
      chart = str.charAt(i); 
 
      string = string + chart; 
 
     } 
 
    } 
 
    return string 
 
} 
 

 
var body = document.querySelector('body'), 
 
    output = kebabToSnake('-'); //First test with '-' in conditional statement 
 

 
body.innerHTML = output; //display to body 
 

 
output = kebabToSnake('Another String'); //Second test with random text triggering ELSE statement 
 

 
body.innerHTML += '<br>' + output; //display to body

+0

Lol私は気付かなかったと信じられない、それは私を夢中にさせていた。ありがとうございました!!! –

+0

@LucasAあなたのために働いた場合、私の答えを受け入れてください。 (チェックマーク)。 – basement

-1

あなたはより簡潔に正規表現を使用することによって、この目標を達成することができます

function kebabToSnake (str) { 
    return str.replace(/-/g, '_'); 
} 
+0

'coz' - 'はRegExpの特殊文字であるため、一致するプレーン文字として使用するときはエスケープする必要があります。 – tibetty

+0

はい、多分あなたは正しいです。しかし、実際には、私はいつも私のプログラマーにそれを逃れるように頼んでいます。正確さに否定的な影響はありません。 – tibetty

関連する問題