世界で闘うプログラミング力を鍛える本_p.107_重複のない文字列_のGithubに書かれているRubyコードと比較したいです。

Question

世界で闘うプログラミング力を鍛える本のp.107の重複のない文字列のコードがGithubにありました。自分で書いたコードとどちらが良いか比較をしたいです。

引用「p.107
重複のない文字列:ある文字列が、すべて固有である(重複する文字がない)
かどうかを判定するアルゴリズムを実装してください。
また、それを実装するのに新たなデータ構造が使えない場合、どのようにすればよいですか？
」

Githubのコード

CtCI-6th-Edition-Ruby/Q1_01_Is_Unique.rb at 8db2b4794a7dec91c31bd655e4076dae84f110ff · careercup/CtCI-6th-Edition-Ruby

def is_unique?(string)
  string.each_char.with_object({}) do |char, hash_table|
    if hash_table[char]
      return false
    else
      hash_table[char] = true
    end
  end

  true
end

書いたコード

def is_unique2?(string)
    a=string.chars
    return a.size == a.uniq.size
end

p is_unique2?("abc")
p is_unique?("abca")

出力

true
false

また、僕の書いたコードがよかったらプルリクを出してもいいのでしょうか？

Answer 1

自分だったらこうかな。2.7以上じゃないと動かないけど。

def is_unique?(str)
  str.each_char.tally.each_value.all?(1)
end

Comments

[DrqYuto]

ありがとうございます！tallyの使い方分からなかったので、助かります…

Answer 2

DrqYuto さんのcodeの方が好きです。
短いだけではなく、何をしているかが一目でわかる

Comments

[DrqYuto]

ありがとうございます！(hashがまだ、理解できてないのでとは言えないっす…