Ajuda com Exceções.

[afcwb]

Boa tarde pessoal, estou com uma duvida e estou há alguns minutos tentando fazer porém sem secesso. A minha duvida é eu tenho uma classe Bicicleta e preciso tratar a entrada de um valor que é passado para um atributo através do método setVelocidade, a condição é que para ser aceito o valor passado ele nao pode ser negativo e maior que 100, porém não estou conseguindo retornar o erro ao compilar o programa, o que há de errado no código?

  public void setVelocidade(int velocidade){

		  this.velocidade = velocidade;
		 try{ 
		  if(this.velocidade > 100 || this.velocidade < 0);
	  }catch(Exception ex){
		  System.out.println("Valor para velocidade invalido");
	  }
  }

[DiF]

Não sei se resolve.. ou pode ser bobagem.. mas tenta separar os valores dentro do if assim:

  if((this.velocidade > 100) || (this.velocidade < 0))

[afcwb]

Já tentei isso também e continua não retornando, fiz um println e os valores que não deveriam ser atribuídos estão sendo.

[DiF]

Tenta adicionar a mensagem de erro para ver o que está retornando:

 if((this.velocidade > 100) || (this.velocidade < 0));
	  }catch(Exception ex){
		  System.out.println("O erro é: " + ex.getMessage());
	  }

só para ver o que retorna..

[afcwb]

Infelizmente ainda não deu, imagino que seja um erro bobo no bloco do try, porque se eu comento todas as linhas dele e do catch o valor passa pelo if e se não for válido ele não atribui.

[LuucasR.]

Tente assim:

public void setVelocidade(int velocidade){

		  this.velocidade = velocidade;
		 try{ 
		  if(this.velocidade > 100 || this.velocidade < 0);
	  }catch(Exception ex){
		  ex.printStackTrace();
	  }
  }

Se não resolver poste o código completo, para que eu possa testar aqui, e se possível encontrar a solução para seu problema

[r4uny]

Se você quer lançar uma exceção ao cair no if, faça o seguinte

public void setVelocidade(int velocidade){

        if(this.velocidade > 100 || this.velocidade < 0)
             throw new Exception("A velocidade deve ser maior que 0 e menor que 1000");

        this.velocidade = velocidade;


  }

Lembrando que não é indicado lançar exceções genéricas (Exception) o ideal é criar suas próprias classes de exceção (crie uma classe e extenda Exception) e então lança-las. Dessa maneira fica mais fácil de tratar posteriormente as exceções lançadas.

public class MyException extends Exception{
}

public void setVelocidade(int velocidade){

        if(this.velocidade > 100 || this.velocidade < 0)
             throw new MyException("A velocidade deve ser maior que 0 e menor que 1000");

        this.velocidade = velocidade;


  }