コードレビュー #9090
完了
ND10190_開発_傭車新規登録 コードレビュー
浦田 光祐 さんが4ヶ月前に追加.
4ヶ月前に更新.
ファイル
|
clipboard-202606231124-unlwo.png (42.9 KB)
clipboard-202606231124-unlwo.png |
|
渡邉 拓英, 2026-06-23 02:24 午前
|
|
|
clipboard-202606231131-fos7f.png (271 KB)
clipboard-202606231131-fos7f.png |
|
渡邉 拓英, 2026-06-23 02:31 午前
|
|
|
clipboard-202606231147-0iqsw.png (34.2 KB)
clipboard-202606231147-0iqsw.png |
|
渡邉 拓英, 2026-06-23 02:47 午前
|
|
|
clipboard-202606231335-5rsos.png (4.82 KB)
clipboard-202606231335-5rsos.png |
|
渡邉 拓英, 2026-06-23 04:35 午前
|
|
|
clipboard-202606231335-isj4m.png (6.77 KB)
clipboard-202606231335-isj4m.png |
|
渡邉 拓英, 2026-06-23 04:35 午前
|
|
|
clipboard-202606231348-bgiuf.png (4.61 KB)
clipboard-202606231348-bgiuf.png |
|
渡邉 拓英, 2026-06-23 04:48 午前
|
|
|
clipboard-202606231348-dtnry.png (5.69 KB)
clipboard-202606231348-dtnry.png |
|
渡邉 拓英, 2026-06-23 04:48 午前
|
|
|
clipboard-202607091109-w0rae.png (5.33 KB)
clipboard-202607091109-w0rae.png |
|
渡邉 拓英, 2026-07-09 02:09 午前
|
|
|
clipboard-202607091114-tnw4v.png (10.6 KB)
clipboard-202607091114-tnw4v.png |
|
渡邉 拓英, 2026-07-09 02:14 午前
|
|
|
clipboard-202607091115-spklx.png (7.15 KB)
clipboard-202607091115-spklx.png |
|
渡邉 拓英, 2026-07-09 02:15 午前
|
|
最新のリビジョン＃3817です。レビューよろしくお願いいたします。
- ファイル clipboard-202606231124-unlwo.png を追加
- ファイル clipboard-202606231131-fos7f.png を追加
DBの更新処理は、dataProcessModeX（データ処理）で行うこと。
物理的に不可ではなく、共通仕様としてのこの案件のルール。(更新処理→トランザクション処理が必要になります。ロールバック・エラー処理など）
#8382
getDataMode1（データ取得処理）で登録処理をしたのち改めて取得してますが、その後使用する処理がありません。のちのokProcessMode1（ＯＫ処理）で使いますが、ここまで持ち越せないので再度取得する必要があります。
103行目の取得処理を126行目の処理直前に移動してほしいです。
と記載がありますが、プロパティ.データに記載されているもは、持ち越せる認識です。XX00000Dataに設定すれば持ち越せる認識ですが、どういったコーディングを想定しているのか？
モードを超えることはできない。モードの間は使用できる。
設計書・実装とも修正。
// RDADRN[住所ＤＢ２].ADRNKNN3[大字．通称＿カナ ＋ 字．丁目＿カナ] <> "" 場合
if (!"".equals(rdadrn.get().getADRNKNN3())) {
DBの定義はNCHARです。
データが取れた場合、""にはなりえません。
設計書・・・トリムした値 <> ""
実装・・・・isBlank()
- ファイル clipboard-202606231147-0iqsw.png を追加
おそらく、各所にトリムが必要。
大字コードが、1文字のみ入力したら例外になりそうですが、大丈夫でしょうか？
// ND3400025 支払税率整数部
// VKCUSTW[顧客マスタ（基本情報）適用前].KZEIPER1[支払料率] / 100 の整数部『下2桁以外と同義』
String siharaiZeiritu = stVkcustw.getKZEIPER1().toString();
form.setND3400025(siharaiZeiritu.substring(0, siharaiZeiritu.length() - 2));
// ND3400026 支払税率小数部
// VKCUSTW[顧客マスタ（基本情報）適用前].KZEIPER1[支払料率] / 100 の小数部『下2桁と同義』
form.setND3400026(siharaiZeiritu.substring(siharaiZeiritu.length() - 2));
『下2桁と同義』・・・同義ではない。
税率は、永久に固定でしょうか？
可変値である内容なので考慮不足
KZEIPER1[支払料率]が0の場合、例外が発生するのではないでしょうか？(substringに変えるのは悪手）
設計書で/100の整数部と書いてある。わざわざ変える必要はない。
- ファイル clipboard-202606231335-5rsos.png を追加
- ファイル clipboard-202606231335-isj4m.png を追加
設計書、実装
入力可能、0埋めされない。
それぞれ「1」「1」の値を入力したら、どんな値が登録されますか？？
- ファイル clipboard-202606231348-bgiuf.png を追加
- ファイル clipboard-202606231348-dtnry.png を追加
9000?
- ステータス を 新規 から フィードバック に変更
- 担当者 を 渡邉 拓英 から 浦田 光祐 に変更
- ステータス を フィードバック から 設計中（第一貨物） に変更
- 担当者 を 浦田 光祐 から 渡邉 拓英 に変更
レビューありがとうございます。指摘を基に設計書、実装、共に見直しましたので再レビューお願いします。
補足
コメント＃2
プロパティ.データに記載されているもは、持ち越せる認識です。XX00000Dataに設定すれば持ち越せる認識ですが、どういったコーディングを想定しているのか？
これまで XX00000Data に設定するデータは全て プロパティ.変数 に記載していたので、プロパティ.データ は対象外と思ってました。
今回更新に必要な処理を全て dataProcessMode1 に集約できそうなのでプロパティに保存せずに修正しました。
- 題名 を ND10190_開発_傭車新規登録 コードレビュー から ND10190_開発_傭車新規登録 コードレビュー V２ に変更
- ファイル clipboard-202607091109-w0rae.png を追加
- ファイル clipboard-202607091114-tnw4v.png を追加
- ファイル clipboard-202607091115-spklx.png を追加
- 題名 を ND10190_開発_傭車新規登録 コードレビュー V２ から ND10190_開発_傭車新規登録 コードレビュー に変更
- ステータス を 設計中（第一貨物） から 終了 に変更
他の形式にエクスポート: Atom
PDF
