Refatoraçao

[Karpov1]

double energiaPotencial ( double massa , double altura ) {
// 9.81 ´e a constante gravitacional .
return massa * 9.81 * altura ;
}

static final double CONSTANTE_GRAVITACIONAL = 9.81;
double energiaPotencial ( double massa , double altura ) {
return massa * CONSTANTE_GRAVITACIONAL * altura ;
}

 por que o segundo codigo é considerado superior?

[craftrod]

Qual o valor da variavel 'massa' e 'altura' ?

[Karpov1]

Isso nao vem ao caso...porque é questao sobre refatoracao, quer saber o que é um codigo melhor e pior

[craftrod]

@Karpov1 Ah, agora que entendi. Você quer saber o porque o segundo código é melhor.

Pelo que sei Refatoração é uma modificação ou melhora no código sem modificar seu comportamento ou funcionamento.

 

No primeiro código, ele não tinha uma variável determinada para a CONSTANTE_GRAVITACIONAL (ele usava um valor 9.81 na multiplicação, nesse caso)

 

Já na segunda ele já tinha um valor para esses 9.81 (a variável CONSTANTE_GRAVITACIONAL)

Assim, não precisando colocar os '9.81' toda hora que você multiplicar a massa, a constante gravitacional e a altura.

 

Por isso que ele é considerado 'superior', mas ele não modifica o comportamento do aplicativo.

[Karpov1]

Ah, ele nao vai precisar escrever 9,81 mas vai precisar escrever CONSTANTE_GRAVITACIONAL o que é bem mais difícil de escrever que 9,81 nao é? e ainda deixa o codigo maior

[Cweiler]

@Karpov1 não entendi o que você quis dizer com "código maior".

 

O segundo trecho de código é melhor pois utiliza uma constante no lugar de um "número mágico". Números mágicos são complicadores em código pois quem for utilizar seu código não sabe o objetivo deste número. Outro problema é quando este número precisa ser utilizado em vários pontos diferentes do código, existe grande chance de erro nas digitações e quando for necessário mudar o valor é necessário varrer o código.