Looks good, although I would add one or two things.

> #assuming target number 15
>
Put that in a variable for the target number

> roll = (result, initial_mins, initial_max)
> if roll[0] > 15:
>    if roll[1] >= 2:

You could even put all the constants in variables

>

       print("Success")
>    elif roll[2] >= 2:
>        print("Critical failure!")
>    else:
>        print("Failure.")
> elif roll[0] <= 15:
>    if roll[1] >= 2:
>        print("Critical success!")
>    elif roll[2] >= 2:
>        print("Failure")
>    else:
>        print("Success")
>
Also need a couple comments, plus everything from the other emails.
_______________________________________________
Tutor maillist  -  Tutor@python.org
To unsubscribe or change subscription options:
http://mail.python.org/mailman/listinfo/tutor

Reply via email to