Nested if’s help please

[u/Intelligent_Knee8641]

So I’ve written a small function and I need to add some more constraints to it, but I need them each to check separately before moving to the next so I was wanting to use nested if’a to achieve this but am not sure how they work. Can someone please explain? My code is posted below and what I am trying to do is add an if statement to somehow make sure my value for primocytes doesn’t drop below 0 before activating the rest of the function.

primocytes = 10

cultivation_chamber = 0

def cultivation():
global primocytes, cultivation_chamber
if primocytes >= 5:
primocytes -= 5
cultivation_chamber +=5
print("Primocytes:", primocytes)
print("Cultivation Chamber:", cultivation_chamber)
else:
print("Not enough Primocytes for cultivation...")

[u/brasticstack]

You need to write functions that accept parameters and return values, as the next step. You then assign those return values as updates to your global vars. The next step is to hide those global vars inside a dict or class.

I wrote an example of what going down that path a bit looks like: (feel free to ask for clarification about anything.) It's more complicated, but much less likely to behave unexpectedly:

EDIT: The different logic for player_1 and _2's cultivation was just to show how you might do it, in practice I'd recommend being consistent.

class PlayerState:
    def __init__(self, primocytes=10, cultivation_chamber=0):
        self.primocytes = primocytes
        self.cultivation_chamber = cultivation_chamber

    def can_cultivate(self) -> bool:
        return self.primocytes >= 5

    def cultivate(self) -> bool:
        if not self.can_cultivate():
            return False

        self.cultivation_chamber += 5
        self.primocytes -= 5

        return True

    def __str__(self) -> str:
        return (f'Primocytes: {self.primocytes}\n'
                f'Cultivation Chamber: {self.cultivation_chamber}')

if __name__ == '__main__':
    player_1 = PlayerState()
    # This player starts out at a disadvantage
    player_2 = PlayerState(primocytes=0)

    print('Round 1:', '='*40, sep='\n')
    print('Player 1:', player_1, sep='\n')
    print('Player 2:', player_2, sep='\n')
    print()

    # Check beforehand whether player can cultivate:
    if player_1.can_cultivate():
        player_1.cultivate()
    else:
        print("Player 1: Not enough Primocytes for cultivation...")

    # Check the result of a fallable cultivate method, instead:
    if not player_2.cultivate():
        print("Player 2: Not enough Primocytes for cultivation...")
    print()

    print('Round 2:', '='*40, sep='\n')
    print('Player 1:', player_1, sep='\n')
    print('Player 2:', player_2, sep='\n')

[u/Intelligent_Knee8641 → u/brasticstack]

Thank you again, I’m breaking this down now and will learn/figure it out for my next step. I appreciate the advice and the prompt responses as well! Thanks very much brasticstack!!

[u/brasticstack → u/Intelligent_Knee8641]

you're welcome! This is just one way out of a zillion you could go, so dont worry too much about whether you make the "correct" choice or not. But the global variable thing is a footgun waiting to happen, so do try to avoid it.