出错时正确退出 While 循环

Question

我很好奇用 while 循环处理这个错误的“正确”Python 方式是什么。在本示例中，我可以简单地检查字符串以查看它是否预先包含 [-1] 字符，但我的问题示例所基于的实际代码要复杂得多。

try:
    while mystring[-1] == '!' #Will throw an error if mystring is a blank string
         print("Exclamation!")
         mystring = mystring[:-1]
    return mystring
except:
    return ""

实际上，我的问题是我的 while 循环取决于检查，偶尔在循环内进行一些处理后，会抛出错误。上面只是该问题的（也许过于）简化的说明。我通过一系列的 try: excepts: 修复了它，但我觉得这不是解决这个问题的“正确”方法。

Answer 1

您当前的代码做了两件事，您不应该这样做：

1. 捕获任何异常，您应该只捕获感兴趣的特定异常
2. 在 try 块中包含整个循环，如果可能的话，您应该只包含引发该异常的语句/表达式如果

您发现自己在 python 中大量使用 while 循环，则表明您没有最有效地利用 python。考虑到 python 的工具集，你几乎应该总是使用某种 for 循环。如果没有看到您的代码的真实示例，我无法判断这是否属实。如果您需要该领域的帮助，请在 http://codereview.stackexchange.com 发布一些代码

此问题的一般解决方案是编写处理异常并使用它的函数。

def last_character(string):
    try:
       return string[-1]
    except IndexError:
       return ' '

while last_character(mystring) == '!'
    mystring = mystring[:-1]
return mystring

事实上，在许多情况下，已经有与标准构造相同的无异常等效项。使用 .endswith() 方法可以轻松编写此循环。通过使用这些或自己制作，您可以最干净地处理异常。

Answer 2

对于您的示例，您可以简单地执行以下操作：

while mystring and mystring[-1] == '!':
    print("Exclamation!")
    mystring = mystring[:-1]
return mystring

这将起作用，因为如果 mystring 为空，它将短路并结束循环，因此您永远不会尝试访问空的 -1 索引string

编辑： 正如 Winston 指出的，您可以使用 str.endswith 删除所有特殊大小写，如以下代码所示

while mystring.endswith('!'):
    print("Exclamation!")
    mystring = mystring[:-1]
return mystring

Answer 3

使用 mystring.rstrip('!') 删除字符串末尾的'!' 字符;-)

如果问题更复杂，正确的方法是捕获操作抛出的IndexError。

try:
    while mystring[-1] == '!' #Will through an error if mystring is a blank string
         print("Exclamation!")
         mystring = mystring[:-1]
    return mystring
except IndexError:
    return ""

另一种方法是检查字符串是否为空并避免使用引发异常的操作：

while mystring and mystring[-1] == '!': # lazy boolean expression evaluation
      mystring = mystring[:-1]
return mystring

没有惰性布尔表达式求值的其他版本：

if not mystring:
    return mystring
while mystring[-1] == '!':
    mystring = mystring[:-1]
    if not mystring:
        break
return mystring

我个人更喜欢第二个版本，特别是如果您更改 mystring[-1] == '!' with mystring.endswith('!') （但在这种情况下，您不需要检查是否为空，因为endswith已经为您完成了这项工作）。