Benötigen Sie Hilfe beim Refactoring meines Python-Skripts

Question

Ich habe ein Python-Skript, das eine Datei Zeile für Zeile verarbeiten, wenn die Zeile eine Regex entspricht, ruft es eine Funktion auf, um es zu behandeln.

Meine Frage ist, gibt es eine bessere schreiben Refactor mein Skript. Das Skript funktioniert, aber so wie es ist, muss ich den Einzug rechts vom Editor beibehalten, da ich mehr und mehr Regex für meine Datei hinzufüge.

Vielen Dank für eine Idee. mein Code nun bis am Ende wie folgt:

 
for line in fi.readlines(): 

     result= reg1.match(line) 

     if result: 
       handleReg1(result) 

     else: 
       result = reg2.match(line) 

       if result: 
         handleReg2(result) 
       else: 
         result = reg3.match(line) 

         if result: 
           handleReg3(result) 
         else: 
           result = reg4.match(line) 

           if result: 
             handleReg4(result) 
           else: 
             result = reg5.match(line) 

             if result: 
               handleReg5(result) 

Answer 1

ich unter Verwendung einer Datenstruktur Mapping Regexes auf Funktionen wechseln würde. Etwas wie:

map = { reg1: handleReg1, reg2: handleReg2, etc } 

Dazu einfach Schleife durch sie:

for reg, handler in map.items(): 
    result = reg.match(line) 
    if result: 
     handler(result) 
     break 

Wenn Sie die Spiele müssen in einer bestimmten Reihenfolge geschehen müssen Sie eine Liste verwenden, anstatt eines Wörterbuchs, aber die Prinzip ist das Gleiche.

Answer 2

Hier ist trivial:

handlers = { reg1 : handleReg1, ... } 

for line in fi.readlines(): 
    for h in handlers: 
     x = h.match(line) 
     if x: 
      handlers[h](x) 

Wenn es eine Linie sein könnte, dass dieser Code mehrere regexps entspricht, wird aus dem Code unterschiedlich sein Sie eingefügt: es wird mehrere Handler aufrufen. Das Hinzufügen von break wird nicht helfen, weil die Regexps in einer anderen Reihenfolge ausprobiert werden, so dass Sie am Ende die falsche aufrufen werden. Also, wenn dies der Fall ist, sollten Sie über die Liste iterieren:

handlers = [ (reg1, handleReg1), (reg2, handleReg2), ... ] 

for line in fi.readlines(): 
    for reg, handler in handlers: 
     x = reg.match(line) 
     if x: 
      handler(x) 
      break 

Answer 3

Ein alternativer Ansatz, die funktionieren könnte für Sie ist es, alle regexps in einem riesigen regexp zu kombinieren und nutzen m.group() zu erkennen, welche abgestimmt. Meine Intuition sagt, das sollte schneller sein, aber ich habe es nicht getestet.

Dies wird kompliziert, wenn die Regexps, gegen die Sie testen, selbst kompliziert sind oder Match-Gruppen verwenden.