Логика бронирования не работает

Question

Логика бронирования не работает

1

Я реализую программу, которая создает и менеджер транспортного средства, с транспортным средством (объектами), и те объекты транспортного средства содержат объекты бронирования (с двумя строками, один для даты начала и один для даты окончания). Так что я сделал некоторые тесты бронирования, и это не Работа. Это должно быть в этом методе.

    public boolean isBookable(Booking wishedBooking) {

    boolean bookable = false;

    if ((wishedBooking.begin.compareTo(begin) != 1
            && wishedBooking.end.compareTo(begin) != 1)
            || (wishedBooking.begin.compareTo(end) != -1
                && wishedBooking.end.compareTo(end) != -1)) {
        bookable = true;
    }

    return bookable;             
}

Я попытался использовать compareTo. Даты имеют формат: ГГГГ/ММ/ДД ЧЧ: ММ

Таким образом, единственный способ, которым может быть совершена попытка, - это когда начало и конец этого рев или выше одного из брони. Если в одном случае это не так, это означает, что он не может быть забронирован, так как в один временной интервал временного интервала уже есть заказ. Я использую! =, Потому что возможно, что он может быть равен, поэтому результат будет равен 0.

Объект транспортного средства теперь повторяется с помощью метода над всеми объектами бронирования, чтобы проверить, может ли бронирование быть сделано, и его можно забронировать.

Где ошибка?

Goldi 27 апр. 2014, в 21:50

Источник

0

Вы уверены, что метод compareTo () работает с вашим форматом даты? в каком объекте вы храните даты, строку или дату?
Sara S 27 апр. 2014, в 19:49
0

Я не уверен, какова точная причина проблемы, но это может быть исправлено путем анализа дат с помощью SimpleDateFormat а затем с использованием Date.before и Date.after . Это, вероятно, также сделает ваш код более читабельным.
andersschuller 27 апр. 2014, в 19:52
0

@SaraSeppola Я попробовал в другом классе, чтобы проверить, работает ли он или нет. Дата хранится в двух строках - начале и конце. Они оба хранятся в объекте бронирования.
yemerra 27 апр. 2014, в 19:52
0

@andersschuller Хорошо, я хотел бы последовать твоему совету, но так как это домашнее задание, я должен следовать инструкциям.
yemerra 27 апр. 2014, в 19:53
0

хорошо, так как даты в формате String, compareTo (), вероятно, просто сравнит даты лексикографически
Sara S 27 апр. 2014, в 19:53
0

@SaraSeppola ммм, а это значит, что он не работает для чисел?
yemerra 27 апр. 2014, в 19:54
0

ну, лексикографически я думаю, что 143> 1222, потому что значения проверяются слева направо ... но так как все ваши даты имеют одинаковый формат, начиная с года, затем месяца и т. д., я думаю, что также может работать строка в этом случае. Если нет, используйте вместо этого объект javas Date (). и проверьте ответ @Mureinik
Sara S 27 апр. 2014, в 19:57

Показать ещё 5 комментариев

Теги:

java

date

iteration

compareto

1 ответ

Ещё вопросы

Вы уверены, что метод compareTo () работает с вашим форматом даты? в каком объекте вы храните даты, строку или дату?
Я не уверен, какова точная причина проблемы, но это может быть исправлено путем анализа дат с помощью SimpleDateFormat а затем с использованием Date.before и Date.after . Это, вероятно, также сделает ваш код более читабельным.
@SaraSeppola Я попробовал в другом классе, чтобы проверить, работает ли он или нет. Дата хранится в двух строках - начале и конце. Они оба хранятся в объекте бронирования.
@andersschuller Хорошо, я хотел бы последовать твоему совету, но так как это домашнее задание, я должен следовать инструкциям.
хорошо, так как даты в формате String, compareTo (), вероятно, просто сравнит даты лексикографически
@SaraSeppola ммм, а это значит, что он не работает для чисел?
ну, лексикографически я думаю, что 143> 1222, потому что значения проверяются слева направо ... но так как все ваши даты имеют одинаковый формат, начиная с года, затем месяца и т. д., я думаю, что также может работать строка в этом случае. Если нет, используйте вместо этого объект javas Date (). и проверьте ответ @Mureinik

Mureinik · Accepted Answer · 2014-04-27T17-44-00.000Z

Контракт a.compareTo(b) должен вернуть значение больше 0, если a "больше", чем b, значение меньше 0, если оно "меньше" или 0, если они равны. Использование 1 и -1 является чисто детализацией реализации, и вы не должны полагаться на него. То есть, ваш код должен изменить ваш код:

public boolean isBookable(Booking wishedBooking) {

    boolean bookable = false;

    if ((wishedBooking.begin.compareTo(begin) <= 0
            && wishedBooking.end.compareTo(begin) <= 0)
            || (wishedBooking.begin.compareTo(end) >= 0
                && wishedBooking.end.compareTo(end) >= 0)) {
        bookable = true;
    }

    return bookable;             
}

Обратите внимание: поскольку вы только возвращаете boolean, if может быть удален, и вы можете просто вернуть его условие:

public boolean isBookable(Booking wishedBooking) {
    return ((wishedBooking.begin.compareTo(begin) <= 0
            && wishedBooking.end.compareTo(begin) <= 0)
            || (wishedBooking.begin.compareTo(end) >= 0
                && wishedBooking.end.compareTo(end) >= 0));
}

Подождите, это значит, что это может быть и другое число, отличное от 1? Если он говорит, что он больше 0, значит, это может быть что-то еще, верно?
Боже мой, большое спасибо. Сейчас все в порядке. Какая глупая ошибка. Большое спасибо!
@DennisvonEich Да, это может быть любое число больше 0 . Например, с версией Java, которую я установил локально (1.7.0_45), "2014/04/27 23:00".compareTo("2011/11/16 23:00") дает 3 .
Ну, у меня есть небольшая проблема, может быть, это не большая проблема, поэтому я спрашиваю здесь. Мой тестовый класс работает только в 50% случаев. Я не вижу никакой причины, почему это работает только в 50% случаев.
Сложно ответить без исходного кода тестового класса. Не могли бы вы открыть новый пост с исходным кодом тестового класса и описанием того, как он терпит неудачу?