這是不好的做法？

Question

在這段代碼中，如果我匹配線上給定的模式，我用自建字符串替換該線。這實際上是好的做法嗎？它看起來有點像重用變量行來輸出，它實際上包含了當前讀取行的內容。

for line in lines: 
    match = re.search(r'@Table\(name = "(.*)"\)', line) 

    if match: 
    line = "".join(['@Table (name = "', prefix, match.group(1)[:max_len], '")', '\n']) 

    f.write(line) 

f.close() 

Answer 1

我想說你的代碼中的意圖是清晰的，代碼簡短，所以沒有錯。如果仍然困擾你要重用的變量，你可以做這樣的事情：

for line in lines: 
    match = re.search(r'@Table\(name = "(.*)"\)', line) 

    if match: 
     output_line = "".join(['@Table (name = "', prefix, match.group(1)[:max_len], '")', '\n']) 
    else: 
     output_line = line 

    f.write(output_line) 

f.close() 

這樣，每個變量名準確地描述其內容全部的時間。