ルビで文のチェックが機能しない場合

Question

私は以下のコードを持っていますが、実行するとIF文のコードは決して実行されません。

変数cとkeyの値が下の太字の出力と同じでも、IF文が実行されない理由はわかりません。 keyがsymbol、ないstringであるためである

text = 'cab' 

letters = { 
    'a': 1, 
    'b': 2, 
    'c': 3, 
} 

text.split("").each do |c| 
    puts c 
    puts "Text #{c}" 
    for key, value in letters 

    puts "Key #{key}, Text #{c}, Value #{value}" 
    if c == key 
     puts "hello #{c}" 
    end 
    end 
end 

出力

 
c 
Text c 
Key a, Text c, Value 1 
Key b, Text c, Value 2 
Key c, Text c, Value 3 
a 
Text a 
Key a, Text a, Value 1 
Key b, Text a, Value 2 
Key c, Text a, Value 3 
b 
Text b 
Key a, Text b, Value 1 
Key b, Text b, Value 2 
Key c, Text b, Value 3 

Answer 1

。文字列をキーとして使用する場合は、代わりに'a' => 1表記を使用する必要があります。あなたのkeyが文字列に変更し、シンボルである

if c == key.to_s

Answer 2

：

if c == key.to_s

Answer 3

あなたがしている問題

また、あなたは、シンボルの.to_sメソッドを呼び出すことができあなたのifステートメントのコンパレータは異なるタイプなので、条件は常にfalseです。この変更を行います。

letters = { 
    'a' => 1, 
    'b' => 2, 
    'c' => 3, 
} 

これは、文字列、代わりのシンボルをキーとあなたのlettersハッシュを行います。ハッシュテーブル=>は、キーをシンボルに変換することなく、ハッシュの値にキーを関連付けるために使用されます。ハッシュキーを文字列にすると、文字列のハッシュキーを文字列の文字と正しく比較することができます。他の回答で提案されているように

またそうのように、インライン変換を行っている可能性：

if c == key.to_s 

しかし、これはランタイムペナルティあなたがチェックをしなければならないたびに発生します。単にハッシュキーを変更するだけでチェックが行われ、比較するたびにあるオブジェクトを別のオブジェクトに手動で変換する必要はありません。 lettersハッシュは、文字との比較を提供するためだけに存在するため、使用されるタイプとしてキーを宣言することが理にかなっています。

この特定のプログラムは動的変換の影響を大幅に受けませんが、不要なコンバージョンを検出し回避することに慣れていれば、コード全体の品質が向上します。早期に優れたコーディング手法を開発すれば、長期的にはより良い開発者になるでしょう。