实现使用Python

Question

我目前有麻烦“自动无聊的东西”完成这个挑战在Collat​​z功能：

我的代码是：

def collatz(number): 
    global seqNum 
    if (seqNum % 2 == 0): 
     return seqNum // 2 
    elif (seqNum % 2 == 1): 
     return 3 * seqNum + 1 


print('What number would you like to use?') 
seqNum = input() 
number = int(seqNum) 
i = number 

while i > 1: 
    collatz(seqNum) 
    print(number) 

而且我得到这个错误：

"Traceback (most recent call last): 
    File "C:/Users/Administrative/AppData/Local/Programs/Python/Python36-32/collatzSeq.py", line 15, in <module> 
    collatz(seqNum) 
    File "C:/Users/Administrative/AppData/Local/Programs/Python/Python36-32/collatzSeq.py", line 3, in collatz 
    if (seqNum % 2 == 0): 
TypeError: not all arguments converted during string formatting" 

我知道我在做SOMETHIN G写错了我的代码，但我不明白它到底是什么。任何和所有的帮助，非常感谢！

我也是使用python 3

Answer 1

1. 你正在做算术的字符串，而不是一个整数。

2. 有没有必要有一个global变量。将一个参数传递给一个函数，并让它相应地返回一个值。

---

def collatz(number): 
    if (number % 2 == 0): 
     return number // 2 

    elif (number % 2 == 1): 
     return 3 * number + 1 

print('What number would you like to use?') 

i = int(input()) 
while i > 1:  
    i = collatz(i) 
    print(i) 

Answer 2

seqNum是一个字符串。

>>> "3" % 2 
Traceback (most recent call last): 
    File "<stdin>", line 1, in <module> 
TypeError: not all arguments converted during string formatting 
>>> 

Answer 3

看来，你应该通过i给你的函数，而不是seqNum。

并在您的函数中删除所有对seqNum的引用，并使用number来代替。

Answer 4

这里有几个问题，但在导致你的例外是，您在功能，这是input()返回使用seqNum。并且input()返回一个字符串（至少在Python 3上）。 strings the %是“格式化操作符”，它也解释了异常消息，它讨论了“字符串格式化”。

如下你可以把它写（使用number代替seqNum）：

def collatz(number): 
    # you pass the number to the function and you return, so no need for global   
    if number % 2 == 0:  # python doesn't need parenthesis for "if"s 
     return number // 2 
    else:      # it can only be even OR odd so no need to calculate the modulo again 
     return 3 * number + 1 

# You can put the question as argument for "input" instead of printing it 
seqNum = input('What number would you like to use?') 
number = int(seqNum) 

while number > 1 : 
    number = collatz(number) # assign the result of the function to "number" 
    print(number)