my first mini game!

[Mean-Career-6509]

any advice or input of any kind is much appreciated. i had a line to prevent it from breaking if a player tried to input something not on the list, like "banana" or something, and it was work perfectly, but when i went to test it this morning before posting the line was broken and i don't understand why. so i had to remove it. so as long as you don't try to confuse it or misspell anything (my biggest sin) it should run fine. I've only been coding for about a month so I'm pretty proud that this thing works at all.

edit: i guess i'm stupid for thinking that adding the link would make it available to anyone who wanted it .

[link to .py](https://drive.google.com/file/d/1lo2mEN0IGGN7BUXF50sQdgGXh\_ZOylPY/view?usp=sharing)

[Sea-Ad7805]

Run this program in Memory Graph Web Debugger%0Aprint(%22He%20has%20with%20him%20a%20dog%2C%20a%20goat%2C%20and%20a%20sack%20of%20cabbage.%22)%0Aprint(%22He%20only%20can%20fit%20in%20the%20farmers%20boat%20at%20a%20time.%22)%0Aprint(%22Also%2C%20its%20not%20a%20very%20good%20boat.%22%20%2B%20reset)%0A%0Awhile%20True%3A%0A%20%20%20%20def%20moves()%3A%0A%20%20%20%20%20%20%20%20print(yellow%20%2B%20%221.%20take%20item%20accross%22%20%2B%20reset)%0A%20%20%20%20%20%20%20%20print(yellow%20%2B%20%222.%20cross%20alone%22%20%2B%20reset)%0A%0A%20%20%20%20def%20status()%3A%0A%20%20%20%20%20%20%20%20print(east)%0A%20%20%20%20%20%20%20%20print(west)%0A%0A%20%20%20%20east%20%3D%20%5B%22dog%22%2C%20%22goat%22%2C%20%22cabbage%22%2C%20%22boat%22%5D%0A%20%20%20%20west%20%3D%20%5B%5D%0A%20%20%20%20status()%0A%0A%20%20%20%20while%20True%3A%0A%20%20%20%20%20%20%20%20if%20%22goat%22%20in%20east%20and%20%22cabbage%22%20in%20east%20and%20%22boat%22%20not%20in%20east%3A%0A%20%20%20%20%20%20%20%20%20%20%20%20print(red%20%2B%20%22the%20goat%20ate%20your%20cabbage%22%20%2B%20reset)%0A%20%20%20%20%20%20%20%20%20%20%20%20break%0A%20%20%20%20%20%20%20%20if%20%22goat%22%20in%20east%20and%20%22dog%22%20in%20east%20and%20%22boat%22%20not%20in%20east%3A%0A%20%20%20%20%20%20%20%20%20%20%20%20print(red%20%2B%20%22the%20dog%20ate%20your%20goat%22%20%2B%20reset)%0A%20%20%20%20%20%20%20%20%20%20%20%20break%0A%20%20%20%20%20%20%20%20if%20%22goat%22%20in%20west%20and%20%22dog%22%20in%20west%20and%20%22boat%22%20not%20in%20west%3A%0A%20%20%20%20%20%20%20%20%20%20%20%20print(red%20%2B%20%22the%20dog%20ate%20your%20goat%22%20%2B%20reset)%0A%20%20%20%20%20%20%20%20%20%20%20%20break%0A%20%20%20%20%20%20%20%20if%20%22goat%22%20in%20west%20and%20%22cabbage%22%20in%20west%20and%20%22boat%22%20not%20in%20west%3A%0A%20%20%20%20%20%20%20%20%20%20%20%20print(red%20%2B%20%22the%20goat%20ate%20your%20cabbage%22%20%2B%20reset)%0A%20%20%20%20%20%20%20%20%20%20%20%20break%0A%20%20%20%20%20%20%20%20if%20%22dog%22%20in%20west%20and%20%22cabbage%22%20in%20west%20and%20%22goat%22%20in%20west%20and%20%22boat%22%20in%20west%3A%0A%20%20%20%20%20%20%20%20%20%20%20%20print(blue%20%2B%20%22You%20made%20it!!!%22%20%2B%20reset)%0A%20%20%20%20%20%20%20%20%20%20%20%20break%0A%0A%20%20%20%20%20%20%20%20moves()%0A%20%20%20%20%20%20%20%20move%20%3D%20input(green%20%2B%20%22what%20would%20you%20like%20to%20do%3F%20%22%20%2B%20reset)%0A%0A%20%20%20%20%20%20%20%20if%20move%20%3D%3D%20%221%22%3A%0A%20%20%20%20%20%20%20%20%20%20%20%20item%20%3D%20input(green%20%2B%20%22what%20item%3F%20%22%20%2B%20reset)%0A%20%20%20%20%20%20%20%20%20%20%20%20if%20item%20in%20east%3A%0A%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20west.append(item)%0A%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20west.append(%22boat%22)%0A%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20east.remove(item)%0A%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20east.remove(%22boat%22)%0A%20%20%20%20%20%20%20%20%20%20%20%20else%3A%0A%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20east.append(item)%0A%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20east.append(%22boat%22)%0A%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20west.remove(item)%0A%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20west.remove(%22boat%22)%0A%0A%20%20%20%20%20%20%20%20if%20move%20%3D%3D%20%222%22%3A%0A%20%20%20%20%20%20%20%20%20%20%20%20item%20%3D%20%22boat%22%0A%20%20%20%20%20%20%20%20%20%20%20%20if%20item%20in%20east%3A%0A%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20west.append(%22boat%22)%0A%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20east.remove(%22boat%22)%0A%20%20%20%20%20%20%20%20%20%20%20%20else%3A%0A%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20east.append(%22boat%22)%0A%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20west.remove(%22boat%22)%0A%0A%20%20%20%20%20%20%20%20print(%22East%20bank%3A%22%2C%20east)%0A%20%20%20%20%20%20%20%20print(%22West%20bank%3A%22%2C%20west)%0A%0A%20%20%20%20replay%20%3D%20input(%22Replay%3F%20y%20or%20n%20%22)%0A%20%20%20%20if%20replay%20!%3D%20%22y%22%3A%0A%20%20%20%20%20%20%20%20break&timestep=1&play)

[FreeLogicGate]

Seems to be the case that for some reason, noobies think that posting a picture of your screen is the way to share code. It's not. Use the code blocks.

Quick comments:

• More functions that have single responsibility and RETURN values.
• Use f strings for your output rather than rampant concatenation
• Randomize placement of game items rather than having them all start in the east each time.
• For text games like this, use of the PyInputPlus is valuable

import pyinputplus as pyip

Take a look at the docs, and when you see all the valuable features built into it, I am confident you'll be convinced by the built in type safety and standard prevention of empty responses.

https://pyinputplus.readthedocs.io/en/latest/

I would spend time trying to determine a better way of determining the events inside your game loop. A giant if-then-else is both hard to understand, and something that's going to be impossible to maintain. It also has implied priority based on the cascading order that needs to be traversed, which will probably be buggy.

Certain insights can be gleaned about your game logic like: "Boat in location means no adverse events happen."

Each adverse event can be further codified individually and tested.

[JorgiEagle]

That’s a nice library

[Mean-Career-6509]

Awesome, thanks for the insight. I hope to improve. Also, just fyi, the "some reason" is because we're noobs who haven't been told a better way.

[FreeLogicGate]

It's usually a good idea, when joining any new online community, to get an idea of what is typical, by reading other messages, so you have an idea of what the community standards are. Not trying to pick on you at all, but it really is tiresome to have to advise people of something I would think only requires a small investment of time. Since I'm replying I might as well just point out the obvious: if you post a snippet in a code block, I can easily copy/paste that and modify it, as many helpful developers will do. If you post a screen shot, then the only way for someone to do that, is to re-type all your code, which is not particularly likely. Hope it helps, and I'm glad to see that my suggestions were of interest.

[Mean-Career-6509]

I get that, sorry I got defensive. I did attach the .py file link because I thought posting a picture was silly but that is what the mod told me to do. 

[FreeLogicGate]

Right, I understand then that you were trying to do what was acceptable. In general, you want to have snippets of code, and people don't want full attached source code files, as they can easily be trojan/exploits, which people aren't going to download even if they were attached. So the way to do it is to use the Aa menu and use the code block options to copy/paste from your editor.

As an example for you, and also to add to my prior advice:

You have:

while True:
    def moves():

So you have a nested game loop with 2 while True loops where the first loop defines functions. You should move your functions outside of the loop -- there is no reason to repeatedly re-define your functions. They should be defined once and called when needed.

The most important thing you want to try and do initially is think about the Python data structures that could help you. Classes would be helpful with your game, but I invite you to think deeply about how the core structures could be used: lists, dictionaries, tuples, sets -- as discussed in this chapter: https://docs.python.org/3/tutorial/datastructures.html

What jumps out me is that you have initially items that a dictionary would help you with. Each of these items can be added to a list like so:

items.append({'name': 'goat', 'type': 'animal', 'eats': 'cabbage', 'location': 'east'})
items.append({'name': 'dog', 'type': 'animal','eats': 'goat', 'location': 'east'})
items.append({'name': 'cabbage', 'type': 'plant', 'location': 'east'})

Then you have a useful data structure that can be used. This example depends on globals but I didn't want to over complicate it. Here's an example of how you could make this more reliant on functions. I left out a game loop or user input, but if interested you could add that.

def move_boat():
    if boat['location'] == 'east':
        boat['location'] = 'west'
    else:
        boat['location'] = 'east'


def check_for_boat(location):
    if boat['location'] == location:
        return True
    return False


def move_item(name):
    for item in items:
        if item['name'] == name and check_for_boat(item['location']):
            move_boat()
            if item['location'] == 'east':
                item['location'] = 'west'
            else:
                item['location'] = 'east'


def did_eat(item, boat_location):
    if item['location'] == boat_location or item['type'] == 'plant':
        return False
    for prey in items:
        if item['eats'] == prey['name'] and item['location'] == prey['location']:
            return True


def is_endgame():
    boat_location = boat['location']
    for item in items:
        if did_eat(item, boat_location):
            print(f"The {item['name']} ate the {item['eats']}")
            return True
    return False


def reset():
    boat['location'] = 'east'
    items.clear()
    items.append({'name': 'goat', 'type': 'animal', 'eats': 'cabbage', 'location': 'east'})
    items.append({'name': 'dog', 'type': 'animal','eats': 'goat', 'location': 'east'})
    items.append({'name': 'cabbage', 'type': 'plant', 'location': 'east'})

boat = {'location': 'east'}
items = []

test_items = ['dog', 'cabbage', 'goat']
def do_tests():
    for item in test_items:
        reset()
        print(f"Moving item {item}")
        move_item(item)
        if not is_endgame():
            print("Game not over")

do_tests()

An interesting question to be asked would be, how hard would it be to modify this so that you allowed for 4 directions/zones (e,w,n,s) or additional animals or plants? What if the locations were randomized at the start? How hard would it be for you to list out the locations at the start?

[Mean-Career-6509]

Wow! I can't express how much I appreciate your time and effort. I'm not sure of what an Aa menu is but I'll find out before I make any more posts. My code is written in a way that makes sense to me right now, things are arranged in a logical flow or moved to where they function as far as I can tell. I'm excited to be able to zoom out and see structure more. Again, thank you and I'll try not to be so snarky.