检查文件是否可以用 Python 读取：try 还是 if/else？

Question

我有以下代码：

import glob, os
for file in glob.glob("\\*.txt"):
    if os.access(file, os.R_OK):
        # Do something
    else:
        if not os.access(file, os.R_OK):
            print(file, "is not readable")
        else:
            print("Something went wrong with file/dir", file)
        break

但我不完全确定这是否是正确的做法。错误使用try 和catch 会更好吗？如果是这样，我该如何尝试以提高可读性？请注意我的 else 语句中的 break。一旦无法读取文件，我就想中止循环。

Answer 1

一种更明确的方法来检查file 是否实际上是一个文件而不是目录，例如，它是可读的：

from os import access, R_OK
from os.path import isfile

file = "/some/path/to/file"

assert isfile(file) and access(file, R_OK), \
       f"File {file} doesn't exist or isn't readable"

Answer 2

在 Python 文化中，ask forgiveness, not permission 更常见，因此最好捕获异常：

for filename in glob.glob('*.txt'):
    try:
        with open(filename) as fp:
            # work with the file

    except IOError as err:
        print "Error reading the file {0}: {1}".format(filename, err)
        break

这样您还可以避免任何双重检查或竞争条件。

Answer 3

对我来说，在与使用 if-else 相同的范围内使用 try-except 不会获得可读性。异常的价值在于它们可以在调用树的更高级别被捕获。

只移出一层，我们避免使用break 声明：

import glob, os
try:
    for file in glob.glob("\\*.txt"):
        with open(file) as fp:
            # do something with file
except IOError:
    print("could not read", file)

但异常的真正天才在于代码消失了：

# Operate on several files
# SUCCESS: Returns None
# FAIL: Raises exception
def do_some_files():
    for file in glob.glob("\\*.txt"):
        with open(file) as fp:
            # do something with file

现在调用程序有责任在失败时显示有用的错误消息。我们已经将处理失败的责任完全从这段代码中移到了另一个领域。

事实上，我们可以将责任完全从我们的程序中转移到解释器中。在这种情况下，解释器将打印一些有用的错误消息并终止我们的程序。如果 Python 的默认消息对您的用户来说足够好，我建议不要检查错误根本。因此，您的原始脚本变为：

import glob, os
for file in glob.glob("\\*.txt"):
    # Do something

Answer 4

try:
        # check to see if file is readable
        with open(filename) as tempFile:





except Exception as e:
        print e
        # here you can modify the error message to your liking

这通常是我所做的。 它健壮且直接