Skip to main content
conteúdo editado
Fonte Link
Maniero
  • 484,4mil
  • 94
  • 1,3mil
  • 2,2mil
adicionou 1 caractere ao conteúdo
Fonte Link
Maniero
  • 484,4mil
  • 94
  • 1,3mil
  • 2,2mil

O problema não é quantidade de if e sim de decisões a serem todas, não tem 34 if, mas tem 34 comparações que geram booleanos. Se quiser reduzir para apenas dois ifs é tranquilo porque todos geram o mesmo resultado, pra falar a verdade precisa de um ifif`ss é tranquilo porque todos geram o mesmo resultado, pra falar a verdade precisa de um ifs (em C# poderia se feito com zero).

O problema não é quantidade de if e sim de decisões a serem todas, não tem 34 if, mas tem 34 comparações que geram booleanos. Se quiser reduzir para apenas dois ifs é tranquilo porque todos geram o mesmo resultado, pra falar a verdade precisa de um if`s (em C# poderia se feito com zero).

O problema não é quantidade de if e sim de decisões a serem todas, não tem 34 if, mas tem 34 comparações que geram booleanos. Se quiser reduzir para apenas dois ifs é tranquilo porque todos geram o mesmo resultado, pra falar a verdade precisa de um ifs (em C# poderia se feito com zero).

removeu 1 caractere do conteúdo
Fonte Link
Maniero
  • 484,4mil
  • 94
  • 1,3mil
  • 2,2mil

Se for por esse caminho eureu iria pelos grupos que já criou, talvez reagrupando um pouco (voc~evocê tinha começado bem), verifique se é null e se é igual junto, o que faz mais sentido até mesmo no caso de manter tudo em um método só, provavelmente será mais eficiente e mais legível.

Dito tudo istoisso, é possível que istoisso realmente possa ser interessante estar em métodos separados que controlam cada um dos campos individualmente, porque eles isoladamente devem ser tratados assim, ou sejam toda engenharia do software precisa ser pensada para lidar com isso, principalmente se cada condição possa ser usada novamente em algum outro local, afinal devemos seguir o DRY, ou seja, fazer a boa e velha abstração, que inclusive tem a ver com orientação a objeto que tanto defendem. Mas não tenho informação suficiente para saber se isto é o mais adequado neste caso. Certamente seria mais testável, o que eu questiono se deveria ser o objetivo principal do código, mas se for para mudar deve ser em primeiro lugar para atender a demanda do problema, só depois pensar no tsteteste, e não pensar no SonarLint mal configurado.

Se for por esse caminho eur iria pelos grupos que já criou, talvez reagrupando um pouco (voc~e tinha começado bem), verifique se é null e se é igual junto, o que faz mais sentido até mesmo no caso de manter tudo em um método só, provavelmente será mais eficiente e mais legível.

Dito tudo isto, é possível que isto realmente possa ser interessante estar em métodos separados que controlam cada um dos campos individualmente, porque eles isoladamente devem ser tratados assim, ou sejam toda engenharia do software precisa ser pensada para lidar com isso, principalmente se cada condição possa ser usada novamente em algum outro local, afinal devemos seguir o DRY, ou seja, fazer a boa e velha abstração, que inclusive tem a ver com orientação a objeto que tanto defendem. Mas não tenho informação suficiente para saber se isto é o mais adequado neste caso. Certamente seria mais testável, o que eu questiono se deveria ser o objetivo principal do código, mas se for para mudar deve ser em primeiro lugar para atender a demanda do problema, só depois pensar no tste, e não pensar no SonarLint mal configurado.

Se for por esse caminho eu iria pelos grupos que já criou, talvez reagrupando um pouco (você tinha começado bem), verifique se é null e se é igual junto, o que faz mais sentido até mesmo no caso de manter tudo em um método só, provavelmente será mais eficiente e mais legível.

Dito tudo isso, é possível que isso realmente possa ser interessante estar em métodos separados que controlam cada um dos campos individualmente, porque eles isoladamente devem ser tratados assim, ou sejam toda engenharia do software precisa ser pensada para lidar com isso, principalmente se cada condição possa ser usada novamente em algum outro local, afinal devemos seguir o DRY, ou seja, fazer a boa e velha abstração, que inclusive tem a ver com orientação a objeto que tanto defendem. Mas não tenho informação suficiente para saber se isto é o mais adequado neste caso. Certamente seria mais testável, o que eu questiono se deveria ser o objetivo principal do código, mas se for para mudar deve ser em primeiro lugar para atender a demanda do problema, só depois pensar no teste, e não pensar no SonarLint mal configurado.

adicionou 139 caracteres ao conteúdo
Fonte Link
Maniero
  • 484,4mil
  • 94
  • 1,3mil
  • 2,2mil
Carregando
adicionou 973 caracteres ao conteúdo
Fonte Link
Maniero
  • 484,4mil
  • 94
  • 1,3mil
  • 2,2mil
Carregando
Fonte Link
Maniero
  • 484,4mil
  • 94
  • 1,3mil
  • 2,2mil
Carregando