Olá

Nunca altere o estado dos objetos passados por parâmetro de um método, utilize uma variável temporária quando alterar o valor de algum atributo do objeto.

Essa prática é para evitar erros diante de uma confusão causada pelo tipo de passagem de valores que o Java adota. Java sempre faz a passagem por valor.

Abaixo um exemplo de erro que levamos a cometer quando alteremos o estado de um objeto.

Nós temos um objeto do tipo ItemPedido que possui os atributos quantidade e preço. Passamos esse objeto para o método totalizarPedido(), que para seu calculo interno subtrai um caso o parâmetro promoLeveUmGratis for true.

if(promoLeveUmGratis){

item.setQuantidade(item.getQuantidade().subtract(BigDecimal.ONE));

}

Observe que estamos atribuindo o resultado da subtração no mesmo objeto que recebemos por parâmetro para o cálculo na linha abaixo.

return item.getQuantidade().multiply(item.getPreco());

Em seguida os métodos calcularFrete() e darBaixaNoEstoque() são invocados.

ex.totalizarPedido(ip, true);
ex.calcularFrete(ip);
ex.darBaixaNoEstoque(ip);

Coloquei alguns sysout´s para ilustrar o problema.

Quantidade de itens no pedido:   10

Quantidade de itens no frete:    9

Quantidade de itens a dar baixa: 9

Note que para os dois métodos seguintes a quantidade que chegou foi alterada, não era essa a intenção. Isso traria sérios problemas para nós.

Downloads: Exemplo.java ItemPedido.java

Até mais,

Olá

Acrescentando e removendo parâmetros, todo desenvolvedor já precisou algum dia fazer essa alteração no código-fonte, esse é o tipo de refatoração mais comum na nossa vida.

Pode parecer simples quando o projeto é pequeno e pouco reaproveitado, mas precisamos ter muito cuidado ao fazer isso em grandes projetos cujo método é altamente reusado para não correr o risco de tomar um NoSuchMethodException em produção.

O procedimento é parecido para os dois tipos de refatoração.

1.       Crie um novo método com o novo parâmetro (ou sem o parâmetro se estiver removendo). Geralmente é um Ctrl+C e Ctrl+V do método existente apenas alterando a lista de parâmetros.

2.       Agora você deve copiar o corpo do método existente para o método novo.

3.       No método que já existia efetue a chamada do método novo. No caso de acréscimo de parâmetro você terá que passar um valor padrão para o parâmetro novo, eu geralmente uso null.

4.       Altere os demais códigos para que chame o método novo. Tente procurar pelo máximo de referencias.

5.       Execute o aplicativo e teste.

6.       Você pode apagar o método antigo, mas se isso não for possível por que o método faz parte de uma facade (interface) mantenha-o e coloque um @deprecated no javadoc dele.

Se precisar alterar o método com mais de um parâmetro, o ideal é fazer tudo de uma vez só para evitar retrabalho depois.

Muitos não têm o costume de retirar parâmetros, mesmo que não sejam mais usados. Eles mantêm o parâmetro boiando (inclusive eu :-D ) com a premissa de que não vai dar problema no comportamento do método, nisso temos razão, mas com certeza dará problema no seu entendimento ou no do pobre coitado alguns dias depois quando for dar manutenção.

Até mais,

Olá,

A dica de hoje é mover números mágicos do código para constantes com nomes sugestivos para facilitar a leitura por quem for dar manutenção futuramente.
No exemplo abaixo, o valor utilizado para converter km/h em m/s e vice-versa é 3.6, ao invés de repetir esse número várias vezes, colocamos em uma constante e usamos ela em todos os lugares que precisamos.

public double converterKMH2MS(double velocidade){
	return velocidade / 3.6;
}

public double converterMS2KMS(double velocidade){
	return velocidade * 3.6;
}

O código melhorado ficaria assim:

//
public static final double FATOR_CONVERSAO_KMH_MS = 3.6;

public double converterKMH2MS(double velocidade){
	return velocidade / FATOR_CONVERSAO_KMH_MS;
}

public double converterMS2KMS(double velocidade){
	return velocidade * FATOR_CONVERSAO_KMH_MS;
}

Dependendo da quantidade de constantes que você tem no seu sistema é bom agrupá-las em uma interface.

Até mais,
Roberto