我是python的新手,我已经阅读了很多关于如何正确编码的教程。不断弹出的一件事是永远不要多次写同一行代码。我不确定长期的elif论点是否算得上,但对我而言,它看起来就像是糟糕的代码。
for exaple:
class answers(object):
def __init__(self):
self.x = 'hello world'
def knight(self):
print('Neee!')
def bunny(self):
print('Rawwww!')
def pesant(self):
print('Witch!')
def dingo(self):
print('Bad, wicked, naughty Zoot!')
foo = answers()
egg = input("Sounds:")
if egg == "knight":
foo.knight()
elif egg == 'bunny':
foo.bunny()
elif egg == 'pesant':
foo.pesant()
elif egg == 'dingo':
foo.dingo()
else:
print("I don't know?")
虽然有效,但我认为以下代码看起来更清晰。
class answers(object):
def __init__(self):
self.x = 'hello world'
def knight(self):
print('Neee!')
def bunny(self):
print('Rawwww!')
def pesant(self):
print('Witch!')
def dingo(self):
print('Bad, wicked, naughty Zoot!')
foo = answers()
responce = {'knight': 'foo.knight()', 'bunny': 'foo.bunny()', 'pesant': 'foo.pesant()', 'dingo': 'foo.dingo()'}
while True:
try:
egg = input('sounds:')
exec(responce[egg])
except KeyError:
print("I don't know")
这两行代码都做同样的事情,我使用的是真的重要还是比另一行更好?
注意,我知道通常不应该使用exec(),但我找不到另一种方法来为字符串分配函数。
答案 0 :(得分:2)
如果跳过()
和参数
responce = {
'knight': foo.knight,
'bunny': foo.bunny,
'pesant': foo.pesant,
'dingo': foo.dingo,
}
然后你可以使用()
(带有预期参数)
responce[egg]()
#responce[egg](arg1, arg2, ...) # if function require arguments
完整代码
class Answers(object): # CamelCase name for class - see PEP8 document
def __init__(self):
self.x = 'hello world'
def knight(self):
print('Neee!')
def bunny(self):
print('Rawwww!')
def pesant(self):
print('Witch!')
def dingo(self):
print('Bad, wicked, naughty Zoot!')
foo = Answers()
responce = {
'knight': foo.knight,
'bunny': foo.bunny,
'pesant': foo.pesant,
'dingo': foo.dingo,
}
while True:
try:
egg = input('sounds:')
responce[egg]() # call function
except KeyError:
print("I don't know")
BTW:这样你甚至可以使用函数名作为另一个函数的参数。
在Tkinter中用于为Button分配功能
Button( ..., text="knight", command=foo.knight)
或将事件分配给事件
bind('<Button-1>', foo.knight)
如果需要使用参数赋值函数,则可以使用lambda
函数。
Python3的版本:
responce = {
'knight': lambda:print('Neee!'),
'bunny': lambda:print('Rawwww!'),
'pesant': lambda:print('Witch!'),
'dingo': lambda:print('Bad, wicked, naughty Zoot!'),
}
Python2的版本:
Python2中的 print
不是函数,因此lambda
不能与print
一起使用 - 所以你必须创建函数。
def show(text):
print text
responce = {
'knight': lambda:show('Neee!'),
'bunny': lambda:show('Rawwww!'),
'pesant': lambda:show('Witch!'),
'dingo': lambda:show('Bad, wicked, naughty Zoot!'),
}
编辑:但我会在字典中没有函数的情况下执行此操作:)
# --- classes ---
class Answers(object):
def __init__(self):
# TODO: read it from file CSV or JSON
#
# import json
#
# with open("data.json") as f:
# self.data = json.load(f)
self.data = {
'knight': 'Neee!',
'bunny': 'Rawwww!',
'pesant': 'Witch!',
'dingo': 'Bad, wicked, naughty Zoot!',
}
def response(self, text):
try:
return self.data[text]
except KeyError:
return "I don't know"
# --- functions ---
# empty
# --- main ---
foo = Answers()
while True:
egg = input('sounds: ').lower()
if egg == 'exit':
break
print(foo.response(egg))
# ---
print("Good Bye!")
答案 1 :(得分:0)
furas'伟大的评论已经很好地涵盖了如何,但至于为什么这样做,有一个比“看起来更干净”更好的理由:
与case / switch语句或if / elif / else语句的长链相比,函数表往往具有更少的代码 - 无论是编写的内容还是编译器或解释器生成的代码。
他们也跑得更快。
所以你对“看起来更干净”的直觉是非常正确的,事实上这是做事的方式,不仅仅是在Python中,而是在任何编程语言中。