This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

classification
Title: Dictionary iterators break old code
Type: Stage:
Components: Library (Lib) Versions: Python 2.2
process
Status: closed Resolution: fixed
Dependencies: Superseder:
Assigned To: loewis Nosy List: gvanrossum, loewis
Priority: normal Keywords:

Created on 2001-08-05 13:27 by loewis, last changed 2022-04-10 16:04 by admin. This issue is now closed.

Files
File name Uploaded Description Edit
dictiter.py loewis, 2001-08-05 13:27
Messages (4)
msg5779 - (view) Author: Martin v. Löwis (loewis) * (Python committer) Date: 2001-08-05 13:27
With the introduction of dictionary iterators, 
iteration over UserDict specializations may silently 
break existing code. The attached code is a 
simplification of the xml.dom.NamedNodeMap class, 
which provides iteration over the values using a 
well-defined order, which is implemented in 
__getitem__. In Python 2.2a1, __getitem__ is not 
called anymore, thus the code breaks.
msg5780 - (view) Author: Guido van Rossum (gvanrossum) * (Python committer) Date: 2001-08-06 20:59
Logged In: YES 
user_id=6380

Aaaaaaargh!!!!!!!

You mean somebody wrote a subclass of UserDict that attempts
to behave like a sequence by checking the type of the
__getitem__ argument??????????

Yuck!!!!!!!!!!!!!!!

You XML weenies are sickos. :-)

Now that I've got that out of the way :-),

Short of totally dropping dictionary iterators, the only
solution I can see it to remove the __iter__ method of
UserDict. That is backwards compatible, but it removes the
ability to iterate over UserDict instances (except for the
sick ones :-), thus making UserDict less like a dictionary.

But since UserDict is likely to become deprecated anyway in
favor of subclassing from the 'dictionary' built-in type,
this solution seems acceptable -- but only if this trick is
also in use outside the XML code.  I would prefer to see the
XML code fixed, since it can now properly define __iter__ to
iterate over whatever it wants to in any sequence it would
like. :-)
msg5781 - (view) Author: Martin v. Löwis (loewis) * (Python committer) Date: 2001-08-07 06:16
Logged In: YES 
user_id=21627

I've already fixed it in PyXML, although the only way I 
could find to fix it was to add an __iter__; I cannot take 
away the __getitem__ since the code has to work in older 
Python versions.

The solution I would prefer to see is that UserDict offers 
a second class, say IterableUserDict, which would be for 
educational purposes only; anybody who really wants a 
dictionary that you can iterate over should inherit from 
dictionary.

That also raises the question whether UserDict 
applications in the standard library should be changed to 
inherit from dictionary. The major incompatibility I can 
see with doing so is that the .data member goes away.
msg5782 - (view) Author: Guido van Rossum (gvanrossum) * (Python committer) Date: 2001-08-07 17:53
Logged In: YES 
user_id=6380

Closed by adding an IterableUserDict to UserDict.py.
History
Date User Action Args
2022-04-10 16:04:17adminsetgithub: 34896
2001-08-05 13:27:10loewiscreate