Support on Payroll Code Error?

Question

Im getting a few errors from this code, can anyone explain what seems to be the issue? Thanks! This program is supposed to calculate the payroll for a declared (counter) amount of employees. Note: This is Python 3.3

Also: please feel free to make any improvements you like! I am looking to learn!

def computeGrossPay(hours, rate):
    if hours > 40:
        grossPay = (40 * rate) + ((hours - 40) * 1.5 * hours)
    else:
        grossPay =  hours * rate
    return grossPay

def computeTaxAmount(status, grossPay):
    if status == 1:
            if grossPay <= 42:
                taxAmount = 0
            elif grossPay > 42 and grossPay <= 214:
                taxAmount = (grossPay - 15) * .1
            elif grossPay > 214 and grossPay <= 739:
                taxAmount = 17.2 + ((grossPay - 214) * .15)
            elif grossPay > 739 and grossPay <= 1732:
                taxAmount = 95.95 + ((grossPay - 739) * .28)
            elif grossPay > 1732 and grossPay <= 2566:
                taxAmount = 344.2 + ((grossPay - 1732) * .28)
            elif grossPay > 3566 and grossPay <= 7703:
                taxAmount = 857.72 + ((grossPay - 3566) * .33)
            elif grossPay > 7703 and grossPay <= 7735:
                taxAmount = 2222.93 + ((grossPay -7703) * .35)
            elif grossPay > 7735:
                taxAmount = 2234.13 + ((grossPay - 7735) * .396)
    elif status == 2:
            if grossPay <= 160:
                taxAmount = 0
            elif grossPay > 160 and grossPay <= 503:
                taxAmount = (grossPay - 160) * .1
            elif grossPay > 503 and grossPay <= 1554:
                taxAmount = 34.3 + ((grossPay - 503) * .15)
            elif grossPay > 1554 and grossPay <= 2975:
                taxAmount = 191.95 + ((grossPay - 1554) * .28)
            elif grossPay > 2975 and grossPay <= 4449:
                taxAmount = 547.2 + ((grossPay - 2975) * .28)
            elif grossPay > 4449 and grossPay <= 7820:
                taxAmount = 959.92 + ((grossPay - 4449) * .33)
            elif grossPay > 7820 and grossPay <= 8813:
                taxAmount = 2072.35 + ((grossPay -7820) * .35)
            elif grossPay > 8813:
                taxAmount = 2419.9 + ((grossPay - 8813) * .396)
    return taxAmount

def computeNetPay(grossPay, taxAmount):
    netPay = grossPay - taxAmount
    return netPay

def main():
    totalGrossPay = 0
    print("Welcome to the Payroll Program")
    counter = eval(input("How many employees will be entered? \n"))
    for i in range(counter):
        name = input("What are the first and last names of the employee? \n")
        id = input("What are the last four numbers of his/her Social Security Number? \n")
        hours = eval(input("How many hours did they work? \n"))
        rate = eval(input("What was the rate? \n"))
        while (True):
            status = eval(input("Are they single(1) or married(2)? \n"))
            if status != 1 or 2:
                break
        grossPay = computeGrossPay(hours, rate)
        taxAmount = computeTaxAmount(status, grossPay)
        netPay = computeNetPay(grossPay, taxAmount)
        totalGrossPay = totalGrossPay + grossPay
        averageGrossPay = totalGrossPay / counter

        print("The ID is ", id)
        print("The name is ", name, ".")
        print("This employee worked ", hours, " hours.")
        print("The hourly rate was ", rate)
        print("The gross pay was $", grossPay)
        print("The tax amount withheld for the week is $", taxAmount)
        print("The net pay is $", netPay)
    print("The total gross pay was $", totalGrossPay)
    print("The average gross pay was $", averageGrossPay)

main()

Answer 1

You "set" the status like this:

status = input("Are they single(1) or married(2)? \\n")

The type() of status is going to be a string. When you attempt to compare it in your computeTaxAmount() method, you either need to turn it into an int or compare it to a string, otherwise it will never be True and taxAmount will never be set.

Ex:

if status == 1:

becomes:

if status == '1':

You have other issues in your code as well. In Python, '&' is the bitwise AND operator, not the logical AND. elif grossPay > 42 & grossPay <= 214 for example needs to become elif grossPay > 42 and grossPay <= 214 .