プロジェクト

全般

プロフィール

コードレビュー #10626

未完了

コードレビュー依頼

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

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

0%

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

ファイル

clipboard-202608311515-sul4p.png (24.9 KB) clipboard-202608311515-sul4p.png 渡邉 拓英, 2026-08-31 03:15 午後
clipboard-202608311518-b9qr7.png (9.23 KB) clipboard-202608311518-b9qr7.png 渡邉 拓英, 2026-08-31 03:18 午後
clipboard-202608311521-ugjim.png (24.5 KB) clipboard-202608311521-ugjim.png 渡邉 拓英, 2026-08-31 03:21 午後
clipboard-202608311521-71las.png (9.16 KB) clipboard-202608311521-71las.png 渡邉 拓英, 2026-08-31 03:21 午後
clipboard-202608311529-f95mq.png (46.7 KB) clipboard-202608311529-f95mq.png 渡邉 拓英, 2026-08-31 03:29 午後
clipboard-202608311538-xxai4.png (70.8 KB) clipboard-202608311538-xxai4.png 渡邉 拓英, 2026-08-31 03:38 午後
clipboard-202608311538-chgzc.png (20.7 KB) clipboard-202608311538-chgzc.png 渡邉 拓英, 2026-08-31 03:38 午後
clipboard-202608311547-krnsr.png (49.1 KB) clipboard-202608311547-krnsr.png 渡邉 拓英, 2026-08-31 03:47 午後
clipboard-202608311515-sul4p.png
clipboard-202608311518-b9qr7.png
clipboard-202608311521-ugjim.png
clipboard-202608311521-71las.png
clipboard-202608311529-f95mq.png
clipboard-202608311538-xxai4.png
clipboard-202608311538-chgzc.png
clipboard-202608311547-krnsr.png

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

  • ファイル clipboard-202608311515-sul4p.png を追加

全件取得であれば、selectAllを使用すればよいのでは?(なにか理由があれば)
全件取得時に、ソート順を指定すること。
(VKKANMSは、全件再構築されるようだが、Updateされると取得順の担保できない)


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

  • ファイル clipboard-202608311518-b9qr7.png を追加

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

  • ファイル clipboard-202608311521-ugjim.png を追加
  • ファイル clipboard-202608311521-71las.png を追加

KACUSTCDは、UK、かつNULL不許可
不要な判定処理。

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

  • ファイル clipboard-202608311529-f95mq.png を追加

DBは、oracle19cです。

LIMIT?

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

ソート順にIDも加える事。
同じソートキーで区切りの場合、
次の取得で取れるデータは、未確定

    /**
     * POSメートマスタをページング取得する。(KS60350P バッチ処理用)
     *
     * @param limit  取得件数上限
     * @param offset 取得開始位置
     * @return VKPFSEQリスト
     */
    @Query("SELECT * FROM VKPFSEQ ORDER BY POSQVKEY, POSFAXKB LIMIT :limit OFFSET :offset")
    List<VKPFSEQ> get1DataPaged(@Param("limit") int limit, @Param("offset") int offset);

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

  • ファイル clipboard-202608311538-xxai4.png を追加
  • ファイル clipboard-202608311538-chgzc.png を追加

Entity側で、空 or nullなら初期値にセットする処理はすでにある。
機能毎、分岐処理を設ける必要はない。
他の項目も同じ。

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

  • ファイル clipboard-202608311547-krnsr.png を追加

三項演算子は、基本使用しない

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

個々に実装しない

toInt()

    /**
     * 整数安全変換処理<br>
     * 文字列を整数に変換し、nullや空文字・変換不可の場合は0を返す。
     *
     * @param s 変換対象文字列
     * @return 処理結果(数値型)
     */
    private Integer parseIntSafe(String s) {

        // ┌ 引数.変換対象文字列 = "" の場合
        // │ 0 を返して処理を終了する。
        // ├ 以外の場合
        // │ 引数.変換対象文字列 を整数に変換して返す。
        // └ 『END IF』
        if (s == null || s.trim().isEmpty()) return 0;
        try {
            return Integer.parseInt(s.trim());
        } catch (NumberFormatException e) {
            // ■■■■エラー処理■■■■
            // エラー発生時
            // 0 を返して処理を終了する。
            return 0;
        }
    }

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

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

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