爲什麼這個ruby代碼不能返回任何東西？

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

您的代碼中存在很多邏輯問題。 這裏的東西，可能只是爲了工作，你

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. 首先，你必須在你的while循環這個條件i < string.length 這是行不通的，因爲字符串的長度不斷變化。所以我使用了一個變量來存儲這個值並且在字符串被更新時更新變量。

2. 如果更新字符串，我們可以確定我們可以跳過下兩個索引。 例如：number inputed - > 122 然後第一次迭代後字符串將是-1-22 因此，我們不想爲下一個索引運行相同的條件，因爲這將再次爲1，因此是無限循環。 （希望你明白了）

3. pop對字符串不起作用，僅僅因爲我們可以使用像數組這樣的索引來訪問字符，所以我們不能使用pop作爲字符串。

4. 爲了確保沒有連續的破折號，我用gsub用單個破折號代替它們。

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