私はアプリケーションを書いていますが、このポイントに到達しました:
private void SomeMethod()
{
if (Settings.GiveApples)
{
GiveApples();
}
if (Settings.GiveBananas)
{
GiveBananas();
}
}
private void GiveApples()
{
...
}
private void GiveBananas()
{
...
}
これは非常に簡単です。いくつかの条件があり、それらが真である場合、メソッドが呼び出されています。しかし、私は考えていました、このようにするのがむしろ良いですか?
private void SomeMethod()
{
GiveApples();
GiveBananas();
}
private void GiveApples()
{
if (!Settings.GiveApples)
{
return;
}
...
}
private void GiveBananas()
{
if (!Settings.GiveBananas)
{
return;
}
...
}
2番目のケースでは、各メソッドはそれ自体を保護するため、これらのメソッドのGiveApples
いずれかまたはGiveBananas
外部から呼び出された場合でもSomeMethod
、設定に正しいフラグがある場合にのみ実行されます。
これは私が実際に問題と見なすべきものですか?
私の現在のコンテキストでは、これらの2つのメソッドがこのメソッドの外部から呼び出されることはほとんどありませんが、だれもそれを保証することはできません。