找回密码
 立即注册

QQ登录

只需一步,快速开始

查看: 2857|回复: 17

写 Python 时的 5 个坏习惯,你有没有?

  [复制链接]
  • 打卡等级:常驻代表
  • 打卡总天数:34
  • 打卡月天数:6
  • 打卡总奖励:9027
  • 最近打卡:2025-12-17 23:15:51

2823

主题

541

回帖

2万

积分

管理员

积分
22569
发表于 2019-4-16 21:05:08 | 显示全部楼层 |阅读模式
今天说说写代码时的几个坏习惯。
有的习惯会让 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}')

这样的话,看起来要明了一些。
上面提到的几点有些带有自己一定的偏见,不要求大家都接受,选择合理的使用就好。

本帖子中包含更多资源

您需要 登录 才可以下载或查看,没有账号?立即注册

×
工控课堂 www.gkket.com

0

主题

144

回帖

336

积分

注册会员

积分
336
发表于 2019-4-16 21:08:25 | 显示全部楼层
论坛有你更精彩!
工控课堂 www.gkket.com

0

主题

178

回帖

424

积分

注册会员

积分
424
发表于 2019-4-16 21:08:30 | 显示全部楼层
看到这帖子真是高兴!
工控课堂 www.gkket.com

0

主题

195

回帖

435

积分

注册会员

积分
435
发表于 2019-4-16 21:11:29 | 显示全部楼层
感恩无私的分享与奉献
工控课堂 www.gkket.com

0

主题

123

回帖

375

积分

注册会员

积分
375
发表于 2019-4-16 21:15:01 | 显示全部楼层
加油,加油,不要沉下去,我是最热贴
工控课堂 www.gkket.com

0

主题

179

回帖

443

积分

注册会员

积分
443
发表于 2019-4-16 21:17:21 | 显示全部楼层
无回帖,不论坛,这才是人道。
工控课堂 www.gkket.com

0

主题

133

回帖

387

积分

注册会员

积分
387
发表于 2019-5-16 13:07:24 | 显示全部楼层
加油,加油,不要沉下去,我是最热贴
工控课堂 www.gkket.com

0

主题

132

回帖

351

积分

注册会员

积分
351
发表于 2019-5-16 13:07:27 | 显示全部楼层
看到这帖子真是高兴!
工控课堂 www.gkket.com

0

主题

101

回帖

143

积分

新手上路

积分
143
发表于 2025-11-13 10:46:27 | 显示全部楼层
这评论区卧虎藏龙,个个都是人才!
工控课堂 www.gkket.com

0

主题

139

回帖

524

积分

中级会员

积分
524
发表于 2025-11-13 15:05:09 | 显示全部楼层
哈哈哈哈笑不活,楼主这脑洞绝了!
工控课堂 www.gkket.com
您需要登录后才可以回帖 登录 | 立即注册

本版积分规则

关闭

站长推荐上一条 /1 下一条

QQ|手机版|免责声明|本站介绍|工控课堂 ( 沪ICP备20008691号-1 )

GMT+8, 2025-12-23 05:45 , Processed in 0.113929 second(s), 26 queries .

Powered by Discuz! X3.5

© 2001-2025 Discuz! Team.

快速回复 返回顶部 返回列表