使用Tkinter绘制美国地图时遇索引越界错误的修复咨询

Let's break down why you're hitting those IndexError issues and fix them step by step.

Root Causes of the Errors #

Your code has two critical issues that lead to index out of range errors:

1. Modifying the list while iterating: Using data.remove(data[i]) changes the length of the data list as you loop. When you delete an element, all subsequent elements shift left—but your i variable keeps incrementing, which can skip elements or send i beyond the new length of the list.
2. No bounds checking: Your while True loop doesn't check if i is still within the valid range of the data list. Eventually, i will exceed len(data) - 1, causing data[i] to throw an error.

Fixed Code #

Here's a revised version of your code that addresses these problems, plus some readability improvements:

from Tkinter import Tk, Canvas
# from PIL import Image, ImageTk  # Uncomment if you plan to use PIL for image handling later

root = Tk()
canvas = Canvas(root, width=640, height=480, bg="white")

# Use a with statement to safely open/close the file
with open("lab313.txt", "r") as f:
    data = f.read().split()

usamain = []
usaisland = []
usa = []
i = 0
makePoly = False

# Loop with bounds checking to avoid index overflow
while i < len(data):
    current_item = data[i]
    
    # Start collecting polygon points when we hit a state abbreviation
    if len(current_item) == 2 and not makePoly:
        makePoly = True
        i += 1
        continue
    
    # Stop collecting a single polygon
    if current_item == "END_ONE_POLY":
        makePoly = False
        i += 1
        continue
    
    # Reset when we finish a state's full polygon set
    if current_item == "END_ALL_POLY":
        usamain.clear()
        usaisland.clear()
        makePoly = False
        i += 2  # Skip END_ALL_POLY and the following element (as your original code did)
        continue
    
    # Exit loop when we reach the end of the file
    if current_item == "END_FILE":
        break
    
    # Collect coordinate data based on current state
    if makePoly:
        usamain.append(float(current_item))
    else:
        usaisland.append(float(current_item))
        usa.append(float(current_item))
    
    i += 1

print("usamain:", usamain)
print("usaisland:", usaisland)
print("usa:", usa)

canvas.pack()
root.mainloop()

Key Improvements #

• No more list modification during iteration: Instead of deleting elements from data, we just move the i pointer to skip over markers we don't need. This keeps the list intact and prevents index shifting bugs.
• Bounds checking: The loop condition while i < len(data) ensures we never try to access an index that doesn't exist.
• Cleaner flow: Using continue in each condition branch makes the logic easier to follow—each marker type has its own clear handling path.
• Safe file handling: The with statement automatically closes the file after reading, preventing resource leaks.
• Clearer state management: Using clear() instead of usamain[:] = [] makes the intent of resetting lists more explicit.

Additional Recommendations #

1. Separate data parsing from drawing: Parse all your state polygon data into a structured format (like a dictionary where keys are state abbreviations and values are lists of coordinate tuples) before you start drawing. This makes debugging and maintenance much easier.
2. Add error handling: Wrap the float(current_item) conversion in a try-except block to handle any malformed data in your text file, e.g.:

   try:
       coord = float(current_item)
   except ValueError:
       print(f"Skipping invalid coordinate: {current_item}")
       i += 1
       continue
   

3. Coordinate mapping: Raw lat/long values won't fit on your 640x480 canvas. You'll need to add a function to map geographic coordinates to pixel positions (e.g., scale and shift the lat/long range to match your canvas dimensions).

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