Funkcja checkWin w nieodpowiednim miejscu

[Bravo555]

Zauważyłem, że funkcja checkWin jest prawie taka sama dla każdego gracza, a jedyne czym się różni to wartość do której porównywana jest zawartość pola.

• Dla PlayerGreen

virtual bool checkWin(int board[3][3])
    {
        if(((board[0][0] == 1) && (board[0][1] == 1) && (board[0][2] == 1)) ||
           ((board[1][0] == 1) && (board[1][1] == 1) && (board[1][2] == 1)) ||
           ((board[2][0] == 1) && (board[2][1] == 1) && (board[2][2] == 1)) ||
           ((board[0][0] == 1) && (board[1][0] == 1) && (board[2][0] == 1)) ||
           ((board[0][1] == 1) && (board[1][1] == 1) && (board[2][1] == 1)) ||
           ((board[0][2] == 1) && (board[1][2] == 1) && (board[2][2] == 1)) ||
           ((board[0][0] == 1) && (board[1][1] == 1) && (board[2][2] == 1)) ||
           ((board[2][0] == 1) && (board[1][1] == 1) && (board[0][2] == 1)))
        {
            return true;
        }
        else
            return false;
    }

• dla PlayerBlue

virtual bool checkWin(int board[3][3])
    {
        if(((board[0][0] == 2) && (board[0][1] == 2) && (board[0][2] == 2)) ||
           ((board[1][0] == 2) && (board[1][1] == 2) && (board[1][2] == 2)) ||
           ((board[2][0] == 2) && (board[2][1] == 2) && (board[2][2] == 2)) ||
           ((board[0][0] == 2) && (board[1][0] == 2) && (board[2][0] == 2)) ||
           ((board[0][1] == 2) && (board[1][1] == 2) && (board[2][1] == 2)) ||
           ((board[0][2] == 2) && (board[1][2] == 2) && (board[2][2] == 2)) ||
           ((board[0][0] == 2) && (board[1][1] == 2) && (board[2][2] == 2)) ||
           ((board[2][0] == 2) && (board[1][1] == 2) && (board[0][2] == 2)))
        {
            return true;
        }
        else
            return false;
    }

Jak widać funkcje są prawie takie same, moim zdaniem należałoby je przenieść gdzieś do klasy Game.