Question

一般来说，我非常尊重模块隐私（如果变量以下划线为前缀，我不使用它）。然而，我有一个角落案例，它似乎相当“安全”。

parser=argparse.ArgumentParser()
parser.add_argument('--point',help='enter a point (e.g. 2,3,4)')
parser.parse_args('--point=-2,5,6'.split())  #works
parser.parse_args('--point -2,5,6'.split())  #doesn't work

这可以通过以下方式解决：

parser=argparse.ArgumentParser()
parser.add_argument('--point',help='enter a point (e.g. 2,3,4)')
parser.parse_args('--point=-2,5,6'.split())  #works
parser._negative_number_matcher = re.compile(r'^-\d+|^-\d*\.\d+')
#Next line works too on Cpython 2.6, 
#got the idea from reading Cpython 3.2 source so it should work there too
parser.parse_args('--point -2,5,6'.split())

在这个例子中，这个与类的内部结构的玩法似乎没问题，因为：

*如果argparse更改了该变量（不太可能）或不同的python实现以不同方式执行，那么这似乎不太可能改变类的行为导致某些内容破坏。

*其他python实现可能会使用cpython的argparse，因为它是用纯python编写的（据我所知）。

*我的代码并不严格依赖于这个“功能”。（用户总是可以恢复为传递'--point = -2,5,6'）

我的问题是：我错过了什么吗？这真是个坏主意吗？我可能会破坏一些东西吗？如果是，为什么？

Answer 1

我认为这是一个坏主意，并不是因为它可能会破坏argparse内部的某些东西，而是因为它错了。 ^-\d+不是负数的有效正则表达式（因为它匹配-23foo），并且在代码中使用明显错误的东西不利于维护。想象一下你自己（或者甚至更糟的是其他人）两年后盯着这段代码并且对于^-\d+如何匹配负数而感到困惑！

尊重argparse的隐私和可用性

1 个答案: