「もしa,b,c,d全ての値が違うならば{何かしらのアクション}を起こす。ただし変数の値が""のものは除く」のような条件式が書きたい

Question

Ruby on Railsを使ってWebサイトを製作中なのですが、以下のような条件の書き方がわかりません。

a = ランダムな文字列
b = ランダムな文字列（ただし""もあり)
c = ランダムな文字列（ただし""もあり)
d = ランダムな文字列（ただし""もあり)

if a,b,c,d全てが違う値の時（ただし、値が""な変数は除く)
   何かしらのアクション
end

変数a,b,c,dを作ってそれぞれにランダムな値が入るのですが、その際に「もしa,b,c,d全ての値が違うならば{何かしらのアクション}を起こす。ただし変数の値が""のものは除く」のような条件式が書きたいです。
ご回答よろしくお願いします。

Comments

[tatsu99]

すみません。
a=""
b=""
c="X"
d="Y"
のケースはアクションを起こすのでしょうか。
a=""
b=""
c=""
d="Y"
のケースもアクションを起こしますか？

[KokiOya]

起こします！わかりにくくて申し訳ありません。

[tatsu99]

すみません。すべて、""の場合は、どうなりますか。
a=""
b=""
c=""
d=""
の場合です。このケースが、回答者の間で問題になっています。

[KokiOya]

軽く質問したつもりがそこまで考えてくださってありがとうございます。実はaには必ず何かしらの値が入るので""が入ることは無いようになっております。上記の質問も編集します。質問に間違いがありましたこと、大変申し訳ございません。

Answer 1

ary = [a,b,c,d].grep_v("")
if !ary.uniq!
  何かしらのアクション
end

Comments

[set0gut1]

uniq! の返り値にそんな使い方があったんですね。勉強になりました。

[winterboum]

これですと、"" を含むときは期待と違ってしまいませんか？
A　B　C　"" のとき　！nil && !true 　となって、結果 false です

[winterboum]

勘違い　A B C "" は大丈夫ですね。
[[1, 2, 3, 4], [1, 2, 3, ""], [1, 2, 2, ""], [1, 2, "", ""]].
each{|a| ary=a.dup; p [a, (!ary.uniq! || !ary.include?(""))] }
=>
[[1, 2, 3, 4], true]
[[1, 2, 3, ""], true]
[[1, 2, 2, ""], false]
[[1, 2, "", ""], false]　<= こいつが期待と違う

[asm]

修正しました。

[KokiOya]

この度はご回答ありがとうございます。非常にわかりやすく完結なコードでしたのでそのまま使わせてもらいました。本当にありがとうございました。またのご回答よろしくお願いします。

Answer 2

Ruby は詳しくないですが…

a～d を配列にして、空文字列を除外して、ソートして
同じのが連続してなきゃokみたいな関数つくるのかなあ。

Comments

[takasima20]

あー、全部が空文字列だった時の結果って true ? false?
それともなんか別の動きすんのかなあ。

[KokiOya]

ご回答ありがとうございます。asmさんのコードでうまくいきましたので、参考程度に確認してください。またのご回答よろしくお願いします

Answer 3

判定処理をメソッドにし、いくつかのケースを試してみました。

a.rb

def check(ary)
    filtered = ary.select{|x| x != ""}
    filtered.length == filtered.uniq.length
end

TESTS = [
  [], [''],
  ['a'], ['a', ''], 
  ['a', 'a'], ['a', 'a', ''], 
  ['a', 'b'], ['a', 'b', ''],
  ['', ''],

  [nil], [nil, ''],
  [nil, nil], [nil, nil, ''],
  [[1, 2], [2, 1]],
  # nil
]

TESTS.each do |ary|
  p("#{check(ary)}: #{ary}")
end

実行例

Answer 4

たくさんのご回答ありがとうございました。今回はasmさんのコードをそのまま使わせてもらいました。

ary = [a,b,c,d].grep_v("")
if !ary.uniq!
   何らかのアクション
end

たくさんご回答いただけたのに採用できないものがあったこと大変申し訳ありませんでした。この度は本当にありがとうございました。またのご回答よろしくお願いします。

Answer 5

!ary.reject{|a| a==""}.uniq!

Comments

[tatsu99]

a = ""
b = ""
c = ""
d = ""
ary = [a,b,c,d]
のとき、trueになってしまいますが、それは、意図された動作でしょうか。

[winterboum]

意図しています。

ここは仕様がはっきりしていないんですね。
で、
「””は無視する。残ったものに重複がなければ true」
をそのまま解釈しました。

[winterboum]

false にするならこうかなぁ。
(compressed = ary.reject{|a| a== ""}).present? && !compressed.uniq!
＃ 無理やりonelinerにするな！

[KokiOya]

ご回答ありがとうございます。前回もご回答していただき、いつもありがとうございます。本当に助かっています。これからも回答していただけると幸いです。本当にありがとうございました。

Answer 6

値が""である要素を削除して、その結果、1件以上のデータがあり、かつ重複がないなら、アクション
を起こします。

ary = [a,b,c,d]
ary.delete("")
if ary.length > 0 && ary.length == ary.uniq.length
  puts "ACTION"
end

Comments

[KokiOya]

ご回答ありがとうございました。大変参考にさせてもらい、一度はこちらの方法でうまくいったのですが、今回はasmさんのコードを使わせてもらいました。またのご回答よろしくお願いします

Answer 7

考えてみました

# frozen_string_literal: true

# NOTE: Rails 上では不要
# スッピンの Ruby コードとして書いているので require している
require 'active_support'
require 'active_support/core_ext'

def validate_uniqueness_check(array)
  # uniq で重複排除した結果と配列の差異は発生しないか
  is_all_unique =
    array.size.eql?(array.uniq.size)

  # 配列中の文字列に空白文字が含まれないか
  is_not_included_empty_string =
    array.exclude?('')

  is_all_unique && is_not_included_empty_string
end

# ランダムな値を生成, a 〜 d までの変数に多重代入
# (適当なアルファベット4文字を生成)
a, b, c, d =
  Array.new(
    4,
    -> { ('a'..'z').to_a.sample(4).join }
  ).map(&:call)

# いくつか テスト用データのパターンを作成
sample_x = [a, b, c, d] # ぜんぶ違う値
sample_y = [a, b, c, ''] # 空白文字が含まれる
sample_z = [a, b, 'foo', 'foo'] # 同じ値が含まれる

# データを渡して結果を確認
p validate_uniqueness_check(sample_x) # => true
p validate_uniqueness_check(sample_y) # => false
p validate_uniqueness_check(sample_z) # => false

素朴に、考えられる条件の通り実装してもいいのですが、「こういう状況のときに、こうしたい」というコード以外の部分について、より詳しく言及されていれば、また違った回答が得られるかもしれません

なにか参考になれば幸いです

Comments

[KokiOya]

ご回答ありがとうございました。とても参考になりました！本当にありがとうございました。これからもよろしくお願いします。

Answer 8

条件式は以下のようになります。

ary = [a,b,c,d]

ary.select{ |e| e !=""}.uniq.count == ary.select{ |e| e !=""}.count

4つの変数を一つの配列にまとめ、""であるような要素を除外した後に「ユニークな要素数=全要素数」になっていれば、すなわち4つの変数の中身がバラバラということになります。
逆に、例えばaとbの変数の中身が一緒であればユニークな要素数は全要素数よりも1小さくなります。

Comments

[tatsu99]

a = ""
b = ""
c = ""
d = ""
のときも、条件が成立しますが、意図された結果でしょうか。

[gnbrganchan]

質問内容からこれが想定されるものと判断しました。

[KokiOya]

ご回答ありがとうございました。とても参考にさせていただきました。またのご回答よろしくお願いします。

Answer 9

「a,b,c,dのうち、空文字列でないもので、値が重複するものが存在しない」ときにtrueとなる条件式です。

[a, b, c, d].
    select{|x| not x.empty?}.  #空文字列を除去
    group_by{|x| x}.           #同じ値でグルーピングする
    select{|k, v| v.size > 1}. #サイズ2以上のグループ（重複要素のグループ）だけ抽出する
    empty?                     #サイズ2以上のグループが存在するかどうか判定

Comments

[KokiOya]

ご回答ありがとうございます。またのご回答よろしくお願いします。