pythonduplicatestext-based

Repeating sections of code multiple times in my text-based game


I'm making a text-based game, which is based largely in if-, elif- and else-statements. However, I also use while loops for different "areas" within the game. (Play_loop, Main_loop, Shop_loop, Fish_loop, etc.).

Lately I've implemented admin commands which I use in-game to change things on the go, and I want these to be available in every loop. I also have some general commands which I want to be available (help, leave, quit, logout, go to another area, check inventory, etc.).

The issue I'm facing is knowing that duplicated code should be avoided, but i'm wondering if this is necessary in this situation. I've already made many functions to make each command code pretty short, about 2-15 lines in general.

Should I add the code blocks into functions that do the same thing, and then just repeat the function, or should I just keep it as is? Or maybe I should do something else that I havent even thought about?

Example code:

elif command == '/user info':
    if user.admin:
        print(f'User-list: {users}')
        who = input('Name of user: ').strip()
        who_user = admin_load_user(who, users)
        if who_user:
            print(who_user.info())
            print(who_user.display_inv())
        else:
            print(DNEError)
    else:
        print(PError)
elif command == '/add coins':
    who = input('Who gets coins? ').strip()
    amount = int(input('How much? ').strip())
    admin_add_coins(who, amount, users)
    save_users(users)

Solution

  • Code that is repeated should typically be put into functions, so that you have better overview and control over what they are doing. There you can also easily give them default arguments and expand on the code without bloating the main function of your program.

    Here is a concise argument for using functions. It's written for C but applies to python as well.

    Then, as Jason Chia pointed out, you should consider thinking about building your game into classes, as they solve some of the problems you mentioned and generally are an important control instrument for bigger programs where you need to coordinate different states (e.g. changing something in a room based on the player's actions).

    This tutorial could help with that.

    Does that about answer your question?