为什么 Spring JPA 双向 OneToMany 和 ManyToOne 不更新外键列？

Question

嗨，我正在使用 OneToMany 和 ManyToOne 双向关系学习 Spring JPA，在某些示例中，当我在两侧编写时，我看到 OneToMany 和 ManyToOne 关系，JPA 添加一个新列作为外键列并从父表。但是当我尝试我的时，该列始终为空白。这是我的代码的样子：

这是我的 Account.java 模型：

@Entity
@Table(name = "msAccount")
public class Account {

    @Id
    @NotBlank(message = "Not Blank")
    @Size(min = 0, max = 20)
    public String accountId;

    @NotBlank(message = "Not Blank")
    public String accountName;

    @NotBlank(message = "Not Blank")
    @Email(message = "Should be the right email")
    public String accountEmail;

    @NotBlank(message = "Not Blank")
    @Size(min = 5, message = "Minimal 5 char")
    public String accountAddress;

    @NotBlank(message = "Not Blank")
    public String town;

    @NotBlank(message = "Not Blank")
    public String npwp;

    @NotBlank(message = "Not Blank")
    public String phoneNumber;

    public String fax;

    public String remarks;

    @NotNull
    public Date entryTime;

    @NotNull
    public Boolean active;

    @OneToMany(mappedBy="account", cascade = CascadeType.ALL, orphanRemoval = true)
    public List<Dealer> dealer;

//getter setter skipped

}

这是我的 Dealer.java 模型：

@Entity
@Table(name = "msDealer")
public class Dealer {

    @Id
    @NotBlank(message = "Tidak Boleh Kosong")
    @Size(min = 0, max = 20)
    public String dealerId;

    @NotBlank(message = "Tidak Boleh Kosong")
    public String dealerName;

    @NotBlank(message = "Tidak Boleh Kosong")
    @Email(message = "Masukkan Email yang bener")
    public String dealerEmail;

    @NotBlank(message = "Tidak Boleh Kosong")
    @Size(min = 5, message = "Minimal 5 karakter")
    public String dealerAddress;

    @ManyToOne(fetch = FetchType.LAZY)
    public Account account;

//getter setter skipped

}

这是我的存储库：

@Repository
public interface AccountRepository extends JpaRepository<Account, Long> {

}

这是我的服务：

@Service
public class AccountService {

    @Autowired
    private AccountRepository accountRepository;

    public Account save(Account account) {
        return accountRepository.save(account);
    }

}

这是我的控制器：

@RestController
@RequestMapping("/api/account")
public class AccountController {
    private final Logger logger = LoggerFactory.getLogger(this.getClass());
    private final int ROW_PER_PAGE = 10;

    @Autowired
    private AccountService accountService;

    @PostMapping("/new")
    public ResponseEntity<Account> addAccount(@Valid @RequestBody Account account) {
        try {
            Account newAccount = accountService.save(account);
            return ResponseEntity.created(new URI("/api/account/" + newAccount.getAccountId()))
                    .body(account);
        } catch(Exception ex) {
            logger.error(ex.getMessage());
            return ResponseEntity.status(HttpStatus.BAD_REQUEST).build();
        }
    }

}

然后我将 JSON 发布到我的保存端点：

{
  "accountId": "USA001",
  "accountName": "string",
  "accountEmail": "string",
  "accountAddress": "string",
  "town": "string",
  "npwp": "string",
  "phoneNumber": "string",
  "fax": "string",
  "remarks": "string",
  "entryTime": "2020-04-07T15:01:29.404Z",
  "active": true,
  "dealer": [
    {
      "dealerId": "MMO001",
      "dealerName": "string",
      "dealerEmail": "string",
      "dealerAddress": "string"
    }
  ]
}

当我保存它时，显示在我的终端中的休眠看起来将查询插入到那 2 个表中，但是当我检查我的数据库表（即 postgresql）时，我发现有一个字段“account_account_id”为空，什么我想念这里吗？

我希望 Hibernate 像这样运行 sql：

insert into account (account_id, account_name, ...etc)
values ('USA001', 1)

insert into dealer (account_account_id, dealer_name, dealer_id, ...etc)
values ('USA001', 'New dealer 1', 'MMO001')

经过一番尝试，这是我的更新模型：

我的帐户.java 我删除 cascade = CascadeType.ALL, orphanRemoval = true

@Entity
@Table(name = "msAccount")
public class Account {

    @Id
    @NotBlank(message = "Tidak Boleh Kosong")
    @Size(min = 0, max = 20)
    public String accountId;

    @NotBlank(message = "Tidak Boleh Kosong")
    public String accountName;

    @NotBlank(message = "Tidak Boleh Kosong")
    @Email(message = "Masukkan Email yang bener")
    public String accountEmail;

    @NotBlank(message = "Tidak Boleh Kosong")
    @Size(min = 5, message = "Minimal 5 karakter")
    public String accountAddress;

    @NotBlank(message = "Tidak Boleh Kosong")
    public String town;

    @NotBlank(message = "Tidak Boleh Kosong")
    public String npwp;

    @NotBlank(message = "Tidak Boleh Kosong")
    public String phoneNumber;

    public String fax;

    public String remarks;

    @NotNull
    public Date entryTime;

    @NotNull
    public Boolean active;

    @OneToMany(mappedBy="account")
    // @JoinColumn(name = "accountId")
    public List<Dealer> dealer;

//getter setter skipped

}

这是我的 Dealer.java。添加了@JoinColumn：

@Entity
@Table(name = "msDealer")
public class Dealer {

    @Id
    @NotBlank(message = "Tidak Boleh Kosong")
    @Size(min = 0, max = 20)
    public String dealerId;

    @NotBlank(message = "Tidak Boleh Kosong")
    public String dealerName;

    @NotBlank(message = "Tidak Boleh Kosong")
    @Email(message = "Masukkan Email yang bener")
    public String dealerEmail;

    @NotBlank(message = "Tidak Boleh Kosong")
    @Size(min = 5, message = "Minimal 5 karakter")
    public String dealerAddress;

    @ManyToOne(fetch = FetchType.LAZY)
    @JoinColumn(name = "account_id")
    public Account account;

//getter setter skipped

}

现在错误越来越奇怪了，我在保存 JSON 数据时遇到了这个错误

> "Unable to find com.api.b2b.Model.Dealer with id MMO001; nested
> exception is javax.persistence.EntityNotFoundException: Unable to find
> com.api.b2b.Model.Dealer with id MMO001"

在某些教程中它有效，但我的不是，我做错了什么？

这是我的 github 仓库：https://github.com/Fly-Away/LearningSpring

Answer 1

为了在双向关系中保存孩子与父母，在孩子实体中设置父母也同步双方。

这里在dealer对象中设置account引用

public Account save(Account account) {
    for (Dealer dealer: account.getDealer()) {
        dealer.setAccount(account);
    }
    return accountRepository.save(account);
}

更新：

但是，如果您想使用单向关系，请删除 Dealer 实体中的 Account 关系。删除这部分

@ManyToOne(fetch = FetchType.LAZY)
public Account account;

然后更新Account表中的关系。

@OneToMany(cascade = CascadeType.ALL, orphanRemoval = true)
@JoinColumn(name = "account_id")
public List<Dealer> dealer;

这里我们删除了mappedBy，因为目前我们删除了Dealerside 中的映射，并添加了@JoinColumn 来定义我们用于帐户引用的列。

Answer 2

如果您在两个实体之间存在双向关系（此处为 Account 和 Dealer），您必须决定哪一方是所述关系的所有者。 默认情况下，One 端是所有者，它导致一个 Join-Table，在修改列表时更新。

由于您定义了 mappedBy 属性 (@OneToMany(mappedBy = "account"))，Many 方是关系的所有者。这意味着msDealer 表中的account 列将保存Account 的外键，然后不再使用Join-Table。 Join-Table 可能是在将 mappedBy 定义添加到注释之前初始化数据库的遗留物。

您的选择：

1. 让Dealer 保留所有者，不要使用连接表。如果您想观察数据库中的副作用，请查看msDealer.account 列。
2. 使用@JoinTable 注释强制使用此类表

Answer 3

您在孩子方面缺少@JoinColumn：

@Entity
@Table(name = "ms_dealer")
public class Dealer {

    @ManyToOne(fetch = FetchType.LAZY)
    @JoinColumn(name = "account_account_id")
    public Account account;

    // other fields

}

你在父端使用了mappedBy，但是在子端没有映射。您需要指出，Dealer 是关系所有者 - 它具有外键。

编辑：如果您保留（不合并）Account 实体及其子实体，则不应传递子实体的 ID。 （实际上，在持久化时传递任何 id 都是代码异味，很可能是性能杀手。）使用的 json 应该如下所示：

{
  "accountName": "string",
  "accountEmail": "string",
  "accountAddress": "string",
  "town": "string",
  "npwp": "string",
  "phoneNumber": "string",
  "fax": "string",
  "remarks": "string",
  "entryTime": "2020-04-07T15:01:29.404Z",
  "active": true,
  "dealer": [
    {
      "dealerName": "string",
      "dealerEmail": "string",
      "dealerAddress": "string"
    }
  ]
}

在保存双方同步之前可能还需要：

account.getDealer().forEach(d -> d.setAccount(account));

编辑：

从Author 编辑必须级联到子级：

@OneToMany(mappedBy = "account", cascade = CascadeType.ALL, orphanRemoval = true)
public List<Dealer> dealer;

您还可以在 Action 或 List<Dealer> 上添加 @JsonIgnore 以避免序列化为 json 时出现堆栈溢出。

Answer 4

正如您所说的您正在学习，我想给您一个详细的答案，以便您理解。你在这里缺少的是@JoinColumn。

@JoinColumn 可以用于关系的双方。这里的重点在于物理信息重复（列名）以及未优化的SQL将产生一些额外的UPDATE 语句的查询。

根据documentation：

由于多对一（几乎）始终是 JPA 规范中双向关系的所有者方，因此一对多关联由 @OneToMany(mappedBy=...)

通过基本代码示例了解

@Entity
public class Troop {
    @OneToMany(mappedBy="troop")
    public Set<Soldier> getSoldiers() {
    ...
}

@Entity
public class Soldier {
    @ManyToOne
    @JoinColumn(name="troop_fk")
    public Troop getTroop() {
    ...
} 

Troop 通过 troop 属性与Soldier 具有双向的一对多关系。您不必（不得）在mappedBy 端定义任何物理映射。

要将双向一对多映射，一对多作为拥有方，您必须删除mappedBy 元素并将多对一@JoinColumn 设置为insertable 和 updatable 为假。此解决方案未经优化，会产生一些额外的UPDATE 语句。

@Entity
public class Troop {
    @OneToMany
    @JoinColumn(name="troop_fk") //we need to duplicate the physical information
    public Set<Soldier> getSoldiers() {
    ...
}

@Entity
public class Soldier {
    @ManyToOne
    @JoinColumn(name="troop_fk", insertable=false, updatable=false)
    public Troop getTroop() {
    ...
}

如果您对给出的解释有任何其他问题，请在下方评论。 :)