Restricoes

[AJBM]

Boas!

Eu estou a tentar eliminar um bebida. Eu tenho um Stock com 6 bebidas e quero eliminar 4. Em vez de so eliminar, elimina a bebida e apresenta esta mensagem A quantidade a remover e maior do que a existente: e nao devia apresentar

public boolean eliminarBebidas(Bebida b) {
       boolean eliminar = false;
           if(b!=null){
           Bebida[] tmp = new Bebida[bebida.length - 1];

           for (int i = 0; i < bebida.length; i++) {
               if ( b.getNome().equals(bebida[i].getNome()) && b.getStock() < bebida[i].getStock()) {
                   bebida[i].setStock(bebida[i].getStock() - b.getStock());
               }

               if (b.getNome().equals(bebida[i].getNome()) && bebida[i].getStock() == 0) {
                   System.arraycopy(bebida, 0, tmp, 0, i);
                   for (int a = i + 1; a < bebida.length; a++) {
                       tmp[a - 1] = bebida[a];
                   }
                   bebida = tmp;
               }
               else{
                   System.out.println("A quantidade a remover e maior do que a existente: ");
               }
           }
           listarStock();
           }
       return eliminar;
   }

Stock:
Nome: vodka    Stock: 4
Nome: 7up    Stock: 6
Nome: cola    Stock: 10

Eliminar:
A quantidade a remover e maior do que a existente:
A quantidade a remover e maior do que a existente:
A quantidade a remover e maior do que a existente:
Nome: vodka    Stock: 4
Nome: 7up    Stock: 3
Nome: cola    Stock: 10

[HappyHippyHippo]

???? eu ando a faltar às aulas de português ....

[joelbrito]

???? eu ando a faltar às aulas de português ....

;D ;D ;D

AJBM,

Duas dicas para ti:

- Quando estiveres a expor um dúvida tenta se o mais claro possível de maneira a que se perceba;

- Tem atenção ás chavetas dos ciclos e das condições

[HappyHippyHippo]

if (b.getNome().equals(bebida[i].getNome()) && bebida[i].getStock() == 0) {

quando a bebida não tem o mesmo nome a condição falha, logo vai aparecer a mensagem

[joelbrito]

if (b.getNome().equals(bebida[i].getNome()) && bebida[i].getStock() == 0) {

quando a bebida não tem o mesmo nome a condição falha, logo vai aparecer a mensagem

...e a mensagem só deveria aparecer quando falhar a condição:

 b.getStock() < bebida[i].getStock()

Quando a quantidade a remover for maior do que a quantidade em stock.

Certo?

[HappyHippyHippo]

apesar de que quando olho para este código da-me cócegas no fundo das costas, sim, deveria ser efectuada uma primeira verificação da bebida do array (para se descartar qualquer tipo de acção numa bebida diferente) e só depois verificar o stock