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 Object-Oriented Python Dice Roller RPG Roller

Tiana Manzano
PLUS
Tiana Manzano
Courses Plus Student 2,248 Points

Hand.roll(2)

I have no idea what I'm doing wrong. It just tells me to try again

dice.py
import random


class Die:
    def __init__(self, sides=2):
        if sides < 2:
            raise ValueError("Can't have fewer than two sides")
        self.sides = sides
        self.value = random.randint(1, sides)

    def __int__(self):
        return self.value

    def __add__(self, other):
        return int(self) + other

    def __radd__(self, other):
        return self + other

class D20(Die):
    def __init__(self):
        super().__init__(sides=20)
hands.py
from dice import D20

class Hand(list):
    @classmethod
    def roll(cls, size=2, die_class=D20):
        die = []
        for __ in range(sum):
            die.append(die_class())
    return cls(die)
    @property   
    def total(self):
        return sum(self)

1 Answer

Steven Parker
Steven Parker
231,275 Points

You're close, but I see two issues:

  • "roll" has a parameter named "size", but "range" is being passed something undefined named "sum"
  • the "return" statement needs to be indented more to be considered part of the method
Tiana Manzano
Tiana Manzano
Courses Plus Student 2,248 Points

I have adjusted accordingly, but now it's telling me that it didn't get D20's in my hand instance. I'm confused because I've specified it as the class being used in the class method. Do I need to put it in the init?