从员工字典中打印员工对象信息

Question

我试图在不使用 for 循环的情况下从员工对象字典中打印员工对象信息。这是我到目前为止所做的：

employee_dict = {}

class Employee:

  def __init__(self, id, salary):
    self.id = id
    self.salary = salary
    self.employee_dictionary(self)

  def info(self):
    return "Employee ID:{} \nSalary:{}".format(self.id, self.salary)

  def employee_dictionary(self):
    employee_dict = {self.id: self}

emp = Employee(1, 10)
emp1 = Employee(2, 5)

employee = employee_dict[1]

print(employee.info())

有人能指出我正确的方向吗？我觉得我很亲近。这段代码给我的错误是：

Traceback (most recent call last):
  File "/home/rob/PycharmProjects/untitled4/sdfghsfdgjrtyrty.py", line 19, in <module>
    employee = employee_dict[1]
KeyError: 1

Answer 1

这里的根本问题是您没有正确使用字典。您不是将新员工添加到现有的全局 employee_dict，而是尝试用一个仅包含新员工的新 dict 替换该全局 dict：

def employee_dictionary(self):
  employee_dict = {self.id: self}

如果你解决了这个问题，你真正要问的问题甚至不会出现：

def employee_dictionary(self):
  employee_dict[self.id] = self

这将在字典中添加一个从self.id 到self 的新映射。因此，在你的两个 Employee 构造之后，ID 为 1 和 2，你最终会在字典中得到两个条目，分别是键 1 和 2，一切都会正常工作。

---

此外，您绝对应该至少考虑另外两个更改：

• 用类属性替换全局变量，也可能用类方法替换employee_dictionary方法，详见scharette's answer。
• 将方法从 employee_dictionary 重命名为 (a) 标记为私有（以 _ 开头的方法按照惯例是私有的，这意味着您的用户知道他们不应该调用它，除非他们有一个奇怪的用例） ，并且（b）更能反映它的作用。也许是_register 或_add_to_dict？

其中的第一个也（同样，在 scharette 的答案中详述）已经解决了您所询问的问题。 （但你仍然需要主要修复。）

---

但您可能还是想了解更直接的范围问题——以及当您无法让它消失时如何解决它。

在 Python 中，您在函数中分配的任何名称都是局部变量。如果函数中有 spam = … （或 with eggs as spam: 或其他类型的东西也算作赋值），则该函数中对 spam 的每个引用都指向该本地。因此，employee_dict = {self.id: self} 创建了一个名为 employee_dict 的局部变量，为其赋值，然后返回，此时所有局部变量都消失了。它碰巧与全局同名这一事实对 Python 来说没有任何意义（尽管对于像你这样的人类读者来说，这显然是令人困惑的）。

你使用的任何名称​​没有在任何地方分配给它是一个本地或封闭或全局或内置变量。当您执行employee_dict[self.id] 时，因为在任何地方都没有employee_dict = …，Python 会搜索本地、封闭、全局和内置范围以查找您所说的employee_dict 的含义，然后它会找到全局。

您可以强制 Python 将名称视为全局变量，即使您已分配给它，也可以在函数顶部使用 global 语句：

def employee_dictionary(self):
  global employee_dict
  employee_dict = {self.id: self}

即使名称已经是全局名称，添加 global 也是安全的。这通常是没有意义的——但是如果你不确定某个东西是否算作赋值（或者只是不确定你的代码的未来读者会确定......），并且你想明确这个变量是一个全局变量，你可以声明它：

def employee_dictionary(self):
  global employee_dict
  employee_dict[self.id] = self

Answer 2

这是一个范围问题。 employee_dict = {} 的行为不像你想象的那样。

您没有在正确的范围内分配值。因此，您实际上从未向其添加Employee。导致KeyError。

您正在寻找的可能是像这样使用 类变量，

class Employee:
    employee_dict = {}
    def __init__(self, id, salary):
        self.id = id
        self.salary = salary
        self.employee_dictonary(self)

    def info(self):
        return "Employee ID:{} \nSalary:{}".format(self.id, self.salary)

    @classmethod
    def employee_dictonary(cls,current_employee):
        cls.employee_dict[current_employee.id]=current_employee

emp = Employee(1, 10)
emp1 = Employee(2, 5)

print(Employee.employee_dict[1].info())

基本上，为了解释起见，您正在创建一个employee_dict，它将在 Employee 的所有实例之间共享。

另外，你必须记住的最后一件事是，如果有人像这样创建 2 名员工，

emp = Employee(1, 10)
emp2 = Employee(1, 100)

调用print(Employee.employee_dict[1].info())会输出

Employee ID:1 
Salary:100

这可能是您想要的，但仍然认为强调可以覆盖员工是相关的。

Answer 3

您应该将employee_dict 定义为全局：

def employee_dictonary(self):
  global employee_dict

如果没有全局关键字，您将分配给局部变量。

第二个问题是使用employee_dict = {self.id: self} 分配给employee_dict。这样，您就可以在每次调用时覆盖变量。那行应该是employee_dict[self.id] = self。

通过该更改，输出为：

Employee ID:1 
Salary:10

在大多数情况下强烈建议不要使用全局变量，请参阅Why are global variables evil?。