Python函数运行时portofolio_value与porto变量值翻倍问题排查

每次运行函数计算结果翻倍的核心原因是全局变量tradesdf存在重复追加数据的问题：
你的calculations()函数没有在内部重新拉取/重置持仓DataFrame，而是直接读取全局作用域下的tradesdf做循环计算。你每次调用函数前的持仓拉取逻辑，是把新获取到的MT5持仓记录追加到原有tradesdf中，而不是覆盖旧数据：第一次运行时tradesdf里有1条对应持仓，第二次运行时同一条持仓被存了2次，第三次存了4次，循环累加得到的portofolio_value自然会跟着翻倍。
你给出的测试场景（持仓数量12、品种价格5）下，这个问题的表现会非常明显：第一次计算portofolio_value为60，第二次为120，第三次为240，完全符合翻倍的异常表现。
另外你的函数只声明了部分全局变量，tradesdf、PnL、start_amount这几个核心计算依赖的变量都直接从全局作用域读取，没有做入参传递，也放大了全局变量被意外篡改的风险。

按优先级操作即可解决问题：

• 修正持仓拉取逻辑：每次获取MT5最新持仓时，直接用全量最新持仓覆盖赋值给tradesdf，禁止在原有tradesdf基础上重复追加同一条持仓记录。
• 优化函数结构：把tradesdf、PnL、start_amount改为函数的显式入参，不要直接在函数内部读取全局变量，从根源避免全局状态污染导致的计算异常。
• （建议）修正变量拼写错误：代码中portofolio为拼写错误，正确拼写为portfolio，避免后续因为拼写不一致引出隐性bug。

修复后核心计算逻辑参考：

# 每次调用函数前，重新拉取全量持仓覆盖tradesdf，不要追加
positions = mt5.positions_get()
tradesdf = pd.DataFrame(list(positions), columns=positions[0]._asdict().keys())

def calculations(tradesdf, start_amount, PnL):
    portofolio_value = 0
    acc_info = mt5.account_info()
    money_end = float(acc_info.balance)
    for i in range(len(tradesdf)):
        symb = str(tradesdf.at[i, 'symbol'])
        price = float(mt5.symbol_info_tick(symb).bid)
        position_volume = float(tradesdf.at[i, 'quantity'])
        portofolio_value += position_volume * price
    RT_portofolio_value = round((money_end + portofolio_value), 2)
    porto = portofolio_value
    profit = round(((porto + money_end) - start_amount), 2)

    len_win = len([i for i in PnL if i > 0])
    len_PnL = len(PnL)
    rw_perc = 0 if len_PnL == 0 else (len_win / len_PnL) * 100
    perc_profit = 0 if start_amount <=0 else round(((profit/start_amount) * 100), 2)

    print(porto)
    print(portofolio_value)
    # 计算结果通过return返回，不要依赖全局变量传递
    return {
        "money_end": money_end,
        "RT_portofolio_value": RT_portofolio_value,
        "porto": porto,
        "profit": profit,
        "rw_perc": rw_perc,
        "perc_profit": perc_profit
    }

修复后只要传入的tradesdf是无重复的最新全量持仓，无论调用多少次函数，计算结果都不会出现翻倍异常。

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