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

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

ただいまの
回答率

90.04%

tableにおける削除について。

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 1
  • VIEW 194

naotaro0704

score 15

前提・実現したいこと

tableの行削除についてです。ラジオボタンで選択した行を削除させるようにしたいのですが、現在選択した行ではない行が削除されてしまっています。削除自体は実行されています。

該当のソースコード

SalesSystemController.java

~中略~

if (RecordManager.getSelectList().size() > 0) {
            /**
             * RecordManagerのselectListを取得し、allListという名前でmodelに登録する。
             */
            model.addAttribute("allList", RecordManager.getSelectList());
            /**
             * RecordManagerにnameListを作成 nameListという名前でmodelに登録する。
             */
            model.addAttribute("nameList", RecordManager.makeNameList());
            /**
             * RecordManagerのtotalを取得しtotalという名前でmodelに登録する。
             */
            model.addAttribute("total", RecordManager.getTotal());
            /**
             * 明細追加画面へ
            */
            return ADD;
        }
        /** 売上明細の残りが0件のときの処理 */
        else {
            /**
             * 初期画面へ
             */
            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));
        list.add(new Item("A00201", "極細ボールペン", 120, 0, 0));
        list.add(new Item("A00301", "蛍光ペン6色セット", 420, 0, 0));
        list.add(new Item("A00401", "シャープペンシル", 100, 0, 0));
        list.add(new Item("A00501", "鉛筆H(1ダース)", 400, 0, 0));
        list.add(new Item("B00101", "無線綴ノートA4", 100, 0, 0));
        list.add(new Item("B00201", "リングノートA4", 120, 0, 0));
        list.add(new Item("B00301", "領収書", 350, 0, 0));
        list.add(new Item("C00101", "はさみ(青)", 128, 0, 0));
        list.add(new Item("C00201", "ステープラー", 338, 0, 0));
        list.add(new Item("C00301", "2穴パンチ", 128, 0, 0));
        list.add(new Item("C00401", "ゼムクリップ", 98, 0, 0));
        list.add(new Item("C00501", "消しゴム", 58, 0, 0));
    }

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

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

            if (id.equals(list.get(i).getId())) {
                // System.out.println(i);
                /** Itemを再定義 */
                Item item = new Item(id, list.get(i).getName(), list.get(i).getPrice(), quantity,
                        quantity * list.get(i).getPrice());

                selectList.add(item);

                /** xを定義 */
                int subtotal = quantity * list.get(i).getPrice();
                total = total + subtotal;

                break;
            }

        }

        return selectList;

    }

    /** nameListの作成 */
    public static String[] makeNameList() {
        String[] nameList = new String[list.size()];
        for (int i = 0; i < list.size(); i++) {
            nameList[i] = (list.get(i)).getName();
        }
        return nameList;
    }
    /** idListの作成 */
    public static String[] makeIdList() {
        String[] idList = new String[list.size()];
        for (int i = 0; i < list.size(); i++) {
            idList[i] = (list.get(i)).getId();
        }
        return idList;
    }


    /** 選択した売上明細を削除 */
    public static void remove(int pos) {
        /** 合計の更新 */
        total = total - selectList.get(pos).getSubtotal();
        System.out.println(pos);
        selectList.remove(pos);

    }



    /** ↓getterとsetter */

    /** itemList取得 */
    public static List<Item> getItemList() {
        return list;
    }

    /** itemLista設定 */
    public static void setItemList() {
        /** 設定した[item」をlist内で検索する。 */
        list.set(list.indexOf(item), item);
    }

    /**
     * @return selectList
     */
    public static List<Item> getSelectList() {
        return selectList;
    }

    /**
     * @param selectList
     *            セットする selectList
     */
    public static void setSelectList(List<Item> selectList) {
        RecordManager.selectList = selectList;
    }


    /**
     * @return pos
     */
    public static int getPos() {
        return pos;
    }

    /**
     * @param pos
     *            セットする pos
     */
    public static void setPos(int pos) {
        RecordManager.pos = pos;
    }

}


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:option path="id" value="${idList}" begin="1"/>
                    <form:options items="${nameList}"/>

                </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="${allList}" var="allList" varStatus="status" >
                        <tr>
                            <th class="itemradiobtn">
                            <form:radiobutton path="name" name="pos" label="" value="${status.index}" /></th>
                            <th class="id">${allList.id}</th>
                            <th class="name">${allList.name}</th>
                            <th class="price">${allList.price}</th>
                            <th class="quantity">${allList.quantity}</th>
                            <th class="subtotal">${allList.subtotal}</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>

試したこと

選択したラジオボタンの1つ上の行が削除されていたので、

public static void remove(int pos) {
        /** 合計の更新 */
        total = total - selectList.get(pos+1).getSubtotal();
        System.out.println(pos+1);
        selectList.remove(pos+1);


というふうにpos+1に変更してみました。
```
System.out.println(pos+1);

補足情報(FW/ツールのバージョンなど)

clipse Java EE IDE for Web Developers.
Version: Mars.2 Release (4.5.2)
ローカルホストTomcat8

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

checkベストアンサー

0

htmlテーブル上の行番号とリストの行(添字)の扱いに差異があるのであれば、どちらかでプラマイするのは特に問題ないと思います。
が、そもそも論としてそのように小細工しなくて済むように見かけの行番号を渡すのではなく、idなどといった削除するデータを特定するキーを渡すべきです。
(そしてidをキーとした削除メソッドを呼ぶ)
例えば今後データとしてはあるが条件により非表示にする場合などで見かけの行と合わないことはいくらでも出てくることが予想されます。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/09/17 17:20

    シーケンシャルな番号はvalue="${status.index}"で振ってあるということにはならないですかね。

    キャンセル

  • 2019/09/17 17:50

    横からすみません
    >指摘していただいたやり方だと、同名の商品を一気に削除してしまうということが起こってしまいそうなのですが

    被りが発生する可能性がある時点でIDの存在意義がないのではないでしょうか…?

    キャンセル

  • 2019/09/19 01:59

    本業忙しく回答修正まではできませんが、解決へのヒントはコメント含め出尽くしていると思います。
    >value="${status.index}"で振ってある
    これは不変ではないことに注意してください。どこかの要素を削除するとIndexが詰めなおされますね。
    1つ削除してリストやビューを作り直して…という場合に問題なく思えますが例えばリストにチェックボックスを付けて2つ削除するとなったらどうなるでしょうか?

    シーケンシャルな番号という記載が紛らわしかったかもしれません、要するに画面上の行であるとかListのIndexであるとか、データを操作したら変わってしまうもの、扱うものでインデックスオリジンが違うものにすべきではないということです。
    元の回答に記載したIDも別にアイテムのIDという意味ではなく、リストを作成した際にリスト用のIDを作ってくださいという意味でした。

    キャンセル

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

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

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