写Python时的5个坏习惯，你有几条？
Yasin 2018-11-26
很多文章都有介绍怎么写好 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}') 