I'm new to LXML, and parsing XML documents in general. I have written a small section of code that appears to do what I need it too do, but it feels like I have overly complicated it. Is there any way I can simplify this?


def importFromXML(self, filename):
    tree = etree.parse(filename)
    for child in tree.getroot():
        if child.tag != 'SLOT':
            print('Tag:', child.tag, ', Text', child.text)
        elif child.tag == 'SLOT':
            for slot, index in child.items():
                for attribute in child:
                    print('Slot Number =', index, ', Tag:', attribute.tag, ', Value:', attribute.text)


<?xml version="1.0" encoding="UTF-8"?>
<Location>Left Wrist</Location>
<ItemName>Band of the Dream Conqueror</ItemName>
<SLOT Number="0">
<SLOT Number="1">
<SLOT Number="2">

I just want to make sure I'm doing this right. I'm using Python 3.x. Eventually I will be doing more with the data I pull, but for right now I'm just trying to pull the information, and figure out the most optimized way of doing this.


1 Answer 1


The code isn't actually doing much, there's little to review to be honest, it's not looking complicated for what it is. Of course don't just have everything in a couple of loops, perhaps have some handlers to dispatch on the tags or so (handlers[child.tag](child) ...).

I suppose if you wanted a relatively quick way to write the importer code, consider XPath. If you wanted quick processing (by not building up the full DOM in memory), consider using a SAX handler.

Finally, there might be a library to automatically map the XML to a class structure, or even parse it directly into dict instances or something, if you prefer that, but I don't any off the top of my hat.

  • \$\begingroup\$ Thank you. I'm just trying to find the most optimized way to pull the information, then I will do something with it later. There is a lot of terminology that I will need to look up (new to both Python and XML parsing) and see how I can apply it. The XML files are at most 25 lines, and it would be ideal if I could load the <SLOT> elements in a dict that gives me { SLOT: {'Index': value, 'Type': value, 'Effect': value, 'Amount': value}} perhaps that will be my next step. \$\endgroup\$
    – artomason
    Feb 4, 2018 at 1:36

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Not the answer you're looking for? Browse other questions tagged or ask your own question.