On Thu, 09 Jul 2009 18:36:05 -0700, inkhorn wrote: > def list_items_in_string(list_items, string): > for item in list_items: > if item in string: > return True > return False ... > Any ideas how to make that function look nicer? :)
Change the names. Reverse the order of the arguments. Add a docstring. Otherwise looks pretty nice to me. Simple, straightforward, and correct. If you're running Python 2.5 or better, then this is even shorter (and probably faster): def contains(s, targets): """Return True if any item of targets is in string s.""" return any(target in s for target in targets) -- Steven -- http://mail.python.org/mailman/listinfo/python-list