0

我有一个 GUI,我要求用户输入一些值。当用户提交数据时,我会进行一些验证:

  1. 首先我检查用户是否为每个输入输入了一个值
  2. 然后我检查每个输入的值是否都是整数

尽量不重复自己,我想出了这个,但验证的第二部分看起来更像是一个 hack。是否有一种更 Pythonic 的方式来重写它,而不是像验证的第一部分那样拼写所有内容?

    errors = []
    # 1) check if values exist
    if not self.startInput.GetValue():
        errors.append("Please provide a start")
    if not self.stopInput.GetValue():
        errors.append("Please provide a stop")
    if not self.valueInput.GetValue():
        errors.append("Please provide a value")
    # 2) check if values are integers
    try:
        self.start = int(self.startInput.GetValue())
        self.stop = int(self.stopInput.GetValue())
        self.value = int(self.valueInput.GetValue())
    except ValueError as err:
        tb = traceback.format_exc()
        func = re.search('self\.(.*) =', tb).groups()[0]
        errors.append("Value for {0} needs to be an integer.".format(func)) 
    if errors:
        raise RuntimeError('\n'.join(errors))
4

3 回答 3

4

由于您正在检查整数而不是浮点数,因此您可以简单地执行以下操作:

if self.start.GetValue().strip().isdigit():
    pass

isdigit()返回False输入为空字符串和输入包含非数字的两种情况。

如果您想为不正确的分配发送特定错误:

startValue = self.start.GetValue().strip()

if not startValue: 
    errors.append("Please provide a start.")

if not startValue.isdigit():
    errors.append("Value of start must be an integer.")
于 2013-07-17T08:29:15.277 回答
1

我认为这try: ... except是完美的 Pythonic。如果需要,我会使用辅助函数而不是搜索get_int_of_name(name, value, error)返回 int 和更新错误的错误消息:

def get_int_of_name(name, value, error):
    try:
        res = int(value)
    except ValueError:
        error.append("...")
        return 0
    else:
        return res
于 2013-07-17T08:30:39.517 回答
1

如果您在名为的字典中有这些输入,inputs则可以执行以下操作:

errors = []
for inputname, inputval in self.inputs.items():
    if not inputval:
        errors.append("Please provide a {}".format(inputname))
    try:
        setattr(self, inputname, int(inputval.GetValue()))
    except:
        errors.append("Value for {0} needs to be an integer.".format(inputname)) 
if errors:
    raise RuntimeError('\n'.join(errors))
于 2013-07-17T08:32:36.887 回答