* src/Utils.py (roman): Proper implementation.

svn: r3869

ChangeLog

@@ -1,3 +1,6 @@
+2005-01-05 Douglas S. Blank  <dblank@brynmawr.edu>
+	* src/Utils.py (roman): Proper implementation.
+
 2005-01-05  Martin Hawlisch  <Martin.Hawlisch@gmx.de>
 	* src/EditSource.py: Patch to fix the list of References of a 
 	Source, because currently the Source for a Person and Sources 

src/Utils.py

@@ -419,24 +419,19 @@ def search_for(name):
 #
 #-------------------------------------------------------------------------
 def roman(num):
-    "Simple-and-dirty way to get roman numerals for small numbers."
+    """ Integer to Roman numeral converter for 0 < num < 4000 """
-
+    if type(num) != type(0):
-    romans = {
+        return "?"
-        1 : 'I', 2 : 'II', 3 : 'III', 4 : 'IV', 5 : 'V', 
+    if not 0 < num < 4000:
-        6 : 'VI', 7 : 'VII', 8 : 'VIII', 9 : 'IX', 10 : 'X', 
+        return "?"
-        11 : 'XI', 12 : 'XII', 13 : 'XIII', 14 : 'XIV', 15 : 'XV', 
+    vals = (1000, 900, 500, 400, 100,  90,  50,  40,  10,   9,   5,   4,   1)
-        16 : 'XVI', 17 : 'XVII', 18 : 'XVIII', 19 : 'XIX', 20 : 'XX', 
+    nums = ( 'M','CM', 'D','CD', 'C','XC', 'L','XL', 'X','IX', 'V','IV', 'I')
-        21 : 'XXI', 22 : 'XXII', 23 : 'XXIII', 24 : 'XXIV', 25 : 'XXV', 
+    retval = ""
-        26 : 'XXVI', 27 : 'XXVII', 28 : 'XXVIII', 29 : 'XXIX', 30 : 'XXX', 
+    for i in range(len(vals)):
-        31 : 'XXXI', 32 : 'XXXII', 33 : 'XXXIII', 34 : 'XXXIV', 35 : 'XXXV', 
+        amount  = int(num / vals[i])
-        36 : 'XXXVI', 37 : 'XXXVII', 38 : 'XXXVIII', 39 : 'XXXIX', 40 : 'XV' 
+        retval += nums[i] * amount
-    }
+        num    -= vals[i] * amount
-    
+    return retval
-    if isinstance(num,type(1)) and num<=41:
-        rnum = romans[num]
-        return rnum
-    else:
-        return '?'
 
 #-------------------------------------------------------------------------
 #