您最好使用已经存在的existing library,而不是试图重新发明轮子:
pip install vobject
然后在python中
>>> import vobject
>>> s = """\
... BEGIN:VCARD
... VERSION:2.1
... N:MEO;Apoio;;;
... FN:Apoio MEO
... TEL;CELL;PREF:0123456789
... TEL;CELL:0123456768
... END:VCARD
... BEGIN:VCARD
... VERSION:2.1
... N:estrangeiro;Apoio MEO;no;;
... FN:Apoio MEO no estrangeiro
... TEL;CELL;PREF:+0123456789
... END:VCARD """
>>> vcard = vobject.readOne(s)
>>> vcard.prettyPrint()
VCARD
VERSION: 2.1
TEL: 1696
TEL: 162 00
FN: Apoio MEO
N: Apoio MEO
你就完成了!
所以如果你想用它制作一本字典,你需要做的就是:
>>> {vcard.contents['fn'][0].value: [tel.value for tel in vcard.contents['tel']] }
{'Apoio MEO': ['1696', '162 00']}
这样你就可以将所有这些变成一个函数:
def parse_vcard(path):
with open(path, 'r') as f:
vcard = vobject.readOne(f.read())
return {vcard.contents['fn'][0].value: [tel.value for tel in vcard.contents['tel']] }
从那里,您可以改进代码以在单个 vobject 文件中处理多个 vcards,并使用更多手机更新 dict。
N.B.:我留给你一个练习,将上面的代码从读取文件中的一个且仅一个 vcard 更改为可以读取多个 vcard 的代码。提示:read the documentation of vobject.
N.B.:我正在使用你的数据,而且我认为无论你写什么,它都是没有意义的。但有疑问,我修改了电话号码。
只是为了好玩,让我们看看你的代码。首先是缩进问题,但我认为这是因为复制/粘贴错误☺。
① import re
② file = open('Contacts.txt', 'r')
③ contacts = dict()
④ for line in file:
⑤ name = re.findall('FN:(.*)', line)
⑥ nm = ''.join(name)
⑦ if len(nm) == 0:
⑧ continue
⑨ contacts[nm] = contacts.get(nm)
⑩ print(contacts)
首先,第②行有两个问题。您正在使用 open() 打开一个文件,但您没有关闭该文件。如果你调用这个函数来打开十亿个文件,你会饿死你系统的可用文件描述符,因为你没有关闭这些文件。作为一个好习惯,您应该始终使用 with 构造:
with open('...', '...') as f:
… your code here …
它会为您处理 fd,并更好地显示您可以在哪里使用已打开的文件。
第二个问题是你调用了你的变量file,它隐藏了file 类型。希望file 类型很少使用,但这是一个坏习惯,因为您可能有一天不理解发生的错误,因为您使用变量遮蔽了类型。千万别用,总有一天会省事的。
第 ⑤ 行和第 ⑥ 行,您在每行上应用 re.findall 正则表达式。您最好使用re.match(),因为您已经遍历了每一行,并且该行中不会有FN: something。这将使您避免不必要的''.join(name),但不要将正则表达式用于这么简单的事情,您最好使用str.split():
if 'FN:' in line:
name = line.split(':')[-1]
第⑦行不仅是多余的——如果你使用上面的if,而且实际上是错误的。因为这样您将跳过其中没有 FN: 的所有行,这意味着您永远不会提取电话号码,只提取名称。
最后第⑧行完全没有意义。基本上,您所做的相当于:
if nm in contacts.keys():
contacts[nm] = contacts[nm]
else:
contacts[nm] = None
总而言之,在您的代码中,您所做的只是提取姓名,而您甚至不用担心电话号码。所以当你说:
有了这个,我得到了一本有名字的字典,但对于数字,我得到了 None
这毫无意义,因为您实际上并没有尝试提取电话号码。
我可以用 re 做到这一点吗?用相同的re.findall 表达式提取姓名和号码?
是的,您可以使用看起来像的东西(未经测试的正则表达式很可能无法正常工作),覆盖整个文件,或者至少对于每个 vcard:
FN:(?P<name>[^\n]*).*TEL[^:]*:(?P<phone>[^\n])
但是,当你有一个完美适合你的库时,何必麻烦呢!