Welcher rekursive Ansatz ist ein besseres Design?

Question

Angenommen, ich habe eine mehrzeilige Zeichenfolge, die Zeilen enthalten kann, die aus einem einzigen Dateinamen bestehen. Ich möchte jede Zeile in der Zeichenfolge drucken, es sei denn, die Zeile ist ein Dateiname (hier definiert als '.txt'). In diesem Fall möchte ich jede Zeile in dieser Datei drucken, es sei denn, die Zeile in dieser Datei ist a Dateinamen usw.

Mein erster Ansatz ist eine Hilfsfunktion wie folgt zu verwenden:

def deep_print_helper(file_path, line_sep): 
    with open(file_path) as f: 
     text = f.read() 
     return deep_print(text, line_sep) 

def deep_print(s, line_sep): 
    lines = s.split(line_sep) 
    for l in lines: 
     if l.endswith('.txt'): 
      deep_print_helper(l, line_sep) 
     else: 
      print(l) 

Aber mit nur line_sep an die Hilfsfunktion zu übergeben es passieren zurück scheint wieder unelegant.

So habe ich versucht, einen Ansatz, der nur eine Funktion verwendet:

def deep_print(line_sep, s='', file_path=''): 
    if file_path: 
     with open(file_path) as f: 
      s = f.read() 
    lines = s.split(line_sep) 
    for l in lines: 
     if l.endswith('.txt'): 
      deep_print(line_sep, file_path=l) 
     else: 
      print(l) 

Diese implizite erforderliches Argument hat (entweder s oder file_path aber nicht beide), aber da der Benutzer der Funktion nur eine Form verwenden (s=) kann es nicht zu klucky sein. Aus der Sicht des Benutzers erscheint es auch ein wenig seltsam, dass line_sep das erste Argument ist.

Welcher Ansatz ist ein besseres Design? Gibt es einen anderen Ansatz, den ich berücksichtigen sollte?

def deep_print(s, line_sep): 

    def deep_print_helper(file_path): 
     with open(file_path) as f: 
      text = f.read() 
      return deep_print(text, line_sep) 

    lines = s.split(line_sep) 
    for l in lines: 
     if l.endswith('.txt'): 
      deep_print_helper(l) 
     else: 
      print(l) 

Answer 1

die line_sep Parameter übergeben Um zu vermeiden, können Sie die Hilfsfunktion innerhalb der rekursiven Funktion definieren. Es hängt mit dem Design zusammen, unabhängig von der Sprache. Wenn Sie mich unter den zwei Ansätzen fragen, werde ich mit 1 gehen. Aber der bessere Weg, um zu erreichen ist über Klasse mit Ihren Funktionen darin. In Python kann tun Sie es mögen:

class DeepPrint(object): 
    def __init__(self, file_path): 
     DeepPrint._deep_print_helper(file_path) 

    @staticmethod 
    def _deep_print_helper(file_path): 
     with open(file_path) as f: 
      return DeepPrint._deep_print(f) 

    @staticmethod 
    def _deep_print(f): 
     for l in f.readlines(): 
      if l.endswith('.txt'): 
       DeepPrint._deep_print_helper(l) 
      else: 
       print(l) 

Answer 2

Ihre Frage sollte nicht in Bezug auf Pythonic Weg, dies zu erreichen:

Answer 3

Ihre Anforderungen an das nicht zulassen, aber str.splitlines verwenden würde die Dinge etwas weniger kompliziert. Gibt es aus dem gleichen Grund einen Grund, warum die Originaldatei nicht als Teil der Rekursion geöffnet wird (d. H. Anstatt eine Zeichenkette in deep_print zu übergeben, könnten Sie einen Dateipfad übergeben)? Wenn diese beiden Einschränkungen aufgehoben werden können, könnten Sie etwas wie folgt tun:

def deep_print(file_path): 
    with open(file_path) as f: 
     s = f.read() 
    for line in [l.strip() for l in s.splitlines()]: 
     if line.endswith('.txt'): 
      deep_print(line) 
     else: 
      print(line)