1

一般来说,我非常尊重模块的隐私(如果变量带有下划线前缀,我不使用它)。然而,我有一个角落案例,它看起来相当“安全”。

这是演示(我以前的问题

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')

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

4

1 回答 1

3

我认为这是一个坏主意,并不是因为它可能会破坏 argparse 内部的某些东西,而是因为它是错误的。^-\d+不是负数的有效正则表达式(因为它匹配-23foo),并且代码中有明显错误的东西不利于维护。想象一下你自己(或者更糟的其他人)两年后盯着这段代码,^-\d+不知道到底应该如何匹配负数!

于 2012-06-14T14:02:43.723 回答