作者Reder (Reder)
看板java
標題Re: [J2SE] enum應用問題
時間Sun Mar 13 15:06:34 2011
我個人是 enum 的愛好者 XD
我是覺得，你的函式的宣告應該要能夠表達你這個函式能夠接受的參數，
所以用 updateStatus(String id, Status status) 會比較好，
這樣要使用這個函式的人就知道你有參數上的限制。
※ 引述《KevinGet (就算我換了ID)》之銘言：
: 問題不是在於其他程式能不能用 其他程式當然是可以使用ENUM
: 我想看看大家有沒有比較好的設計想法建議我
: -----------------------------------------
: 舉個例子
: Web系統的修改使用者狀態
: 使用者狀態有兩種值 open/close
: 原本設計提供這種方法 updateStatus(String id,String Status)
: 前端可能是一個表單 有下拉選單或Radio Box
: 所以傳入後端是
: String id
: String status //value is open/close
基本上目前比較新的 web framework 都應該要支援 enum 的轉換，
也就是你後端的 vo 直接寫 Status status 他就會幫你塞好。
如果沒有的話，應該就要丟掉… 啊不是，
應該要自己寫 converter 。（我有寫過 Struts1 可以用的 converter ）
: ****第一種寫法***********
: 程式可能會呼叫
: updateStatus(id,status);
: 但是如果要處理當status值不在預期內的情況 那就需要做檢查
: 但是用一般判斷式檢查String 以後status要增加或更改 那要加寫程式
: 所以我想到改利用Enum做檢查 以後維護修改都很方便
: ****第二種寫法***********
: 但是如果提供的方法改成
: updateStatus(String id,Status status)
: 那就可能會呼叫
: updateStatus(id,Status.valueOf(status));
: 但這樣有可能會拋出例外 如果該頁PG撰寫者沒有處理 那系統就會停在這
這個狀況只會發生在開發時期或者是 Status 變動，
那麼該頁 PG 撰寫者一定得處理這個問題，
所以我覺得不是大問題。
（事實上爆炸了反而好，就是知道該處理了）
: 他當然是可以直接使用enum
: 但是他必須還加寫判斷傳入的status值 他才知道要傳入哪個ENUM 不是嗎?
: 可能變成這樣
: if(status.equals(Status.open)){
: updateStatus(id.Status.open);
: }else if(status.equals(Status.close)){
: updateStatus(id,Status.close);
: }else if(...){
: ....
: ...
: 以後要加status 就要加寫判斷
: 狀態越多 程式會越來越長
: 而且這些是散落在各處呼叫這之程式的地方
: 可能就會有N份 相同的程式
: 只是為了判斷要傳入哪個ENUM值
所以只應該用 valueOf() 來做轉換，如果沒有自然會拋出錯誤。
附帶一提的是跟 enum 關係最好的是 switch 不是 if else 喔，
你可以這樣寫：
switch(status){
case open:
// do something when open
break;
case close:
// do something when close
break;
default: throw new IllegalArgumentException("The handler of this status does not exist.");
}
--
※ 發信站: 批踢踢實業坊(ptt.cc)
◆ From: 218.168.216.200