J'écris une candidature et j'en suis arrivé à ce point:
private void SomeMethod()
{
if (Settings.GiveApples)
{
GiveApples();
}
if (Settings.GiveBananas)
{
GiveBananas();
}
}
private void GiveApples()
{
...
}
private void GiveBananas()
{
...
}
Cela semble assez simple. Il existe certaines conditions et si elles sont vraies, les méthodes sont appelées. Cependant, je pensais, est-il préférable de faire comme ceci:
private void SomeMethod()
{
GiveApples();
GiveBananas();
}
private void GiveApples()
{
if (!Settings.GiveApples)
{
return;
}
...
}
private void GiveBananas()
{
if (!Settings.GiveBananas)
{
return;
}
...
}
Dans le second cas, chacune des méthodes se garde, donc même si l'une de ces méthodes GiveApples
ou GiveBananas
est appelée de l'extérieur SomeMethod
, elles ne seront exécutées que si elles ont le bon indicateur dans Paramètres.
Est-ce quelque chose que je devrais réellement considérer comme un problème?
Dans mon contexte actuel, il est très peu probable que ces deux méthodes soient appelées de l'extérieur de cette méthode, mais personne ne peut jamais garantir cela.