リファクタリングのメソッド

Question

私はTic Tac Toeのゲームを作りましたが、2つの同様のメソッドを1つにリファクタリングしたいと思います。

def computer_offense(brd, square='') 
    WINNING_LINES.each do |line| 
    square = find_at_risk_square(line, brd, COMPUTER_MARKERR) 
    break if square 
    end 
    square 
end 

def computer_defense(brd, square) 
    WINNING_LINES.each do |line| 
    square = find_at_risk_square(line, brd, PLAYER_MARKER) 
    break if square 
    end 
    square 
end 

Answer 1

あなたはこのようにリファクタリングすることができます

def computer_offense_or_defense(brd, square='', offense=true) 
    WINNING_LINES.each do |line| 
    square = find_at_risk_square(line, brd, offense ? COMPUTER_MARKERR : PLAYER_MARKER) 
    break if square 
    end 
    square 
end 

だからここで以前に今あなたがcomputer_offense_or_defense(brd, square,true) を呼び出すことができますし、computer_defenseのためにあなたはあなたがこのようにそれをリファクタリングすることができます

computer_offense_or_defense(brd, square,false) 

Answer 2

を呼び出すことができますcomputer_offenseと呼ばれます

def computer_offense(brd, square='') 
    get_square(brd, square, COMPUTER_MARKERR) 
end 

def computer_defense(brd, square) 
    get_square(brd, square, PLAYER_MARKER) 
end 

def get_square(brd, square, marker) 
    WINNING_LINES.each do |line| 
    break if square = find_at_risk_square(line, brd, marker) 
    end 
    square 
end