Refactor code to be more readable #46
@@ -1,26 +1,20 @@
|
|||||||
#===========================================================================
|
|
||||||
#
|
|
||||||
# Parse XML messages into an object.
|
|
||||||
#
|
|
||||||
#===========================================================================
|
|
||||||
import defusedxml.ElementTree as ET
|
import defusedxml.ElementTree as ET
|
||||||
from . import messages
|
from . import messages
|
||||||
|
|
||||||
#==========================================================================
|
def parse(xmlText):
|
||||||
|
"""
|
||||||
|
Parse XML messages into an object.
|
||||||
|
|
||||||
# <rainForest ...>
|
Args:
|
||||||
# <[Message]>...</[Message]>
|
xmlText (str): The XML text to parse.
|
||||||
# </rainForest>
|
|
||||||
def parse( xmlText ):
|
|
||||||
root = ET.fromstring( xmlText )
|
|
||||||
assert( root.tag == "rainforest" )
|
|
||||||
|
|
||||||
child = root[0]
|
Returns:
|
||||||
|
messages.Message: The parsed message object.
|
||||||
|
"""
|
||||||
|
root = ET.fromstring(xmlText)
|
||||||
|
assert root.tag == "rainforest"
|
||||||
|
|
||||||
msgClass = messages.tagMap.get( child.tag, None )
|
child = root[0]
|
||||||
if not msgClass:
|
|
||||||
return None
|
|
||||||
|
|
||||||
return msgClass( child )
|
msgClass = messages.tagMap.get(child.tag, None)
|
||||||
|
return msgClass(child) if msgClass else None
|
||||||
#==========================================================================
|
|
||||||
Reference in New Issue
Block a user