基本上,我想打开针对我的slackbot的以下任何命令的选项:
@torbot
@torbot [command]
@torbot [command] [arguments]
以下是我现在使用的内容,但它看起来很难看。根据我的经验,通常当某些事情看起来很丑陋时,这意味着可能有更直观的方式来做同样的事情。
class TorbotCommand(object):
def __init__(self, input):
self.__input = input
# TODO: There has to be a better way..
try:
self.__command_string = self.__input['text'].split(' ', 1)[1].strip().lower()
except:
self.__command_string = None
pass
try:
self.__command = self.__command_string.split(' ', 1)[0]
except:
self.__command = None
pass
try:
self.__text = self.__command_string.split(' ', 1)[1]
except:
self.__text = None
pass
def getCommand(self): return self.__command
def getText(self): return self.__text
def getInput(self): return self.__input
答案 0 :(得分:4)
只需拆分一次,然后测试列表是否足够长:
def __init__(self, input):
self.__input = input
parts = input['text'].split(None, 1)
self.__command = parts[0] if parts else None
self.__command_string = parts[1].lower() if len(parts) > 1 else None
self.__text = None
if self.__command_string:
self.__text = self.__command_string.partition(' ')[-1]
我使用None
作为str.split()
的第一个参数,让它在任意空格上分割;这也为你提供了一个自动条带。
如果您仍然需要处理异常,请不使用一揽子except:
块。仅针对索引表达式IndexError
捕获特定的异常,其中列表可能不够长。
另外,我建议不要使用__
双下划线名称。仅当您的目标是通过第三方代码对您的类进行子类化时才使用这些内容,其中重要的是内部实现不会被子类定义的任何内容实际意外影响。