質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.00%

未選択でのposの値が0でわたってしまう。

解決済

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 135

naotaro0704

score 15

前提・実現したいこと

削除処理の工程で躓いています。
ラジオボタン未選択のときにエラー処理を行いたいのですが、今現在未選択のときはposに値として0がわたっています。これですと削除処理が正常?に行われてしまいます。未選択のときに削除処理を行わせないようにするにはどうしたらよいでしょうか。

プログラミング初心者でへんなこと聞いていたらすみません。

該当のソースコード

SalesSystemController.java

@Controller
@RequestMapping(value = "/system")
public class SalesSystemController {

    private static final Logger logger = LoggerFactory.getLogger(SalesSystemController.class);

    /** Stringの定数として次の3つを定義する */
    private static final String INIT = "init";
    private static final String ADD = "add";

    /** メッセージと定義 */

    private static final String DELMSG = "選択された明細行を削除しました。";
    private static final String REMMSG = "明細行を選択してください。";



    /** 明細追加画面での削除押下時の処理 */
    @RequestMapping(params = "remove")
    public String remove(SalesForm form, BindingResult result, Model model) {

        /**
         * RecordManagerのitemListを取得 itemListという名前でmodelに登録する。
         */
         model.addAttribute("ItemList", RecordManager.getItemList());
        /**
         * formのquantityを取得し、addDataという名前でmodelに登録する。
         */
         model.addAttribute("addData", form.getQuantity());

        /** ラジオボタン選択による条件分岐 */

        /** ラジオボタンを選択せずに削除ボタンを押下した時の処理 */
         if(result == null) {
         /**
          * RecordManagerのselectLisitを取得し、allListという名前でmodelに登録する。
          */
         model.addAttribute("selectList",RecordManager.getSelectList());
        /**
         * RecordManagerにnameListを作成 nameListという名前でmodelに登録する。
         */
         model.addAttribute("ItemList", RecordManager.getItemList());
        /**
         * formのquantityを取得し、addDataという名前でmodelに登録する。
         */
         model.addAttribute("addData", form.getQuantity());
        /**
         * RecordManagerのtotalを取得しtotalという名前でmodelに登録する。
         */
         model.addAttribute("total", RecordManager.getTotal());
        /**
         * REMOVEをmessage5という名前でmodelに登録する。
         */
         model.addAttribute("message5", REMMSG);
         /**
          * 明細追加画面へ
          */
         return ADD;

         }
         /** ラジオボタンを選択して削除ボタンを押下した時の処理 */
         else {
        /** 削除処理 */

        System.out.println(form.getPos()+"from");

        /**
         * fromのposを取得しmodelに格納する。
         */
        model.addAttribute("pos",form.getPos());
        /**
         * RecordManaagerのremoveメソッドを実行させる。
         */
        RecordManager.remove(form.getPos());

        /** 商品の残りの数による条件分岐 */

        /** 売上明細の残りが1件以上の処理 */
        if (RecordManager.getSelectList().size() > 0) {
            /**
             * DELETEをmessage4という名前でmodelに登録する
             */
            model.addAttribute("message4", DELMSG);
            /**
             * RecordManagerのselectListを取得し、allListという名前でmodelに登録する。(売上明細)
             */
            model.addAttribute("selectList", RecordManager.getSelectList());
            /**
             * RecordManagerのitemListを取得 itemListという名前でmodelに登録する。
             */
             model.addAttribute("ItemList", RecordManager.getItemList());
            /**
             * RecordManagerのtotalを取得しtotalという名前でmodelに登録する。(合計)
             */
            model.addAttribute("total", RecordManager.getTotal());
            /**
             * 明細追加画面へ
            */
            return ADD;
        }
        /** 売上明細の残りが0件のときの処理 */
        else {
            /**
             * DELETEをmessage4という名前でmodelに登録する
             */
            model.addAttribute("message4", DELMSG);
            /**
             * 初期画面へ
             */
            return INIT;
        }
    }

    }

}


RecordManager.java

public final class RecordManager<selectList> {

    /** 商品データ */
    private static List<Item> list = new ArrayList<>();
    private static List<Item> selectList = new ArrayList<>();
    private static Item item;
    /** totalを定義 */
    private static int total = 0;
    private static long salesId;
    private static int pos;

    static {
        list.add(new Item("A00101", "油性ボールペン", 60, 0, 0, 0));
        list.add(new Item("A00201", "極細ボールペン", 120, 0, 0, 0));
        list.add(new Item("A00301", "蛍光ペン6色セット", 420, 0, 0, 0));
        list.add(new Item("A00401", "シャープペンシル", 100, 0, 0, 0));
        list.add(new Item("A00501", "鉛筆H(1ダース)", 400, 0, 0, 0));
        list.add(new Item("B00101", "無線綴ノートA4", 100, 0, 0, 0));
        list.add(new Item("B00201", "リングノートA4", 120, 0, 0, 0));
        list.add(new Item("B00301", "領収書", 350, 0, 0, 0));
        list.add(new Item("C00101", "はさみ(青)", 128, 0, 0, 0));
        list.add(new Item("C00201", "ステープラー", 338, 0, 0, 0));
        list.add(new Item("C00301", "2穴パンチ", 128, 0, 0, 0));
        list.add(new Item("C00401", "ゼムクリップ", 98, 0, 0, 0));
        list.add(new Item("C00501", "消しゴム", 58, 0, 0, 0));
    }

    /** 商品名 */
    public static List<Item> selectItem(String id, int quantity,int pos) {

        for (int i = 0; i < list.size(); i++) {

            if (id.equals(list.get(i).getId())) {
                pos = selectList.size();

                /** Itemを再定義 */
                Item item = new Item(id,list.get(i).getName(),  list.get(i).getPrice(), quantity,
                        quantity * list.get(i).getPrice(),pos);

                selectList.add(item);

                /** subtotalを定義 */
                int subtotal = quantity * list.get(i).getPrice();
                total = total + subtotal;
                System.out.println(pos+"I");

                break;
            }

        }

        return selectList;

    }



    /** 選択した売上明細を削除 */
    public static  void remove(int pos) {
        /** int型jを定義しselectListの要素内で繰り返し処理*/
        for (int j = 0; j < selectList.size();j++) {


            /** 選択したposと同じ順番の要素をselectList内から探す*/
            if (pos == selectList.get(j).getPos()){
//System.out.println(pos+"I");


                /** 合計からpos番目の小計を引く */
                total = total - selectList.get(j).getSubtotal();

System.out.println(selectList.get(j).getPos()+"R");

            /** selectListのj番目を削除 */
                selectList.remove(selectList.get(j).getPos());
                for(int k = 0; k<selectList.size();k++) {
                    selectList.get(k).setPos(k);
                }
                break;

            }

        }
    }

}


add.jsp

<body>
    <form:form modelAttribute="salesForm" action="/sales/system">
        <div class="header">
            <span class="titleName">直太郎オンラインショップ</span>
            <div class="date"><%=LocalDate.now()%></div>
        </div>
        <div class="main">
            <div>
                商品:
                <form:select path="id">

                <form:options items="${ItemList}" itemValue="id" itemLabel="name" />

                </form:select>

                <br />
            </div>
            <div>
                <span class="itemQuantity">点数:</span>
                <form:input path="quantity" size="20" />
            </div>

            <div class="add">
                <div>
                    <input type="submit" name="addition" value="明細追加" />
                </div>
            </div>
        </div>

        <br>
        <div class="message">
            <font color="#ff0000"> <c:out value="${message1}" />
            </font> <font color="#1e90ff"> <c:out value="${message2}" /> <font
                color="#1e90ff"> <c:out value="${message4}" />
            </font>
            </font> <font color="#ff0000"> <c:out value="${message5}" />
            </font>
        </div>
        <br>



        <div class="">
            <span class="details">売上明細</span>

            <table class="product">

                <tr>
                    <th>削除</th>
                    <th>商品ID</th>
                    <th>商品名</th>
                    <th>単価</th>
                    <th>点数</th>
                    <th>小計</th>
                </tr>
                <c:forEach items="${selectList}" var="selectList" varStatus="status">
                    <tr>
                        <th class="itemradiobtn"><form:radiobutton path="pos"
                                label="" value="${selectList.pos}" /></th>
                        <th class="id">${selectList.id}</th>
                        <th class="name">${selectList.name}</th>
                        <th class="price">${selectList.price}</th>
                        <th class="quantity">${selectList.quantity}</th>
                        <th class="subtotal">${selectList.subtotal}</th>
                        <th class="pos">${selectList.pos}</th>
                    </tr>
                </c:forEach>
            </table>
        </div>

        <div class="total">
            合計:
            <fmt:formatNumber groupingUsed="true">
                <c:out value="${total}" />
            </fmt:formatNumber></div>


        <div class="footer">

            <input type="submit" name="remove" value="削除"
                onclick="remove_row(this)" /> <input type="submit" name="firm"
                value="確定" />
        </div>

    </form:form>

</body>


SalesForm.java

    /** 商品id */
    private String id;
    /** 商品名 */
    private String name;
    /** 単価 */
    private int price;
    /** 個数 */
    @NotNull
    @Min(1)
    private int quantity=1;
    /** 小計 */
    private int subtotal;
    @NotNull
    private int pos;




public SalesForm() {

}


Item.java

public class Item {

    /** 商品id */
    private String id;
    /** 商品名 */
    private String name;
    /** 単価 */
    private int price;
    /** 個数 */
    private int quantity;
    /** 小計 */
    private int subtotal;

    private int pos;
}

試したこと

未選択時の値変更させればいいと思ったので、posの初期値をnullに使用としましたがint型ではnullは解決できないとエラーが出てしまいました。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

0

posをInteger型に変更。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/10/07 10:09

    なんかしばらく前に
    このままじゃ未選択時に0が渡るからrequiredかcheckedつければ…みたいな話したような気もしますが。
    ユーザ側が悪意ある行動をとらないの前提ならrequiredにするだけで解決するんですけどね。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.00%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

同じタグがついた質問を見る