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), \
       "File {} doesn't exist or isn't readable".format(file)

Answer 2

对我来说，使用try-except与使用if-else获得的范围相同的范围没有可读性。异常的值是它们可以在调用树中的更高级别捕获。

只移出一个级别，我们避免使用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 3

在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 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

这通常就是我的工作。这是强大而直接的

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

4 个答案: