我希望这是一个重复的问题,但我找不到答案。对不起。
我编写了一个简单的函数,它应该返回一个空字符串或一个字符串。但是,有时会返回无。即使应该匹配其中一个操作数,如果不匹配,仍会输出一个字符串。
baseconvert = digits+ascii_lowercase+"0"
def checker(operand, factorlist): # hardcoded interpreter of the calculus converted in base 10. i is equal to the base.
if (operand == '+'):
if (factorlist[0] + factorlist[1] == factorlist[2]):
return baseconvert[i]
elif (operand == '-'):
if (factorlist[0] - factorlist[1] == factorlist[2]):
return baseconvert[i]
elif (operand == '*'):
if (factorlist[0] * factorlist[1] == factorlist[2]):
return baseconvert[i]
elif (operand == '/'):
if (factorlist[0] / factorlist[1] == factorlist[2]):
return baseconvert[i]
else:
return ""
因此,在输入这篇文章时,我发现我可以删除else语句:
# else: is not necessary
return ""
工作正常。然而,我仍然想知道为什么这句话从未被触发过。
P.S。:有没有更简洁的方法来解释字符串,例如“ 10 - 7 = 3 ”以获得布尔结果?我对python比较陌生,无法想出任何东西。
以下代码(参数取消上下文)打印无:
from string import digits, ascii_lowercase
baseconvert = digits+ascii_lowercase+"0"
def checker(operand, factorlist): # hardcoded interpreter of the calculus converted in base 10. i is equal to the base.
if (operand == '+'):
if (factorlist[0] + factorlist[1] == factorlist[2]):
return baseconvert[i]
elif (operand == '-'):
if (factorlist[0] - factorlist[1] == factorlist[2]):
return baseconvert[i]
elif (operand == '*'):
if (factorlist[0] * factorlist[1] == factorlist[2]):
return baseconvert[i]
elif (operand == '/'):
if (factorlist[0] / factorlist[1] == factorlist[2]):
return baseconvert[i]
else:
return ""
i=17
print(checker('+', [1987,222,2227]))
预期结果:
if (operand == "+")
是真的。if (factorlist[0] + factorlist[1] == factorlist[2])
是假的。else:
被调用。我仍然想知道为什么我所期望的这种行为没有显示出来,但无论如何。只使用一个return语句以及离开else:
都是很好的解决方案。
我还发现了以下作品,这些作品最接近我想要完成的作品:
def checker(operand, factorlist): # hardcoded interpreter of the calculus converted in base 10. i is equal to the base.
if ((operand == '+') & (factorlist[0] + factorlist[1] == factorlist[2])):
return baseconvert[i]
elif ((operand == '-') & (factorlist[0] - factorlist[1] == factorlist[2])):
return baseconvert[i]
elif ((operand == '*') & (factorlist[0] * factorlist[1] == factorlist[2])):
return baseconvert[i]
elif ((operand == '/') & (factorlist[0] / factorlist[1] == factorlist[2])):
return baseconvert[i]
else:
return ""
我已经多次对所有解决方案进行了计时,这个最终解决方案似乎比使用拆分if语句和使用带有一次返回的变量快15%。
答案 0 :(得分:2)
您的父if
中嵌套了if...elif
个子句,当控件没有流出时,它不会返回父else
(即如果函数未退出)通过任何嵌套的 if ,执行在函数结束时完成。您应该在函数末尾添加一个返回值以正确处理它并避免返回默认的None
:
def checker(operand, factorlist):
if operand == '+':
if (factorlist[0] + factorlist[1] == factorlist[2]):
...
elif operand == '-':
if (factorlist[0] - factorlist[1] == factorlist[2]):
...
elif operand == '*':
if (factorlist[0] * factorlist[1] == factorlist[2]):
...
elif operand == '/':
if (factorlist[0] / factorlist[1] == factorlist[2]):
...
return ""
您还可以安全地删除最终else
,因为最终返回在您的函数中充当全局返回默认值。
答案 1 :(得分:0)
这是主观的,但很多人会说在一个函数中只有一个return
。在这种情况下可以提供帮助。问题是如果另一个条件为True,则只从每个elif
返回。它可以通过以下方式重构:
baseconvert = digits+ascii_lowercase+"0"
def checker(operand, factorlist):
# hardcoded interpreter of the calculus converted in base 10. i is equal to the base.
final_value = ""
if operand == '+':
if factorlist[0] + factorlist[1] == factorlist[2]:
final_value = baseconvert[i]
elif operand == '-':
if factorlist[0] - factorlist[1] == factorlist[2]:
final_value = baseconvert[i]
elif operand == '*':
if factorlist[0] * factorlist[1] == factorlist[2]:
final_value = baseconvert[i]
elif operand == '/':
if factorlist[0] / factorlist[1] == factorlist[2]:
final_value = baseconvert[i]
return final_value
请注意,您只需要在python中围绕条件使用括号来获得优先级,大多数情况下它们不是必需的。