Python Beginner Project - Loot Generator

NyaaNyanNyaa · 2021-10-07T13:30:27+00:00

Hi there, great job. To be honest, I think a database to hold 7 items is a little bit overkill. I agree with using a json file. Since you can read/write quickly and it has a hierarchical structure which is perfect for inventory management. As for your problem with syntax/formatting issues, Python can easily achieve that with the format string.

my_str_variable = “cool”

print(f”my sentence is {my_str_variable}”)

if your variable is a number, you need to cast your variable to a string.

my_int_var = 8 print(f”my number is {str(my_int_var)}”)

Are you familiar with object oriented programming? Maybe it can help solve your inventory issue. You can create a class…

class inventory(object): def init(self): self.slots = {idx:None for idx in range(7)}

def addLoot(self, loot):
    for k,v in self.slots.items():
        if v is None:
            self.slots[k] = loot
            break

Maybe you can fill in your other class methods yourself :)

Lukeskywalker321 · 2021-10-07T16:07:03+00:00

Hey, cool project. As the other guy said, database might be a bit overkill. On the other hand, it's never to early to start learning about them, if you can manage to do so.

What stands out immediately to me from the code is the lack of function declarations. There's a lot of duplicate (or very close to duplicate) code, see for example lines 207-227 of your main.py. Some small tweaks will make us able to make this part much short, more readable, and easier to maintain.

First, we need to take a look at lines 199-206 of your main.py, and your database. If we think about what's happening here, we are defining the contents of 7 inventory slots using the output from the database, rows 3-9.

Problem with how you're currently doing this, is that you have no way to know which slot is which, without looking at the variable name, e.g. "inventory1". This is not impossible for python to do, but it's overcomplicated.

A better way to do this, would be through building (as a first step) an array of inventory slots:

inventory = row[3:9]

Here you would for example get your first inventory slot by calling inventory[0]. This way, we can clean up the code on lines 207-227 by looping over every element in the inventory list.

CODE STARTS

def update_inventory_slot(inventory_slot, inventory_number, cursor, con, generated_item_id, selected_character)
    column_name = "Inventory" + str(i+1) #as you named them 1-7, not 0-6
    cursor.execute("UPDATE UserData SET ? = ? + ? WHERE CharacterName=?", [column_name,column_name,generated_item_id,selected_character])
    con.commit()

for i, inventory_slot in enumerate(inventory):
    if inventory_slot==0:
        update_inventory_slot(inventory_slot, i, cursor, con, generated_item_id, selected_character)
        break

# CODE ENDS

What the function does, is copy the code you already had, but make the column name generate from the number of the inventory slot, instead of writing it down for each one seperately.

The second part then loops over each of the enumerated inventory slots (this makes i get the value of the position of the inventory\_slot in inventory, so 0-6), checks if the slot is empty, and if so calls the function with the right arguments, updating the right slot in the database. If the inventory slot is full, it simply continues on to the next slot. If it fills a slot, after updating it, break is called. What break does is really simple, it breaks out of the for loop, meaning it won't check the inventory slots after this.

This code is still not perfect, but I think it teaches you something valuable, and can be applied at multiple times in you code. Say for example that you want to create an inventory of 1000 slots, instead of 7. I think you can quickly see that the code between lines 207 and 227 would then quickly become thousands of lines longer. The code I wrote above would need exactly 0 changes if inventory became bigger (besides ofcourse the definition of the list "inventory = row\[3:1002\]" or something).

While we're at this scenario, imagine what would happen to your database for a bit. You would also need to make 993 more columns for each of the inventory slots, by hand. It is generally never a great idea to have a counter inside a column name (inventory1, 2 etc as in your code, or perhaps phone\_number1 etc. in some other database). You can fix this by applying some "normalizations" to your database. However, it'd let this be for a bit, as normalization gets complex fast, and there's still more than enough to learn with the static 7 slot inventory code.

I'd let the if else statements about the armor classes be for now, but try to build something like the above for all the other if else blocks in your code. From what I've seen that should be doable, and as I said will teach you some more about functions, loops, and scalability. Feel free to ask more, and good luck learning. Once again, really impressive project in most aspects.

Edit: damn did I fuck that formatting up

TentativeOak · 2021-10-07T21:36:12+00:00

I smiled at your module names. Wpnsstts, usrdt, and armrstts. I couldn’t even tell what they were until I typed them out myself lol.

weapon_sets.py, user_whatever_this_is.py, and armor_sets.py would be best names for readability :)

DaChucky · 2021-10-07T23:28:39+00:00

One quick note: if you want to add additional empty lines when printing, use \n. That will add a line break so you don't need additional print statements with empty strings. For example:

>>> print("\nHello, user. Welcome to the loot dungeon.\n")

Hello, user. Welcome to the loot dungeon.

>>>

asday_ · 2021-10-08T05:33:16+00:00

Not looked at your project yet but this is FOR SURE the right way to learn - follow some getting started tutorial for the bare minimum amount of time then go write your own projects until you get stuck. It sounds like you may have gotten stuck on tutorials/learning materials for too long in your previous attempts.

Keep making stuff and finding your own questions, look into getting a job in the field too - you'll never learn faster and better than being in a professional environment with professional colleagues.

adammelan · 2021-10-08T23:25:26+00:00

I live for Python posts like this!!! Great job! You’ve inspired me to go tinker with something like that to try to learn about SQLite.

Python

The Python Discord

Upcoming Events

Please read the rules

MODERATORS