変数の基本的な問題

Question

私は何か非常に根本的な欠点があるように感じます。なぜなら、より多くの髪を引き裂かないようにするために、p1、p1t、p1t2が "hand = newgame" p1を見る。私はなぜp1が変更されているのか分からないので、私はここで完全に困惑しています。どんな助けでも大いに感謝します。

# encoding: utf-8 

class Cards 

def view hand 
    x = 0 
    hand.each do |card| 
     if card[0] == 's' 
      card[0] = '♠' 
     elsif card[0] == 'd' 
      card[0] = '♦' 
     elsif card[0] == 'h' 
      card[0] = '♥' 
     elsif card[0] == 'c' 
      card[0] = '♣' 
     else 
      #nil 
     end 
     hand[x] = card 
     x = x + 1 
    end 
    prettyhand = '' 
    hand.each do |card| 
     prettyhand = prettyhand + card[0] + card[1] + ' ' 
    end 
    return prettyhand 
end#view 

end#Cards 
########### 

deck = 0 
up = 0 
p1 = 0 
p2 = 0 
newgame = Cards.new 
p1 = [["c", "4"], ["s", "4"], ["d", "q"], ["c", "8"], ["s", "5"], ["s", "q"], ["h", "3"], ["c", "10"]] 
    p1t = [] 
    p1t2 = [] 
    hand = '' 
    p1.each do |card| 
     p1t.push card 
    end 
    p1t.each do |card| 
     p1t2.push card 
    end 
    p '----------------------------------' 
    p 'fresh p1:' 
    p p1 
    p p1t 
    p p1t2 
    hand = newgame.view p1 
    p 'unfresh p1:'#why is this changing?? 
    p p1 
    p p1t 
    p p1t2 
    p '----------------------------------' 

出力：あなたは括弧もメソッドシグネチャでは、あなたは常にあなたのコードでそれらを省略する理由はわからないいいです、hand（BTWパラメータを取る方法viewを定義している

"----------------------------------" 
"fresh p1:" 
[["c", "4"], ["s", "4"], ["d", "q"], ["c", "8"], ["s", "5"], ["s", "q"], ["h", "3"], ["c", "10"]] 
[["c", "4"], ["s", "4"], ["d", "q"], ["c", "8"], ["s", "5"], ["s", "q"], ["h", "3"], ["c", "10"]] 
[["c", "4"], ["s", "4"], ["d", "q"], ["c", "8"], ["s", "5"], ["s", "q"], ["h", "3"], ["c", "10"]] 
"unfresh p1:" 
[["♣", "4"], ["♠", "4"], ["♦", "q"], ["♣", "8"], ["♠", "5"], ["♠", "q"], ["♥", "3"], ["♣", "10"]] 
[["♣", "4"], ["♠", "4"], ["♦", "q"], ["♣", "8"], ["♠", "5"], ["♠", "q"], ["♥", "3"], ["♣", "10"]] 
[["♣", "4"], ["♠", "4"], ["♦", "q"], ["♣", "8"], ["♠", "5"], ["♠", "q"], ["♥", "3"], ["♣", "10"]] 
"----------------------------------" 

Answer 1

配列はコピーされずに参照渡しされます。このスニペットをご覧ください。

a1 = [[1], [2], [3]] 
a2 = [] 

a2 << a1.first # push a reference to the array 
a2 # => [[1]] 


a1[0][0] += 1 # => 2 
a1 # => [[2], [2], [3]] # a1 has changed 
a2 # => [[2]] # a2 changed too! 

# now push a copy of the array 
a2 = [a1.first.dup] # => [[2]] 

a1[0][0] += 1 # => 3 
a1 # => [[3], [2], [3]] # a1 has changed 
a2 # => [[2]] # a2 has not 

Answer 2

）。 viewメソッドは、その引数handを変更します。あなたはこの

hand = newgame.view p1 

としてそれを呼び出す、 p1は法 viewによって変更され、それが変わる理由です。

p1を0に設定して、それを配列の配列に設定するのも、私は興味があります。それをする理由はありません。また、あなた自身と他の人のために、記述的な変数名を使用しようとしてください。

Answer 3

FWIWは、あなたのview方法は、多くのことを簡単にすることができます。

class Cards 
    Suits = { 'c' => '♣', 'd' => '♦', 'h' => '♥', 's' => '♠' } 

def view(hand) 
    hand.map do |suit, rank| [ Suits[suit], rank ] end 
end 
end 

このバージョンでは何も変更されませんので、問題を回避できます。