如何重構此Java方法以更好地使用OOP

我試圖重構此代碼以使其更清潔並使用更好的OOP實踐。這種方法需要一堆無線電/複選框和文本框響應，並將它們更新到數據庫表中，然後更新另一個數據庫表中的清單本身。如何重構此Java方法以更好地使用OOP

我覺得這個方法試圖做的很多。但是我需要確定其他方法和類使用的一些東西，例如，如果存在缺陷（無線電值= 2），是否推進workFlow（processUpdateCheckbox中的advanceWorkflow布爾值確定），根據狀態currentActionItem和advanceWorkflow布爾值，以及保留響應。

setFormFeedback不屬於此處，因爲該方法是從另一個處理表單數據的Servlet調用的，並且此消息丟失。

任何重構幫助，非常感謝。

public ChecklistInstance updateYesNoNAChecklistTogles(HttpServletRequest request, ChecklistInstance ci) throws DAOException { 
    String work_item_class_id = request.getParameter("work_item_class_id"); 
    String work_action_class_id = request.getParameter("work_action_class_id"); 

    String paramName; 
    String attribute_id; 
    String radioValue; 
    String textValue; 
    String strStatus = "1"; 
    String strStoredNo = ""; 
    Date dateNow = new Date(); 

    YesNoNAAnswerDAO ynnDao = new YesNoNAAnswerDAO(); 
    ChecklistInstanceDAO ciDao = new ChecklistInstanceDAO(); 

    WorkflowInstanceWorkItemAction currentActionItem = new WorkflowInstanceWorkItemAction(); 
    currentActionItem.setWork_item_class_id(work_item_class_id); 
    currentActionItem.setWork_action_class_id(work_action_class_id); 

// Put the form check list responses into a list 
    List answer_attribute_list = new ArrayList(); 

    java.util.Enumeration enum2 = request.getParameterNames(); 
     while (enum2.hasMoreElements()) { 
      paramName = (String) enum2.nextElement(); 
      boolean isNewQ = paramName.startsWith("qID_"); 

      if (isNewQ) { 
       attribute_id = paramName.replaceAll("qID_", ""); 
       YesNoNAAnswer clr = new YesNoNAAnswer(); 

       if (request.getParameter("radio_" + attribute_id) != null) { 
        radioValue = request.getParameter("radio_" + attribute_id); 
       } else { 
        radioValue = "0"; 
       } 

       if (request.getParameter("textbox_" + attribute_id) != null) { 
        textValue = request.getParameter("textbox_" + attribute_id); 
       } else { 
        textValue = ""; 
       } 

       if (request.getParameter("check_" + attribute_id) != null) { 
        radioValue = request.getParameter("check_" + attribute_id); 
       } else { 
        // checkValue = ""; 
       } 

       if ("0".equals(radioValue) || "2".equals(radioValue)) { 
        strStatus = "0"; 
       } 

       strStoredNo = request.getParameter("stored_" + attribute_id); 
       if ("2".equals(radioValue) && !"yes".equals(strStoredNo)) { 
        deficiencyFound = true; 
       } 

       clr.setWorkflow_instance_id(ci.getWorkflow_instance_id()); 
       clr.setWfi_work_item_action_id(ci.getWfi_work_item_action_id()); 
       clr.setFail_reason(textValue); 
       clr.setAttribute_id(attribute_id); 
       clr.setToggle_value(radioValue); 
       answer_attribute_list.add(clr); 
      } 
     } 

     ci.setChecklist_state(strStatus); 
     ci.setLast_update(dateNow); 
     ci.setAdditional_info(FormUtil.getFieldValue(request, FIELD_ADDITIONAL_INFO)); 

     processUpdateCheckbox(request, ci, currentActionItem); 

      // Update the base check list 
      ciDao.updateInstance(ci, authenticatedUser); 

      // Update the check list question responses 
      ynnDao.updateToggles(answer_attribute_list, authenticatedUser); 

      // update the work flow 
      WorkflowInstanceDAO wfiDao = new WorkflowInstanceDAO(); 
      WorkflowInstanceForm wfiForm = new WorkflowInstanceForm(wfiDao, authenticatedUser); 
      WorkflowInstance wfi = (WorkflowInstance) wfiForm.view(ci.getWorkflow_instance_id(), authenticatedUser); 
      wfiForm.updateWorkFlowInstance(wfi, currentActionItem); 

      setFormFeedback("You have successfully updated the checklist."); 
      triggerUpdateEmail(request, ci, wfi, currentActionItem); 

    return ci; 
}

來源

2012-09-07 jeff

您可能會在http://codereview.stackexchange.com/上獲得更好的答案。 –

我是否在此處轉發問題或者版主如何移動？不想因交叉發帖而被罵！ – jeff

首先第一件事情：你在做面向對象編程，不是程序編程，這就是爲什麼你應該考慮的手哪些課前應採取哪些責任。

那麼，這裏的責任是什麼？我們可以列出必須獨立完成以下任務：用戶輸入數據的

驗證：檢查的有效性（價值觀，無效的值，安全問題（注保護）的範圍...）提供的值由您的客戶通過HTTP。

控制器它將只包含調用域模型方法的列表。

域模型：一組表示您的業務數據和提供操作方法的類。

平均到堅持您的域模型：一個數據庫，以XML文件...

不要忘記管理交易（如果需要）從開始到結束全部處理。

刪除無用的依賴關係：您的域模型不應該知道HTTP以及數據來自何處。您的控制器不應該知道數據如何持久。

不要重複發明輪子：使用MVC框架（例如Struts 2）來滿足您的驗證和控制器需求。使用持久性框架，如hibernate/JPA。

來源

2012-09-07 14:38:56

謝謝。你的第一個陳述是現貨。我來自FORTRAN背景。我打算最終使用持久化框架，但我必須一次重構小塊，我的Dao類對我來說工作得非常好。這是需要立即關注的程序性內容。 – jeff

快速修復：

手工製作所發生的事情在這個函數
寫所採取行動的每一個類型的小功能的簡單示意圖。
創建一個包含從HttpServletRequest
所有有價值的參數值創建從一個HttpServletRequest創建這種類的對象方法的類（構造函數，靜態方法，制定者，如你所願）
您可以使用狀態爲您的UI組件。

的僞代碼看起來像

HttpServletRequest req; 
    RequestParameters params = new RequestParameters(req); 
    //make some processing here (database) 
    for(UIState state : stateList){ 
    state.makechanges(params) 
    } 
    /* UIState can be an interface, each UI item has it own subclass*/

來源

2012-09-07 14:37:56 UmNyobe

我不是已經在做第3步和第4步了嗎？YesNoNAAnswer clr = new YesNoNAAnswer（）; clr.setToggle_value（radioValue）; – jeff

如何重構此Java方法以更好地使用OOP

回答

相關問題