错误索引错误加载文件配置

Question

我的类config中有一个函数可以加载配置文本文件。在加载数据之前，该功能会检查文件以避免用户手动修改文件而造成的故障。

def load_config(self):
    list_dat = []
    list_str = []
    with open(self.path, "r") as file:
        for conf in file:
            conf = conf.replace(": ", "|")
            conf = conf.replace("\n", "")
            try:
                conf1, conf2 = conf.split("|")
            except ValueError:
                self.new_config()
                break
            try:
                conf2 = ast.literal_eval(conf2)
            except (ValueError, SyntaxError):
                self.new_config()
                break
            if isinstance(conf2, tuple):
                self.new_config()
                break
            list_str.append(conf1)
            list_dat.append(conf2)

    for x in range(0, len(list_str)):
        if list_str[x] != self.conf_str[x]:
            self.change_config(list_str[x], self.conf_str[x], x, text=True)
            list_str[x] = self.conf_str[x]

    for y in range(0, len(list_dat)):
        if type(list_dat[y]) != type(self.list_default[y]):
            self.change_config(list_dat[y], self.list_default[y], y)
            list_dat[y] = self.list_default[y]

    if len(list_dat) == 0 and len(list_str) == 0:
        self.new_config()

    app = wx.App(False)
    width, height = wx.GetDisplaySize()

    if list_dat[1] != width:
        if list_dat[1] == self.list_default[1]:
            if list_dat[0] == False:
                self.change_config(list_dat[1], width, 1)
                self.change_config(list_dat[2], height, 2)
                list_dat[1] = width
                list_dat[2] = height
        else:
            self.change_config(list_dat[1], width, 1)
            self.change_config(list_dat[2], height, 2)
            list_dat[1] = width
            list_dat[2] = height
    if list_dat[2] != height:
        if list_dat[2] == self.list_default[2]:
            if list_dat[0] == False:
                self.change_config(list_dat[1], width, 1)
                self.change_config(list_dat[2], height, 2)
                list_dat[1] = width
                list_dat[2] = height
        else:
            self.change_config(list_dat[1], width, 1)
            self.change_config(list_dat[2], height, 2)
            list_dat[1] = width
            list_dat[2] = height
    if list_dat[4] > 100:
        self.change_config(list_dat[4], 100, 4)
        list_dat[4] = 100
    else:
        if list_dat[4] < 0:
            self.change_config(list_dat[4], 0, 4)
            list_dat[4] = 0
    if list_dat[5] > 100:
        self.change_config(list_dat[5], 100, 5)
        list_dat[5] = 100
    else:
        if list_dat[5] < 0:
            self.change_config(list_dat[5], 0, 5)
            list_dat[5] = 0
    if list_dat[6] > 100:
        self.change_config(list_dat[6], 100, 6)
        list_dat[6] = 100
    else:
        if list_dat[6] < 0:
            self.change_config(list_dat[6], 0, 6)
            list_dat[6] = 0
    if list_dat[7] > 100:
        self.change_config(list_dat[7], 100, 7)
        list_dat[7] = 100
    else:
        if list_dat[7] < 0:
            self.change_config(list_dat[7], 0, 7)
            list_dat[7] = 0
    if list_dat[8] > 100:
        self.change_config(list_dat[8], 100, 8)
        list_dat[8] = 100
    else:
        if list_dat[8] < 0:
            self.change_config(list_dat[8], 0, 8)
            list_dat[8] = 0

    self.main.id[20], self.main.id[9] = list_dat[0], list_dat[3]
    self.main.id[4], self.main.id[5] = list_dat[1], list_dat[2]
    self.main.id[11], self.main.id[13], self.main.id[15], self.main.id[17], self.main.id[19] = list_dat[4], list_dat[5], list_dat[6], list_dat[7], list_dat[8]


def change_config(self, old, new, n, text=False):
    with open(self.path, "r") as file:
        temp = file.readlines()
        if text:
            temp1, temp2 = temp[n].split(": ")
            temp1 = temp1.replace(str(old), str(new))
            temp[n] = temp1 + ": " + temp2
        else:
            temp[n] = temp[n].replace(str(old), str(new))
    with open(self.path, "w") as file:
        file.writelines(temp)

这是配置文件：

Display Set: True
Display Width: 1024
Display Height: 576
Fullscreen: False
Music Volume: 50
Sound Volume: 50
Voice Volume: 50
Ambient Volume: 50
Other Volume: 50

现在，我收到一个错误消息，说实话我不明白是什么原因引起的。例如，如果我以这种方式更改文件：

Display Set: True
Display Width: gfgfdd
Display Height: 576
Fullscreen: False
Music Volume: 50
Sound Volume: 50
Voice Volume: 50
Ambient Volume: 50
Other Volume: 50

我得到了错误：

if list_dat[4] > 100:
IndexError: list index out of range

如果我更改Display Height，Music Volume，Sound Volume，Voice Volume，Ambient Volume和Other Volume，也会发生同样的事情。

得到错误后，我去查看配置文件，并已正确纠正它。这就是为什么我不明白为什么我会不断收到这些错误的原因。

不断给我错误的代码是：

if list_dat[1] != width:
    if list_dat[1] == self.list_default[1]:
        if list_dat[0] == False:
            self.change_config(list_dat[1], width, 1)
            self.change_config(list_dat[2], height, 2)
            list_dat[1] = width
            list_dat[2] = height
    else:
        self.change_config(list_dat[1], width, 1)
        self.change_config(list_dat[2], height, 2)
        list_dat[1] = width
        list_dat[2] = height
if list_dat[2] != height:
    if list_dat[2] == self.list_default[2]:
        if list_dat[0] == False:
            self.change_config(list_dat[1], width, 1)
            self.change_config(list_dat[2], height, 2)
            list_dat[1] = width
            list_dat[2] = height
    else:
        self.change_config(list_dat[1], width, 1)
        self.change_config(list_dat[2], height, 2)
        list_dat[1] = width
        list_dat[2] = height
if list_dat[4] > 100:
    self.change_config(list_dat[4], 100, 4)
    list_dat[4] = 100
else:
    if list_dat[4] < 0:
        self.change_config(list_dat[4], 0, 4)
        list_dat[4] = 0
if list_dat[5] > 100:
    self.change_config(list_dat[5], 100, 5)
    list_dat[5] = 100
else:
    if list_dat[5] < 0:
        self.change_config(list_dat[5], 0, 5)
        list_dat[5] = 0
if list_dat[6] > 100:
    self.change_config(list_dat[6], 100, 6)
    list_dat[6] = 100
else:
    if list_dat[6] < 0:
        self.change_config(list_dat[6], 0, 6)
        list_dat[6] = 0
if list_dat[7] > 100:
    self.change_config(list_dat[7], 100, 7)
    list_dat[7] = 100
else:
    if list_dat[7] < 0:
        self.change_config(list_dat[7], 0, 7)
        list_dat[7] = 0
if list_dat[8] > 100:
    self.change_config(list_dat[8], 100, 8)
    list_dat[8] = 100
else:
    if list_dat[8] < 0:
        self.change_config(list_dat[8], 0, 8)
        list_dat[8] = 0

你能帮我吗？

编辑：

我添加了简化的类版本，仅包含错误所需的功能。

import ast, os, wx
from pathlib import Path

class config_lib():

    def __init__(self):
        # IdDataDefaultConfig
        self.id1 = True  # SetDisplayDefault
        self.id2 = 1024  # DisplayWidthDefault
        self.id3 = 576  # DisplayHeightDefault
        self.id4 = 1024  # DisplayWidthUser
        self.id5 = 576  # DisplayHeightUser
        self.id6 = 0  # LocationDisplayX
        self.id7 = 50  # LocationDisplayY
        self.id8 = False  # FullscreenDefault
        self.id9 = False  # FullscreenUser
        self.id10 = 50  # MusicVolumeDefault
        self.id11 = 50  # MusicVolumeUser
        self.id12 = 50  # SoundVolumeDefault
        self.id13 = 50  # SoundVolumeUser
        self.id14 = 50  # VoiceVolumeDefault
        self.id15 = 50  # VoiceVolumeUser
        self.id16 = 50  # AmbientVolumeDefault
        self.id17 = 50  # AmbientVolumeUser
        self.id18 = 50  # OtherAudioVolumeDefault
        self.id19 = 50  # OtherAudioVolumeUser
        self.id20 = True  # SetDisplayUser

        self.conf_str = ["Display Set", "Display Width", "Display Height", "Fullscreen", "Music Volume", "Sound Volume", "Voice Volume", "Ambient Volume", "Other Volume"]
        self.list_default = [self.id1, self.id2, self.id3, self.id8, self.id10, self.id12, self.id14, self.id16, self.id18]

        self.path = os.path.join("dat.bw3")
        self.conf = Path(self.path)
        try:
            lc = self.conf.resolve()
        except FileNotFoundError:
            self.new_config()
        else:
            self.load_config()

    def new_config(self):
        list_dat = [[self.conf_str[0], self.id1], [self.conf_str[1], self.id2], [self.conf_str[2], self.id3], [self.conf_str[3], self.id8], [self.conf_str[4], self.id10], [self.conf_str[5], self.id12], [self.conf_str[6], self.id14], [self.conf_str[7], self.id16], [self.conf_str[8], self.id18]]
        with open(self.path, "wt") as file:
            for conf in list_dat:
                print(conf[0], conf[1], sep=': ', file=file)
        self.load_config()

    def load_config(self):
        list_dat = []
        list_str = []
        with open(self.path, "r") as file:
            for conf in file:
                conf = conf.replace(": ", "|")
                conf = conf.replace("\n", "")
                try:
                    conf1, conf2 = conf.split("|")
                except ValueError:
                    self.new_config()
                    break
                try:
                    conf2 = ast.literal_eval(conf2)
                except (ValueError, SyntaxError):
                    self.new_config()
                    break
                if isinstance(conf2, tuple):
                    self.new_config()
                    break
                list_str.append(conf1)
                list_dat.append(conf2)

        for x in range(0, len(list_str)):
            if list_str[x] != self.conf_str[x]:
                self.change_config(list_str[x], self.conf_str[x], x, text=True)
                list_str[x] = self.conf_str[x]

        for y in range(0, len(list_dat)):
            if type(list_dat[y]) != type(self.list_default[y]):
                self.change_config(list_dat[y], self.list_default[y], y)
                list_dat[y] = self.list_default[y]

        if len(list_dat) == 0 and len(list_str) == 0:
            self.new_config()

        app = wx.App(False)
        width, height = wx.GetDisplaySize()

        if list_dat[1] != width:
            if list_dat[1] == self.list_default[1]:
                if list_dat[0] == False:
                    self.change_config(list_dat[1], width, 1)
                    self.change_config(list_dat[2], height, 2)
                    list_dat[1] = width
                    list_dat[2] = height
            else:
                self.change_config(list_dat[1], width, 1)
                self.change_config(list_dat[2], height, 2)
                list_dat[1] = width
                list_dat[2] = height
        if list_dat[2] != height:
            if list_dat[2] == self.list_default[2]:
                if list_dat[0] == False:
                    self.change_config(list_dat[1], width, 1)
                    self.change_config(list_dat[2], height, 2)
                    list_dat[1] = width
                    list_dat[2] = height
            else:
                self.change_config(list_dat[1], width, 1)
                self.change_config(list_dat[2], height, 2)
                list_dat[1] = width
                list_dat[2] = height
        if list_dat[4] > 100:
            self.change_config(list_dat[4], 100, 4)
            list_dat[4] = 100
        else:
            if list_dat[4] < 0:
                self.change_config(list_dat[4], 0, 4)
                list_dat[4] = 0
        if list_dat[5] > 100:
            self.change_config(list_dat[5], 100, 5)
            list_dat[5] = 100
        else:
            if list_dat[5] < 0:
                self.change_config(list_dat[5], 0, 5)
                list_dat[5] = 0
        if list_dat[6] > 100:
            self.change_config(list_dat[6], 100, 6)
            list_dat[6] = 100
        else:
            if list_dat[6] < 0:
                self.change_config(list_dat[6], 0, 6)
                list_dat[6] = 0
        if list_dat[7] > 100:
            self.change_config(list_dat[7], 100, 7)
            list_dat[7] = 100
        else:
            if list_dat[7] < 0:
                self.change_config(list_dat[7], 0, 7)
                list_dat[7] = 0
        if list_dat[8] > 100:
            self.change_config(list_dat[8], 100, 8)
            list_dat[8] = 100
        else:
            if list_dat[8] < 0:
                self.change_config(list_dat[8], 0, 8)
                list_dat[8] = 0

        self.id20, self.id9 = list_dat[0], list_dat[3]
        self.id4, self.id5 = list_dat[1], list_dat[2]
        self.id11, self.id13, self.id15, self.id17, self.id19 = list_dat[4], list_dat[5], list_dat[6], list_dat[7], list_dat[8]

    def change_config(self, old, new, n, text=False):
        with open(self.path, "r") as file:
            temp = file.readlines()
            if text:
                temp1, temp2 = temp[n].split(": ")
                temp1 = temp1.replace(str(old), str(new))
                temp[n] = temp1 + ": " + temp2
            else:
                temp[n] = temp[n].replace(str(old), str(new))
        with open(self.path, "w") as file:
            file.writelines(temp)

conf = config_lib()

EDIT2：

在类的__init__中，它检查文件是否存在。如果不存在，则调用new_config()函数，该函数将使用默认数据创建一个新文件，然后调用load_config()函数。如果文件存在，则直接调用load_config()函数。

load_config()函数必须先检查文件数据，然后再将其加载到程序中。

load_config()函数的第一块：

要做的第一件事是将文件中写入的内容转换为程序可以处理的内容。然后读取该文件，对于以后用有用符号替换的每一行（“：”）（“ |”），然后删除回车符。现在我们必须将其分成两部分，以便在一侧有数据名称，而在另一侧有数据本身。通过划分先前插入符号（“ |”）的位置来完成此操作。这种划分是在try中完成的，因此，如果文件内部有错误，可以通过使用new_config()创建一个新文件来覆盖文件。第二件事是将数据（当前为字符串）转换为有效数据。这是由ast.literal_eval()完成的。再次，将所有内容放入try中，其原因与以前相同。最后，检查数据是单个数据而不是元组。数据名称和数据分别在list_dat和list_str中输入。

load_config()函数的第二个块：

要做的第一件事是检查名称的默认列表，如果数据名称正确，则错误，则将其更正。数据相同，但在这种情况下，将检查类型是否与其默认对应类型相同。很少（例如，每100个中有1个），我注意到，如果文件为空，则不会从Except块或split()块引发ast.literal_eval()。所以我加了：

if len(list_dat) == 0 and len(list_str) == 0:
            self.new_config()

为避免此问题。

load_config()函数的第三块（这是给我带来问题的块）：

这将检查屏幕的宽度和高度是否与数据的宽度和高度相同。如果它们不相等，则检查数据是否与默认值相同（1024、576）。如果它们相同，则最后检查默认加载模式是否为false。执行此检查是为了在创建程序屏幕时没有任何问题。如果这些检查中的任何一个失败（否则），则宽度和高度数据将更改为默认数据。

if list_dat[1] != width:
    if list_dat[1] == self.list_default[1]:
        if list_dat[0] == False:
            self.change_config(list_dat[1], width, 1)
            self.change_config(list_dat[2], height, 2)
            list_dat[1] = width
            list_dat[2] = height
    else:
        self.change_config(list_dat[1], width, 1)
        self.change_config(list_dat[2], height, 2)
        list_dat[1] = width
        list_dat[2] = height
if list_dat[2] != height:
    if list_dat[2] == self.list_default[2]:
        if list_dat[0] == False:
            self.change_config(list_dat[1], width, 1)
            self.change_config(list_dat[2], height, 2)
            list_dat[1] = width
            list_dat[2] = height
    else:
        self.change_config(list_dat[1], width, 1)
        self.change_config(list_dat[2], height, 2)
        list_dat[1] = width
        list_dat[2] = height

类似地，进行相同的音量控制，唯一的区别是该数字不超过100（最大音量）或小于0（最小音量）。

if list_dat[4] > 100:
    self.change_config(list_dat[4], 100, 4)
    list_dat[4] = 100
else:
    if list_dat[4] < 0:
        self.change_config(list_dat[4], 0, 4)
        list_dat[4] = 0
if list_dat[5] > 100:
    self.change_config(list_dat[5], 100, 5)
    list_dat[5] = 100
else:
    if list_dat[5] < 0:
        self.change_config(list_dat[5], 0, 5)
        list_dat[5] = 0
if list_dat[6] > 100:
    self.change_config(list_dat[6], 100, 6)
    list_dat[6] = 100
else:
    if list_dat[6] < 0:
        self.change_config(list_dat[6], 0, 6)
        list_dat[6] = 0
if list_dat[7] > 100:
    self.change_config(list_dat[7], 100, 7)
    list_dat[7] = 100
else:
    if list_dat[7] < 0:
        self.change_config(list_dat[7], 0, 7)
        list_dat[7] = 0
if list_dat[8] > 100:
    self.change_config(list_dat[8], 100, 8)
    list_dat[8] = 100
else:
    if list_dat[8] < 0:
        self.change_config(list_dat[8], 0, 8)
        list_dat[8] = 0

load_config()函数的第四个块：

将受控数据分配给它们的变量，以供程序使用。

EDIT3：

很不幸，我之前忘记说些什么。我知道ast.literal_eval()出现了异常，这就是为什么我将其放在try中的原因。发生这种情况时，将调用new_config()函数，并且循环应该重新开始。因此，列表返回空白，并由检查再次填充。至少在理论上是这样。取而代之的是，虽然该文件被创建为新文件（带有默认数据），但该函数的其余部分将在第二次执行时分解。同样也是因为该错误仅由我输入的数据引起。对于其他具有布尔值且在第三个块中没有控制权的其他人（全屏等），则不会发生此问题。

Answer 1

将脚本简化为这样的内容后，

import ast
with open(self.path, "r") as file:
    for conf in file:
        conf = conf.replace(": ", "|")
        conf = conf.replace("\n", "")
        conf1, conf2 = conf.split("|")
        conf2 = ast.literal_eval(conf2)
        print(conf2)

很明显ast.literal_eval（）方法抛出ValueError异常：

Traceback (most recent call last):
  File "/usr/local/Cellar/python3/3.6.3/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/IPython/core/interactiveshell.py", line 2910, in run_code
    exec(code_obj, self.user_global_ns, self.user_ns)
  File "<ipython-input-214-f993e6585bcd>", line 7, in <module>
    conf2 = ast.literal_eval(conf2)
  File "/usr/local/Cellar/python3/3.6.3/Frameworks/Python.framework/Versions/3.6/lib/python3.6/ast.py", line 85, in literal_eval
    return _convert(node_or_string)
  File "/usr/local/Cellar/python3/3.6.3/Frameworks/Python.framework/Versions/3.6/lib/python3.6/ast.py", line 84, in _convert
    raise ValueError('malformed node or string: ' + repr(node))
ValueError: malformed node or string: <_ast.Name object at 0x111f50f28>

由于您在这里进行处理：

    try:
        conf2 = ast.literal_eval(conf2)
    except (ValueError, SyntaxError):
        self.new_config()
        break

您获得的配置条目少于预期，因此出现InvalidIndex错误。

现在ast.literal_eval()引发异常，因为您的输入无法被评估为任何已知的python类型或方法。

请检查the docs，以了解如何正确使用它