Skip to main content
2 de 5
adicionou 90 caracteres ao conteúdo
Maniero
  • 484,4mil
  • 94
  • 1,3mil
  • 2,2mil

Resolvi responder porque as respostas atuais não estão corretas. Acho que não precisa dizer que uma delas faz a mesma coisa com sintaxe diferente e o AP já sabe que não funciona. O outro considera que as variáveis não tem valores.

A primeira coisa que tem que avaliar sobre isto é entender o processo de desenvolvimento de software e como escrever códigos. Um código deve expressar bem o que está fazendo de forma clara e objetiva. Ser curto é uma característica boa, ser curto demais, ser esquisito, usar truques para parecer curto não é uma característica boa.

Dito isto não vejo problema algum no que fez originalmente, este código está simples e claro, expressa bem o que deve fazer, é organizado e facilitará a manutenção no futuro.

Se tentar fazer algo que fuja disto ficará um código esquisito e poderá trazer problemas no futuro. Você pode estar começando e não ter essa visão clara, eu faço isto há 35 anos, e já tentei fazer tudo o mais curto possível. Hoje eu vejo os erros que cometi.

Dá para reduzir ligeiramente:

void resetStage() {
    _fullLife = _halfLife = _almostDead = false;
}

Não gosto disto porque isto gera efeito colateral, mudança de estado, e é melhor deixar isto explícito e claro. Mudança de estado é o que tem de pior em um código, quase todos problemas sérios de algoritmo estão relacionados com mudança de estado. Inclusive em alguns casos pode dificultar o processo de debug. Mas é o máximo que deveria fazer.

Acho que deveria ler sobre o DRY. Muitas pessoas acham que é sobre escrever menos código. O DRY é sobre ser canônico e não escrever menos. Escrever menos código só é bom quando o código é desnecessário ou claramente repetitivo, não é o seu caso, este código esta ótimo para todos padrões conhecidos. tentar algo que parece mais simples na verdade é complicar mais, é expressar o código da forma errada, e código é expressão.

Qualquer idiota pode escrever código que um computador entenda. Bons programadores escrevem código que humanos podem entender

-- Martin Fowler

A solução que passa variáveis por referência parece inteligente, mas no momento que você começa usar verá que não é tão boa assim. Sem falar que ela está errada. Você terá que escrever tantos overloads que não compensará. O trabalho que dará para fazer essas opções será maior que o uso. A complicação não compensa, e e esquisitice torna o código menos legível sem ganho algum real. E mudanças vão te trazer problemas.

Se ainda insistisse pelo menos poderia fazer certo:

void boolReset(bool resetTo, ref bool b1, ref bool b2, ref bool b3) {
    b1 = b2 = b3 = resetTo;
}

Lembrando que se pode ser uma variável terá mais um overload, se puder ser duas, terá precisará mais um, se tiver 4 outro. E considerando que tudo é o mesmo tipo.

Depois chama assim (vou colocar abaixo o original "por extenso" logo abaixo pra comparar o tamanho):

void resetStage() {
    boolReset(false, ref _fullLife, ref _halfLife, ref _almostDead);
}
void resetStage() {
    _fullLife = _halfLife = _almostDead = false;
}

Mas ainda insisto, mesmo não conseguindo convencer, faça um em cada linha. Aumenta um ; um ENTER e repete o false duas vezes, só isso, é muito pouco.

Tem um jeito que simplifica um pouco, mas não muito a sintaxe, mas aí a estrutura de dados ficará tão errada que nem vale a pena cogitar, e ainda bem que ninguém sugeriu.

Está olhando para as coisas erradas no código. Comece prestar atenção ao que importa.

Maniero
  • 484,4mil
  • 94
  • 1,3mil
  • 2,2mil