写Python时的5个坏习惯，你有几条？

[[250473]]

很多文章都有介绍怎么写好 Python，我今天呢，相反，说说写代码时的几个坏习惯。有的习惯会让 Bug 变得隐蔽难以追踪，当然，也有的并没有错误，只是个人觉得不够优雅。

注意：示例代码在 Python 3.6 环境下编写

1 用列表作函数的默认参数

看下面这个例子 

def func(a, b=[]):  
      b.append(a)  
      print(f'a: {a}')  
      print(f'b: {b}')  
  func(1)  
  func(2) 

正常我们期望的结果应该是这样的   

a: 1  
   b: [1]  
   a: 2  
   b: [2] 

但当我们执行代码后，只会得到这样的结果   

a: 1  
   b: [1]  
   a: 2  
   b: [1, 2] 

与预期不一致。为什么呢？因为 Python 列表是可变对象，而且函数传参又是传的引用，所以当第二次调用 func 方法前，b 中已经有了元素 1，调用后 b 最终有两个元素 1 和 2。

示例中 func 方法比较简单，当发现问题的时候简单看下就能找到根源。但是，如果是在一个比较复杂的方法里面，你有可能会粗心的忽略这一点，从而会碰到一些莫名其妙的问题。

所以，当我们要为函数设置默认参数的时候，不要使用可变对象。

上面的代码改成这样就 OK 了 

def func(a, b=None):  
     if b is None:  
         b = []  
     b.append(a)  
     print(f'a: {a}')  
     print(f'b: {b}') 

执行后得到预期结果   

a: 1  
   b: [1]  
   a: 2  
   b: [2] 

2 文件操作

很多刚接触 Python 的伙伴做文件操作的时候很容易写类似的代码   

file = open('file_name')  
    try:  
        for line in file:  
            print(line)  
    finally:  
        file.close() 

这没有问题，不过文件资源我们没有必要手动去维护，像关闭这样的操作交给上下文管理器做就好。   

with open('file_name') as file:  
       for line in file:  
           print(line) 

这样看起来不是清爽很多。

3 捕获所有异常   

try:  
       pass  # 做一些操作  
   except Exception as e:  
       print(f'Exception {e}') 

就像上面一样，有时我们为了能够快速的完成功能，很容易不管三七二十一，就捕获 Exception 异常。这可能会捕捉到键盘中断（KeyboardInterrupt）（CTRL + C）或断言错误（AsstionError）等异常。捕获不确定的异常，有时也会让我们的程序出现莫名其妙的问题，我们应该避免这样做。

准确的做法是根据上下文捕获 ValueError 、AttributeError 、TypeError 等比较具体的异常，然后做适当的错误处理，比如打印日志等。

4 忽略 Python 的 for...else 语法

开发中我们很容易碰到类似的需求，在一个列表中，确定某个特定的元素是否存在。比如，下面的代码便是确定列表中有没有奇数存在 

numbers = [1, 2, 3, 4, 5]  
  is_odd_exist = False  
  for n in numbers:  
      if n % 2 == 1:  
          is_odd_exist = True  
          break  
  if is_odd_exist:  
      print('Odd exist')  
  else:  
      print('Odd not exist') 

这里，我们使用了一个标识 is_odd_exist，默认为 False。当找到奇数时，将其置为 True，然后跳出循环。这样写并没有问题，但是我们可以换种方式   

numbers = [1, 2, 3, 4, 5]  
   for n in numbers:  
       if n % 2 == 1:  
           print('Odd exist')  
           break  
   else:  
       print('Odd not exist') 

先介绍下 Python 的 for...else 语法，当 for 循环是正常结束时（即不是通过 break 跳出结束的），会执行 else 中的语句。

这里，我们使用了相对于其他语言如 C、PHP 等不同的一种方式，完成了相同的功能，看起来代码也简洁了不少。

5 使用键遍历字典

初学 Python 的伙伴，可能容易写出这样的代码   

member = {'name': 'xiaoming',  
              'age': 18,  
              'mobile': '18312341234'}  
    for key in member:  
        print(f'{key}: {member[key]}') 

同样，这也是没有问题的，但看起来并不直观。字典遍历的时候，其实可以直接取出键值信息，像这样 

member = {'name': 'xiaoming',  
            'age': 18,  
            'mobile': '18312341234'}  
  for key, val in member.items():  
      print(f'{key}: {val}') 

这样的话，看起来要明了一些。

上面提到的几点有些带有自己一定的偏见，不要求大家都接受，选择合理的使用就好。