Help with lists and class objects

[stullis]

On line 30, the indentation is incorrect. __str__() should not be nested under __init__(). Also, you still need the __repr__() method. These two often seem interchangeable, but they are called by different functions; for instance, print() calls obj.__repr__() whereas str() calls obj.__str__().

[snippsat]

print() calls obj.__repr__() whereas str() calls obj.__str__().

A correction here,print() call __str__() the same do str().
So in my code before in Thread.

>>> # Call __str__
>>> print(sword)
The item is: sword
>>> str(sword)
'The item is: sword'

>>> # Call __repr__
>>> sword
Items('sword')
>>> repr(sword)
"Items('sword')"

__str__ method should primarily be human readable.

__repr__ goal is to be unambiguous,and should be as explicit as possible about what this object is.
The resulting string is intended more as help for developers and debugging.

[Marbelous]

In addition to too much indentation on your __str__() definition you are using format strings wrong. What you have now will always return the string literal "self.item". On line 31 you are returning self.item which is already a string so just return it, remove the f and the two double quotes:

def __str__(self):
     return self.item

Also on line 351 you are not calling your Starter.list_items method, you are printing it. Should be:

def inventory():
    Starter.list_items()

While I applaud your attempt to make this game Object Oriented you must have realized that simply creating several classes with the exact same code probably isn't right. The whole point of OOP would be to make a base class called "Crate" with all the common functionality and then inheriting from it in more specific classes which should only contain the things that are different from the base class. Inheritance is the first "Pillar of OOP" and it makes coding games like this a lot easier so you need to understand how to use it.

[AlluminumFoil]

Thank you guys! you've helped me a lot! sorry if I was annoying, I am still attempting to learn Python because I'd like to do it as a job eventually :)

[Marbelous]

No need to apologize. Learning and helping others learn is what a lot of us are here for. That and the spam, spam, spam. Lovely Spam!

Take a look at class inheritance next and code a proper base class for your Crates which can become the parent class for all the crate types.

I suggest you work in a sandbox environment with just the classes and some test code for now rather than trying to incorporate everything directly into your game. OOP is one of those things you really to need grok before you try to use it.

[AlluminumFoil]

Thanks a lot! I'll definitely look into that :)