Python：在创建新对象之前检查对象是否已经存在

Question

请帮我看看我在下面缺少什么。我首先创建了我班级的三个对象并将它们添加到集合列表中。在创建任何其他对象之前，我想 检查 以确保该人不存在于列表中。如果该人已经存在，则不应再次创建该人。我希望通过if prompt_fname == person.fname and prompt_lname == person.lname: 来实现这个检查。显然，我没有正确执行它，因为程序仍然运行并创建了列表中已经存在的同一个人。它两次创造了这个人。我如何修改才能捕捉到这一点，以便不再创建列表中已经存在的人。此外，不应在循环的每次迭代中一次又一次地创建任何新人。我是编程新手，所以请不要在答案中遗漏太多细节。非常感谢。

class Person(object):

    personslist = []
    '''Creates a person object'''
    def __init__(self, firstname, lastname):
        self.lname = lastname.title()
        self.fname = firstname.title()
        Person.personslist.append(self)

    def __str__(self):
        return "{0} {1}".format(self.fname, self.lname)

    def __repr__(self):
        return "{0} {1}".format(self.fname, self.lname)


Person("Adamu", "Emeka")
Person("Femi", "Ojukwu")
Person("Wole", "Jonathan")


prompt_fname = "Adamu"
prompt_lname = "Emeka"

print(Person.personslist)

for person in Person.personslist:
    if prompt_fname == person.fname and prompt_lname == person.lname:
        pass
    else:
        Person(prompt_fname, prompt_lname)

print(Person.personslist)

产量

[Adamu Emeka, Femi Ojukwu, Wole Jonathan]
[Adamu Emeka, Femi Ojukwu, Wole Jonathan, Adamu Emeka, Adamu Emeka]

使用 Python 3.4.1

Answer 1

您的循环正在检查每个条目，如果该特定条目不相等，它会创建另一个实例。但是如果那个特定的条目 是 相等，它不会停止，它只是继续到下一个不相等的条目。这就是为什么你在最后得到了两个额外的条目。

您可以通过保留标志来解决此问题：

found = False
for person in Person.personslist:
    if prompt_fname == person.fname and prompt_lname == person.lname:
        found = True
        break
if not found:
    Person(prompt_fname, prompt_lname)

但是，有一种更好的方法可以做到这一点：您的方法效率非常低，因为它每次都需要进行线性扫描。相反，保留一个以全名为键的对象字典：

class Person(object):

    persons_dict = {}
    '''Creates a person object'''
    def __init__(self, firstname, lastname):
        self.lname = lastname.title()
        self.fname = firstname.title()
        fullname = "%s %s" % (self.fname, self.lname)
        Person.persons_dict[fullname] = self

现在您可以直接签到：

if "%s %s" % (prompt_fname, prompt_lname) not in Person.persons_dict:

Answer 2

每次添加名称前检查完整列表：

name = prompt_fname + prompt_lname

if not any(person.fname + person.lname == name for person in Person.personslist):
    Person(prompt_fname, prompt_lname)

Answer 3

这段代码没有做它应该做的事情：

for person in Person.personslist:
    if prompt_fname == person.fname and prompt_lname == person.lname:
        pass
    else:
        Person(prompt_fname, prompt_lname)

使用for person in Person.personslist:，它会遍历您已经创建的三个 Person() 对象。第一个是“Adamu Emeka”，因此名称相同，“if”语句达到“pass”。但是，personslist 中的下一项名为“Femi Ojukwu”，而if 中的名称不相等，因此它到达else 子句并创建一个新对象。第三个名字也是如此。这就是为什么你有两个额外的 Adamu Emeka 副本。

查看 Daniel Roseman 的替代解决方案的答案

Answer 4

尝试：

if prompt_fname.title() == person.fname and prompt_lname.title() == person.lname:

因为你在Person的init中使用了title()方法

Answer 5

在您的代码中，一旦您找到一个不是 Adamu Emeka 的人，您就会将另一个 Adamu Emeka 添加到列表中。这就是添加两个 Adamu Emeka 的原因。请尝试以下操作：

alreadyExists = False;
for person in Person.personslist:
    if prompt_fname == person.fname and prompt_lname == person.lname:
        alreadyExists = True;
        break;
if not alreadyExists:
    Person(prompt_fname, prompt_lname)

更好的方法可能是使用set，这样您就不必执行线性搜索。要使设置正常工作，您必须implement __hash__ and __eq__ methods。