A pergunta é pertinente sim caso o bloco de código no if contenha alta complexidade computacional, escritas no banco de dados ou em arquivos, etc..
O ideal, entre os dois, seria o (A).
Porque Performance(A) >= Performance(B)
Caso 1: a execução do programa tenha sempre exibirAdeus = true
, Performance(A) = Performance(B). Tanto (A) como (B), acabarão sempre tendo o mesmo fluxo:
Algoritmo (A), Algoritmo (B):
1. Atribui "" ou "Oi" para `message`
2. Verifica `exibirAdeus == true`
3. Atribui "Adeus" para `message`
Caso 2: a execução do programa tenha 1 ou mais exibirAdeus = false
, Performance(A) > Performance(B). Porque para esses casos (B) será mais custoso:
Algoritmo (A):
1. Atribui "Oi" para `message`
2. Verifica `exibirAdeus == true`: falso, não faz nada (message já é "Oi"). continua adiante.
Algoritmo (B):
1. Atribui "" para `message`
2. Verifica `exibirAdeus == true`: falso, precisa atribuir "Oi".
3. Atribui "Oi" para `message`
Mundo Real
No entanto, caso esse bloco seja pequeno e de baixa complexidade, isso se torna extremamante desnecessário.
O caminho que tu iria seguir é: O quão esse código está legível e fácil de dar manutenção (legibilidade & manutenabilidade). Digo isso porque há um consenso* que nesse caso apresentado, deverias se preocupar mais em colocar as chaves ({}) para demarcar os blocos do if-else.
string mensagem = "OI"; // padrão: caso mais provável...
if (exibirAdeus) {
mensagem = "Adeus";
}
ou ainda, caso o código comece a ficar mais complicado como lembrado por C. E. Gesser:
private bool Teste(){
[..]
string mensagem = initMensagem(condicao);
[..]
}
private string initMensagem(bool condicao){
if (condicao) {
return "Adeus";
}
return "OI";
}
Outro caso comum
string message = "";
if (condicao == 1){
message = "blablbla";
}
else if (condicao == 2){
message = "Lorem";
}
else if (condicao == 3){
message = "Ipsum";
}
[..]
else {
message = "Dolor";
}
Seria melhor escrito assim** por performance, legibilidade e manutenabilidade:
string message = "";
switch (condicao) {
case 1:
message = "blablbla";
break;
case 2:
message = "Lorem";
break;
case 3:
message = "Ipsum";
break;
[..]
default:
message = "Dolor";
}
*
https://softwareengineering.stackexchange.com/questions/16528
https://softwareengineering.stackexchange.com/questions/2715
https://stackoverflow.com/questions/15786949
**
https://stackoverflow.com/questions/445067
https://stackoverflow.com/questions/767821
https://stackoverflow.com/questions/395618