Python代码整洁之道勘误

代码清单1-10 以函数形式对字典排序

def get_sorted_dictionary(users):
"""Sort the nested dictionary"""
	if not isinstance(users, dict):
		 raise ValueError("Not a correct dictionary")
	if not len(users):
		 raise ValueError("Empty dictionary")
	users_by_name = sorted(users, key=get_user_name)
	return users_by_name

这里的users此时是list实例, 所以应该是isinstance(users, list)

代码清单1-12 以更具可读性的代码读取一个csv文件

import csv
with open('employee.txt', mode='r') as csv_file:
 	csv_reader = csv.DictReader(csv_file)
	line_count = 0
 	process_salary(csv_reader)

这里是一个employee.txt的example, 可以写入后改为csv文件
第一行第表头columns,其余行是表信息

name,country,age,salary
clarence,US,23,20k
bob,Canada,29,40k
ruby,Japan,28,16k

这里打开文件应该是employee.csv文件, 原著作者应该是创建并写入employee.txt然后修改后缀名为csv, 所以写代码的时候忽略了

7 比较类型时更偏向使用isintance()方法而不是type()

user_ages = {"Larry": 35, "Jon": 89, "Imli": 12}
type(user_ages) == dict:

这里最好是type(user_ages) is dict

1.3.2 不要使用复杂的列表推导

ages = [1, 34, 5, 7, 3, 57, 356]
old = [age for age in ages if age > 10 and age < 100 and age is
not None]

如果要校验，那么写在and逻辑运算符前面，注意短路

1.3.3 应该使用lambda吗
关于函数功能描述有误(原版和译版)
这里如果是找到列表中含有最小值的元素,应该这样

def min_val(data):
	"""Find minimum value from the data list."""
	return min(data, key=lambda x:len(x))

改为return min(data, key=lambda x: min(x))

1.3.6 为什么range函数在Python3中更好
译者翻译有误

原文是:

If you have worked with Python 2, you might have used xrange. In Python 3,
xrange has been renamed to range with some extra features. range is
similar to xrange and generate an iterable.

译文里range类似于xrange并生成一个迭代器

应该是range与xrange相似, 并且产生一个可迭代对象, 这也许会对初中级的python使用者产生困扰.

待续…