2

loop1（check1）にネストされたloop2（check2）がありますが、ネストされたloop2（check2）は1回しか実行されないようです。

両方のループに同じ配列が含まれています。このスクリプトは、check1で重複するIDをチェックするために使用されます。

check1=["0", "0", "0", "1", "1", "2", "3", "4", "5", "100", "4294967294", "9", "11", "6", "200", "7", "201", "811", "202", "204", "3000", "205", "3001", "3001", "3001"]
check2 =["0", "0", "0", "1", "1", "2", "3", "4", "5", "100", "4294967294", "9", "11", "6", "200", "7", "201", "811", "202", "204", "3000", "205", "3001", "3001", "3001"]

コード：

check1.each do |check1|
  counter=0
  puts "checking against:"+check1.to_s
  check2.each do |check2|
    puts "checking:"+check1.to_s+"|"+check2.to_s
    if check1 == check2
      counter += 1
    end
  end
  if counter > 1
    dupUID << check1
  end
end

結果：

checking against:0 <- checking the 1st element in check1
checking:0|0
checking:0|0
checking:0|0
checking:0|1
checking:0|1
checking:0|2
checking:0|3
checking:0|4
checking:0|5
checking:0|100
checking:0|4294967294
checking:0|9
checking:0|11
checking:0|6
checking:0|200
checking:0|7
checking:0|201
checking:0|811
checking:0|202
checking:0|204
checking:0|3000
checking:0|205
checking:0|3001
checking:0|3001
checking:0|3001
checking against:0<- checking the 2nd element in check1
checking:0|3001   <- nested loop2(check2) is not looping again on the 2nd element of loop 1
checking against:0
checking:0|3001   <- loop2 stops at the last element for the remaining elements in check1
checking against:1
checking:1|3001
checking against:1
checking:1|3001
checking against:2
checking:2|3001
checking against:3
checking:3|3001
checking against:4
checking:4|3001
checking against:5
checking:5|3001
checking against:100
checking:100|3001
checking against:4294967294
checking:4294967294|3001
checking against:9
checking:9|3001
checking against:11
checking:11|3001
checking against:6
checking:6|3001
checking against:200
checking:200|3001
checking against:7
checking:7|3001
checking against:201
checking:201|3001
checking against:811
checking:811|3001
checking against:202
checking:202|3001
checking against:204
checking:204|3001
checking against:3000
checking:3000|3001
checking against:205
checking:205|3001
checking against:3001
checking:3001|3001
checking against:3001
checking:3001|3001
checking against:3001
checking:3001|3001

誰かが私の間違いを指摘できますか？どうもありがとう。解決済み：すべてに感謝します！

check1.each do |ch1|
  counter=0
  check2.each do |ch2|
    if ch1 == ch2
      counter += 1
    end
  end
  if counter > 1
    dupUID << ch1
  end
end

puts dupUID
4

5 に答える 5

2

私はあなたがそのようなコードを書くべきではないと思います。より良い解決策があります：

x = [0, 10, 15]
y = [0, 20, 15]
x & y # => [0, 15]

このメソッドは、指定された2つの配列に共通の要素を返します。

（更新） 1つのアレイ内でそれを行う別の方法があります：

[0, 10, 10, 15, 20].inject({}) 
{ 
    |a, c| a[c] ||= 0; a[c] = a[c].next; a 
}.delete_if { |k, v| v == 1 }.keys
于 2010-07-30T02:03:51.350 に答える
2

ブロック変数の名前が同じであるため、配列check1と配列をシャドウイングしています。check2do

内側のdoブロックの後check2は、配列自体ではなく、配列の最後の要素を参照します。

ch1これを修正するには、ブロック変数の名前をandのような名前に変更しますch2。

したがって、これは、ネストされたループが期待どおりに実行されない理由を説明しています。実際、アルゴリズム自体にも欠陥があります。@floatlessの答えは、問題へのより良いアプローチを提供します。

于 2010-07-30T02:09:36.990 に答える
1

短くなりますが、理解しやすくなりません。

check.inject(Hash.new(0)) { |a, x| a[x] += 1; a }.reject { |k, v| v <= 1 }.keys
于 2010-07-30T02:32:10.457 に答える
0

単一の配列がcheckあり、その中のすべての重複要素を検索したいとします。Ruby1.9を使用します。

check.group_by {|v| v}.map { |k, v| v.size > 1 ? k : nil }.compact

説明：

  • group_byキーが数値で、値が各オカレンスの配列であるハッシュを返します。
  • map値が1回だけ発生する場合はnilを返し、複数回発生する場合は値を返します。
  • compactすべての値をクリーンアップしnilます。

ステップバイステップの結果は次のとおりです。

# after group_by
{"204"=>["204"], "6"=>["6"], "11"=>["11"], "205"=>["205"], "7"=>["7"], "811"=>["811"], "9"=>["9"], "4294967294"=>["4294967294"], "0"=>["0", "0", "0"], "100"=>["100"], "1"=>["1", "1"], "200"=>["200"], "2"=>["2"], "201"=>["201"], "3"=>["3"], "3000"=>["3000"], "202"=>["202"], "4"=>["4"], "3001"=>["3001", "3001", "3001"], "5"=>["5"]}>

# after map
[nil, nil, nil, nil, nil, nil, nil, nil, "0", nil, "1", nil, nil, nil, nil, nil, nil, nil, "3001", nil]

# after compact
["0", "1", "3001"]
于 2010-07-30T03:10:26.360 に答える
0

（）に警告があった場合は$VERBOSE = true、間違いがあったことを通知します。

ruby 1.9.1のIRBでは、コマンドラインで$ VERBOSEをオンにすることはできませんが、1.9.2ではオンにできます。

更新：この質問により、rubyのこのバグ/機能の改善を提出することになりました。ありがとう！

于 2010-07-30T03:50:08.343 に答える