Irgendwelche Nachteile beim Ändern eines ungültigen Rückgabetyps auf etwas anderes in einer häufig verwendeten Dienstprogrammfunktion?

Question

wenn ich eine Funktion wie

public void usefulUtility(parameters...) { 
    string c = "select * from myDB"; 
    do_a_database_call(c); 
} 

haben, die in eine Menge von Orten verwendet wird, gibt es bei der Veränderung es zu einem möglichen Schaden ist:

public bool usefulUtility(parameters...) { 
    string c = "select * from myDB"; 
    bool result = do_a_database_call(c); 
    return result; 
} 

Könnte dies möglicherweise beliebigen Code brechen?

Ich kann an nichts denken ... aber es ist möglich?

Answer 1

Ja, praktisch alles, was Sie möglicherweise tun könnten, das Ihre öffentliche Schnittstelle betrifft, ist eine bahnbrechende Änderung. Es könnte klein genug sein, dass es Ihnen egal ist, und dass niemand oder fast niemand die Eckfälle tatsächlich trifft, aber Eric Lippert explains, dass es Randfälle gibt (von denen viele Typinferenz beinhalten), die sogar verursachen können Diese scheinbar harmlosen Änderungen brechen.

Für Ihr spezielles Beispiel würde dieser Code durch diese Änderung gebrochen werden.

Action a = usefulUtility; 

Answer 2

Sie müssten tatsächlich eine Zeichenfolge oder mindestens Null für die zu kompilierende Methode zurückgeben. Sie können die Methode nicht mehr ohne Rückgabe beenden.

Answer 3

Es ist sicherlich möglich , dass es einen anderen Code brechen könnte. Wie Lippert hervorhebt, fast Every public change is a breaking change in einer bizarren Situation.

Die wichtigere Frage ist, ist es wahrscheinlich zu etwas zu brechen, und die Antwort ist nein. Sie sollten ziemlich sicher sein, diese Änderung vorzunehmen, weil die Leute wahrscheinlich nicht die bizarren Dinge tun, die sie tun müssten, um Probleme zu verursachen. Das bedeutet nicht, dass es unmöglich ist, einen anderen Code zu brechen, aber es liegt außerhalb des Verantwortungsbereichs.

Answer 4

Wenn Sie absolut sicher sein wollen, nichts zu brechen, geben Sie der Methode, die einen String zurückgibt, einen anderen Namen und lassen Sie die Signatur der void-Methode gleich.

Answer 5

Eine solche Änderung könnte definitiv etwas kaputt machen, sowohl hinsichtlich der Binärkompatibilität als auch hinsichtlich der Quellenkompatibilität.

Sie können sich vielleicht einen Blick auf this StackOverflow answer und den zugehörigen Thread werfen, in dem die API-breaking-Änderungen ausführlich behandelt werden.