[CUBBY-56] APIの再度検討 Created: 2008-01-02  Updated: 2008-01-30  Resolved: 2008-01-12

Status: Closed
Project: Cubby
Component/s: None
Affects Version/s: None
Fix Version/s: 1.0.0-RC1

Type: Improvement Priority: Major
Reporter: agata Assignee: baba
Resolution: Fixed Votes: 0
Labels: None


 Description   

以前も話が出ましたが、以下のAPIがやっぱり気になってきました。
1.0リリース前なので、最後にチャンスとして検討して、変更できたらやりたいです。

Priority High

  • @URL -> @Path
  • @Form -> @ActionMethod or @ ActionBinding

Priority Low

  • Actionの継承 -> @Actionアノテーションと、DefaultActionなどに分割


 Comments   
Comment by baba [ 2008-01-12 ]

対応しました

Comment by jfut [ 2008-01-12 ]

了解です。良いと思います。

Comment by agata [ 2008-01-11 ]

今日馬場さんと会う機会があったので、以下で以降と言うことになりました。

>- Actionの継承 -> @Actionアノテーションと、DefaultActionなどに分割

これはシンプルに現在の仕様のままでいきましょうということになりました。

なのでAPI変更は@Path、@Accept対応になります。

Comment by jfut [ 2008-01-11 ]

@URL -> @Path

ServletのcontextPath,servletPath,pathInfoを考えると@Path良さそうです。
@Accept も賛成です。

Actionの継承 -> @Actionアノテーションと、DefaultActionなどに分割

こちらも賛成です。

Comment by baba [ 2008-01-11 ]

@Acceptが省略された場合は、GETとPOSTとか?

そうですね。そんなかんじ。

Comment by agata [ 2008-01-11 ]

たしかにわかりやすいかも。
@Acceptが省略された場合は、GETとPOSTとか?

Comment by baba [ 2008-01-11 ]
  • @Url -> @Path
    ですが、accept属性とRequestMethodを違うアノテーションに分割するとしっくりくるような気がしました。
    @Path("/foo/{id,.*}")
    @Accept(RequestMethod.POST)
    public ActionResult bar() {
    ....
    

どうでしょ?

Comment by agata [ 2008-01-10 ]

言い出しておいてなんですが、手元で少し変えてみると、
前のほうが良く見えたりして、堂々巡りです・・・悩むなあ。

Comment by agata [ 2008-01-10 ]

*@URL -> @Path
これは賛成。

やりましょう。

*@Form -> @ActionMethod or @ ActionBinding
@Formは「リクエストパラメータはどのオブジェクトにバインドされるか」なので今のままでも違和感がないかも。
アクションメソッドかどうかはメソッドの戻り値で判定していたはず。

それではこちらは、なしで。

*Actionの継承 -> @Actionアノテーションと、DefaultActionなどに分割

では、こちらもやりましょ。

Comment by baba [ 2008-01-10 ]

*@URL -> @Path
これは賛成。

*@Form -> @ActionMethod or @ ActionBinding
@Formは「リクエストパラメータはどのオブジェクトにバインドされるか」なので今のままでも違和感がないかも。
アクションメソッドかどうかはメソッドの戻り値で判定していたはず。

*Actionの継承 -> @Actionアノテーションと、DefaultActionなどに分割
賛成。

Generated at Sat Apr 27 05:00:20 JST 2024 using Jira 9.15.0#9150000-sha1:9ead8528714127d8cfabf2446010d7e62c0a195c.