爲什麼`String＃delete`在我的ruby代碼中返回錯誤？

Question

目標是取一個字符串並返回最常見的字母及其數量。對於字符串'hello'，它將返回['l', 2]。

我寫了以下內容：

def most_common_letter(string) 
list = [] 
bigcount = 0 
while 0 < string.length 
    count = 0 
    for i in 0..string.length 
     if string[0] == string[i] 
      count += 1 
     end 
    end 
    if count > bigcount 
     bigcount = count 
     list = (string[0]) 
     string.delete[string[0]] 
    end 
end 
return [list,bigcount] 
end 

我得到以下錯誤：

wrong number of arguments (0 for 1+) 
(repl):14:in `delete' 
(repl):14:in `most_common_letter' 
(repl):5:in `initialize' 

請幫助我瞭解我在做什麼毛病delete聲明，或還有什麼導致這返回一個錯誤。

我以另一種方式完成解決方案，但我認爲這將工作得很好。

Answer 1

您使用的是delete功能失常

使用string.delete(string[0])，而不是string.delete[string[0]]

編輯

至於你提到的無限循環。 您的狀態爲0 < string.length

而且您期望string.delete[string[0]]語句一次實際刪除一個字符。

但是它究竟做了什麼，它刪除了一個字符並返回新的字符串，但它實際上並沒有改變/改變實際的字符串。

因此，嘗試將其更改爲string = string.delete[string[0]]

Answer 2

你可以這樣做在一個更簡單的方法，如果你讓我說。

def most_common_letter(string) 
    h = Hash.new 
    string.chars.sort.map { |c| 
    h[c] = 0 if (h[c].nil?) 
    h[c] = h[c] + 1 
    } 
    maxk = nil 
    maxv = -1 
    mk = h.keys 
    mk.each do |k| 
    if (h[k] > maxv) then 
     maxk = k 
     maxv = h[k] 
    end 
    end 
    [ maxk , maxv ] 
end 

如果你測試這個與

puts most_common_letter("alcachofra") 

結果將是

[ 'a', 3 ] 

最後，記住你不需要在Ruby方法的最後一個回報。最後分配的值會自動返回。

以Ruby的方式做Ruby！

Answer 3

除了使用delete()代替delete[]這已經回答了...

你最需要的Ruby的String類實現原生的東西。 each_char和count。

def most_common_letter(string) 
    max = [ nil, 0 ] 
    string.each_char {|char| 
    char_count = string.count(char) 
    max = [ char, char_count ] if char_count > max[1] 
    } 
    return max 
end