To your battle stations! Just a little thing


#1
from random import randint

board = []

for x in range(0, 5):
    board.append(["O"] * 5)

def print_board(board):
    for row in board:
        print " ".join(row)

print_board(board)

def random_row(board):
    return randint(0, len(board) - 1)

def random_col(board):
    return randint(0, len(board[0]) - 1)

ship_row = random_row(board)
ship_col = random_col(board)


for turn in range(3):
    guess_row = int(raw_input("Guess Row:"))
    guess_col = int(raw_input("Guess Col:"))
    if (guess_row == ship_row) and (guess_col == ship_col):
        print "Congratulations! You sank my battleship!"
    else:
        if guess_row not in range(5) or guess_col not in range(5):
            print "Oops, that's not even in the ocean."
        elif board[guess_row][guess_col] == "X":
            print "You guessed that one already."
        else:
            print "You missed the battleship"
            print board[guess_row][guess_col] == "X"
            print print_board(board)

It isnĀ“t changing Os for Xs, so you can instert the same coords over and over again. Please help


#2

Put you code for the guess inside the loop.

Put that into your for loop.


#3

i noticed that just as i posted it but it wont change the Os for the Xs, so you can insert the same coordinates over and over again


#4

The question then is, are you printing the board before or after you change the item in your list? Or did you not change the location's O into an X?

Check your code and ensure that you did both of the previous.


#5

It is printing after changing the Os for Xs, but it just prints False


#6

Did you read all lines of your code?

Go to the IDE and enter the following

PYTHON SNIPPETS

1 == 0
a = 1
a == 1

Notice anything?

Need any more help feel free to ask.