在 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
我正在尝试过滤一个由多个词典组成的列表,以查找无用的条目和不理想的字段。如果 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