第一貨物 [Mirror]

記述

のような場合、
画面行数の「10」

final static で定数を宣言し機能全体で使用する。
・値の変更を防げる（安全性）
・「意味のある名前」を付けられる（可読性）
・一箇所の変更で全体を反映（保守性）

例

  // 一番上に書いて
  private final static Integer ROW_COUNT_10 = 10;
  
  /**
   * 活性制御 disp1<br>
   * setDisabledListDisp1
   */
  private List<String> setDisabledListDisp1(MPND160Form form) {

    List<String> disabledList = new ArrayList<String>();

    // labal項目は非活性

    // ■MPND160
    disabledList.add("ND1600001"); // タイトル
    disabledList.add("ND1600002"); // 顧客コード
    disabledList.add("ND1600003"); // 顧客名
    disabledList.add("ND1600004"); // 試算
    disabledList.add("ND1600005"); // 宅配料金適用区分内容１
    disabledList.add("ND1600006"); // 宅配料金適用区分内容２
    disabledList.add("ND1600007"); // 宅配料金適用区分内容３
    disabledList.add("ND1600008"); // 契約方面コード
    disabledList.add("ND1600009"); // 契約方面名
    disabledList.add("ND1600010"); // 扱方区分
    disabledList.add("ND1600011"); // 扱方区分名
    disabledList.add("ND1600012"); // 発着区分
    disabledList.add("ND1600013"); // 発着区分名
    disabledList.add("ND1600014"); // 元着区分
    disabledList.add("ND1600015"); // 元着区分名
    // 明細部
    for (int i = 0; i < ROW_COUNT_10; i++) {
      disabledList.add("ND1600016_" + i); // 上限区分名
      disabledList.add("ND1600018_" + i); // 単価区分名
      disabledList.add("ND1600019_" + i); // 加算記号
      disabledList.add("ND1600025_" + i); // 高低名
    }

    disabledList.add("ND1600016"); // 上限区分名
    disabledList.add("ND1600018"); // 単価区分名
    disabledList.add("ND1600019"); // 加算記号
    disabledList.add("ND1600025"); // 高低名

    disabledList.add("ND1600029"); // 確認メッセージ

    // 単価区分 = "5"の場合、入力不可
    for (int i = 0; i < ROW_COUNT_10; i++) {
      if ("5".equals(getFormValue("ND1600017", form, i))) {
        disabledList.add("ND1600020_" + i); // 加算
        disabledList.add("ND1600021_" + i); // 単価整数部
        disabledList.add("ND1600022_" + i); // 単価小数部
      }
    }

    return disabledList;
  }

不要なコメント？！

同じ処理ですし、設計書は『1行づつ値を取得してリストに纏めてからプロパティに設定』と大きな指示、nd10100Dataでやり取りできる内容のため
メソッド化（共通化）しましょう！

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

プログラム修正いたしましたので、確認お願いします。