2017年DBFlute Fessでお話した内容です。 DBFlute Fess : https://connpass.com/event/70648/ DBFlute : http://dbflute.seasar.org/ Pull Request : https://github.com/dbflute/dbflute-core/pull/54

2. 目次

3. ● 舟木 初 (ふなき はじめ)
● U-NEXT新卒1年目
● ずっとLastaFluteで開発
● 今回が初めてのオープンソースコミット

4. DBFluteにコミットをしました。
カラムの連結検索で、
nullが含まれていても大丈夫なようにした

5. 複数カラムに分かれた住所に対する曖昧検索
市町村区 丁目番地 建物部屋番号 検索値
渋谷区 神宮前3-35-2 クローチェ6F ← 渋谷区神宮前
世田谷区 北沢1-2-3 メゾン201 ← 北沢1-2-3メゾン

6. (DBとしては検索カラムがあったほうがよかったかもしれないが・・・)
LikeSearchOptionで連結検索できるので・・！
cb.query().setAddress1_LikeSearch(body.address, op -> {
UnregisteredUserCB dreamCruiseCB = cb.dreamCruiseCB();
op.addCompoundColumn(dreamCruiseCB.specify().columnAddress2());
op.addCompoundColumn(dreamCruiseCB.specify().columnAddress3());
op.likeContain();
});

7. 連結するとこうなる
さすがDBFlute! 一件落着!
市町村区 丁目番地 建物部屋番号 → 連結した値
渋谷区 神宮前3-35-2 クローチェ6F → 渋谷区神宮前3-35-2クローチェ6F
世田谷区 北沢1-2-3 メゾン201 → 世田谷区北沢1-2-3メゾン201

8. あれ・・・？
連結したカラムの値にひとつでもNULLがあると・・・？
市町村区 丁目番地 建物部屋番号 → 連結した値
渋谷区 神宮前3-35-2 クローチェ6F → 渋谷区神宮前3-35-2クローチェ6F
世田谷区 北沢1-2-3 NULL → NULL

9. NULLが空文字になればいいのに。
市町村区 丁目番地 建物部屋番号 → 連結した値
渋谷区 神宮前3-35-2 クローチェ6F → 渋谷区神宮前3-35-2クローチェ6F
世田谷区 北沢1-2-3 NULL → 世田谷区北沢1-2-3””

10. ふなき「カラムを連結するとき、
NULLを空文字として扱ってくれる機能はありますか?」
くぼさん「ないねえ」
ふなき「了解です。まあ大丈夫です。」

11. くぼさん「別に、ふなきくん、
プルリクしても
いいんだからね笑」

12. プルリクしよう
1. dbflute-coreとdbflute-test-active-docksideをフォークして
2. dbflute-coreでcoalesceを使えるように修正して
3. テストコードをdbflute-test-active-dockside書いて
4. プルリクだす！
5. レビューしてもらう！

13. 発行されるSQLを見てみた。
select dfloc.ADDRESS1 as ADDRESS1, dfloc. ADDRESS2 as ADDRESS2
from USER dfloc
where dfloc.ADDRESS1 || dfloc.ADDRESS2 like ‘%渋谷区西参道%' escape '|'

14. こうなってほしい
select dfloc.ADDRESS1 as ADDRESS1, dfloc. ADDRESS2 as ADDRESS2
from USER dfloc
where dfloc.ADDRESS1 || dfloc.ADDRESS2 like ‘%渋谷区西参道%' escape '|'
coalesce(dfloc. ADDRESS1,'') || coalesce(dfloc. ADDRESS2,'')

15. DBFluteのなか
このあたりでcoalesceを足せばよさそう。
public abstract class ConditionKey implements Serializable {
・
・
protected ColumnRealName resolveCompoundColumn(ColumnRealName baseRealName, ConditionOption option) {
・
・
final List<SpecifiedColumn> compoundColumnList = option.getCompoundColumnList();
final List<ColumnRealName> realNameList = new ArrayList<ColumnRealName>();
realNameList.add(baseRealName); // already cipher
for (SpecifiedColumn specifiedColumn : compoundColumnList) {
realNameList.add(doResolveCompoundColumn(option, specifiedColumn));
}
final OnQueryStringConnector stringConnector = option.getStringConnector();
final String connected = stringConnector.connect(realNameList.toArray());
return ColumnRealName.create(null, new ColumnSqlName(connected));
}
・
・
}

16. public abstract class ConditionKey implements Serializable {
・
・
protected ColumnRealName resolveCompoundColumn(ColumnRealName baseRealName, ConditionOption option) {
・
・
final List<SpecifiedColumn> compoundColumnList = option.getCompoundColumnList();
final List<ColumnRealName> realNameList = new ArrayList<ColumnRealName>();
realNameList.add(baseRealName); // already cipher
realNameList.add(doResolveCompoundColumnOption(option, baseRealName)); // already cipher
for (SpecifiedColumn specifiedColumn : compoundColumnList) {
realNameList.add(doResolveCompoundColumn(option, specifiedColumn));
realNameList.add(doResolveCompoundColumnOption(option, doResolveCompoundColumnCipher(option, specifiedColumn)));
}
・
・
}
protected ColumnRealName doResolveCompoundColumnOption(ConditionOption option, ColumnRealName columnRealName) {
if (option.isNullCompoundedAsEmpty()) {
return toColumnRealName("coalesce(" + columnRealName + ",'')");
}
return columnRealName;
}
・
・
}

17. cb.query().setAddress1_LikeSearch("渋谷区西参道", op -> {
op.likeContain();
op.addCompoundColumn(dreamCruiseCB.specify().columnAddress2());
op.compoundsNullAsEmpty();
});
optionで呼び出す
coalesce(dfloc. ADDRESS1,'') || coalesce(dfloc. ADDRESS2,'')

18. // ## Assert ##
assertTrue(Srl.containsIgnoreCase(popCB().toDisplaySql(), "coalesce"));
assertHasAnyElement(vendorCheckList);
assertTrue(vendorCheckList.size() >= 3); // inserted 3 entities
for (VendorCheck vendorCheck : vendorCheckList) {
assertContains(vendorCheck.getTypeOfChar() + vendorCheck.getTypeOfVarchar(), "bcd");
}
テストコードも書いた
※VendorCheckというテスト用のテーブルがある

19. レビュー

20. ● https://github.com/dbflute/dbflute-core/pull/54/files
● https://github.com/dbflute-test/dbflute-test-active-
dockside/pull/11/files

21. 目次