More release preparation. Docstrings and consistency work.
[python-collate.git] / collate / strings.py
index 1dffdac..8d6af99 100644 (file)
@@ -1,3 +1,7 @@
+"""String utility functions for collation."""
+
+__all__ = ["sortemes", "numeric", "normalize_number"]
+
 import unicodedata
 
 CONTINUE_ON = frozenset([
@@ -13,17 +17,29 @@ UNKNOWN, LETTER, NUMBER = range(3)
 BREAKER = u"\u2029" # Paragraph break character
 INFINITY = float('inf')
 
+KEEP_IN_NUMBERS = u"'.,"
+ALLOWED_IN_NUMBERS = KEEP_IN_NUMBERS + u"_"
+
+def stripends(word):
+    """Strip punctuation and symbols from the ends of a string."""
+    while word and unicodedata.category(word[0])[0] in "PS":
+        word = word[1:]
+    while word and unicodedata.category(word[-1])[0] in "PS":
+        word = word[:-1]
+    return word
+
 def sortemes(string, key=lambda s: s):
     """Generate a list of sortemes for the string.
 
     A sorteme, by analogy with grapheme/morpheme/etc. is an atom of
     sort information. This is larger than a word boundry but smaller
     than a sentence boundry; roughly, a sorteme boundry occurs between
-    letters and numbers, between numbers and numbrs if 'too much'
+    letters and numbers, between numbers and numbers if 'too much'
     punctuation exists in between, between lines.
 
     There is no formal specification for sortemes; the goal of this
     function is to provide good output for Collator.sortemekey.
+
     """
 
     words = []
@@ -34,23 +50,17 @@ def sortemes(string, key=lambda s: s):
     string = unicode(string)
     categories = map(unicodedata.category, string)
     previous = UNKNOWN
-    types = []
-
-    def stripends(word):
-        while word and unicodedata.category(word[0])[0] in "PS":
-            word = word[1:]
-        while word and unicodedata.category(word[-1])[0] in "PS":
-            word = word[:-1]
-        return word
 
     def aletters(letters):
+        """Add a group of letters to the word list."""
         words.append((INFINITY, stripends(letters)))
     def adigits(digits):
+        """Add a group of digits to the word list."""
         words.append((numeric(digits), u''))
 
     # TODO(jfw): This kind of evolved over time, there's probably a much
     # faster / more concise way to express it now.
-    for i, (c, category) in enumerate(zip(string, categories)):
+    for i, (uchar, category) in enumerate(zip(string, categories)):
 
         if letters and previous == LETTER and words:
             word = stripends(words.pop()[1].strip()) + BREAKER
@@ -60,7 +70,7 @@ def sortemes(string, key=lambda s: s):
         # Split at the first letter following a number or
         # non-continuing character.
         if category[0] == "L":
-            letters.append(c)
+            letters.append(uchar)
             if digits:
                 adigits(u"".join(digits).strip())
                 digits = []
@@ -69,14 +79,14 @@ def sortemes(string, key=lambda s: s):
         # Split at the first number following a non-number or
         # non-continuing character.
         elif category[0] == "N":
-            digits.append(c)
+            digits.append(uchar)
             if letters:
                 aletters(u"".join(letters))
                 letters = []
                 previous = LETTER
 
         # Only certain punctuation allowed in numbers.
-        elif digits and c not in "',._":
+        elif digits and uchar not in ALLOWED_IN_NUMBERS:
             adigits(u"".join(digits))
             digits = []
             previous = NUMBER
@@ -106,9 +116,9 @@ def sortemes(string, key=lambda s: s):
 
         else:
             if digits:
-                digits.append(c)
+                digits.append(uchar)
             elif letters:
-                letters.append(c)
+                letters.append(uchar)
 
     if letters and previous == LETTER and words:
         word = stripends(words.pop()[1].strip()) + BREAKER
@@ -122,13 +132,21 @@ def sortemes(string, key=lambda s: s):
 
     return [(i, key(w) if w else u'') for i, w in words]
 
-def numeric(orig, invalid=float('inf')):
+def numeric(orig, invalid=INFINITY):
+    """Parse a number out of a string.
+
+    This function parses a unicode number out of the start of a
+    string. If a number cannot be found at the start, the 'invalid'
+    argument is returned.
+        
+    """
+
     if not orig:
         return invalid
 
     string = unicode(orig)
-    for c in string:
-        if c.isnumeric():
+    for uchar in string:
+        if uchar.isnumeric():
             break
     else:
         return invalid
@@ -140,24 +158,18 @@ def numeric(orig, invalid=float('inf')):
         string = string[1:]
 
     if not string[:1].isnumeric():
-        return (invalid, orig)
-
-    string = normalize_punc(string)
+        return invalid
 
-    # Early out if possible.
-    try:
-        return float(string) * mult
-    except ValueError:
-        pass
+    string = normalize_number(string)
 
-    # Otherwise we need to do this the hard way.
     def _numeric(string):
+        """Interpreter a number as base 10."""
         total = 0
-        for c in string:
-            v = unicodedata.numeric(c)
-            if v >= 1 or v == 0:
+        for uchar in string:
+            number = unicodedata.numeric(uchar)
+            if number >= 1 or number == 0:
                 total *= 10
-            total += v
+            total += number
         return total
 
     try:
@@ -168,9 +180,21 @@ def numeric(orig, invalid=float('inf')):
     except ValueError:
         return mult * _numeric(string)
 
-def normalize_punc(string):
-    string = unicode(string.strip(u",.'"))
-    string = filter(lambda u: u.isnumeric() or u in u",.'", string)
+def normalize_number(string):
+    """Normalize punctuation in a number.
+
+    This function attempts to guess which characters in a number
+    represent grouping separators and which represent decimal
+    points. It returns a string that is valid to pass to Python's
+    float() routine (potentially, NaN, if nothing like a number is
+    found).
+
+    """
+
+    string = unicode(string)
+    string = filter(lambda u: u.isnumeric() or u in KEEP_IN_NUMBERS, string)
+    string = string.strip(KEEP_IN_NUMBERS)
+
     commas = string.count(u",")
     stops = string.count(u".")
     quotes = string.count(u"'")
@@ -187,7 +211,7 @@ def normalize_punc(string):
         quotes = 0
 
     def normalize_two(a, b, string):
-        # One of each - assume the first is grouping, second is point.
+        """One of each - assume the first is grouping, second is point."""
         a_idx = string.rindex(a)
         b_idx = string.rindex(b)
         if a_idx > b_idx:
@@ -248,4 +272,4 @@ def normalize_punc(string):
         # Single stop, but no decimal - probably grouping.
         string = string.replace(u".", u"")
 
-    return string
+    return string or "NaN"