第一貨物 [Mirror]

・プロパティ一覧.入力店所コードについて
　以前にも伝えましたが、設計書とPGは内容を合わせてください。
　（レベル感は案件によりますが、一般的に考えて「同じ」はずです）

　設計書に「プロパティ一覧.入力店所コード」と記載があるのであれば、コーディングすべき。
　でなければ、設計書が「コミュエリア.事業所コード」であるべき

　プロパティ一覧.入力店所コードの値を設定する（取得元）の記載が一切ありません。
　この設計書から、このコーディングは不可能です。

・・・OK

プロパティ一覧　諸料金不可条件適用除外料金_有無フラグ

設計書の出現ヶ所

コード上では、下記が存在する。

・・・OK

プロパティ一覧　一律の運賃変更区分

実装とコーディングが合っていない。

・・・OK

フォーマット方法が指定されていないが、フォーマット処理がある。
フォーマット方法が他の機能と異なる。今回の方針に乗っ取っていない。
画面項目.編集を使用する。
→動けばいいわけではない。

'NOCD1'
(コードなし)
の「(コードなし)」とはどんな意味でしょうか？
記載ルールに違反していませんか？　他の誰が実装しても伝わる内容でしょうか？

コメントの場合は『』ですよね？

・・・OK

設計書とPGが異なる。

以下のチェックは赤線部分だと思いますが。
未入力チェックのタイミングで「以下は行なわない」と指示であれば
設計書通りだと下記になると思います。

これを踏まえ、設計書・PGのあるべき形を考え直してください。全体的に。

ちなみに、「チェック時の処理」は、何時、どんな条件でやるのか記載してください。
今回の場合、チェック前なのか、あとなのか、未入力時なのか、記載がありません。
（処理に曖昧さが発生しています。Aさんはチェック前に行うかもしれません、Bさんはチェック後に一律行うかもしれません。）

・・・OK

      // 妥当性チェック
      if ("0".equals(form.getND1200009()) || "1".equals(form.getND1200009()) || "2".equals(form.getND1200009())
          || "3".equals(form.getND1200009())) {
      } else {
        // それ以外の場合
        baseAddError(bindingResult, "ND1200009", "ERR01");
      }

無駄にコードが長いのは、理解するのに時間がかかりますし。
何より綺麗でないです。
Arraysを使用し完結に書いてください。

また、必要以上に１行で行ったり、処理を詰め込むのも、見難いです。
適切な加減を覚えていきましょう。
（案件やメンバー間のレベルにより書き方も変わってきます。）
書きやすい・読みやすい・改修しやすいは、重要なファクターです。

・・・OK

' 'なのか、''なのか何が正解かわかりません。
設計書に「※未入力時、この項目の以下のチェックを行わない。」とありますが、実装に無い。
チェック内容では' '(設計書では) ''(実装では)がOKだと記載がある。
いろいろ辻褄が合わない

    // ND1200024 上限全国適用区分
    // 形式チェック
    if ("true".equals(LSS1010.isEmptyNull(form.getND1200024()))) {
      // 初期表示の値をセット
      form.setND1200024(stvkcun1w.getKNO2NO1K());
    }

    // 文字数チェック
    baseCharCountCheck(form, bindingResult, errors, "ND1200024", "上限全国適用区分", 1, "ERR01");

    // 妥当性チェック
    if ("".equals(form.getND1200024()) || "0".equals(form.getND1200024()) || "1".equals(form.getND1200024())) {
    } else {
      // それ以外の場合
      baseAddError(bindingResult, "ND1200024", "ERR01");
    }

・・・OK

「P1」とはなんなのか？
突然、発生したが？
プロパティやメソッド等に記載が無ければ誰もわかりません。
どこから発生して、何をするのでしょうか？
第三者に理解できる内容でお願いします。

『』はコメントです。思いはわかりましたが、
必要な処理を記述してください。

変数.S1 ←　メッセージの左からN文字目～N文字
変数.P1 ←　メッセージの左からN文字目～N文字
・・・
処理
・・・

処理が多いのであればメソッドを作成すればいいですし。

さすがに、このレベルはチェック用のメソッドや、別シートでも作成してください。
エクセルで縮小しないと見えないですよね。

・・・OK

コメント頂いた箇所修正いたしました。

また、設計書・PG共に見直して記述を修正しました。
まだ不備を見落としているかもしれませんが引き続きレビューをお願いいたします。

ND1200003　運賃契約区分

任意入力で「0埋め」しない
判定が、統一されていない。

入力パターン

「]     空
「 ]    半角スペース
「  ]   半角スペース半角スペース
「0]    0
「0 ]   0半角スペース
「 0]   半角スペース0
「00]   00

＞＞＞おそらく、トリムした値を左0埋めするのが正解では？
また、空の場合、0として扱ってもよいか？エラーとなるべきか？空で良い＆0と違う扱いか？（必須入力？）

ND1200015　重量換算適用KG

ここも上記に似ている
形式：Cではなく、「9」であるべき？

文字列でcompareToすると、恐らく期待の値ではない＞＞＞数値として比較すべき？？

昇順ソート
["01", "02", "1", "10", "2", "20"]

ND1200024 上限全国適用区分

今回はこのままでOKです。（今後の設計・実装時に意識してください）

      // 上限全国適用区分 = ' ' OR '0' OR '1' 以外の場合
      if (Arrays.asList(" ", "0", "1").contains(form.getND1200024())) {

のところ

今回、入力項目が、全て半角スペースの場合、空として扱うのがほとんどのパターンになると思います。
LSS1010.isEmptyNullは、下記で"true"
・null
・空（文字数：0)
・半角スペースが何文字か
" "のパターンは未入力とみなしています。
また、「" "と等しい場合」等と書かれると、空のチェックなのか、半角スペース１文字の判定かわかりません。

など、参考にしてください。

ND1200034 引取税転嫁率 整数
ND1200035 引取税転嫁率 小数

フォーマット指定なし。

小数部で入力されるパターン

「」
「 」
「  」
「9」
「9 」
「 9」
「99」

全て大丈夫？

編集（フォーマット）
ND1200035 →9
ND1200034 →99 では？

form.setND1200035("0");
↓
form.setND1200035("00");
では？

整数部 + 小数部 → BigDecimal
"1" + "23" → 123
合っている？
1.23になるべきでは？

整数部がスペース
小数部もスペースありのパターンは想定されているか？

例：" " + "." + " 1"

ND1200044 サーチャージ掛率 整数
ND1200045 サーチャージ掛率 小数

        if (!Integer.valueOf("78").equals(Integer.valueOf(form.getND1200041()))
            && !Integer.valueOf("1000").equals(Integer.valueOf(form.getND1200044()) + Integer.valueOf(form.getND1200045()))) {

この場合、整数*10 + 小数　？？？

ここも(スペースを入力しても大丈夫？)

ND1200015 重量換算適用KG
３桁入力可能　対応が足りない

ここも

form.getND1200035() が ""（空文字）→ 例外？
form.getND1200035() が " "（空白のみ）→ 例外？

設計書と実装と実装のコメントが合っていない

new BigDecimal("") は 必ず例外では？

calValueがどんな値になるかはわかりませんが。
calValueが「1」の場合、

XXXX + "1"
で５文字になる。空白（0?!)で埋める必要がありそうです。

運賃再算定メソッド

設計書は戻り値が処理結果になっているが実装は違う。
＞＞＞try　catchで例外を拾うようになっているが
new reculUntinPropertie(rup.stvkcun2w, "1");している。

設計書と実装が合っていないので合わせる。
設計書でも戻り値を見ていない。　　　　　　　　　　　　←仕様として良いのか？
このままでは、例外時も気が付くことが出来ずに進む　　　←仕様として良いのか？

例外処理等で不明なら、誰かに確認する事。

レビューありがとうございます。
設計書とPGを修正しましたので再度レビューをお願いいたします。

＃20,24,28 空白の入力について補足
setInputManで入力形式が '9' または 'C' に設定されていると空白が入力されないので、空白を含む入力パターンは対応の必要がなく桁不足のパターンの対応を追加しました。

◆ND1200003　運賃契約区分
0埋めの件　記述方法が違う。「設計書作成ルール.xlsx」に従う事。

こうなるべき　※例

（レビュー時、どのタイミングで0埋めしているかは、みてはいない）

"0"と比較している。
0埋めしているので、"000"では？

modeXの先頭で行い。値の比較は一貫して行うべき。

使用しないのであれば削除

レビューありがとうございます。

指摘箇所設計書とPGそれぞれ修正しましたので、お手数をおかけしますが確認お願いいたします。