在 Python 中简化列表过滤器

Simplifying a list filter in Python

我正在尝试过滤一个由多个词典组成的列表,以查找无用的条目和不理想的字段。如果 Mem['visibility'] == "redacted" 条目无用,则应从列表中删除。当任何字段为空时,条目是不理想的,这些应该用 0 代替,或者如果 Mem['~whatever~'] 是一个列表,在这种情况下,列表应该转换为一个字符串,其中包含所有对象那个清单。为此,我编写了以下代码:

class Filter:

    @staticmethod
    def members(memberlist):
        for Mem in memberlist:
            for Item in Mem:
                if not Mem[Item]:
                    Mem[Item] = 0
                if (type(Mem[Item]) is list):
                    Mem[Item] = ', '.join(Mem[Item])
            if(Mem['visibility'] == "redacted"):
                memberlist.remove(Mem)
        return(memberlist)

成员列表示例:

[
    {'roles': [],
     'rank': 'No SCB account',
     'type': 'main',
     'stars': 2,
     'visibility': 'visible',
     'sid': 'imperium',
     'handle': 'freakyeagle'
    },
    {'roles': [],
     'rank': 'Fleet Member',
     'type': 'main',
     'stars': 1,
     'visibility': 'visible',
     'sid': 'imperium',
     'handle': 'cadimus'},
    {'roles': [],
     'rank': 'Fleet Member',
     'type': 'main',
     'stars': 1,
     'visibility': 'visible',
     'sid': 'imperium',
     'handle': 'belleal'}
]

我怎样才能使这个尽可能simple/efficient?

此外,出于某种原因,当我从上面代码中的列表中删除无用的 Mem 时,它会中断我在数据库中的插入,这发生在以下函数中(上面函数中的成员列表等同于此函数中的成员):

def member(conn, cursor, member):
    try:
        if type(member) is not list:
            # TODO: Eventually implement single member insert here.
            print(member)
        cursor.executemany("INSERT INTO `Citizens` (`Handle`,`Org`,`Role`, "
                           "`Rank`,`Visibility`,`Stars`,`Type`) VALUES "
                           "(%(handle)s,%(sid)s,%(roles)s,%(rank)s,"
                           "%(visibility)s,%(stars)s,%(type)s);", member)
        conn.commit()

有谁知道为什么会这样?

您不应该在迭代列表时从列表中删除项目 - 如果在删除项目后跳过项目,您会遇到意外行为。

相反,复制列表以进行迭代:

for mem in memberlist[:]:
    ...

或者将您想要的结果保存到另一个列表,然后 return 改为:

keep = []
for mem in memberlist:
    if ...:
       keep.append(mem)
return keep