一般的に使用されているユーティリティ関数の中で、void戻り値の型を変更すると何か問題がありますか？

Question

私は場所の多くで使われています

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

のような機能を持っている場合、それを変更することで任意の害を及ぼす可能性があります：

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

これはおそらく任意のコードを壊してもらえますか？

私は何も考えることができません...しかし、それは可能かもしれませんか？

Answer 1

はい、事実上、パブリックインターフェイスに影響を与える可能性のある事柄は大きな変化です。あなたが気にしないほど小さくても、ほとんど誰も、あるいはほとんど誰もコーナーケースを打つことは実際には起こらないかもしれませんが、Eric Lippert explainsはエッジケース（多くはタイプ推論を伴う）があり、これらの一見無害な変化が壊れる。

特定の例では、このコードはその変更によって壊れます。

Action a = usefulUtility; 

Answer 2

実際にコンパイルするメソッドにはStringまたはnullを返す必要があります。あなたはもはや戻りなしでメソッドを終了することはできません。

Answer 3

それは確かに、それはいくつかの他のコードを壊す可能性があり可能です。 Lippertが指摘するように、ほぼEvery public change is a breaking changeという奇妙な状況では、

さらに重要な質問は、は何かを壊す原因となり、答えはノーです。この変更を行うのはかなり安全です。人々が問題を引き起こすためにしなければならない奇妙な種類のことを行う可能性は低いからです。それは他のコードを破ることは不可能ではありませんが、合理的な責任の範囲外です。

Answer 4

何も壊さないようにしたい場合は、Stringを別の名前に戻すメソッドを与え、voidメソッドのシグネチャをそのままにします。

Answer 5

このような変更は、バイナリ互換性とソースの互換性の両方に関して、間違いなく不具合を引き起こす可能性があります。

this StackOverflow answerとそれに関連するスレッドをご覧になりたい場合があります。ここでは、APIの変更点について深く説明します。