为什么我得到这个错误“IndexError：字符串索引超出范围”

Question

我们被要求编写验证GTIN-8代码的程序。 的valdation如下：

1. 乘第一7位交替由3然后1

2. 添加起来

3. 减去该数量从10

的相等或更高的多

4. 生成的数字是第八位数字

这里是我的代码：

def validgtin(): 

    gtin = input("Enter the 8 digits of GTIN-8 product code: ") 
    valid = False 
    while valid == False: 
     if gtin.isdigit():    
      gtin = str(gtin) 
      if len(gtin) == 8:   
       valid = True 
      else: 
       print("That is not correct, enter 8 digits, try again: ")  
       gtin = input("Enter the 8 digits of GTIN-8 product code: ") 
     else: 
      print("That is not correct, type in numbers, try again: ") 
      gtin = input("Enter the 8 digits of GTIN-8 product code: ")  


    sumdigit = 3*(int(gtin[0])) + 1*(int(gtin[1])) + 3*(int(gtin[2])) + 1*(int(gtin[3])) + 3*(int(gtin[4])) + 1*(int(gtin[5])) + 3*(int(gtin[6])) #sum of the digits 

    gtin = str(gtin) 

    valid1 = False 
    while not valid1: 
     if sumdigit%10 == 0:  
      eightdigit = 0 
     else: 
      eightdigit = (((sumdigit + 10)//10)*10) - sumdigit 

     if eightdigit == (gtin[7]): 
      valid1 = True 
      print("Your GTIN-8 product code is valid.") 

     else: 
      print("Your GTIN-8 product code is not valid.") 
      gtin = input("Enter the 8 digits of GTIN-8 product code: ") 


    return 

validgtin() 

当我运行此代码，然后键入一个无效的GTIN-8代码它说，该代码是无效的，并提示我输入一个新的GTIN-8编码

但

后，我在一个新的和有效的GTIN-8编码键入它仍然说，这是无效的

和

之后，出现这种情况：

Traceback (most recent call last): 



File "C:\Users\Yash Dwivedi\Documents\Year 10\GCSE Computing\Assignment\Task 1 v2.py", line 29, in validgtin 
    if eightdigit == (gtin[7]): 
IndexError: string index out of range 

我不明白为什么我 会感谢任何帮助。

Answer 1

这是我的快速实施。可悲的是，我没有任何测试数据来检查它是正确的！

def _round_up_ten(number): 
    if number % 10 == 0: 
     return number 
    return 10 * (1 + (number/10)) 


def validate_gtin(gtin): 
    if not gtin.isdigit() or len(gtin) != 8: 
     raise ValueError("GTIN must be an 8-digit number") 

    digits = [int(digit) for digit in gtin[:-1]] 
    check_digit = int(gtin[-1]) 

    multiplied_digits = (
     digits[0] * 3 
     + digits[1] 
     + digits[2] * 3 
     + digits[3] 
     + digits[4] * 3 
     + digits[5] 
     + digits[6] * 3 
    ) 

    expected_check_digit = _round_up_ten(multiplied_digits) - multiplied_digits 

    if check_digit!= expected_check_digit: 
     raise ValueError("Incorrect check digit ({}) (expected {})".format(check_digit, expected_check_digit)) 

Answer 2

我会建议做一个“is_valid_gtin”函数，它只检查GTIN是否有效，没有I/O。然后一个简单的 “主（）” 来检查代码：

def is_valid_gtin(gtin): 
    if len(gtin) != 8 or not gtin.isdigit(): 
     return False 
    sum = 0 
    for i in list(gtin)[0:6:2]: 
     sum += 3*int(i) 
    for i in list(gtin)[1:6:2]: 
     sum += int(i) 
    checksum = (10 - sum % 10) % 10 
    return checksum == int(gtin[7]) 


def main(): 
    while (True): 
     gtin = input("Enter the 8 digits of GTIN-8 product code: ") 
     if is_valid_gtin(gtin): 
      print("Your GTIN-8 product code is valid.") 
      break 
     else: 
      print("That is not correct, try again.") 

if __name__ == '__main__': 
    main() 

Answer 3

该缺陷是在该行

if eightdigit == (gtin[7]): 

eightdigit是int但gtin[7]是一个字符串。因此，这种比较是始终为 false - 因此您处于无限循环（只要您输入至少包含8个字符的字符串）。你越来越沮丧，然后只是按下回车键 - 它将您的代码传递给空字符串，缺少第八个字符会触发索引超出范围错误。

因此，你将需要：

if eightdigit == int(gtin[7]): 

来解决特定的漏洞，尽管这仍然会留下一个逻辑上的错误 - 因为在您的代码底部的循环不会验证输入和您正在尝试使用之前的输入计算checkdigit来检查新的候选gtins。您应该遵循@JacquesSupik的出色想法并重构代码，以便将验证逻辑从I/O逻辑中分离出来。