startDateとapplyDateが混在しているようなので統一した方がよいと思います

[ecormaksin]

/webapp/src/main/java/example/infrastructure/datasource/contract/ContractMapper.xmlでは、insertContractHistoryのみapplyDate.valueとなっているので、startDateに統一した方がよいと思います。（DBテーブルの日本語カラム「適用開始日」に対応させるのであれば、applicationStatDateに修正した方がなおよいと思います。）

isolating-the-domain/webapp/src/main/java/example/application/repository/ContractRepository.java

Line 17 in 6c494be

void registerHourlyWage(EmployeeNumber employeeNumber, Date startDate, WageCondition wageCondition);

isolating-the-domain/webapp/src/main/java/example/infrastructure/datasource/contract/ContractDataSource.java

Lines 23 to 28 in 6c494be

	public void registerHourlyWage(EmployeeNumber employeeNumber, Date applyDate, WageCondition wageCondition) {
	mapper.deleteContractData(employeeNumber, applyDate);
	Integer hourlyWageId = mapper.newHourlyWageIdentifier();
	mapper.insertContractHistory(employeeNumber, hourlyWageId, applyDate, wageCondition);
	mapper.insertContract(employeeNumber, applyDate, wageCondition);

isolating-the-domain/webapp/src/main/java/example/infrastructure/datasource/contract/ContractMapper.java

Line 16 in 6c494be

@Param("applyDate") Date applyDate,

isolating-the-domain/webapp/src/main/java/example/infrastructure/datasource/contract/ContractMapper.java

Line 22 in 6c494be

@Param("startDate") Date applyDate,

isolating-the-domain/webapp/src/main/java/example/infrastructure/datasource/contract/ContractMapper.java

Line 25 in 6c494be

void deleteContractData(@Param("employeeNumber") EmployeeNumber employeeNumber, @Param("startDate") Date startDate);

[masuda220]

なるほど。パラメータ名が揺らいでいますね。
合わせたほうがよいと思います。

名前ですが、startDateよりは、effectiveDateのほうが、契約の適用開始日という意味ではよいかもしれません。

ご提案いただいた applicationStartDateは、冗長に感じます。
パラメータ名とか変数名は、可能であれば２単語以内で表現したいと思っています。

[ecormaksin]

ご回答ありがとうございます。

パラメータ名とか変数名は、可能であれば２単語以内で表現したい

今後の設計、実装でも留意します。

[irof]

effectiveDate にしました

[ecormaksin]

ご対応くださり、ありがとうございます。