pythonpython-3.xisbncheck-digit

Getting type error in python when using for loop


I'm sure this will be a very basic question for you, but I'm making a ISBN check digit calculator for my computing class. My current code is this:

isbn = []
results = []

print("Please input your ISBN 1 number at a time")

def isbn_input():
    isbn.append(int(input("ISBN character 1: ")))
    isbn.append(int(input("ISBN character 2: ")))
    isbn.append(int(input("ISBN character 3: ")))
    isbn.append(int(input("ISBN character 4: ")))
    isbn.append(int(input("ISBN character 5: ")))
    isbn.append(int(input("ISBN character 6: ")))
    isbn.append(int(input("ISBN character 7: ")))
    isbn.append(int(input("ISBN character 8: ")))
    isbn.append(int(input("ISBN character 9: ")))
    isbn.append(int(input("ISBN character 10: ")))

isbn_input()

results.append(isbn[0] * 11)
results.append(isbn[1] * 10)
results.append(isbn[2] * 9)
results.append(isbn[3] * 8)
results.append(isbn[4] * 7)
results.append(isbn[5] * 6)
results.append(isbn[6] * 5)
results.append(isbn[7] * 4)
results.append(isbn[8] * 3)
results.append(isbn[9] * 2)

results = sum(results)
results = results % 11
results = 11 - results
print("Your ISBN is'",
      isbn[0],isbn[1],isbn[2],isbn[3],
      isbn[4],isbn[5],isbn[6],isbn[7],
      isbn[8],isbn[9],results,"'")

I know this is an insanely inefficient way of doing it, and you're probably crying even looking at it.

The first thing I did to try making it more efficient is by using a for loop. So, I changed:

def isbn_input():
    isbn.append(int(input("ISBN character 1: ")))
    isbn.append(int(input("ISBN character 2: ")))
    isbn.append(int(input("ISBN character 3: ")))
    isbn.append(int(input("ISBN character 4: ")))
    isbn.append(int(input("ISBN character 5: ")))
    isbn.append(int(input("ISBN character 6: ")))
    isbn.append(int(input("ISBN character 7: ")))
    isbn.append(int(input("ISBN character 8: ")))
    isbn.append(int(input("ISBN character 9: ")))
    isbn.append(int(input("ISBN character 10: ")))

to a for list, but I haven't figured out how to do this yet. This is the bit I need help on (Also, I know the function is pointless. Our teacher demanded we have one in, but if anyone can find a better way of doing it then that would be amazing too)

Thanks a lot for the help. Again, if you spot anything which could be made better, then please do. (Also, this is python 3.)


Solution

  • To simplify the input:

    isbn = [int(input("ISBN character {0}: ".format(i))) 
            for i in range(1, 11)] # or 'format(i+1)' and 'range(10)'
    

    This uses a list comprehension to simultaneously loop and built the list of integers.

    Alternatively, take the whole input at once and convert it into individual integers:

    isbn = list(map(int, input("Enter ISBN: ")))
    

    Here map applies int() to convert each of the characters in the input string in turn.