Rubyコードで `String＃delete`がエラーを返すのはなぜですか？

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

あなたが言及した無限ループのよう

利用string.delete(string[0])代わりのstring.delete[string[0]]

EDIT

間違っdelete機能を使用しています。 中止の状態は0 < string.length

となります。そして、実際にはstring.delete[string[0]]ステートメントで一度に1文字ずつ削除するとします。

しかし正確には、文字を削除して新しい文字列を返しますが、実際には実際の文字列を変更/変更することはありません。

だから、あなたは私が言うことを許可している場合は、はるかに簡単な方法でこれを行うことが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で実行してください！別に、すでにあなたが必要なもののほとんどは、ネイティブRubyのStringクラスで実装されている...

に答えてきたdelete()代わりのdelete[]を使用してから

Answer 3

。 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