Ist es eine schlechte Übung, einen Fehler in Golang zurückzusetzen?

Question

Wenn ich eine Zeichenfolge als Text analysiere, möchte ich, dass jede nicht analysierbare Zeichenfolge zu einer Nullzeit führt, und mache dann weiter.

passwordLastUsed, err = time.Parse(time.RFC3339, record[lastUsed]) 
if err != nil { 
    err = nil 
    passwordLastUsed = time.Time{} 
} 

Dieser Code sieht ein bisschen chaotisch und die 'ineffassign' Linter gibt diese für die 'err = nil' Aussage:

Warnung: ineffektiv Zuordnung irren (ineffassign)

Gibt es einen besseren Weg dies zu handhaben oder sollte ich den Linter einfach ignorieren?

Answer 1

Es ist nicht schlecht, eine Fehlervariable zurückzusetzen, wenn Sie das möchten.

Es ist eine schlechte Methode, Variablen festzulegen, die später nicht mehr verwendet werden - und darum geht es in der Warnung.

Wahrscheinlich gibt es nie einen Grund, um einen Fehler zurückgesetzt wie Sie tun, da nach dem if Block err wird immernil sein.

Es macht Sinn, wenn Sie nur in einigen Fällen zurückgesetzt werden. Ein gängiges Beispiel:

result, err := db.Query(...) 
if err == sql.ErrNoRows { 
    err = nil // Ignore not-found rows 
} 
if err != nil { 
    return err // But return all other errors 
}