像你在v2ex里面对用户输入进行判断,原本可以不用麻烦的,直接用一个form库去处理会简化很多代码,可读性更好。
很抱歉的说,连续几天都在删你的代码,一个函数几页的代码,其实简化或者重构后,最多不会超过30行,极少数会超过。这几天删得特痛苦,唯一的收获就是你的v2ex很有重构的必要。
if site_use_topic_types is None:
s = '<select name="use_topic_types"><option value="1">Enabled</option><option value="0" selected="selected">Disabled</option></select>'
else:
if site_use_topic_types == '1':
s = '<select name="use_topic_types"><option value="1" selected="selected">Enabled</option><option value="0">Disabled</option></select>'
else:
s = '<select name="use_topic_types"><option value="1">Enabled</option><option value="0" selected="selected">Disabled</option></select>'
另外我觉得像这种代码,放在函数里面实在不妥,其实用form库,也就是几行代码的事情,连模板那边也可以用。
很抱歉的说,连续几天都在删你的代码,一个函数几页的代码,其实简化或者重构后,最多不会超过30行,极少数会超过。这几天删得特痛苦,唯一的收获就是你的v2ex很有重构的必要。
if site_use_topic_types is None:
s = '<select name="use_topic_types"><option value="1">Enabled</option><option value="0" selected="selected">Disabled</option></select>'
else:
if site_use_topic_types == '1':
s = '<select name="use_topic_types"><option value="1" selected="selected">Enabled</option><option value="0">Disabled</option></select>'
else:
s = '<select name="use_topic_types"><option value="1">Enabled</option><option value="0" selected="selected">Disabled</option></select>'
另外我觉得像这种代码,放在函数里面实在不妥,其实用form库,也就是几行代码的事情,连模板那边也可以用。