<br><br><div class="gmail_quote"><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br>
Hi Raymond,<br>
<br>
* I'm not a native speaker, but why use the word "Counter"? A "counter"<br>
to my ear sounds like a number that is increased each time an event<br>
occurs; the website counter, eg, comes to mind. I can understanda its<br>
meaning probably stretches to "an object that counts", but I really can't<br>
think of it as a group of object, or a container of object. Moreover, I<br>
find it a much more useful abstraction the idea of a "multi-set" (that<br>
is, a set where elements can appear with multiple cardinality), rather<br>
than stressing the concept of "counting" how many times each element<br>
appears in the set.<br>
<br>
* I find it *very* confusing c.items() vs c.elements(). Items and<br>
elements are synonymous (again, in my understanding of English).<br>
<br>
All in all, I think I prefer your previous bag class:<br>
<a href="http://code.activestate.com/recipes/259174/" target="_blank">http://code.activestate.com/recipes/259174/</a><font color="#888888"><a href="http://mail.python.org/mailman/listinfo/python-list" target="_blank"></a></font></blockquote>
<div><br>Well, I am a native English speaker and I agree with your points completely. Also, wouldn't it be better to either have a least_common method or allow most_common to take a negative integer rather than having to use most_common()[:-n:-1]? For a large Counter, this could take up a lot of memory.<br>
</div></div><br>