Question

mystring = ’14| “Preprocessor Frame Count Not Incrementing; Card: Motherboard, Port: 2″|minor’

So I have 3 elements (id, message and level) divided by pipe (“|”). I want to get each element so I have written these little functions:

    def get_msg(i):
        x = i.split("|")
        return x[1].strip().replace('"','')

    def get_level(i):
        x = i.split("|")
        return x[2].strip()
 #testing
print get_msg(mystring ) # Missing Input PID,   PID: 20 : Port 4 of a static component
print get_level(mystring )# major

Right now it works well but I feel like this is not pythonic way to solve it, how could these 2 functions can be improved? Regular expression feels like fitting here but I’m very naive at it so couldn’t apply.

Answer 1

class MyParser(object):
    def __init__(self, value):
        self.lst = value.split('|')
    def id(self):
        return self.lst[0]
    def level(self):
        return self.lst[2].strip()
    def message(self):
        return self.lst[1].strip(' "')