Patutkah saya membuat fungsi kemas kini untuk setiap harta, atau patutkah saya membenarkan kemas kini separa?

Kandungan soalan

Bagaimanakah proses kemas kini harus dilaksanakan? Adakah disyorkan untuk melakukan kemas kini paksa semua sifat (contoh pertama) atau patutkah saya menambah syarat semakan nol untuk membenarkan kemas kini separa (contoh kedua)?

Contoh pertama

@transactional
    @override
    public teamdto updateteambyid(long id, teamdto teamdto) throws apiexception {
        optional<team> teamoptional = teamrepository.findbyid(id);

        if (teamoptional.ispresent()) {
            team existingteam = teamoptional.get();

            // if objects id are sent in json, set attributes via id.
            if (teamdto != null) {

                if(teamdto.getcity() != null){
                    if (teamdto.getcity().getid() != null) {
                        citydto citydto = cityserviceimp.getcitybyid(teamdto.getcity().getid());
                        existingteam.setcity(citymapper.citydtotocity(citydto));
                    } else if(teamdto.getcity().getname() != null){
                        existingteam.setcity(citymapper.citydtotocity(cityserviceimp.createcity(teamdto.getcity())));
                    }
                }
                if(teamdto.getstadium() != null){
                    if (teamdto.getstadium().getid() != null) {
                        stadiumdto stadiumdto = stadiumserviceimp.getstadiumbyid(teamdto.getstadium().getid());
                        existingteam.setstadium(stadiummapper.stadiumdtotostadium(stadiumdto));
                    } else if(teamdto.getstadium().getname() != null){
                        existingteam.setstadium(stadiummapper.stadiumdtotostadium(stadiumserviceimp.createstadium(teamdto.getstadium())));
                    }
                }
                if(teamdto.getdivision() != null){
                    if (teamdto.getdivision().getid() != null) {
                        divisiondto divisiondto = divisionserviceimp.getdivisionbyid(teamdto.getdivision().getid());
                        existingteam.setdivision(divisionmapper.divisiondtotodivision(divisiondto));
                    } else if(teamdto.getdivision().getname() != null){
                        existingteam.setdivision(divisionmapper.divisiondtotodivision(divisionserviceimp.createdivision(teamdto.getdivision())));
                    }

                }

                return teammapper.teamtoteamdto(teamrepository.save(existingteam));
            }
        }
        throw new teamnotfoundexception(id);
    }

Contoh kedua

@Transactional
    @Override
    public TeamDTO updateTeamById(Long id, TeamDTO teamDTO) throws ApiException {
        Team team = teamMapper.teamDTOToTeam(teamDTO);
        Optional<Team> teamOptional = teamRepository.findById(id);

        if (teamOptional.isPresent()) {
            Team existingTeam = teamOptional.get();
            
            existingTeam.setName(team.getName());
            existingTeam.setDivision(team.getDivision());
            existingTeam.setCity(team.getCity());
            existingTeam.setStadium(team.getStadium());
            existingTeam.setHomeMatches(team.getHomeMatches());
            existingTeam.setAwayMatches(team.getAwayMatches());
            
            return teamMapper.teamToTeamDTO(teamRepository.save(existingTeam));
            }
        throw new TeamNotFoundException(id);
    }

Ini adalah projek peribadi yang kasar. Saya mempunyai soalan yang sama tentang kaedah pos.

Jawapan betul

Dengan kelas dan perhubungan yang ditakrifkan dengan betul, anda boleh menggunakan Kaedah Kedua, yang bersih dan mudah difahami.

Dengan takrifan perhubungan yang betul, saya maksudkan pemetaan antara kelas (sama ada mereka satu-dengan-satu atau satu-dengan-banyak dll.) dan jenis lata mereka . Dalam kes anda, anda boleh menggunakan CascadeType.ALL yang akan menyebarkan semua operasi pangkalan data yang dilakukan pada kelas induk kepada kelas anak. Jadi apabila anda menyimpan objek induk, objek anak anda juga disimpan. Walaupun subkelas anda tidak wujud dalam pangkalan data, ia akan mencipta satu untuk anda.

Sekarang, pendekatan pertama anda juga berfungsi dalam beberapa kes. Sebagai contoh, katakan anda menghantar permintaan yang mengandungi beberapa data bandar, di mana untuk ID yang diberikan, nama yang diberikan adalah berbeza daripada nama dalam pangkalan data. Jadi untuk pendekatan anda, anda tidak mengemas kini nama yang berbeza dalam pangkalan data tetapi dapatkan data sebenar daripada pangkalan data dan tetapkannya. Tetapi jika anda mengikuti kaedah kedua, ia akan menimpa data sedia ada nama bandar.
Untuk mengatasi isu dalam pendekatan kedua, anda boleh menukar lata kepada CascadeType.PERSIST, yang hanya akan menyebarkan operasi simpan kepada subkelas, bukan mengemas kini atau memadam operasi.

Atas ialah kandungan terperinci Patutkah saya membuat fungsi kemas kini untuk setiap harta, atau patutkah saya membenarkan kemas kini separa?. Untuk maklumat lanjut, sila ikut artikel berkaitan lain di laman web China PHP!