2016-04-23 2 views
1

Итак, я должен сделать подключение к четырем консолям, и по какой-то причине моя плата отображает 0s вместо «.» S. Я не уверен, что я сделал не так. Есть 3 модуля, но я покажу только это, так как я предполагаю, что это основная проблема.Совет не правильно печатает

import connectfour 

def _print_board_num(): 
    for i in range(len(connectfour.new_game().board)): 
     print(str(i+1), end = ' ') 
    print() 

def print_board(game: 'connectfour.GameState')->[(str)]: 
    _print_board_num() 
    for row in range(connectfour.BOARD_ROWS): 
     for column in range(connectfour.BOARD_COLUMNS): 
      if game.board[column][row] == ' ': 
       print('.', end = ' ') 
      else: 
       print(game.board[column][row], end = ' ') 

     print() 

def move()->str: 
    input('Drop or Pop? ') 

def column1()->int: 
    int(input('What number of column? ')) 

Моя плата печати как:

1 2 3 4 5 6 7 
0 0 0 0 0 0 0 
0 0 0 0 0 0 0 
0 0 0 0 0 0 0 
0 0 0 0 0 0 0 
0 0 0 0 0 0 0 
0 0 0 0 0 0 0 

, но это, как предполагается печатать, как:

1 2 3 4 5 6 7 
. . . . . . . 
. . . . . . . 
. . . . . . . 
. . . . . . . 
. . . . . . . 
. . . . . . . 

Это функция игрового поля для модуля Четыре в ряд

def _new_game_board() -> [[int]]: 
    ''' 
    Creates a new game board. Initially, a game board has the size 
    BOARD_COLUMNS x BOARD_ROWS and is comprised only of integers with the 
    value NONE 
    ''' 
    board = [] 

    for col in range(BOARD_COLUMNS): 
     board.append([]) 
     for row in range(BOARD_ROWS): 
      board[-1].append(NONE) 

    return board 



def _copy_game_board(board: [[int]]) -> [[int]]: 
    '''Copies the given game board''' 
    board_copy = [] 

    for col in range(BOARD_COLUMNS): 
     board_copy.append([]) 
     for row in range(BOARD_ROWS): 
      board_copy[-1].append(board[col][row]) 

Я бы предпочел e это либо один из этих двух. В основном функция drop, так как я прокомментировал ошибку игрока, но она появляется для функции drop.

def player(game: 'connectfour.GameState') -> None: 
    while True: 
     player = input('Would you like to drop(d) or pop(p)?') 
     if player == 'd': 
      drop(game) 
      return 
     elif player == 'p': 
      pop(game) 
      return 
     else: 
      connectfour.InvalidMoveError(Exception) 
      #print('Invalid Move') 

def drop(game: 'connectfour.GameState') -> bool: 
    try: 
     col = connectfouroverlap.column1() 
     board = connectfour.drop(game,col-1) 
     connectfouroverlap.print_board(board) 
     if gameover(board) != connectfour.NONE: 
      return 
     else: 
      player(board) 
    except: 
     connectfour.InvalidMoveError(Exception) 
     print('Invalid Move') 

Это то, что результат выглядит, когда я запускаю его.

1 2 3 4 5 6 7 
. . . . . . . 
. . . . . . . 
. . . . . . . 
. . . . . . . 
. . . . . . . 
. . . . . . . 
Would you like to drop(d) or pop(p)?d 
What number of column? 2 
1 2 3 4 5 6 7 
. . . . . . . 
. . . . . . . 
. . . . . . . 
. . . . . . . 
. . . . . . . 
. 1 . . . . . 
Invalid Move 
+0

Что имеется в игре.board? – vmonteco

+0

@ vmonteco Я не думаю, что эта часть имеет значение, так как модуль connectfour для нас подготовлен. – accelerate

+0

Это потому, что нам нужно знать, что вы манипулируете, чтобы помочь вам. – vmonteco

ответ

3

После проверки коды, я предполагаю, что game.board является двойным массивом, который содержит только 0 (целые числа?) Значение.

Если это так, то попробуйте заменить:

 if game.board[column][row] == ' ': 
      print('.', end = ' ') 
     else: 
      print(game.board[column][row], end = ' ') 

с:

 if game.board[column][row] == 0: # <--- change the condition here 
      print('.', end = ' ') 
     else: 
      print(game.board[column][row], end = ' ') 

EDIT:->[str] аннотаций в определении функции print_board предполагает, что эта функция предназначена для возврата массив строк , Но это не так.

Вы создаете совершенно новую доску для печати номеров колонок. Это плохая практика, вместо этого используйте существующую.

Вы также можете использовать петли for in непосредственно на массиве, а не использовать индексы. Это потребует гораздо меньше кода и будет намного проще и проще писать:

for row in game.board: # <-- since game.board is your [[int]] 
    for number in row: 
     do_stuff_on(number) 
+0

Еще раз взглянув в оригинальный модуль, я заметил это NONE = 0 RED = 1 YELLOW = 2 Может быть поэтому? – accelerate

+0

@accelerate Итак, кажется, я был прав о том, что такое доска. Предлагает ли решение, которое я предлагаю? – vmonteco

+0

Я просто сделал то, что вы сделали, и заменил 0s на .s, но теперь он отображает его вертикально, а не как обычно. – accelerate

Смежные вопросы