Como no seu código tem a constante MIDNIGHT
, estou assumindo que a classe Time
representa as horas do dia.
Portanto, simplesmente somar 1 aos minutos pode trazer valores inválidos quando você tiver um Time
que representa 15:59, por exemplo. Ao simplesmente somar 1 aos minutos, você acabará com o valor correspondente a 15:60 (e depois 15:61, 15:62, e assim por diante), quando o correto seria ter o valor 16:00 (e depois 16:01, 16:02, etc).
Por isso você deve somar 1 aos minutos e ajustar o valor, caso ele seja 60. Para isso usamos o operador %
, que retorna o resto da divisão. Para garantir que o próximo minuto esteja entre 0 e 59, você pode fazer (this.minutes + 1) % 60
.
Em seguida, se o valor dos minutos for zero, o valor das horas deve ser ajustado para a próxima hora. Estou assumindo que as horas também não podem ultrapassar os valores válidos, e que depois das 23:59, o próximo valor é 00:00. Sendo assim, eu também uso o operador %
, mas agora com o valor 24 (garantindo que as horas tenham valores entre 0 e 23).
Sendo assim, o método tick()
fica assim:
public Time tick() {
int proximaHora = this.hours;
int proximoMinuto = (this.minutes + 1) % 60;
// se o próximo minuto é zero, deve ir para a próxima hora
if (proximoMinuto == 0) {
proximaHora = (proximaHora + 1) % 24;
}
return new Time(proximaHora, proximoMinuto);
}
Além disso, seria interessante o construtor validar os valores recebidos, para evitar que você crie horas inválidas como 99:99, por exemplo. Caso os valores sejam inválidos, você pode lançar um IllegalArgumentException
(que é a exceção nativa que serve para indicar que um parâmetro inválido foi passado):
public Time(int h, int m) {
if (h < 0 || h > 24) {
throw new IllegalArgumentException("Valor inválido para as horas: " + h);
}
if (m < 0 || m > 59) {
throw new IllegalArgumentException("Valor inválido para os minutos: " + m);
}
hours = h;
minutes = m;
}