if循环中向类实例字典添加DataFrame仅存一个的问题排查

Hey there! Let's break down why your myINST.dataframeDict only has one DataFrame instead of two after running the loop twice. This is a super common gotcha, so let's go through the most likely culprits and fix them up.

1. You're using the same dictionary key every time in the loop #

This is the #1 reason for this issue. If you're assigning each DataFrame to a fixed key (like 'df' or a hardcoded string), each loop iteration will overwrite the previous entry instead of adding a new one.

Example of the mistake: #

import pandas as pd
import os

class DataHandler:
    def __init__(self):
        self.dataframeDict = {}
    
    def load_and_format(self, file_path):
        # 1. Create DataFrame from file
        df = pd.read_csv(file_path)
        # 2. Format DataFrame (example)
        df['timestamp'] = pd.to_datetime(df['timestamp'])
        # 3. Add to dict with fixed key - this overwrites every time!
        self.dataframeDict['processed_df'] = df

# Usage
myINST = DataHandler()
file_paths = ['sales_jan.csv', 'sales_feb.csv']
for path in file_paths:
    myINST.load_and_format(path)

# Result: dataframeDict only contains the last month's data under 'processed_df'

Fix: #

Use a unique key for each file. You can use the filename itself, a loop index, or a custom identifier:

def load_and_format(self, file_path):
    df = pd.read_csv(file_path)
    df['timestamp'] = pd.to_datetime(df['timestamp'])
    # Use the filename (stripped of path) as the key
    unique_key = os.path.basename(file_path)
    self.dataframeDict[unique_key] = df

# Or use loop index if you prefer:
for idx, path in enumerate(file_paths):
    myINST.load_and_format(path)
    # Modify the method to accept idx as an argument if you want numeric keys

2. You're reinitializing the dataframeDict inside your method #

If your class method resets the dictionary every time it runs, you'll only ever keep the last entry. This happens when you initialize self.dataframeDict = {} inside the processing method instead of the class constructor.

Example of the mistake: #

class DataHandler:
    def load_and_format(self, file_path):
        # Oops! This resets the dict to empty every time the method is called
        self.dataframeDict = {}
        df = pd.read_csv(file_path)
        self.dataframeDict[os.path.basename(file_path)] = df

Fix: #

Initialize the empty dictionary once in the class's __init__ method:

class DataHandler:
    def __init__(self):
        # Initialize empty dict here - runs only once when creating the instance
        self.dataframeDict = {}
    
    def load_and_format(self, file_path):
        df = pd.read_csv(file_path)
        # Format df...
        self.dataframeDict[os.path.basename(file_path)] = df

3. Quick debug check to confirm the issue #

If you're still unsure, add print statements inside the loop to track what's happening:

for path in file_paths:
    myINST.load_and_format(path)
    print(f"Current keys in dict: {list(myINST.dataframeDict.keys())}")
    print(f"Total entries: {len(myINST.dataframeDict)}\n")

This will show you if entries are being added (then overwritten) or never added at all, helping you narrow down the problem.

Once you fix these, your dataframeDict should have two distinct DataFrames, one for each input file. Feel free to share snippets of your actual code if you need further help!

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