boolean time format

[arunjib]

what is the error of the following code? why it always returns true?

    public boolean validate()
    {
        boolean valid = false;       
        if((hh>=0 || hh<=23)&&(mm>=0 || mm<=59)&&(ss>=0 || ss<=59))
        valid=true;
        return valid;
    }

[Petr]

Hi. what did you enter for variables mm, ss, hh?
Now It looks correct.

[JosAH]

what is the error of the following code? why it always returns true?

    public boolean validate()
    {
        boolean valid = false;       
        if((hh>=0 || hh<=23)&&(mm>=0 || mm<=59)&&(ss>=0 || ss<=59))
        valid=true;
        return valid;
    }

If, say, ss == 4242 it is larger than zero ...

kind regards,

Jos

[Toll]

Also, you can just "return ((hh>=0......". No need to create a variable to hold it, if all it does is return it. And if you really want the variable to return, you can use "boolean valid=((hh>=0...".

[JosAH]

Also, you can just "return ((hh>=0......". No need to create a variable to hold it, if all it does is return it. And if you really want the variable to return, you can use "boolean valid=((hh>=0...".

That doesn't address the problem at all and doesn't solve it either. Read my first reply for a hint of the problem (and the solution).

kind regards,

Jos