Openpyxl保存已损坏/不可读的文件。没有错误报告,只需重新启动外壳

问题描述 投票:4回答:1

我已经学习Python几个月了,以使一些无聊的工作自动化(主要是Excel),并且在大多数情况下我都成功了。但是两周以来,我一直在尝试解决openpyxl遇到的问题,同时用从其他电子表格收集的数据填充每周报告。根据openpyxl.__version__,openpyxl版本为3.0.2,我使用的是内置IDLE的Python 3.6.7。编辑:这是完整代码https://pastebin.com/MrBZ5Usu

程序应该做的是:

  1. 从一周的报告中收集数据并将其放在列表中
  2. 从电子表格中收集数据,日期作为键,小时作为字典值
  3. 打开模板电子表格,用此数据填充并将其另存为另一个文件。

现在,当我运行程序时,没有错误报告。 Python Shell重新启动。该文件在那里,但是它是0字节,无法打开。我已经进行了一些测试,直到保存工作簿之前,一切似乎都还不错。调用时,新工作簿中的单元格将显示我输入的值。每条数据都是希望的标准化格式/类型。当我调用wb.save(filename)方法时,shell重新启动。我尝试了不同的方法将数据放入单元格(使用f字符串循环,使用单元格坐标的预定义列表循环,对单元格和数据进行硬编码),但都无济于事。 Shell重新启动-0字节的电子表格。我确保我机器上的每个模块都是最新的等等。我已经成功地将数据写入了一个搁置的模块,然后使用另一个脚本提取了它们。第二个脚本(只有几行,用相同的代码填充单元格)成功保存了一个有效的工作簿,但前提是该文件是相同的。我尝试过在主程序中更改它,但是没有保存选项(作为其他文件,相同文件,文件的常规副本(!))授予成功。我的代码显然有问题(除了新手的所有错误),但我无法把手放在上面。这是代码-有人有任何建议吗?根据要求,我可以提供基本上整个脚本(大约120行)。

endlist = load_workbook(f"Endlist_{monat1}_2019.xlsx", data_only=True)
endlistws = endlist.active


#creating an empty dict, to store the dates and hours from endlist
endlist_hrs = {}

#creating a dict with dates as keys and empty lists as values
for cell in endlistws['A']:
    if cell.value != None:
        if weeknum(dateConverter(cell.value)) == kw_num:
            if dateConverter(endlistws[f'A{cell.row}'].value) in endlist_hrs.keys(): #is the date already in the dict?
                pass # it is, so pass
            else:
                endlist_hrs[dateConverter((endlistws[f'A{cell.row}'].value))] = [] #its not, so add it
    else:
        pass #does not match

# iterating over keys in the endlist_hrs dict, checking the dates in A column - not the best solution, iterating every time over whole A column - to be upgraded
for key in endlist_hrs.keys():
    for cell in endlistws['A']:
        if cell.value != None:
            if dateConverter(cell.value) == key:
                endlist_hrs[key].append(czasownik(endlistws[f'J{cell.row}'].value))


endlist.close() #closing the endlist workbook

#creating a dict with dates as keys and sum of hours as values - ready to be inserted into cells in the Check workbook
full_endlist_data = {k:sum(v) for (k,v) in endlist_hrs.items()}

#copying the dailycheck workbook and producing the final output

faylneym = f"DC{kw_num}.xlsx"
paf = os.path.join(values['Browse0'], faylneym)

shutil.copy2(values['Browse1'], paf)

dcwb = load_workbook(paf, write_only=True)
dcws = dcwb['KW_XX']
dcws.title = str(kw)
dcwb.save(paf)
dcwb = load_workbook(paf)
dcws = dcwb.active

for x,y in enumerate(strdate, start=2):
    dcws[f'A{x}'].value = y
for x,y in enumerate(strdate, start=12):
    dcws[f'A{x}'].value = y
for x,y in enumerate(hours_from_eos2, start=2):
    dcws[f'E{x}'].value = y
for x,y in enumerate(full_endlist_data.values(), start=2):
    dcws[f'D{x}'].value = y

之后,我只是保存工作簿。

python excel openpyxl
1个回答
0
投票

我重建了我的代码,但具有比结构字符更多的功能。我还将代码的填写部分分为4个函数。

  1. 复制电子表格并为其重新命名。
  2. 打开电子表格,并使用先前生成的列表(6个项目,给定WK中的日期范围)进行填写,然后保存工作簿。
  3. 打开工作簿,遍历6个项目的清单,然后将它们放到位。保存工作簿。
  4. 打开工作簿,遍历字典值并将其放入计划的单元格中。保存工作簿。

以这种方式进行操作,不会遇到任何错误,崩溃或意外行为。工作簿中填充了预期的数据并且没有错误。不幸的是,我的能力不足以查明确切原因。这是最终的代码:

def newWB(source, target, kw_num):
    faylneym = f"DC{kw_num}.xlsx"
    paf = os.path.join(target, faylneym)
    shutil.copy2(source, paf)
    return paf



def filling1(wbpaf, strdata):
    try:
        dcwb = load_workbook(wbpaf)
        dcws = dcwb.active

        for x, y in enumerate(strdate, start=2):
            dcws[f'A{x}'].value = y
        for x, y in enumerate(strdate, start=12):
             dcws[f'A{x}'].value = y
        # for x, y in enumerate(hours_from_eos, start=2):
        #     dcws[f'E{x}'].value = y
        # for x, y in enumerate(endlist_hrs.values(), start=2):
        #     dcws[f'D{x}'].value = y
        dcwb.save(wbpaf)

    except:
        sg.Popup("Filling1 with data failed")

def filling2(wbpaf, hours_from_eos):
    try:
        dcwb = load_workbook(wbpaf)
        dcws = dcwb.active
        for x, y in enumerate(hours_from_eos, start=2):
            dcws[f'E{x}'].value = y
        dcwb.save(wbpaf)
    except:
        sg.Popup("Filling2 with data failed")

def filling3(wbpaf, endlist_hrs):
    try:
        dcwb = load_workbook(wbpaf)
        dcws = dcwb.active
        for x, y in enumerate(endlist_hrs.values(), start=2):
            dcws[f'D{x}'].value = y
        dcwb.save(wbaf)
    except:
        sg.Popup("Filling3 failed")'
© www.soinside.com 2019 - 2024. All rights reserved.