]> granicus.if.org Git - python/commitdiff
Improve error messages
authorRaymond Hettinger <python@rcn.com>
Tue, 16 Oct 2007 19:18:30 +0000 (19:18 +0000)
committerRaymond Hettinger <python@rcn.com>
Tue, 16 Oct 2007 19:18:30 +0000 (19:18 +0000)
Lib/collections.py

index 4515759b786330da3fc81cf673eb00decb9b30f5..fbc00d1fce3d80270d6be152e106cf72595d88af 100644 (file)
@@ -32,16 +32,17 @@ def named_tuple(typename, field_names, verbose=False):
     if isinstance(field_names, basestring):
         field_names = field_names.replace(',', ' ').split() # names separated by whitespace and/or commas
     field_names = tuple(field_names)
-    if not ''.join((typename,) + field_names).replace('_', '').isalnum():
-        raise ValueError('Type names and field names can only contain alphanumeric characters and underscores')
+    for name in (typename,) + field_names:
+        if not name.replace('_', '').isalnum():
+            raise ValueError('Type names and field names can only contain alphanumeric characters and underscores: %r' % name)
+        if name[0].isdigit():
+            raise ValueError('Type names and field names cannot start with a number: %r' % name)
     seen_names = set()
     for name in field_names:
         if name.startswith('__') and name.endswith('__'):
-            raise ValueError('Field names cannot start and end with double underscores: %s' % name)
-        if name[:1].isdigit():
-            raise ValueError('Field names cannot start with a number: %s' % name)
+            raise ValueError('Field names cannot start and end with double underscores: %r' % name)
         if name in seen_names:
-            raise ValueError('Encountered duplicate field name: %s' % name)
+            raise ValueError('Encountered duplicate field name: %r' % name)
         seen_names.add(name)
 
     # Create and fill-in the class template