なぜこのルビコードは何も返しませんか？

Question

私は数値を取って、奇数の周りにダッシュを入れた文字列を返そうとしています。また、文字列はダッシュで開始または終了してはいけません。

私は次のことを書いたが、それは何も返しません：

def dasherize_number(num) 
    string = num.to_s 
    i = 0 

    while i<string.length 
    if (string[i].to_i % 2) != 0 
     string[i] = '-' + string[i] + '-' 
    end 
    i += 1 
    end 

    if string[0] == '-' 
    string.pop(1) 
    end 

    if (string.length - 1) == '-' 
    string.pop(1) 
    end 

    string 
end 

私が正しく理解していればそれが無限ループしているように見えます。コンソールは出力を表示せず、リフレッシュしない限り何もすることができません。私はそれぞれのキャラクターでコードを見直しましたが、どこが間違っているのか分かりません。

Answer 1

コードには多くの論理的な問題がありました。 は、ここで、文字列の長さがあるため、まず、あなたは、あなたのwhileループ機能しませんi < string.length でこの条件を持っていたあなたのためだけに

def dasherize_number(num) 
    string = num.to_s 
    str_len = string.length 

    i = 0 

    while i < str_len 
    next if string[i] == '-' 
    if (string[i].to_i % 2) != 0 
     string[i] = '-' + string[i] + '-' 
     str_len = string.length 
     i += 3 
    else 
     i += 1 
    end 
    end 

    if string[0] == '-' 
    string = string[1..-1] 
    end 

    if (string[string.length - 1]) == '-' 
    string = string[0..-2] 
    end 

    string.gsub('--', '-') 
end 

、について説明

1. を働くかもしれない何か変化し続けるだから私は値を格納し、文字列が更新された場合は、変数を更新する変数を使用しました。

2. 文字列が更新された場合、次の2つのインデックスをスキップできることを確認できます。 例：数値入力 - > 122 最初の反復後、文字列は-1-22 になるので、次のインデックスに対して同じ条件を実行しないようにします。つまり、1、つまり無限ループになります。

3. popは、配列のようにインデックスを使用して文字にアクセスできるため、文字列には使用できません。popを文字列に使用できません。

4. ダッシュが連続していないことを確認するために、gsubを使用してダッシュを1つ置き換えました。

Answer 2

問題は、このコードの一部であるように思わ：

while i<string.length 
    if (string[i].to_i % 2) != 0 
     string[i] = '-' + string[i] + '-' 
    end 
    i += 1 
end 

あなたstringが奇数が含まれている場合、それは2つの以上文字（2倍-）によってその長さを増加させるが、1（i+=1）で、それをインクリメント。

初期文字列の長さをvarに割り当て、その長さをwhileループでチェックします。

string_length = string.length 

while i < string_length 
    if ((string[i].to_i % 2) != 0) 
     string[i] = '-' + string[i] + '-' 
    end 
    i += 1 
end