Welcome to the Treehouse Community

The Treehouse Community is a meeting place for developers, designers, and programmers of all backgrounds and skill levels to get support. Collaborate here on code errors or bugs that you need feedback on, or asking for an extra set of eyes on your latest project. Join thousands of Treehouse students and alumni in the community today. (Note: Only Treehouse students can comment or ask questions, but non-students are welcome to browse our conversations.)

Looking to learn something new?

Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and a supportive community. Start your free trial today.


Pratham Mishra
Pratham Mishra
13,537 Points

Incorrect format of rows and columns in Memory Game (OOP)

My final output doesn't show the exact format of rows and columns as demonstrated in the video.

This is my game.py file:

from cards import Card
import random

class Game:
    def __init__(self):
        self.size = 4
        self.card_options = ['Add', 'Boo', 'Cat',
                             'Dev',  'Egg', 'Far', 'Gum', 'Hut']
        self.columns = ['A', 'B', 'C', 'D']
        self.cards = []
        self.locations = []
        for column in self.columns:
            for num in range(1, self.size + 1):
                location = f'{column}{num}'

    def set_cards(self):
        used_locations = []
        for card in self.card_options:
            for i in range(2):
                random_location = random.choice(
                    list(set(self.locations) - set(used_locations)))
                game_card = Card(card, random_location)

    def create_row(self, num):
        row = []
        for column in self.columns:
            for card in self.cards:
                if card.location == f'{column}{num}':
                    if card.matched:
                        row.append('   ')
        return row

    def create_grid(self):
        header = ' |  ' + '  |  '.join(self.columns) + '  |'
        for row in range(1, (self.size + 1)):
            print_row = f'{row}| '
            get_row = self.create_row(row)
            print_row += ' | '.join(get_row) + ' |'

    def check_match(self, loc1, loc2):
        cards = []
        for card in self.cards:
            if card.location == loc1 or card.location == loc2:
        if cards[0] == cards[1]:
            cards[0].matched = True
            cards[1].matched = True
            return True
            for card in cards:
                print(f'{card.location}: {card}')
            return False

    def check_win(self):
        for card in self.cards:
            if card.matched is False:
                return False
        return True

    def check_location(self, time):
        while True:
            guess = input(f"What's the location of your {time} card? ")
            if guess.upper() in self.locations:
                return guess.upper()
                    "That's not a valid location. Location should be column name then row name like A1.")

    def start_game(self):
        game_running = True
        print('Memory Game')
        while game_running:
            guess1 = self.check_location('first')
            guess2 = self.check_location('second')
            if self.check_match(guess1, guess2):
                if self.check_win:
                    print("Congrats! You've guessed them all")
                    game_running = False
                input("Those cards are not a match. Please enter to continue")
        print("GAME OVER!")

if __name__ == '__main__':
    game = Game()
Here's a look at my output:

Memory Game
 |  A  |  B  |  C  |  D  |
1|   |
2|  |
3|  |
4|  |
What's the location of your first card?
Steven Parker
Steven Parker
217,577 Points

It's not clear what course and lesson you are referring to.
You might want to take a look at this video about Posting a Question.

1 Answer

Steven Parker
Steven Parker
217,577 Points

Line 80 is apparently intended to invoke self.set_cards, but the parentheses after the method name are missing.