Help with optimisation

[special_dragonfly]

Hello,
I know this might be a little cheeky, and if it is, please say, but I need a
little hand optimising some code. For the simple reason that this is
'company' code and I have no idea what I'm allowed to release and not as the
case may be I've changed anything that could give an indication of the
company - if that makes any sense...

for the code below:
text_buffer is a single record from an XML stream. I can't read in the
entire XML at once because it isn't all available straight away, so I
capture line by line, and when a full message is available I use parseString
under the minidom API.
The SQL version is SQLite. It was recommended to me, and is adequate for the
uses I put it to.
The function doesn't return anything, but it's called often enough and
depending on the optimisation I'll be able to use the same style in other
areas of the program.

previous code:
def CreatePerson(te xt_buffer):
dom=xml.dom.min idom.parseStrin g(text_buffer)
reflist = dom.getElements ByTagName('Coun try')
Country = reflist[0].firstChild.nod eValue
reflist = dom.getElements ByTagName('Age' )
Age = reflist[0].firstChild.nod eValue
reflist = dom.getElements ByTagName('Surn ame')
Surname = reflist[0].firstChild.nod eValue
reflist = dom.getElements ByTagName('Fore name')
Forename = reflist[0].firstChild.nod eValue
cursor.execute( 'INSERT INTO Person VALUES(?,?,?)', (Forename + "-" +
Surname, Age, Country))
connection.comm it()

I've changed it now to this:
def CreatePerson(te xt_buffer):
dom=xml.dom.min idom.parseStrin g(text_buffer)
elements=['Country','Age' ,'Surname','For ename']
Values=[]
for element in elements:
reflist=dom.get ElementsByTagNa me(element)
Values.append(r eflist[0].firstChild.nod eValue)
# I can get away with the above because I know the structure of the
XML
cursor.execute( 'INSERT INTO Person
VALUES(?,?,?)', (Forename+"-"+Surname,Age,C ountry))
connection.comm it()

They both seem ugly IMO (read: longer than intuitively necessary), and so I
was wondering whether there was any way to combine Forename and Surname
together within the Values list (think merge cells with the '-' in between)
so I could use the unary(?) operator within the SQL?

I suppose if this is a cheeky request then I won't get any replies.
Thank you for any help
Dominic

[Alex Martelli]

special_dragonf ly <Do*****@PLEASE ASK.co.ukwrote:
...

dom=xml.dom.min idom.parseStrin g(text_buffer)

If you need to optimize code that parses XML, use ElementTree (some
other parsers are also fast, but minidom ISN'T).
Alex

[Bruno Desthuilliers]

special_dragonf ly a écrit :

Hello,

(snip)

The function doesn't return anything, but it's called often enough and
depending on the optimisation I'll be able to use the same style in other
areas of the program.

previous code:
def CreatePerson(te xt_buffer):
dom=xml.dom.min idom.parseStrin g(text_buffer)
reflist = dom.getElements ByTagName('Coun try')
Country = reflist[0].firstChild.nod eValue
reflist = dom.getElements ByTagName('Age' )
Age = reflist[0].firstChild.nod eValue
reflist = dom.getElements ByTagName('Surn ame')
Surname = reflist[0].firstChild.nod eValue
reflist = dom.getElements ByTagName('Fore name')
Forename = reflist[0].firstChild.nod eValue
cursor.execute( 'INSERT INTO Person VALUES(?,?,?)', (Forename + "-" +
Surname, Age, Country))
connection.comm it()

I've changed it now to this:
def CreatePerson(te xt_buffer):
dom=xml.dom.min idom.parseStrin g(text_buffer)
elements=['Country','Age' ,'Surname','For ename']
Values=[]
for element in elements:
reflist=dom.get ElementsByTagNa me(element)
Values.append(r eflist[0].firstChild.nod eValue)
# I can get away with the above because I know the structure of the
XML
cursor.execute( 'INSERT INTO Person
VALUES(?,?,?)', (Forename+"-"+Surname,Age,C ountry))
connection.comm it()

A common python optimisation trick is to stote local references to save
on attribute lookup time, ie:

# local ref to parseString
import dom
dom_parseString =xml.dom.minido m.parseString

def CreatePerson(te xt_buffer):
dom = dom_parseString (text_buffer)
elements=['Country','Age' ,'Surname','For ename']
values=[]
getElementByTag Name = dom.getElements ByTagName
for element in elements:
reflist = getElementsByTa gName(element)
values.append(r eflist[0].firstChild.nod eValue)
But as Alex already pointed out, you'd be better using (c)ElementTree.

They both seem ugly IMO (read: longer than intuitively necessary),

I'd say this is a common problem with XML :-/