rubyのメソッドの基本的な仕様について。

Question

現在入れた文字列の射影を取るメソッドを使っているのですが、その際に引数に指定した配列(ソースのa)がなぜだか変化してしまいます。
ソースを見ても原因がわからず、dupやcloneを使っても変化してしまうためお手上げ状態です。
ものすごく基本的な質問ですが、良ければ原因をお教え下さい。

def projection(db , series)
  
  wo=0
  
  while wo<db.length				#dbの配列数
  
    wi=0
    while wi < series.length 		#系列の配列数だけ回す。
      
      wa = 0
      while wa < db[wo].length						#db[wo]の配列数だけ回す。
      
        
        if db[wo][wa].index(series[wi]) != nil  	#db配列要素の中に系列要素があった場合、if文の中に入る。
          
          wu=0				#系列がある配列要素より前のdb配列要素をすべてカット
          while wu < db[wo].length
            db[wo][wu]=db[wo][wa+wu]
            wu=wu+1
          end
          
          wu=0				#配列要素内の系列までの文字をすべてカット
          we=db[wo][0].index(series[wi])			#dbの一つ目の系列がある場所
          we = we+(series[wi].length-1)
          while wu <= we
            db[wo][0].slice!(0)
            wu=wu+1
          end
          
          db[wo][0].insert(0,"_") 			#カットした場所に_を置く。
          
          if db[wo][0]=="_\n" 				#カットしたことで配列の中身が_と改行のみになった場合、カットする。。
            wu=0
            while wu < db[wo].length
              db[wo][wu]=db[wo][wu+1]
              wu=wu+1
            end
          end
          
          
          break
        end
        wa=wa+1
      end
      wi=wi+1
    end
    db[wo].compact!							#空になった配列要素をカット
    
    
    wo=wo+1
  end
  db.compact!								#空になった配列要素をカット
  return db
end





a=[["1234567890","123456789"],["1234567890","123456789"]]
b=["3"]

c=projection(a , b)

p a

Answer 1

2次元配列は、 dup や clone では、完全には 複製をつくることはできません。

• Rubyで二次元配列のコピー http://simanman.hatenablog.com/entry/2013/06/20/161437

次のプログラムを試してみてください。

# coding: utf-8

def my_func(db)
  db[0] = 'x'
  db[0][0] = "y"
  return db
end

def my_func_1(db)
  work = db.clone
  work[0] = "x"
  work[1][0] = "y"
  return work
end

def my_func_2(db)
  work = Marshal.load(Marshal.dump(db))
  work[0] = "x"
  work[1][0] = "y"
  return work
end

a = [["1234567890","123456789"],["1234567890","123456789"]]
c = my_func(a)
puts "--- after my_func"
p a
p c

a = [["1234567890","123456789"],["1234567890","123456789"]]
c = my_func_1(a)
puts "\n--- after my_func_1"
p a
p c

a = [["1234567890","123456789"],["1234567890","123456789"]]
c = my_func_2(a)
puts "\n--- after my_func_2"
p a
p c

実行すると次のようになります。

$ ruby array.rb
--- after my_func
["y", ["1234567890", "123456789"]]
["y", ["1234567890", "123456789"]]

--- after my_func_1
[["1234567890", "123456789"], ["y", "123456789"]]
["x", ["y", "123456789"]]

--- after my_func_2
[["1234567890", "123456789"], ["1234567890", "123456789"]]
["x", ["y", "123456789"]]

my_func, my_func_1, my_func_2 での配列の複写方法と、その複写した配列へに代入の結果の差がわかるとおもいます。
my_func_2 の方法で配列を複写すると、解決するとおもいます。

Comments

[fei2003]

ありがとうございます！
こちらの方法も試してみます。

Answer 2

projectionの中で、仮引数のdbが指すオブジェクトを書き換えているので、実引数のaにも影響が及びます。
Rubyの変数は、C等で言うとすべてポインタ(リファレンス)だと思ってください。

あと、オブジェクトを内部に含んだオブジェクト～～例えば["A","B"]～～の場合、cloneやdupをしても複製されるのは外側のオブジェクト(例だと[ ])だけで、内部に含まれたオブジェクト("A"と"B")は複製されず共有されます。

a=["A","B"]
puts a.object_id     #=>21104256
puts a[0].object_id  #=>21104280
puts a[1].object_id  #=>21104268

b=a.clone
puts b.object_id     #=>21104160 aとは別オブジェクト
puts b[0].object_id  #=>21104280 a[0]と同一オブジェクト
puts b[1].object_id  #=>21104268 a[1]と同一オブジェクト

浅いコピー(shallow copy)と深いコピー(deep copy)に付いて調べてみてください。

db2 = db.cloneは浅いコピーなので、深いコピーならこの場合は、

db2 = db.map{|x| x.map{|y| y.clone}}

ですね。

Comments

[fei2003]

ありがとうございます。
よく理解出来ました。

[otn]

とりあえずの解決策を追記しておきました。