Question

我对编程，Python和面向对象编程一般都很陌生。对于学校作业，我必须编写一个定义“多项式”类的代码，然后找到多项式的根。由于代码没有按预期运行，我开始分析它并意识到一个全局变量，即表示多项式的系数列表，即多项式类的“输入”，正在被修改。问题是我似乎无法弄清楚是什么（部分代码）导致了这种操作。下面是我使用的代码（的相关部分）：

#set input list
il=[1,1,-1]

#define B!/(O-1)!, note that O is an "oh":
def pr(O,B):
  p=1
  for i in range(O,B+1):
    p*=i
  return p

#polynomial
class pol:

  #init:
  def __init__(self,L=[0]):
    self.l=L
    self.d=len(L)
    self.n=self.d-1

  #evaluate:
  def ev(self,X=0):
    if X==0:
      return self.l[0]
    else:
      s=self.l[0]
      for i in range(1,self.d):
        s+=self.l[i]*X**i
      return s

  #N-th derivative:
  def der(self,N=1):
    if self.n < N:
      return pol([0])
    else:
      lwork=self.l
      for i in range(N,self.d):
        lwork[i]*=pr(i-N+1,i)
      return pol(lwork[N:])

#define specific polynomial and take derivative:
#---here I have put some extra prints to make clear what the problem is---
f=pol(il)
print(il)
fd=f.der()
print(il)
fd2=f.der(2)
print(il)

现在这应该评估（至少它在我的机器上这样做）

[1,1,-1]
[1,1,-2]
[1,1,-4]

虽然我希望它只是第一个列表三次，因为在方法“der”的定义中，我不操纵输入列表，或者在我看来。

有人可以解释发生了什么吗？我错过了（简单）细节，还是我在这里滥用（某些方面）课程？

要执行我的代码，我使用在Python 3.5.2上运行的在线编译器（repl.it）。

Answer 1

一：永远不要使用mutable default argument for a function/method of any kind。

二：从一个名称分配到另一个名称，如：

lwork=self.l

只是别名，lwork成为与list相同的self.l的引用。如果你不想改变self.l，（浅）复制它，例如：对于像list这样的简单序列：

lwork = self.l[:]

这会使新的list与self.l具有相同的值。由于这些值都是不可变的，所以浅拷贝就足够了;如果值可能是可变的，则您希望使用copy模块copy.deepcopy来确保副本与原始list无关。

同样，如果您不想在传递给list初始值设定项的pol与实例上存储的list之间保持平局，请复制一份，例如：

self.l = list(L)

在这种情况下，我使用了list(L)而不是L[:]，因为它从任何输入可迭代类型中获得了保证类型（list）。这实际上使得可变默认参数安全（因为你总是浅拷贝它，所以没有人真正改变它），但即便如此，可变默认值通常被认为是代码气味，因此最好避免它们。 / p>

修复整个__init__方法，您最终得到：

# Mostly to avoid code smell, use immutable default (list constructor converts to list)
def __init__(self, L=(0,)):
    self.l = list(L)      # Create list from arbitrary input iterable
    self.d = len(self.l)  # Get length of now guaranteed list (so iterator inputs work)
    self.n = self.d-1

Python 3.5.2：一个类意外地更改了一个全局变量

1 个答案: