Блог

У меня есть класс домена, который работает нормально. Но теперь я добавил Long свойство, и оттуда все идет на юг.

Это случайное определение класса домена:

 class Accident {

    String fullDescription
    Date dateFrom
    Date dateTo
    Long lastUpdated

    Address address
    GpsPosition gpsPosition
    String providerDescription
    String link

    static constraints = {
        address nullable: true
        gpsPosition nullable: true
        providerDescription nullable: true
        link nullable: true
        fullDescription maxSize: 1024
        lastUpdated nullable: true
    }
}
  

И вот тут я получаю ошибку:

 @Transactional
class DatabaseCleanupService {

    static final long timeNotUpdatedToDelete = 1 * 1000 // milliseconds

    void cleanupDatabase() {
        Accident accident

        Accident.each {
            /*ERROR AT THIS LINE*/ if (System.currentTimeMillis() - it.lastUpdated > timeNotUpdatedToDelete) { 
              // do stuff
        }
    }
}
  

Я могу получить доступ к свойству из других частей кода, я успешно печатаю отладочную информацию, но на данный момент она просто вылетает. Что с ним не так?

Редактировать: в сообщении об ошибке говорится следующее:

Такого свойства нет: lastUpdated для класса: com.quarky.nehodyprahaserver.Accident Возможные решения: последнее обновление

Сначала вам нужно получить список аварий, изменив:

     Accident.each {
  

Для

     Accident.list().each {
  

Должно это исправить 🙂

В зависимости от того, сколько Accident экземпляров имеется в вашей базе данных, вероятно, было бы значительно эффективнее выполнять фильтрацию по дате на уровне базы данных, а не каждый раз извлекать всю таблицу в Groovy objects и фильтровать post-hoc:

 import groovy.time.TimeCategory

@Transactional
class DatabaseCleanupService {

    static final long timeNotUpdatedToDelete = 1 * 1000 // milliseconds

    void cleanupDatabase() {
        use(TimeCategory) {
            Accident.findAllByLastUpdatedLessThan(
              timeNotUpdatedToDelete.milliseconds.ago).each {

                // do stuff
            }
        }
    }
}
  

(или просто встроить константу как 1.second.ago — это выглядит неправильно, когда вы читаете ее как английское «последнее обновление менее одной секунды назад», но это правильно, поскольку LessThan означает «раньше, чем» при применении к Date объектам)