public int getNumeroItem() { return this.numeroItem=numeroItem; }
public String getDescricao() { return this.descricao=descricao; }
public int getQuantidadeComprada() { if (quantidadeComprada < 0) { return this.quantidadeComprada = 0; } else { return this.quantidadeComprada = quantidadeComprada; }
}
public double getValorUnitario() { if (valorUnitario < 0) { return this.valorUnitario = 0.00; } else { return this.valorUnitario = valorUnitario;
} }
}
Lista de comentários
bokomoko
Isso aqui tá errado public int getNumeroItem() { return this.numeroItem=numeroItem; }
O correto seria public int getNumeroItem() { return this.numeroItem; }
public String getDescricao() { return this.descricao=descricao; } também errado. O correto seria public String getDescricao() { return this.descricao; }
Aqui também está errado public int getQuantidadeComprada() { if (quantidadeComprada < 0) { return this.quantidadeComprada = 0; } else { return this.quantidadeComprada = quantidadeComprada; }
O correto seria public int getQuantidadeComprada() { if (quantidadeComprada < 0) { return 0; } else { return this.quantidadeComprada ; }
Outra coisa geral. Se vai criar/usar os métodos get e set então é bom tornar os atributos (campos) private.
1 votes Thanks 0
PaulinhaBela
oi , este ai da quantidade comprada no meu , não foi não .
bokomoko
return quantidadeComprada = 0 ; não faz sentido.
Quando se tem um método para obter informaçào, no caso, quer saber quanto foi a qtde comprada, não se deve atribuir valores. APenas consultar.
Se a quantidade comprada for negativa, é melhor retornar 0 apenas.
Lista de comentários
public int getNumeroItem()
{
return this.numeroItem=numeroItem;
}
O correto seria
public int getNumeroItem()
{
return this.numeroItem;
}
public String getDescricao()
{
return this.descricao=descricao;
}
também errado. O correto seria
public String getDescricao()
{
return this.descricao;
}
Aqui também está errado
public int getQuantidadeComprada()
{
if (quantidadeComprada < 0)
{
return this.quantidadeComprada = 0;
} else
{
return this.quantidadeComprada = quantidadeComprada;
}
O correto seria
public int getQuantidadeComprada()
{
if (quantidadeComprada < 0)
{
return 0;
} else
{
return this.quantidadeComprada ;
}
Outra coisa geral. Se vai criar/usar os métodos get e set então é bom tornar os atributos (campos) private.