Welcome to the Treehouse Community

Want to collaborate on code errors? Have bugs you need feedback on? Looking for an extra set of eyes on your latest project? Get support with fellow developers, designers, and programmers of all backgrounds and skill levels here with the Treehouse Community! While you're at it, check out some resources Treehouse students have shared here.

Looking to learn something new?

Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and join thousands of Treehouse students and alumni in the community today.

Start your free trial

Python

Posted by Tristan Rodgers
Tristan Rodgers
8,137 Points

Lost on the second task for CapitalismHand

help please,

hands.py 
from dice import D6


class Hand(list):
    def __init__(self, size=0, die_class=None, *args, **kwargs):
        if not die_class:
            raise ValueError("You must provide a die class")
        super().__init__()

        for _ in range(size):
            self.append(die_class())
        self.sort()

    def _by_value(self, value):
        dice = []
        for die in self:
            if die == value:
                dice.append(die)
        return dice


class CapitalismHand(Hand):
    def __init__(self, *args, **kwargs):
        super().__init__(size=2, die_class=D6, *args, **kwargs) 

    @property
    def doubles(self, hand):
        current_die_value = sum(hand)
        for next_die in hand:
            if next_die == current_die_value:
                return True
            else:
                return False

    @property
    def ones(self):
        return self._by_value(1)

    @property
    def twos(self):
        return self._by_value(2)

    @property
    def threes(self):
        return self._by_value(3)

    @property
    def fours(self):
        return self._by_value(4)

    @property
    def fives(self):
        return self._by_value(5)

    @property
    def sixes(self):
        return self._by_value(6)

    @property
    def _sets(self):
        return {
            1: len(self.ones),
            2: len(self.twos),
            3: len(self.threes),
            4: len(self.fours),
            5: len(self.fives),
            6: len(self.sixes)
        }

2 Answers

Steven Parker
Steven Parker
231,269 Points
Steven Parker
Steven Parker
231,269 Points

You have a good idea, but a few implementation issues:

  • the method won't need an argument (the instance "self" is a hand)
  • in doubles, the individual dice will equal each other (not their sum)
  • there are only two dice in the hand, so you only need one comparison (no loop)
Jeff Muday
MOD
Jeff Muday
Treehouse Moderator 28,720 Points
Jeff Muday
Jeff Muday
Treehouse Moderator 28,720 Points

Good work on getting task 1 done! For task 2, the challenge is asking you to create a property that checks if both dice in the hand have equal value.

I am going to leave the coding up to you since this can be done in multiple correct ways. Below is a template for your property.

Note:

Since "self" is a LIST... you can check self[0]==self[1] (if True, doubles property should return True else return a False)

    @property
    def doubles(self):
        # your code goes here...
        # return True or False (depending on the outcome of self[0] compared to self[1]