alain walter, 28.10.2010 11:37:
dom = parseString(toxml) self.ApplicationWhitespaceRemoving(dom) print toxmldef ApplicationWhitespaceRemoving(self,ele) : from xml.dom import Node for c in ele.childNodes: if c.nodeType == c.TEXT_NODE: if c.nodeValue == "xxx_toremove_xxx": ???.removeChild(???) elif c.nodeType == ele.ELEMENT_NODE: self.ApplicationWhitespaceRemoving(c)
Just a comment on code style here. Functions and procedures should have names that strongly refer to a descriptive verb, consequently starting with a lower case letter by convention. So a better name for your procedure above could be "removeApplicationSpecificWhitespace".
When using constants defined on a module or class, try to refer to them in a consistent way, either by importing the names globally into your module namespace, or by referencing them from the same namespace (e.g. class object) everywhere. Simple, recurrent patterns help in reading your code.
Then, I don't see what the "Node" import is used for, and it looks like your function is actually a method (called as self.xxx). When presenting code snippets, try to make them consistent, or make it clear that (and where) things are missing.
Hope that helps, Stefan -- http://mail.python.org/mailman/listinfo/python-list
