第一貨物 [Mirror]

動作の確認

popBackId(PROGRAM_ID);で履歴から最新（ND10040）を取り出し処理を行っている。

// ND10010[顧客台帳入力選択]に遷移する。
return "redirect:" + "/" + "ND10010" + "/u/" + id;

↑などで戻っている場合は、辻褄は合っていると思うが

最後までたどり着いた場合、ND10040が履歴に存在しないが、認識通りか？

・リダイレクトの時のみ取り出す処理が必要では？
参考：取り出すだけなら「getRecentBackId」

CRT_BACK_FUNC_ID_Xは、適切になっているか？

       int cal_kbn;

ローワースネークケースは今回使用しない

ローワーキャメルケース（lowerCamelCase）
camelCase
ローワーキャメルケースは、最初の単語を小文字にし、それ以降の単語の先頭を大文字にする形式です。JavaScriptやPHPなどでよく使われる印象があります。

・エラー処理がありません。

※設計書は書き方は逸脱していますが・・・置いておいて・・・
　データ処理で行う処理です。

複数テーブルの更新処理は必ずトランザクション管理してください。

※例　A、BをInsertする　→　Bでエラーとなる場合
・AをInsertし、Bをロールバックし、処理成功
・AをInsertし、Bをロールバックし、処理失敗
・A・Bをロールバックし、処理失敗
等が考えられます。

今回は、特に明記が無ければ、全て成功 or １つでもエラーなら全てロールバックし、処理失敗となります。

普通に
if else ifで書いてください。

setVKCUN2WByCommVKCUN2Wをまとめなくて良い。
→マッピングが二つある→実装も２つにすべき

  private VKCUN2W setVKCUN2WByCommVKCUN2W(STVKCUN2W_VKMSMEN stvkcun2w, Boolean isInsert) {
    VKCUN2W vkcun2w = new VKCUN2W();

↓
Insertにしてください。→処理はしたで説明します。　明記として

isInsert・・・いまいち

◆更新処理について
・vkcun2wService.update(vkcun2wKousin);
・vkcun2wService.Insert(vkcun2wKousin);
今回の内部的な動作として同じです。
ID・VERSIONを見て、新規か更新か自動で切り替わります。

つまり、今回は、どちらも

VKCUN2W vkcun2wKousin = new VKCUN2W();

なので、Insertしか発生しません。

いろいろ確認してください・・・

Arrays.asList(XXXXX).contains(YYYYY)

「// 変数.ループフラグ = False の場合」　とか

コメントが直ってない（全体確認）

// 『全角換算すると 顧客名(最大6文字) + 'スペース(1)' + LSS1060[顧客台帳共通].プロパティ.運賃契約区分名』

設計書にそんな記述はない
コメントを追加するのはいいが、処理と合っていない→混乱を招く

Optionalじゃないでしょうか？

// COMM_AREA明細.ND10050.COMM_VKCUN2W[顧客マスタ(運賃方面別情報)_変更情報].データ部.KRGKZGKM[粁程 増減粁程] > 0 OR

設計書と実装が異なるように見えますが・・・合ってますか？

"0".compareTo(Integer.toString(XXX)) < 0

"0".compareTo(Integer.toString("000"))はどうなるのか？
"0".compareTo(Integer.toString("-1")) はどうなるのか？
"0".compareTo(Integer.toString(" "))はどうなるのか？

なぜ、文字列で比較しているのか？　桁数を確認しているのか？　動作確認は１行づつ、内容と値をチェックしているのか？

修正完了しました。確認願います。

・コメント#1について
ND10040からの遷移の場合、289行目のif文の中のみで処理が完結します。
ND10010に遷移かND10040に戻るかのいずれかの結果になるのでpopBackId で問題ありません。

※トランザクションについて

記述方法等について、説明等不足していましたので、とりあえずwikiにサンプルのみ記載します。

http://10.104.253.197/projects/3-dos/wiki/%E3%83%88%E3%83%A9%E3%83%B3%E3%82%B6%E3%82%AF%E3%82%B7%E3%83%A7%E3%83%B3

また、簡易的に使用できるようにBaseProgramにrunWithRollbackを準備しています。
（仕様とマッチする場合のみ使用してください。）

0と比較したい時にわざわざtoInt("0")するのは良くない

リテラル(0や1や固定の値）との比較は 「==」で良い。
変数と変数の場合、「==」はNG

また、NullPointerException対策なのであれば
Objects.equals(value, 0)
これで書いておけば大体大丈夫。

トランザクションのサンプルありがとうございます。

実装してみましたので確認をお願いします。

ここは　変数と変数の比較なので「equals」で！！！

下記、今回はこのままでいいです。次回から！
空の比較

"" == xxxxx.trim()

↓

xxxxx.isBlank()

で良いです。

コードレビュー　ワークフローについて

コードレビューのチケットは開発者でCloseしないようにしてください。
指摘（渡邉）→修正（開発者）→確認（渡邉）までがレビューのタスクになります。

また、SVNのコミットも各チケットや内容に合った粒度で行うようにしてください。

関連するバグで複数チケットを１コミット　　・・・OK
関連しない複数チケットを１コミット　　　　・・・NG
１つのチケットで関連する複数の画面（機能）・・・OK
関連が無い複数の画面（機能）　　　　　　　・・・NG

申し訳ございません。ワークフローを読み違えました。

コメント#20 の修正はリビジョン2682 でまとまってコミットしてしまいましたので、改めて確認をお願いします。

コミットの粒度も以後気を付けます。