why is it crushing ?

[Cheetos]

I've modified j.crater's code (the 'print' functions) so no conversion is necessary (still in Python 3)

Hide/Show

import time

global coins
global common
global garbage
global uncommon
common = 0
uncommon = 0
garbage = 0
coins = 100


def start():
    print("Hello my friend!")
    name = input("what's your name :")
    print("Welcome, {}!".format(name))
    print("This game is just like Tatsumaki Discord Bot fishing system, i hope that you enjoy it.")
    print("the gole of the game is to collect as much fishs as possible then selling them for coins.")
    print("Common fishs worth 12 coins/uncommon fishs worth 20 coins/garbage worth 8 coins.")
    print("you have to pay 10 coins as a fee for fishing.")

    choice = input("Do you want to play Y/N ?:")
    if choice == "Y":
        print("To fish you have to use the command fish")
        begin()
    if choice == "N":
        print("Goodbye")
        time.sleep(2)


def begin():
    global common
    global garbage
    global uncommon
    global coins
    print("Let's start!")
    if coins > 500:
        print("You have won the game!")
        play = input("Do you want to play again?:")
        if play == "Y":
            begin()
        if play == "N":
            print("Congrazts again")
            time.sleep(2)
    fish = input("Do you want to fish ?:")
    if fish == "fish":
        print("You caught a common fish!")
        common = common + 1
        coins = coins - 10
        print("You currently have, {} common fishs, {} uncommon fishs, {} garbage fishs".format(common, uncommon, garbage))
        print("your coins are now: {}".format(coins))
        begin()
    elif fish == "fish":
        print("You caught an uncommon fish!")
        uncommon = uncommon + 1
        coins = coins - 10
        print(
            "You currently have, {} common fishs, {} uncommon fishs, {} garbage fishs".format(common, uncommon, garbage))
        print("your coins are now: {}".format(coins))
        begin()
    elif fish == "fish":
        print("You caught a garbage fish!")
        garbage = garbage + 1
        coins = coins - 10
        print(
            "You currently have, {} common fishs, {} uncommon fishs, {} garbage fishs".format(common, uncommon, garbage))
        print("your coins are now: {}".format(coins))
        begin()
    if fish == "N":
        sell = input("Do you want to sell your fishs ? Y/N:")
        if sell == "Y":
            print("You currently have, {} common fishs, {} uncommon fishs, {} garbage fishs".format(common, uncommon, garbage))
            print("You have sold your fishs.")
            coins = common * 12 + uncommon * 20 + garbage * 8
            common = 0
            uncommon = 0
            garbage = 0
            print("your coins are now: {}".format(coins))


start()

Your other problem, is your initial "if" statement if fish == "fish": will always evaluate as true, so none of the "elif's" will be looked at. You could create a list of options, say choices = ['common', 'uncommon', 'garbage'] then randomly select one of those (Hint: Random), you could then have:

if choices == 'common':
    do something
elif choices == 'uncommon':
    do something
elif choices == 'garbage':
    do something

Another problem, is I can fish myself into debt as you have made no action to be taken if I reach 0 coins.

There are other smaller items that could be addressed, but I think you'll be fine if you handle the big problems first.

As to the error you are receiving, it's because in your code you've used both tabs and spaces for indentation. In Python, you should always use 4 spaces for indentation

And yes, by all means switch to Python 3 (3.6.4 if possible)

thanks very much it's working great now gonna do what you said about choice atm