プロジェクト

全般

プロフィール

コードレビュー #10628

未完了

コードレビュー依頼

宮川 美佐 さんが2ヶ月前に追加. 2ヶ月前に更新.

ステータス:
フィードバック
優先度:
通常
担当者:
開始日:
期日:
進捗率:

0%

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

ファイル

clipboard-202608311049-pe27a.png (49.8 KB) clipboard-202608311049-pe27a.png 渡邉 拓英, 2026-08-31 10:49 午前
clipboard-202608311051-hadm5.png (2.84 KB) clipboard-202608311051-hadm5.png 渡邉 拓英, 2026-08-31 10:51 午前
clipboard-202608311054-vnxho.png (9.02 KB) clipboard-202608311054-vnxho.png 渡邉 拓英, 2026-08-31 10:54 午前
clipboard-202608311056-wyqsu.png (27.5 KB) clipboard-202608311056-wyqsu.png 渡邉 拓英, 2026-08-31 10:56 午前
clipboard-202608311058-fvos5.png (2.22 KB) clipboard-202608311058-fvos5.png 渡邉 拓英, 2026-08-31 10:58 午前
clipboard-202608311049-pe27a.png
clipboard-202608311051-hadm5.png
clipboard-202608311054-vnxho.png
clipboard-202608311056-wyqsu.png
clipboard-202608311058-fvos5.png

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

  • ファイル clipboard-202608311049-pe27a.png を追加
  • ファイル clipboard-202608311051-hadm5.png を追加

        // 共通関数[LSS1000].Entityを別のEntityにコンバートする をコールする。
        //   引数 ソース:引数.管理台帳
        // 変数.管理台帳 ← 共通関数[LSS1000].Entityを別のEntityにコンバートする.戻り値 をセット
        out = LSS1000.convertEntity(vkkanms, VKKANMS.class);

処理の意図がわからない。
convertEntityは不要? 削除してよいのでは?

Insert時に、インスタンスを引き継ぎたくない意図であれば、「vkkanms.deepCopy()」メソッドを使用してください。
createdVkkanmsでnewしているため、Loop中のインスタンスは切り分けられているのでは?

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

  • ファイル clipboard-202608311054-vnxho.png を追加

システムエラーログ出力が実装されていない

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

  • ファイル clipboard-202608311056-wyqsu.png を追加
  • ファイル clipboard-202608311058-fvos5.png を追加

設計観点だが、

上では、 データ.管理台帳リストと記述がある。

管理台帳を全件削除する。
は
「データ.管理台帳リストを全件削除する」と記述すべきでは?

この設計書で、管理台帳はVKKANMSでは?

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

  • ステータス を 新規 から フィードバック に変更
  • 担当者 を 大場 淳 から 宮川 美佐 に変更

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