Python使用win32com时遭遇无限循环问题求助

Hey there! Let’s dig into why your win32com Word automation code is stuck in an infinite loop with no error messages. First, let’s spot the key issues in your code, then fix them step by step:

Major Issues in Your Original Code #

• Broken Data Definition & Loop Structure: You tried to define the data list and start the for loop in a single line, which creates invalid syntax (it’s neither a proper list assignment nor a valid generator expression). This messed up the loop’s behavior and could lead to unexpected infinite iteration.
• Incorrect Dictionary Key Access: Your data uses the key filename, but your code tries to access i["[filename]"] (with extra square brackets). This would throw a KeyError for items like {'name':'John'}, but without error handling, the code might hang instead of crashing.
• Repeated Word Instance Creation: You’re creating and quitting a new Word instance every loop iteration. This is inefficient and can leave orphaned Word processes running, which might cause the code to freeze or loop unexpectedly.
• Lack of Error Handling: No checks for missing files, failed document openings, or save errors mean the code can silently hang instead of telling you what’s wrong.

Fixed & Improved Code #

Here’s the corrected version with explanations embedded:

import os
import win32com.client
from datetime import datetime

# Initialize the date once (no need to re-calculate every loop)
now = datetime.now()

# Properly define your data list (separate from the loop)
data = [
    {'filename':'example1.docx'},
    {'name':'John'},
    {'filename':'example2.docx'},
    {'name':'Jan'}
]

# Create ONE Word instance (reuse it for all documents)
word = win32com.client.gencache.EnsureDispatch("Word.Application")
word.Visible = False  # Hide Word window to avoid unexpected user interaction prompts

for item in data:
    # Skip items that don't have a 'filename' key
    if 'filename' not in item:
        print(f"Skipping item without filename: {item}")
        continue
    
    # Correctly access the filename key (no extra brackets!)
    original_filename = item['filename']
    new_filename = now.strftime("%d%m%Y") + '__' + original_filename
    
    try:
        original_path = os.path.abspath(original_filename)
        # Check if the file exists before trying to open it
        if not os.path.exists(original_path):
            print(f"File not found: {original_path}")
            continue
        
        # Open the document and save with the new name
        doc = word.Documents.Open(original_path)
        # Use FileFormat=12 to ensure we save as .docx (avoids format conflicts)
        doc.SaveAs(os.path.abspath(new_filename), FileFormat=12)
        doc.Close()  # Cleanly close the document
    
    except Exception as e:
        # Catch and print any errors so you know what went wrong
        print(f"Error processing {original_filename}: {str(e)}")
        # Make sure we close the document even if there's an error
        if 'doc' in locals():
            doc.Close(False)  # False = don't save changes if error occurs

# Clean up the Word instance properly
word.Quit()
del word  # Release COM resources to avoid orphaned processes

Why This Fixes the Infinite Loop #

1. Fixed Loop Structure: Separating the data definition from the for loop ensures Python iterates over your list correctly, eliminating invalid syntax that caused unexpected loops.
2. Single Word Instance: Reusing one Word instance avoids the overhead and potential hangs from repeatedly starting/stopping Word processes.
3. Error Visibility: The try-except block and file existence checks mean you’ll get clear messages if something fails (instead of silent hanging).
4. Key Validation: Skipping items without a filename key prevents KeyError crashes that could cause the code to freeze.

内容的提问来源于stack exchange，提问作者stella