Spring JPA Probleme beim SPeichern von Sets

[8u3631984]

Hallo ich bin dabei ein Photo Album App zu programmieren.
Kurz zur Idee :

• Das Programm soll aus einem Verzeichnis jpg Bilder auszulesen und daraus Meta Data zu ermittel und in einem PhotoItem zu speichern. Dieses PhotoItem soll dann in einem PhotoAlbum Object abgespeichert werden.

Hier die entsprechenden Klassen :

@AllArgsConstructor
@NoArgsConstructor
@Builder(setterPrefix = "with")
@Getter
@Entity
@Table(name = "PHOTO_ITEM")
@ToString
public class PhotoItem implements Item {

    @Id
    @GeneratedValue
    @Column(updatable = false)
    private long ID;

    @NonNull
    @Builder.Default
    private String hashValue = Strings.EMPTY;

    @NonNull
    @OneToOne(cascade = CascadeType.ALL)
    private PhotoMetaData metaData;

    @NonNull
    @OneToOne(cascade = CascadeType.ALL)
    private PhotoLocation location;
}

@AllArgsConstructor
@NoArgsConstructor
@Builder(setterPrefix = "with")
@Getter
@Entity
@Table(name = "PHOTO_METADATA")
@ToString
public class PhotoMetaData implements DatabaseEntity {

        @Id
        @GeneratedValue
        @Column(updatable = false)
        private long ID;

        private int height;

        private int width;

        @NonNull
        @Builder.Default
        private String creationTimeStampAsString = Strings.EMPTY;

        private double latitude;

        private double longitude;
}

@AllArgsConstructor
@NoArgsConstructor
@Builder(setterPrefix = "with")
@Getter
@Entity
@Table(name = "PHOTO_LOCATION")
@ToString
public class PhotoLocation implements Location {

    @Id
    @GeneratedValue
    @Column(updatable = false)
    private long ID;

    @NonNull
    @Builder.Default
    private String country = Strings.EMPTY;

    @NonNull
    @Builder.Default
    private String city = Strings.EMPTY;

    @NonNull
    @Builder.Default
    private String postCode = Strings.EMPTY;

    @NonNull
    @Builder.Default
    private String street = Strings.EMPTY;

    @NonNull
    @Builder.Default
    private String houseNumber = Strings.EMPTY;
}

@AllArgsConstructor
@NoArgsConstructor
@Builder(setterPrefix = "with", toBuilder = true)
@Getter
@Entity
@Table(name = "PHOTO_ALBUM")
@ToString
public class PhotoAlbum implements Album {

        @Id
        @GeneratedValue
        @Column(updatable = false)
        private long ID;

        @NonNull
        @Builder.Default
        private String name = Strings.EMPTY;

        @NonNull
        @Singular
        @ManyToMany(fetch = FetchType.EAGER, cascade = CascadeType.ALL)
        private Set<PhotoItem> albumItems;
}

hier die Stelle wo die Bilder aus dem Verzeichnis gelesen und in eine Photo Item gebaut werden :

        var photoFiles = FileUtils.listFiles(new File(sourceFolder), new String[] { "jpg" }, true);
        var photoItems = Sets.newHashSet(photoFiles.stream().map(photoItemService::buildPhotoItem).toList());

        var photoAlbum = PhotoAlbum.builder()
                .withName(photoAlbumName)
                .withAlbumItems(photoItems)
                .build();

        var existingPhotoAlbum = getIfExists(photoAlbumName);
        if (existingPhotoAlbum.isPresent()) {
            log.info("{}", kv("existing photo album", photoAlbum));
            var existingPhotoItems = existingPhotoAlbum.get().getAlbumItems();

            photoAlbum = photoAlbum.toBuilder()
                    .withAlbumItems(existingPhotoItems)
                    .build();
        }

        var savedPhotoAlbum = repository.save(photoAlbum);
        log.info("{}", kv("saved photo album", savedPhotoAlbum));
        return savedPhotoAlbum;

Ich hoffe, der Code ist einigermaßen verstädnis.
Die zeile var savedPhotoAlbum = repository.save(photoAlbum); führt dann zu einer Excepption :
Caused by: org.hibernate.PersistentObjectException: detached entity passed to persist: my.photoalbum.photoItem.PhotoItem

Der Fehler tritt im folgenden Fall auf :

     var photoAlbumName = "TestPhotoAlbum";
        var service = new PhotoAlbumService(repository, photoItemService);

        var photoAlbum1 = service.buildAndSavePhotoAlbum(photoAlbumName,
                Paths.get("src", "test", "resources", "TestPhotoAlbum", "Album1").toString());
     var photoAlbum2 = service.buildAndSavePhotoAlbum(photoAlbumName,
                Paths.get("src", "test", "resources", "TestPhotoAlbum", "Album2").toString());

Wenn das PhotoAlbum 2 gespeichert werden soll tritt der Fehler auf.
Ich hoffe ihr konntet einigermaßen nachvollziehen, was ich meine und habt evtl eine Idee wie ich das Problem lösen kann

 

[KonradN]

Nach meinem Verständnis ist das Problem hier, dass Du doch bereits ein PhotoAlbum hast. Dieses lädst Du. Und dann erzeugst Du ein neues mit den Elementen des anderen PhotoAlbum.

Wieso hast Du das denn überhaupt so gemacht? Wieso verwendest Du nicht direkt das geladene PhotoAlbum Element? Dein Code macht so aus meiner Sicht keinen Sinn!

a) Die Prüfung, ob das PhotoAlbum bereits existiert, ist doch zuerst zu machen.
b) ist es nicht da, dann erzeugst Du es neu
c) ist es bereits da, dann nutzt Du das geladene Element.

Also - hier im Editor mal schnell editiert also ggf. mit Fehlern:

        var existingPhotoAlbum = getIfExists(photoAlbumName);
        if (existingPhotoAlbum.isPresent()) {
            log.info("{}", kv("existing photo album", photoAlbum));
            return existingPhotoAlbum.get();
        } else {
            var photoFiles = FileUtils.listFiles(new File(sourceFolder), new String[] { "jpg" }, true);
            var photoItems = Sets.newHashSet(photoFiles.stream().map(photoItemService::buildPhotoItem).toList());

            photoAlbum = PhotoAlbum.builder()
                .withName(photoAlbumName)
                .withAlbumItems(photoItems)
                .build();
            return repository.save(photoAlbum);
        }

Das dürfte von der Funktionalität dem entsprechen, das Du in deinem bisherigen Code hättest ... abgesehen davon, dass Dein Code nicht funktioniert, weil Du versuchst ein neues PhotoAlbum mit den gleichen Elementen zu erzeugen, die bereits in einem anderen PhotoAlbum vorhanden sind.

 

[8u3631984]

Hallo vielen Dank für deine Hilfe.
Der Gedanke war, dass ein PhotoAlbum aus mehrereren Verzeichnissen Bilder beinhalten kann. Außerdem kann auch in einem Verzeichnis ein neues bild hinzukommen. Ich scanne die Verzechnisse regelmäßig und aktualisieren dann das Album.
Ich habe meinen Code nun angepasst und nun klappt es :

        var photoAlbum = PhotoAlbum.builder()
                .withName(photoAlbumName)
                .build();
        var photoFiles = FileUtils.listFiles(new File(sourceFolder), new String[] { "jpg" }, true);
        log.info("{} in {}", kv("number of photo files", photoFiles.size()), kv("source folder", sourceFolder));
        var photoItems = Sets.newHashSet(photoFiles.stream().map(photoItemService::buildPhotoItem).toList());

        var existingPhotoAlbum = getIfExists(photoAlbumName);
        if (existingPhotoAlbum.isPresent()) {
            photoAlbum = existingPhotoAlbum.get();
            log.info("{}", kv("existing photo album", photoAlbum));
        }

        photoAlbum = photoAlbum.toBuilder().withAlbumItems(photoItems).build();

        var savedPhotoAlbum = repository.save(photoAlbum);
        log.info("{}", kv("saved photo album", savedPhotoAlbum));
        return savedPhotoAlbum;

Noch eine Nachfrage zu den Cascade Type : Ich habe bisher im mit ALL gearbeitet. Ist dies sinnvoll ?

 

[KonradN]

Ich habe meinen Code nun angepasst und nun klappt es :

Sorry, aber das wage ich zu bezweifeln...

        var existingPhotoAlbum = getIfExists(photoAlbumName);
        if (existingPhotoAlbum.isPresent()) {
            photoAlbum = existingPhotoAlbum.get();
            log.info("{}", kv("existing photo album", photoAlbum));
        }

        photoAlbum = photoAlbum.toBuilder().withAlbumItems(photoItems).build();

Wenn das PhotoAlbum vorhanden ist, dann setzt Du die Variable photoAlbum.
Aber im Anschluss überschreibst Du den Wert mit einem neuen photoAlbum.

Du bekommst also den Fehler nicht mehr, weil Du die ursprünglichen Elemente nicht mehr hast. Der Code von Dir würde das gleiche machen, wenn Du diesen oben gezeigten Block bis auf die letzte Zeile entfernst.

Was Du willst, ist evtl. den Code von mir oben mit eine Anpassung wie:

        var photoFiles = FileUtils.listFiles(new File(sourceFolder), new String[] { "jpg" }, true);
        var photoItems = Sets.newHashSet(photoFiles.stream().map(photoItemService::buildPhotoItem).toList());
        PhotoAlbum photoAlbum;

        var existingPhotoAlbum = getIfExists(photoAlbumName);
        if (existingPhotoAlbum.isPresent()) {
            log.info("{}", kv("existing photo album", photoAlbum));
            photoAlbum = existingPhotoAlbum.get();
            photoAlbum.getAlbumItems().addAll(photoItems);
        } else {
            photoAlbum = PhotoAlbum.builder()
                .withName(photoAlbumName)
                .withAlbumItems(photoItems)
                .build();
        }

        return repository.save(photoAlbum);

Sprich: Wenn das PhotoAlbum geladen werden konnte, dann fügst Du nur noch die neuen photoItems hinzu. (So hatte ich Dich jetzt verstanden...).

Wichtig ist: Wenn es das PhotoAlbum bereits gibt, dann erzeugst Du kein Neues mehr!