プロジェクト

全般

プロフィール

コードレビュー #8222

完了

ND10180_開発_基本、運賃・経費契約照会選択 コードレビュー

浦田 光祐 さんが5ヶ月前に追加. 4ヶ月前に更新.

ステータス:
終了
優先度:
通常
担当者:
開始日:
期日:
進捗率:

0%

予定工数:
実開始日:
DD名:
INCLUDE:
DSN:
バグ区分:
QA分析:
方針(第一貨物):
実終了日:
ツールのバージョン:
AsIsSource行:
進捗計画:
設計・開発者:
画面ID:
機能区分:
機能ID:
レビュー指摘区分:
検査完了日:
検査依頼日:

ファイル

clipboard-202605211539-jiokf.png (4.34 KB) clipboard-202605211539-jiokf.png 渡邉 拓英, 2026-05-21 03:39 午後
clipboard-202605211539-kwlwp.png (21.8 KB) clipboard-202605211539-kwlwp.png 渡邉 拓英, 2026-05-21 03:39 午後
clipboard-202605211624-g3y6m.png (12.1 KB) clipboard-202605211624-g3y6m.png 渡邉 拓英, 2026-05-21 04:24 午後
clipboard-202605211626-fzhh7.png (33 KB) clipboard-202605211626-fzhh7.png 渡邉 拓英, 2026-05-21 04:26 午後
clipboard-202605211645-9mzjc.png (9.52 KB) clipboard-202605211645-9mzjc.png 渡邉 拓英, 2026-05-21 04:45 午後
clipboard-202606091322-ojgrr.png (18.6 KB) clipboard-202606091322-ojgrr.png 渡邉 拓英, 2026-06-09 01:22 午後
clipboard-202606091323-4isw5.png (33.5 KB) clipboard-202606091323-4isw5.png 渡邉 拓英, 2026-06-09 01:23 午後
clipboard-202606091346-idhje.png (10.6 KB) clipboard-202606091346-idhje.png 渡邉 拓英, 2026-06-09 01:46 午後
clipboard-202606091352-kurzv.png (10.8 KB) clipboard-202606091352-kurzv.png 渡邉 拓英, 2026-06-09 01:52 午後
clipboard-202606091352-xdw7t.png (62.2 KB) clipboard-202606091352-xdw7t.png 渡邉 拓英, 2026-06-09 01:52 午後
clipboard-202606091352-go4tt.png (81.8 KB) clipboard-202606091352-go4tt.png 渡邉 拓英, 2026-06-09 01:52 午後
clipboard-202606091354-2h6en.png (2.91 KB) clipboard-202606091354-2h6en.png 渡邉 拓英, 2026-06-09 01:54 午後
clipboard-202606091354-azafe.png (3.12 KB) clipboard-202606091354-azafe.png 渡邉 拓英, 2026-06-09 01:54 午後
clipboard-202606091355-zkcmf.png (28.5 KB) clipboard-202606091355-zkcmf.png 渡邉 拓英, 2026-06-09 01:55 午後
clipboard-202606091355-3q1qc.png (14.1 KB) clipboard-202606091355-3q1qc.png 渡邉 拓英, 2026-06-09 01:55 午後
clipboard-202606091407-6imga.png (5.83 KB) clipboard-202606091407-6imga.png 渡邉 拓英, 2026-06-09 02:07 午後
clipboard-202606091407-rh8vt.png (4.07 KB) clipboard-202606091407-rh8vt.png 渡邉 拓英, 2026-06-09 02:07 午後
clipboard-202606091408-qr5ho.png (23.7 KB) clipboard-202606091408-qr5ho.png 渡邉 拓英, 2026-06-09 02:08 午後
clipboard-202606091408-1qiep.png (2.56 KB) clipboard-202606091408-1qiep.png 渡邉 拓英, 2026-06-09 02:08 午後
clipboard-202606091408-r6afe.png (12.3 KB) clipboard-202606091408-r6afe.png 渡邉 拓英, 2026-06-09 02:08 午後
clipboard-202606091409-yhntd.png (9.25 KB) clipboard-202606091409-yhntd.png 渡邉 拓英, 2026-06-09 02:09 午後
clipboard-202606091409-sdi4q.png (5.57 KB) clipboard-202606091409-sdi4q.png 渡邉 拓英, 2026-06-09 02:09 午後
clipboard-202606091409-ruo69.png (2.49 KB) clipboard-202606091409-ruo69.png 渡邉 拓英, 2026-06-09 02:09 午後
clipboard-202606091409-z1bmw.png (3.35 KB) clipboard-202606091409-z1bmw.png 渡邉 拓英, 2026-06-09 02:09 午後
clipboard-202606091410-bpvbd.png (3.35 KB) clipboard-202606091410-bpvbd.png 渡邉 拓英, 2026-06-09 02:10 午後
clipboard-202606111700-g3vuy.png (158 KB) clipboard-202606111700-g3vuy.png 渡邉 拓英, 2026-06-11 05:00 午後
clipboard-202606111706-jks28.png (47.8 KB) clipboard-202606111706-jks28.png 渡邉 拓英, 2026-06-11 05:06 午後
clipboard-202605211539-jiokf.png
clipboard-202605211539-kwlwp.png
clipboard-202605211624-g3y6m.png
clipboard-202605211626-fzhh7.png
clipboard-202605211645-9mzjc.png
clipboard-202606091322-ojgrr.png
clipboard-202606091323-4isw5.png
clipboard-202606091346-idhje.png
clipboard-202606091352-kurzv.png
clipboard-202606091352-xdw7t.png
clipboard-202606091352-go4tt.png
clipboard-202606091354-2h6en.png
clipboard-202606091354-azafe.png
clipboard-202606091355-zkcmf.png
clipboard-202606091355-3q1qc.png
clipboard-202606091407-6imga.png
clipboard-202606091407-rh8vt.png
clipboard-202606091408-qr5ho.png
clipboard-202606091408-1qiep.png
clipboard-202606091408-r6afe.png
clipboard-202606091409-yhntd.png
clipboard-202606091409-sdi4q.png
clipboard-202606091409-ruo69.png
clipboard-202606091409-z1bmw.png
clipboard-202606091410-bpvbd.png
clipboard-202606111700-g3vuy.png
clipboard-202606111706-jks28.png

渡邉 拓英 さんが5ヶ月前に更新 · 編集済み

  • ファイル clipboard-202605211539-jiokf.png を追加
  • ファイル clipboard-202605211539-kwlwp.png を追加
clipboard-202609241555-8umzb.png
clipboard-202609241555-e7hgn.png

半角スペースではなく、全角スペースでお願いします。

Ecilpseの成形のフォーマットで、コメントの全角スペースは半角スペースにしてしまっている。(いまいち・・・)
ツール出力後のコメント部分は、100%信じないで、全角スペースかどうかは設計書を見てください。

渡邉 拓英 さんが5ヶ月前に更新 · 編集済み

  • ファイル clipboard-202605211624-g3y6m.png を追加
  • ファイル clipboard-202605211626-fzhh7.png を追加
clipboard-202503241404-elrnk.png
clipboard-202503271427-5qbnp.png

ここの実装になると思いますが。

まず、設計書と実装があっていません。
VKCUNK1[顧客マスタ(運賃基本情報)]は設計書上、どこにも記載はありません。

記載が不足しているからと言って、想定で設計書に記載がない処理を行わないこと!!
>getKUNKCODEByVKCUN1Wのようなメソッド自体は、上記に該当しません。(これは便利にするメソッドですよね?)

// 『方面コードを取得』
houmenCodeList = getKUNKCODEsFromVKCUN1W(vkcun1w.get(), vkcun1w.get().getKUNKAKOS(), houmenCodeList);
// 『VKCUN2W[顧客マスタ(運賃方面別情報)適用前]を取得する。』
List<VKCUN2W> vkcun2wList = vkcun2wService.get4Data(getCommAreaDt("ND10010", "COMM_CUSTCD"), houmenCodeList);

「方面コードを取得し、顧客マスタ(運賃方面別情報)適用前を取得する」記載は一切ないため、NGです。
きっと最終的には、そのような処理になりますが、設計書に書いていない&コーディングしていることがNGです。

→設計書が正しくない、不足しているのであれば、対応してもらう必要があります。
(※全体的に見直してください。)

処理は、おそらくは合っていますが、実装者が決めるところではありません。

渡邉 拓英 さんが5ヶ月前に更新 · 編集済み

  • ファイル clipboard-202605211645-9mzjc.png を追加
clipboard-202503281105-g7uod.png

「FIXME」が残っています。

渡邉 拓英 さんが5ヶ月前に更新 · 編集済み

一旦、全体を見直してください。

渡邉 拓英 さんが5ヶ月前に更新 · 編集済み

  • ステータス を 新規 から フィードバック に変更
  • 担当者 を 渡邉 拓英 から 浦田 光祐 に変更

浦田 光祐 さんが5ヶ月前に更新 · 編集済み

  • ステータス を フィードバック から 設計中(第一貨物) に変更
  • 担当者 を 浦田 光祐 から 渡邉 拓英 に変更

現行処理を確認し、設計書とプログラム全体を見直し修正しました。

改めてコードレビューよろしくお願いいたします。(修正のリビジョン 3680)

渡邉 拓英 さんが5ヶ月前に更新 · 編集済み

  • ファイル clipboard-202606091322-ojgrr.png を追加
  • ファイル clipboard-202606091323-4isw5.png を追加
clipboard-202609281107-ht6rp.png
clipboard-202609281112-mrxrm.png

if文 階層が違う。

渡邉 拓英 さんが5ヶ月前に更新 · 編集済み

全角スペース

        case "Y" -> {
          // 変数.エラーコード ← "Y LOG"
          errCd = "Y LOG";
          // 変数.エラーメッセージ ← " ロジ収入計上対象コード【Yロジ】"
          errMsg = " ロジ収入計上対象コード【Yロジ】";
        }
        // ・"S" の場合
        case "S" -> {
          // 変数.エラーコード ← "S LOG"
          errCd = "S LOG";
          // 変数.エラーメッセージ ← " ロジ収入計上対象コード【Sロジ】"
          errMsg = " ロジ収入計上対象コード【Sロジ】";
        }

・"Y" の場合																									
	変数.エラーコード ← "Y LOG"																								
	変数.エラーメッセージ ← " ロジ収入計上対象コード【Yロジ】"																								
・"S" の場合																									
	変数.エラーコード ← "S LOG"																								
	変数.エラーメッセージ ← " ロジ収入計上対象コード【Sロジ】"																								

渡邉 拓英 さんが5ヶ月前に更新 · 編集済み

        // VKCUN1W[顧客マスタ(運賃基本情報)適用前].VK20COPY[運賃契約コピー元顧客コード]の先頭2桁 = "88" の場合
        String msg = vkcun1w.get().getVK20COPY();

変数名がおかしい。

msgメッセージは、使用するときの使い方で、変数として保持するのは、「運賃契約コピー元顧客コード」
運賃契約コピー元顧客コードに由来する名前であるべき。

渡邉 拓英 さんが5ヶ月前に更新 · 編集済み

  • ファイル clipboard-202606091346-idhje.png を追加
clipboard-202609281301-ck2qf.png

半角スペースの文字数が違う

      // プロパティ.顧客獲得従業員名 ← " ---- "
      nd10180Data.setKakutokuJugyoinNm(" ---- ");

渡邉 拓英 さんが5ヶ月前に更新 · 編集済み

半角スペースが異なる

          // プロパティ.顧客担当者所属等 ← "退職 "
          nd10180Data.setTantousyaSyozoku("退職");

渡邉 拓英 さんが5ヶ月前に更新 · 編集済み

全角スペース

・プロパティ.マスタ従業員職種コード = "   " の場合																				『ここだけ=』					
	プロパティ.顧客担当者所属等 ← "?? "																								
・プロパティ.マスタ従業員職種コード = "000" の場合																				『ここだけ=』					
	プロパティ.顧客担当者所属等 ← "?? "																								
																					


            // プロパティ.顧客担当者所属等 ← "?? "
            nd10180Data.setTantousyaSyozoku("?? ");
            // ・プロパティ.マスタ従業員職種コード = "000" の場合 『ここだけ=』
          } else if ("000".equals(jobCd)) {
            // プロパティ.顧客担当者所属等 ← "?? "
            nd10180Data.setTantousyaSyozoku("?? ");

渡邉 拓英 さんが5ヶ月前に更新 · 編集済み

  • ファイル clipboard-202606091352-kurzv.png を追加
clipboard-202609281303-i43jz.png

渡邉 拓英 さんが5ヶ月前に更新 · 編集済み

  • ファイル clipboard-202606091352-xdw7t.png を追加
  • ファイル clipboard-202606091352-go4tt.png を追加
clipboard-202609281303-aapzt.png
clipboard-202609281303-4lxbg.png

全角スペースが足りない



プロパティ.マスタ従業員職種コードで処理を分岐。																													
|	・プロパティ.マスタ従業員職種コード = "   " の場合																				『ここだけ=』								
|		プロパティ.顧客担当者所属等 ← "?? "																											
|	・プロパティ.マスタ従業員職種コード = "000" の場合																				『ここだけ=』								
|		プロパティ.顧客担当者所属等 ← "?? "																											
|	・プロパティ.マスタ従業員職種コード <= "107" の場合																												
|		プロパティ.顧客担当者所属等 ← "管理者"																											
|	・プロパティ.マスタ従業員職種コード <= "205" の場合																												
|		プロパティ.顧客担当者所属等 ← "事務 "																											
|	・プロパティ.マスタ従業員職種コード <= "222" の場合																												
|		プロパティ.顧客担当者所属等 ← "事務 "																											
|	・プロパティ.マスタ従業員職種コード <= "265" の場合																												
|		プロパティ.顧客担当者所属等 ← "事務 "																											
|	・プロパティ.マスタ従業員職種コード <= "310" の場合																												
|		プロパティ.顧客担当者所属等 ← "その他"																											
|	・プロパティ.マスタ従業員職種コード <= "422" の場合																												
|		プロパティ.顧客担当者所属等 ← "その他"																											
|	・プロパティ.マスタ従業員職種コード <= "556" の場合																												
|		プロパティ.顧客担当者所属等 ← "その他"																											
|	・プロパティ.マスタ従業員職種コード <= "601" の場合																												
|		プロパティ.顧客担当者所属等 ← "その他"																											
|	・プロパティ.マスタ従業員職種コード <= "603" の場合																												
|		プロパティ.顧客担当者所属等 ← "集配 "																											
|	・プロパティ.マスタ従業員職種コード <= "605" の場合																												
|		プロパティ.顧客担当者所属等 ← "作業 "																											
|	・プロパティ.マスタ従業員職種コード <= "613" の場合																												
|		プロパティ.顧客担当者所属等 ← "その他"																											
|	・プロパティ.マスタ従業員職種コード <= "631" の場合																												
|		プロパティ.顧客担当者所属等 ← "集配 "																											
|	・プロパティ.マスタ従業員職種コード <= "660" の場合																												
|		プロパティ.顧客担当者所属等 ← "その他"																											
|	・プロパティ.マスタ従業員職種コード <= "662" の場合																												
|		プロパティ.顧客担当者所属等 ← "集配 "																											
|	・プロパティ.マスタ従業員職種コード <= "664" の場合																												
|		プロパティ.顧客担当者所属等 ← "作業 "																											
|	・プロパティ.マスタ従業員職種コード <= "704" の場合																												
|		プロパティ.顧客担当者所属等 ← "その他"																											
|	・プロパティ.マスタ従業員職種コード <= "705" の場合																												
|		プロパティ.顧客担当者所属等 ← "作業 "																											
|	・プロパティ.マスタ従業員職種コード <= "706" の場合																												
|		プロパティ.顧客担当者所属等 ← "その他"																											
|	・プロパティ.マスタ従業員職種コード <= "722" の場合																												
|		プロパティ.顧客担当者所属等 ← "作業 "																											
|	・上記以外																												
|		プロパティ.顧客担当者所属等 ← プロパティ.マスタ従業員職種コード & "  "																											

渡邉 拓英 さんが5ヶ月前に更新 · 編集済み

  • ファイル clipboard-202606091354-2h6en.png を追加
  • ファイル clipboard-202606091354-azafe.png を追加
clipboard-202609281306-qtbvs.png
clipboard-202609281357-yd9ve.png

渡邉 拓英 さんが5ヶ月前に更新 · 編集済み

  • ファイル clipboard-202606091355-zkcmf.png を追加
  • ファイル clipboard-202606091355-3q1qc.png を追加
clipboard-202609281357-xhcdy.png
clipboard-202609281530-dv6cz.png

全角スペース スペースの数

『基本_入力責任者 氏名・所属店名取得』																																				
数値型に変換(プロパティ.基本_入力責任者コード) > 0 の場合																																				
|	従業員名・所属取得メソッドを実行。																																			
|		引数)			従業員コード ← プロパティ.基本_入力責任者コード																															
|																																				
|	プロパティ.基本_入力責任者名 ← プロパティ.マスタ従業員名																																			
|																																				
|	従業員名・所属取得メソッド.戻り値 = 1 の場合																																			
|	|	プロパティ.基本_入力責任者所属名 ← "   " & プロパティ.マスタ店所名 & "所属"																																		
|	上記以外																																			
|	|	プロパティ.基本_入力責任者所属名 ← "   " & プロパティ.マスタ店所名																																		
上記以外																																				
	プロパティ.基本_入力責任者名 ← "   ----        "																																			

渡邉 拓英 さんが5ヶ月前に更新 · 編集済み

のこりも全部

『運契_入力責任者 氏名・所属店名取得』
数値型に変換(プロパティ.運契_入力責任者コード) > 0 の場合
| 従業員名・所属取得メソッドを実行。
| 引数) 従業員コード ← プロパティ.運契_入力責任者コード
|
| プロパティ.運契_入力責任者名 ← プロパティ.マスタ従業員名
|
| プロパティ.運契_入力責任者所属名 = "" の場合
| | 従業員名・所属取得メソッド.戻り値 = 1 の場合
| | | プロパティ.運契_入力責任者所属名 ← "   " & プロパティ.マスタ店所名 & "所属"
| | 上記以外
| | | プロパティ.運契_入力責任者所属名 ← "   " & プロパティ.マスタ店所名
| 上記以外
| (何もしない)
上記以外
プロパティ.運契_入力責任者名 ← " ---- "

『経費_入力責任者 氏名・所属店名取得』
数値型に変換(プロパティ.経費_入力責任者コード) > 0 の場合
| 従業員名・所属取得メソッドを実行。
| 引数) 従業員コード ← プロパティ.経費_入力責任者コード
|
| プロパティ.経費_入力責任者名 ← プロパティ.マスタ従業員名
|
| 従業員名・所属取得メソッド.戻り値 = 1 の場合
| | プロパティ.経費_入力責任者所属名 ← "   " & プロパティ.マスタ店所名 & "所属"
| 上記以外
| | プロパティ.経費_入力責任者所属名 ← "   " & プロパティ.マスタ店所名
上記以外
プロパティ.経費_入力責任者名 ← " ---- "

プロパティ.データステータス = 9 の場合
変数.エラーコード ← ""
変数.エラーメッセージ ← "削除済の顧客です。  (削除日付" & プロパティ.更新年月日 & ")"

渡邉 拓英 さんが5ヶ月前に更新 · 編集済み

  • ファイル clipboard-202606091407-6imga.png を追加
  • ファイル clipboard-202606091407-rh8vt.png を追加
  • ファイル clipboard-202606091408-qr5ho.png を追加
  • ファイル clipboard-202606091408-1qiep.png を追加
  • ファイル clipboard-202606091408-r6afe.png を追加
  • ファイル clipboard-202606091409-yhntd.png を追加
  • ファイル clipboard-202606091409-sdi4q.png を追加
  • ファイル clipboard-202606091409-ruo69.png を追加
  • ファイル clipboard-202606091409-z1bmw.png を追加
  • ファイル clipboard-202606091410-bpvbd.png を追加
clipboard-202609281549-dy4oa.png
8.png
clipboard-202609281630-memaw.png
clipboard-202609291442-vobq4.png
clipboard-202609281634-dmphg.png
clipboard-202609281651-rou9i.png
clipboard-202609281651-9h52c.png
clipboard-202609281657-xmexf.png
clipboard-202609281658-yb6y4.png
clipboard-202609281659-bqxhp.png

insertXXXXXXX

下記の処理は、設計書のどこに記載がなりますか???

例えば、

VKCUSTW[顧客マスタ(基本情報)適用前]の内容を																																	
OSCOMADT[COMM_AREA明細].ND10010.COMM_VKCUSTW[顧客マスタ(基本情報)_変更情報]に登録する。																																	

で、VKCUSTWをOSCOMADTに保存する内容は読み取れますが、どの構造体を使用して書込みするかは、わからないと思いますが???

また、"1"や"・"は、どこから出てきたのでしょうか?


insertSTVKCUNK1ByVKCUNK1全体

insertSTVKCUI2WfromVKCUIT2全体

渡邉 拓英 さんが5ヶ月前に更新 · 編集済み

  • ステータス を 設計中(第一貨物) から フィードバック に変更
  • 担当者 を 渡邉 拓英 から 浦田 光祐 に変更

浦田 光祐 さんが5ヶ月前に更新 · 編集済み

  • 担当者 を 浦田 光祐 から 渡邉 拓英 に変更

レビューありがとうございます。コメント#7~17について再度修正を加えました。

<<コメント#18について>>
"1"と"・"などは入力画面のプログラムから同じ処理を引用したままで実装し消し損ねてました。現行での処理と比較の上、削除しても問題ないと確認できたので削除しました。

CommAreaへJSON形式での登録の場合、使用する構造体はCOMM_AREA仕様書をキーで参照すると以前お聞きしました。
10種のinsertXXXXXXXも該当しますが、設計書にどこまで明記が必要でしょうか?

渡邉 拓英 さんが5ヶ月前に更新 · 編集済み

  • ファイル clipboard-202606111700-g3vuy.png を追加
  • ファイル clipboard-202606111706-jks28.png を追加
clipboard-202605281732-0ep6f.png
clipboard-202604231742-rfsuq.png
で、VKCUSTWをOSCOMADTに保存する内容は読み取れますが、どの構造体を使用して書込みするかは、わからないと思いますが???

>>>CommAreaへJSON形式での登録の場合、使用する構造体はCOMM_AREA仕様書をキーで参照すると以前お聞きしました。

00.共通仕様\COMM_AREA仕様書.xlsxに構造体が載っていましたね。>構造体どの構造体かは、わかりますね。

10種のinsertXXXXXXXも該当しますが、設計書にどこまで明記が必要でしょうか?
>>>構造体がわかればいい。→「00.共通仕様\COMM_AREA仕様書.xlsx」 セットする値がわかればいいです。(以前は、"1"と"・"が、設計書に無い処理だったのでNG)
>>>insertXXXXXXX自体は設計書外なので特に記載はいらないです。(設計書に)
   同じフィールド名同士をマッピングするのですよね?(ここ重要) 
   マッピング仕様として設計書に記載してもらっていいですが、手間ですよね。なので、「convertEntity」 or 「convertEntityNoError」を使ってもらっていいですよ。(このままでもいいです。)
   同じフィールド名でないものは、記載が必要です。

設計書では、XxxxをCommAreaに保存する。とのことなので、XxxxをStXxxxに入れて保存すればOK。
細かくかくのであれば、マッピング仕様を書く。ざっくりであれば、今のままかと。

例えば、

      // VKCUI1W[顧客マスタ(委託料情報(1))適用前]を取得する。
      Optional<VKCUI1W> vkcui1w = vkcui1wService.get1Data(getCommAreaDt("ND10010", "COMM_CUSTCD"));
      // VKCUI1W[顧客マスタ(委託料情報(1))適用前]の内容を
      // OSCOMADT[COMM_AREA明細].ND10140.COMM_VKCUI1W[顧客マスタ(委託料情報(1))_変更情報]に登録する。
      insertSTVKCUN1WByVKCUI1W(vkcui1w.get()); ←ここで細かくマッピングしている

↓

      // VKCUI1W[顧客マスタ(委託料情報(1))適用前]を取得する。
      Optional<VKCUI1W> vkcui1w = vkcui1wService.get1Data(getCommAreaDt("ND10010", "COMM_CUSTCD"));
      // VKCUI1W[顧客マスタ(委託料情報(1))適用前]の内容を
      // OSCOMADT[COMM_AREA明細].ND10140.COMM_VKCUI1W[顧客マスタ(委託料情報(1))_変更情報]に登録する。
      var stVkcui1w = LSS1000.convertEntityNoError(vkcui1w, STVKCUI1W_VKMSMEN.class);
      CommAreaUtility.createOrUpdateCommAreaDtJson(getCommArea().get(), "ND10140", "COMM_VKCUI1W", stVkcui1w, PROGRAM_ID);

でもいいかなと思います。

渡邉 拓英 さんが5ヶ月前に更新 · 編集済み

上記を踏まえ、


  /**
   * COMM_AREA明細 顧客マスタ(運賃基本情報)_変更情報 登録(VKCUNK1)
   * 
   * @param VKCUNK1 顧客マスタ(運賃基本情報)_変更処理
   * @return STVKCUN1W COMM_AREA明細.顧客マスタ(運賃基本情報)_変更情報
   */
  private void insertSTVKCUNK1ByVKCUNK1(VKCUNK1 vkcunk1) {
    STVKCUN1W_VKMSMEN stvkcun1w = new STVKCUN1W_VKMSMEN();

    // VKCUSTM[顧客マスタ(基本情報)]を取得する。
    VKCUSTM vkcustm = vkcustmService.select(getCommAreaDt("ND10010", "COMM_CUSTCD")).get();

    // VKCUNK1[顧客マスタ(運賃基本情報)]の内容を COMM_VKCUN1W[顧客マスタ(運賃基本情報)_変更情報] に移す
    // 顧客コード
    stvkcun1w.setKUNKKOCD(vkcunk1.getKUNKKOCD());
    // 運賃契約区分
    stvkcun1w.setKUNKAKEI(vkcunk1.getKUNKAKEI());
    // 運賃契約適用区分
    stvkcun1w.setKUNKBKEI(vkcunk1.getKUNKBKEI());
    // 端数処理区分
    stvkcun1w.setKUNKATAN(vkcunk1.getKUNKATAN());
    // 端数区分1
    stvkcun1w.setKUNKBTAN(vkcunk1.getKUNKBTAN());
    // 端数区分2
    stvkcun1w.setKUNKCTAN(vkcunk1.getKUNKCTAN());
    // 重量換算有無区分
    stvkcun1w.setKUNKAKAN(vkcunk1.getKUNKAKAN());
    // 換算重量(kg)
    stvkcun1w.setKUNKBKAN(vkcunk1.getKUNKBKAN());
    // 方面の個数
    stvkcun1w.setKUNKAKOS(vkcunk1.getKUNKAKOS());
    // 上限の個数
    stvkcun1w.setKUNKBKOS(vkcunk1.getKUNKBKOS());
    // 持込み割戻し区分
    stvkcun1w.setKRGKWRMD(vkcunk1.getKRGKWRMD());
    // 上限区分1
    stvkcun1w.setKUNKKUBN1(vkcunk1.getKUNKKUBN1());
    // 上限区分2
    stvkcun1w.setKUNKKUBN2(vkcunk1.getKUNKKUBN2());
    // 上限区分3
    stvkcun1w.setKUNKKUBN3(vkcunk1.getKUNKKUBN3());
    // 上限区分4
    stvkcun1w.setKUNKKUBN4(vkcunk1.getKUNKKUBN4());
    // 上限区分5
    stvkcun1w.setKUNKKUBN5(vkcunk1.getKUNKKUBN5());
    // 上限区分6
    stvkcun1w.setKUNKKUBN6(vkcunk1.getKUNKKUBN6());
    // 上限区分7
    stvkcun1w.setKUNKKUBN7(vkcunk1.getKUNKKUBN7());
    // 上限区分8
    stvkcun1w.setKUNKKUBN8(vkcunk1.getKUNKKUBN8());
    // 上限区分9
    stvkcun1w.setKUNKKUBN9(vkcunk1.getKUNKKUBN9());
    // 上限区分10
    stvkcun1w.setKUNKKUBN10(vkcunk1.getKUNKKUBN10());
    // 上限区分11
    stvkcun1w.setKUNKKUBN11(vkcunk1.getKUNKKUBN11());
    // 上限区分12
    stvkcun1w.setKUNKKUBN12(vkcunk1.getKUNKKUBN12());
    // 上限区分13
    stvkcun1w.setKUNKKUBN13(vkcunk1.getKUNKKUBN13());
    // 上限区分14
    stvkcun1w.setKUNKKUBN14(vkcunk1.getKUNKKUBN14());
    // 上限区分15
    stvkcun1w.setKUNKKUBN15(vkcunk1.getKUNKKUBN15());
    // 上限区分16
    stvkcun1w.setKUNKKUBN16(vkcunk1.getKUNKKUBN16());
    // 上限区分17
    stvkcun1w.setKUNKKUBN17(vkcunk1.getKUNKKUBN17());
    // 上限区分18
    stvkcun1w.setKUNKKUBN18(vkcunk1.getKUNKKUBN18());
    // 上限区分19
    stvkcun1w.setKUNKKUBN19(vkcunk1.getKUNKKUBN19());
    // 上限区分20
    stvkcun1w.setKUNKKUBN20(vkcunk1.getKUNKKUBN20());
    // 方面ポインター(一般)
    stvkcun1w.setKUPOINT1(vkcunk1.getKUPOINT1());
    // 方面ポインター(地方)
    stvkcun1w.setKUPOINT2(vkcunk1.getKUPOINT2());
    // 方面ポインター(粁程)
    stvkcun1w.setKUPOINT3(vkcunk1.getKUPOINT3());
    // 方面ポインター(全国一円)
    stvkcun1w.setKUPOINT4(vkcunk1.getKUPOINT4());
    // 方面ポインター(全国一円全て)
    stvkcun1w.setKUPOINTZ(vkcunk1.getKUPOINTZ());
    // 方面コード1
    stvkcun1w.setKUNKCODE1(vkcunk1.getKUNKCODE1());
    // 方面コード2
    stvkcun1w.setKUNKCODE2(vkcunk1.getKUNKCODE2());
    // 方面コード3
    stvkcun1w.setKUNKCODE3(vkcunk1.getKUNKCODE3());
    // 方面コード4
    stvkcun1w.setKUNKCODE4(vkcunk1.getKUNKCODE4());
    // 方面コード5
    stvkcun1w.setKUNKCODE5(vkcunk1.getKUNKCODE5());
    // 方面コード6
    stvkcun1w.setKUNKCODE6(vkcunk1.getKUNKCODE6());
    // 方面コード7
    stvkcun1w.setKUNKCODE7(vkcunk1.getKUNKCODE7());
    // 方面コード8
    stvkcun1w.setKUNKCODE8(vkcunk1.getKUNKCODE8());
    // 方面コード9
    stvkcun1w.setKUNKCODE9(vkcunk1.getKUNKCODE9());
    // 方面コード10
    stvkcun1w.setKUNKCODE10(vkcunk1.getKUNKCODE10());
    // 方面コード11
    stvkcun1w.setKUNKCODE11(vkcunk1.getKUNKCODE11());
    // 方面コード12
    stvkcun1w.setKUNKCODE12(vkcunk1.getKUNKCODE12());
    // 方面コード13
    stvkcun1w.setKUNKCODE13(vkcunk1.getKUNKCODE13());
    // 方面コード14
    stvkcun1w.setKUNKCODE14(vkcunk1.getKUNKCODE14());
    // 方面コード15
    stvkcun1w.setKUNKCODE15(vkcunk1.getKUNKCODE15());
    // 方面コード16
    stvkcun1w.setKUNKCODE16(vkcunk1.getKUNKCODE16());
    // 方面コード17
    stvkcun1w.setKUNKCODE17(vkcunk1.getKUNKCODE17());
    // 方面コード18
    stvkcun1w.setKUNKCODE18(vkcunk1.getKUNKCODE18());
    // 方面コード19
    stvkcun1w.setKUNKCODE19(vkcunk1.getKUNKCODE19());
    // 方面コード20
    stvkcun1w.setKUNKCODE20(vkcunk1.getKUNKCODE20());
    // 方面コード21
    stvkcun1w.setKUNKCODE21(vkcunk1.getKUNKCODE21());
    // 方面コード22
    stvkcun1w.setKUNKCODE22(vkcunk1.getKUNKCODE22());
    // 方面コード23
    stvkcun1w.setKUNKCODE23(vkcunk1.getKUNKCODE23());
    // 方面コード24
    stvkcun1w.setKUNKCODE24(vkcunk1.getKUNKCODE24());
    // 方面コード25
    stvkcun1w.setKUNKCODE25(vkcunk1.getKUNKCODE25());
    // 方面コード26
    stvkcun1w.setKUNKCODE26(vkcunk1.getKUNKCODE26());
    // 方面コード27
    stvkcun1w.setKUNKCODE27(vkcunk1.getKUNKCODE27());
    // 方面コード28
    stvkcun1w.setKUNKCODE28(vkcunk1.getKUNKCODE28());
    // 方面コード29
    stvkcun1w.setKUNKCODE29(vkcunk1.getKUNKCODE29());
    // 方面コード30
    stvkcun1w.setKUNKCODE30(vkcunk1.getKUNKCODE30());
    // 方面コード31
    stvkcun1w.setKUNKCODE31(vkcunk1.getKUNKCODE31());
    // 方面コード32
    stvkcun1w.setKUNKCODE32(vkcunk1.getKUNKCODE32());
    // 方面コード33
    stvkcun1w.setKUNKCODE33(vkcunk1.getKUNKCODE33());
    // 方面コード34
    stvkcun1w.setKUNKCODE34(vkcunk1.getKUNKCODE34());
    // 方面コード35
    stvkcun1w.setKUNKCODE35(vkcunk1.getKUNKCODE35());
    // 方面コード36
    stvkcun1w.setKUNKCODE36(vkcunk1.getKUNKCODE36());
    // 方面コード37
    stvkcun1w.setKUNKCODE37(vkcunk1.getKUNKCODE37());
    // 方面コード38
    stvkcun1w.setKUNKCODE38(vkcunk1.getKUNKCODE38());
    // 方面コード39
    stvkcun1w.setKUNKCODE39(vkcunk1.getKUNKCODE39());
    // 方面コード40
    stvkcun1w.setKUNKCODE40(vkcunk1.getKUNKCODE40());
    // 方面コード41
    stvkcun1w.setKUNKCODE41(vkcunk1.getKUNKCODE41());
    // 方面コード42
    stvkcun1w.setKUNKCODE42(vkcunk1.getKUNKCODE42());
    // 方面コード43
    stvkcun1w.setKUNKCODE43(vkcunk1.getKUNKCODE43());
    // 方面コード44
    stvkcun1w.setKUNKCODE44(vkcunk1.getKUNKCODE44());
    // 方面コード45
    stvkcun1w.setKUNKCODE45(vkcunk1.getKUNKCODE45());
    // 上限区分2-1
    stvkcun1w.setKUNKKUB21(vkcunk1.getKUNKKUB21());
    // 上限区分2-2
    stvkcun1w.setKUNKKUB22(vkcunk1.getKUNKKUB22());
    // 上限区分2-3
    stvkcun1w.setKUNKKUB23(vkcunk1.getKUNKKUB23());
    // 上限区分2-4
    stvkcun1w.setKUNKKUB24(vkcunk1.getKUNKKUB24());
    // 上限区分2-5
    stvkcun1w.setKUNKKUB25(vkcunk1.getKUNKKUB25());
    // 上限区分2-6
    stvkcun1w.setKUNKKUB26(vkcunk1.getKUNKKUB26());
    // 上限区分2-7
    stvkcun1w.setKUNKKUB27(vkcunk1.getKUNKKUB27());
    // 上限区分2-8
    stvkcun1w.setKUNKKUB28(vkcunk1.getKUNKKUB28());
    // 上限区分2-9
    stvkcun1w.setKUNKKUB29(vkcunk1.getKUNKKUB29());
    // 上限区分2-10
    stvkcun1w.setKUNKKUB210(vkcunk1.getKUNKKUB210());
    // 上限区分2-11
    stvkcun1w.setKUNKKUB211(vkcunk1.getKUNKKUB211());
    // 上限区分2-12
    stvkcun1w.setKUNKKUB212(vkcunk1.getKUNKKUB212());
    // 上限区分2-13
    stvkcun1w.setKUNKKUB213(vkcunk1.getKUNKKUB213());
    // 上限区分2-14
    stvkcun1w.setKUNKKUB214(vkcunk1.getKUNKKUB214());
    // 上限区分2-15
    stvkcun1w.setKUNKKUB215(vkcunk1.getKUNKKUB215());
    // 上限区分2-16
    stvkcun1w.setKUNKKUB216(vkcunk1.getKUNKKUB216());
    // 上限区分2-17
    stvkcun1w.setKUNKKUB217(vkcunk1.getKUNKKUB217());
    // 上限区分2-18
    stvkcun1w.setKUNKKUB218(vkcunk1.getKUNKKUB218());
    // 上限区分2-19
    stvkcun1w.setKUNKKUB219(vkcunk1.getKUNKKUB219());
    // 上限区分2-20
    stvkcun1w.setKUNKKUB220(vkcunk1.getKUNKKUB220());
    // ABC地区区分
    stvkcun1w.setKNADABCK(vkcunk1.getKNADABCK());
    // 地区割増区分
    stvkcun1w.setKNADWARK(vkcunk1.getKNADWARK());
    // 地方コード(2桁)
    stvkcun1w.setKNADCODT(vkcunk1.getKNADCODT());
    // 上限最大の時に全国一円
    stvkcun1w.setKNO2NO1K(vkcunk1.getKNO2NO1K());
    // 上限SORT区分(台帳入力時に使用)
    stvkcun1w.setKINSRTKB(vkcunk1.getKINSRTKB());
    // 方面サーチ別粁指定区分
    stvkcun1w.setKHOMSRKM(vkcunk1.getKHOMSRKM());
    // 引取税転嫁料率 %
    stvkcun1w.setKOILPARS(vkcunk1.getKOILPARS());
    // 運賃集約計算区分
    stvkcun1w.setKKBNDUMM(vkcunk1.getKKBNDUMM());
    // サーチャージ区分
    stvkcun1w.setKSERKBNA(vkcunk1.getKSERKBNA());
    // 基準価格
    stvkcun1w.setKSERTANA(vkcunk1.getKSERTANA());
    // 単価期間
    stvkcun1w.setKSERKBNB(vkcunk1.getKSERKBNB());
    // 顧客折衝単価
    stvkcun1w.setKSERTANB(vkcunk1.getKSERTANB());
    // 更新年月日
    stvkcun1w.setKMENDAT2(vkcunk1.getKMENDAT2());
    // 包括保険適用区分
    stvkcun1w.setKHKTHKKB(vkcunk1.getKHKTHKKB());
    // サーチャージ掛率
    stvkcun1w.setKSERPARS(vkcunk1.getKSERPARS());
    // 館内配送手数料収受区分
    stvkcun1w.setKUNKMALL(vkcunk1.getKUNKMALL());

    // 顧客扱店
    stvkcun1w.setVK20ATCD(vkcustm.getKNADATCD1());
    // 事業区分
    stvkcun1w.setVK20KBNK(vkcustm.getKDEXKBNK());
    // 顧客扱店2
    stvkcun1w.setVK20ATCD2(vkcustm.getKNADATCD2());
    // 顧客扱店3
    stvkcun1w.setVK20ATCD3(vkcustm.getKNADATCD3());
    // 顧客扱店4
    stvkcun1w.setVK20ATCD4(vkcustm.getKNADATCD4());

    CommAreaUtility.createOrUpdateCommAreaDtJson(getCommArea().get(), "ND10050", "COMM_VKCUN1W", stvkcun1w, PROGRAM_ID);
  }

の上と下の

vkcustmの取得と、vkcustmの値を使用しているところは、設計に無い処理ではないでしょうか?

渡邉 拓英 さんが5ヶ月前に更新 · 編集済み

  • 担当者 を 渡邉 拓英 から 浦田 光祐 に変更

浦田 光祐 さんが5ヶ月前に更新 · 編集済み

  • ステータス を フィードバック から 設計中(第一貨物) に変更
  • 担当者 を 浦田 光祐 から 渡邉 拓英 に変更

insertXXXXXXXX の箇所を convertEntity を用いて実装しなおしてテストしたところ、10箇所全てで設計通りの動作を確認できました。
よって、マッピング用のメソッドを削除し convertEntityで設計書に合わせしました。

渡邉 拓英 さんが4ヶ月前に更新 · 編集済み

  • ステータス を 設計中(第一貨物) から 終了 に変更

他の形式にエクスポート: Atom PDF