playframework 1.2.7 delete();の使用方法について

Question

webアプリケーションを作成しています。
使用のフレームワークはplayframework1.2.7です。
ブラウザのチェックボックスから、チェックボックスをつけたボックスのIDを
検索して削除する仕組みを考えています。
どこが悪いか分かる方がいらっしゃいましたらご教示頂きたいと思います。
下記の様に実装しています。

【controller】

package controllers;
import java.util.List;

import models.DBregist;
import play.mvc.Controller;

public class Application extends Controller {
// DB情報を削除する
public static void delete(Long id) {
DBregist dbdel = DBregist(id);
dbdel.all();
List<DBregist> dbdels = DBregist.findAll();
render(dbdels);}
}

【views】

#{extends 'main.html' /}
#{set title:'Play Framework' /}

<h1>ユーザー削除</h1> <form method="get" action="@{Application.delete}"> 名前： <input type="text" name="name" /><br/> <table border="1"> <tr><th>名前</th></tr> #{list items:dbdels, as:'dbdel'} <tr> <td><input type="checkbox"></td> <td>${dbdel.name}</td> </tr> #{/list} </table> <input type="submit" value="削除" /> </form>

【model】

package models;

import javax.persistence.Entity;
import javax.persistence.Id;

import play.db.jpa.Model;

@Entity
public class DBregist extends Model {
@Id
public Long id;
public String name;

public DBregist(String name, String message, Long id) {
this.id = id;
this.name = name;
}

}

【route】

GET /delete Application.delete

Answer 1

いまのやり方だとviewからcontrollerに削除対象となるidが飛ばされないのではないでしょうか。
実際に動作させて検証していませんが、恐らく下のような形で実現できると思います。

まず削除対象となるidをviewに持たせます。

<input type="checkbox" name="deleteId[]" value="${dbdel.id}">

controllerでidを受け取って削除します。

public class Application extends Controller {
     public static void delete(List<Long> deleteId) {
          for(Iterator<Long> it = deleteId.iterator; it.hasNext();){
               DBregist dbdel = DBregist.findbyId(it.next());
               dbdel.delete();
          }
          render(dbdels);
     }
}

また現在の実装だとURLさえわかれば誰でもデータを削除できてしまいますので、アプリケーションの目的に応じて何らかのチェックを入れたほうがよいかもしれません。

routeについては/deleteはGETでも動きますが、DELETEとするのが正しい？かもです。
その場合はviewのmethodも変更してください。
https://www.playframework.com/documentation/1.2.7/routes

Comments

[kuriboros]

ありがとうございました。参考にさせて頂きます。再度、実装して動作確認してみます。