如何改进这个number2words脚本

Question

import sys

words = {
    1 : 'one',
    2 : 'two',
    3 : 'three',
    4 : 'four',
    5 : 'five',
    6 : 'six',
    7 : 'seven',
    8 : 'eight',
    9 : 'nine',
    10 : 'ten',
    11 : 'eleven',
    12 : 'twelve',
    13 : 'thirteen',
    14 : 'fourteen',
    15 : 'fifteen',
    16 : 'sixteen',
    17 : 'seventeen',
    18 : 'eighteen',
    19 : 'nineteen'
}

tens = [
    '',
    'twenty',
    'thirty',
    'forty',
    'fifty',
    'sixty',
    'seventy',
    'eighty',
    'ninety',
]

placeholders = [
    '',
    'thousand',
    'million',
    'billion',
    'trillion',
    'quadrillion'
]

# segMag = segment magnitude (starting at 1)
def convertTrio(number):
    return ' '.join([words[int(number[0])],  'hundred',  convertDuo(number[1:3])]) # convertDuo(number[1:3])


def convertDuo(number):
    #if teens or less
    if int(number[0]) == 1:
        return words[int(number)]
    #twenty-five
    else:
        return tens[int(number[0]) - 1] + '-' + words[int(number[1])]


if __name__ == "__main__":

    string = []
    numeralSegments = []
    numeral = sys.argv[1]

    if int(numeral) < 100:
        print convertDuo(numeral)
    else:

        # split number into lists, grouped in threes
        for i in range (0, len(numeral), 3):
            numeralSegments.append(numeral[i:i+3])

        numeralSegments.reverse()

        # for every segment, convert to trio word and append thousand, million, etc depending on magnitude
        for i in range (len(numeralSegments)):
            string.append(convertTrio(numeralSegments[i]) + ' ' + placeholders[i])

        # reverse the list of strings before concatenating to commas
        string.reverse()        
        print ', '.join(string)

警告：我完全是个蟒蛇新手。我知道有很多更有效的方法来做事情。如果能给他们指点的话我会很感激的。

编辑：该代码目前只适用于数字计数为三倍的数字。我也希望有一个优雅的方法来解决这个问题。谢谢。

Answer 1

你不能把数字分成从左到右的“段”。range(0,len(),3)不会有好结果的。您必须编写相同的算法来插入数字分隔符。你从右边开始，从数字段开始。

还剩下什么(左边，明白吗？)将是1，2或3位数。convertTrio和convertDuo分别处理3位和2位数字。在某个地方有一个转换一位数的函数(看不见)。

如果不是作业，那么，这里有一个适当的数字聚类算法

def segment( n ):
   segList= []
   while len(n) > 3:
       segList.insert( 0, n[-3:] )
       n= n[:-3]
   segList.insert( 0, n )
   return segList

编辑

要更多地使用Pythonic，请将其打包为一个整洁、可重用的模块。if __name__ == "__main__"内部的东西做两件事，应该分开。

您的命令行解析(与sys.argv相关的任何内容都是一回事。实际的“转换一个数字”函数完全是另外一回事。你想看上去更像这样。

if __name__ == "__main__":
    import sys
    for number in sys.argv[1:]:
        print number2string( number )

然后，您的number2string函数成为该模块的一个容易重用的部分。

Answer 2

我想到了两个改进：

• 40拼写为“40”，而不是“40”。
• 您的程序需要单元测试。

请看一下Python和博士考试模块。

Answer 3

而不是切片数字，使用模块化算法分离单位。此函数将使用给定的数据结构转换小于100的数字。

def convert(n):
    q, r = divmod(n, 10)
    if q < 2:
        return words[n]
    result = tens[q-1] # offset because tens is missing first null value
    if r:
        result += '-' + words[r]
    return result

然后使用递归转换来支持更大的数字，例如，从divmod(n，100)开始，等等。