[Python] Primo programmino

[andbin]

Ho realizzato il mio primo programmino in Python. Serve per convertire un numero dalla numerazione decimale alla numerazione romana.

#! /usr/bin/env python

class BadNumberException(Exception):
    def __init__ (self, value):
        self.value = value
    def __str__ (self):
        return repr (self.value)

def format_roman_digit (digit, roman1, roman5, roman10):
    if digit >= 1 and digit <= 3:
        return roman1 * digit
    elif digit == 4:
        return roman1 + roman5
    elif digit >= 5 and digit <= 8:
        return roman5 + roman1 * (digit-5)
    elif digit == 9:
        return roman1 + roman10
    else:
        return ""

def roman_number (num):
    if num < 1 or num > 3999:
        raise BadNumberException ("bad number " + str(num))

    roman = format_roman_digit (num/1000, "M", "", "")
    num %= 1000
    roman += format_roman_digit (num/100, "C", "D", "M")
    num %= 100
    roman += format_roman_digit (num/10, "X", "L", "C")
    num %= 10
    roman += format_roman_digit (num, "I", "V", "X")

    return roman


if __name__ == '__main__':
    import sys

    if len (sys.argv) == 2:
        strnum = sys.argv[1]
        try:
            r = roman_number (int (strnum))
            print "roman =", r
        except ValueError:
            print "invalid input: " + strnum
        except BadNumberException, err:
            print err

Il programma funziona bene. Vi sembra ben scritto?? C'è qualcosa che si può migliorare?
Grazie.

[andbin]

Originariamente inviato da andbin

C'è qualcosa che si può migliorare?

Mi rispondo da solo .... stavo leggendo adesso il libro Dive Into Python e c'è proprio, tra gli esempi, la conversione in numeri romani .... ed usa un algoritmo molto semplice "a sottrazione" .... il mio quindi è 'na schifezza

[Stiwy.NET]

Originariamente inviato da andbin

Mi rispondo da solo .... stavo leggendo adesso il libro Dive Into Python e c'è proprio, tra gli esempi, la conversione in numeri romani .... ed usa un algoritmo molto semplice "a sottrazione" .... il mio quindi è 'na schifezza

Non abbatterti...spesso si fanno/scrivono cose inutili e ripetitive, ma almeno servono a capire come funzionano... eppoi farsi le cose aiuta ad imparare e tenere in esercizio la mente... io mi sono fatto un ftp-treeview in vb.net... è inutile... però è stato carino farlo... ora soddisfa quasi tutte le mie necessità...