5

我对编程很陌生，并制作了一个程序来从 Team Fortress 2 玩家那里获取库存数据，并将库存项目放入字典中，其中 steamid 作为键，项目列表作为值。

我遇到的问题是，在字典中输入了大约 6000 个条目后，该程序基本上耗尽了我系统上的所有 RAM 并关闭。

我猜字典只是变得太大了，但根据我从类似问题中读到的内容，6000 个条目的字典不应该占用我那么多的 RAM。

我一直在寻找其他解决方案，但我可以为我的代码使用一些具体示例。

import re, urllib.request, urllib.error, gzip, io, json, socket, sys

with open("index_to_name.json", "r", encoding=("utf-8")) as fp:
    index_to_name=json.load(fp)

with open("index_to_quality.json", "r", encoding=("utf-8")) as fp:
    index_to_quality=json.load(fp)

with open("index_to_name_no_the.json", "r", encoding=("utf-8")) as fp:
    index_to_name_no_the=json.load(fp)

with open("steamprofiler.json", "r", encoding=("utf-8")) as fp:
    steamprofiler=json.load(fp)

inventory=dict()
playerinventories=dict()
c=0

for steamid in steamprofiler:
    emptyitems=[]
    items=emptyitems
    try:
        url=urllib.request.urlopen("http://api.steampowered.com/IEconItems_440/GetPlayerItems/v0001/?key=XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX&steamid="+steamid+"&format=json")
        inv=json.loads(url.read().decode("utf-8"))
        url.close()
    except (urllib.error.HTTPError, urllib.error.URLError, socket.error) as e:
        c+=1
        print("URL/HTTP error, continuing")
        continue
    try:
        for r in inv["result"]["items"]:
            inventory[r["id"]]=r["quality"], r["defindex"]
    except KeyError:
        c+=1
        print(steamid, "didn't have an inventory")
        continue
    for key in inventory:
        try:
            if index_to_quality[str(inventory[key][0])]=="":
                items.append(
                    index_to_quality[str(inventory[key][0])]
                    +""+
                    index_to_name[str(inventory[key][1])]
                    )
            else:
                items.append(
                    index_to_quality[str(inventory[key][0])]
                    +" "+
                    index_to_name_no_the[str(inventory[key][1])]
                    )
        except KeyError:
            print("Key error, uppdate def_to_index")
            c+=1
            continue
playerinventories[int(steamid)]=items
items=emptyitems
c+=1
print(c, "inventories fetched")

在保留字典外观的同时，我真的不知道有任何其他方法可以做到这一点，这非常重要，因为我希望能够知道它是谁的库存。如果我对此有任何不清楚的地方，请直说，我会尽力解释

4

2 回答 2

4

我认为您的代码中有一些逻辑错误。例如，您将每个玩家的库存项目添加到inventory字典中，然后对其进行迭代以填充其他内容。

但是，您永远不会重置inventory字典，因此它会继续积累物品（因此第二个玩家似乎除了他们自己的物品之外还有第一个人的物品栏）。

您items稍后使用的字典也有类似的问题。您将它重置emptyitems为最初是一个空列表，但是因为 Python 中的赋值是通过引用进行的，所以这没有效果（items已经是与 相同的对象emptyitems）。

通过这两个修复程序，您可能有更好的机会不使用所有系统内存。

另一个杂项代码改进（可能与内存使用无关）：

在您的循环中inventory，您重复访问相同的两个值而不使用keyfor 任何内容。而不是for key in inventory尝试for value1, value2 in inventory.itervalues()（或者in inventory.values()如果您使用的是 Python 3）。然后使用value1代替inventory[key][0]和value2代替inventory[key][1]（或者更好的是，给他们更有意义的名字）。

编辑：这是循环的外观（我有点猜测以前在inventory[key][0]和中的两个值的名称inventory[key][1]）：

for quality, name in inventory.itervalues():
    try:
        if index_to_quality[str(quality)]=="":
            items.append(
                index_to_quality[str(quality)]
                +""+
                index_to_name[str(name)]
                )
        else:
            items.append(
                index_to_quality[str(quality)]
                +" "+
                index_to_name_no_the[str(name)]
                )
于 2012-12-07T10:54:51.967 回答
1

我相信这说明了您的代码存在问题：

>>> emptyitems=[]
>>> a=emptyitems
>>> a.append("hello")
>>> a.append("bar")
>>> a
['hello', 'bar']
>>> emptyitems
['hello', 'bar']

换句话说，您正在捕获对列表的引用，该emptyitems列表确实会变得非常大。这可能不是您的意思，我可以想象处理一个非常大的列表会变得非常耗费内存。

于 2012-12-07T10:52:00.127 回答