Switch语句导致重复的代码

Question

我是一名初级开发人员，希望更好地组织我的代码。

现在，我正在处理表单编号。我检索表单编号的哈希图，根据表单编号，我需要调用不同的方法。每个方法都接受相同的参数，但做的事情不同。

例如：

var formDetails = new inferForms.buildFormsMap

for(form in formDetails){

switch(form.formNumber){
          case "A1345":
            getExclusionDetails(account, state, form, businessDealing)
            break
          case "B254":
            getExclusionDetails(account, state, form, businessDealing)
            break
          case "B297":
            getPartnershipDetails(account, state, form, businessDealing)
            break
          case "C397":
            getBrokerageDetails(account, state, form, businessDealing)
            break
          case "D972":
            getBrokerageDetails(account, state, form, businessDealing)
            break
          case "E192":
            getBrokerageDetails(account, state, form, businessDealing)
            break
          case "E299":
            getBrokerageDetails(account, state, form, businessDealing)
            break
          case "F254":
            getLocationDetails(account, state, form, businessDealing)
            break
          case "F795":
            getLocationDetails(account, state, form, businessDealing)
            break
          case "G642":
            getContractDetails(period, wcmJurisdiction, newForm, wcmBusiness, frm)
            break
          case "G979":
            getContractDetails(period, wcmJurisdiction, newForm, wcmBusiness, frm)
            break
   }
}

几点注意事项

-方法是由另一个开发人员构建的。他辞职了，所以我承担了他的工作，并希望重构以使其变得更好。

-起点是表单编号的哈希映射。我生成HashMap，然后循环遍历它，根据HashMap中的每个表单编号收集详细信息。

-即使我要将这些方法转换为对象继承结构，我仍然需要一个switch语句来知道要实例化哪个子类，不是吗？switch语句与上面的语句类似吗？

-其中一些case语句正在调用完全相同的方法。有没有避免重复的方法？

谢谢你的帮助。我正在努力想办法更好地重新设计它。请让我知道，如果我可以提供更多的细节。

Answer 1

您可以将开关情况替换为if，否则如果，因为有多个条件具有相同的结果，它将减少重复。

        var formDetails = new inferForms.buildFormsMap

        for(form in formDetails){
            var formNumber = form.formNumber
            if(formNumber.equals("A1345") || formNumber.equals("A1345")){
                getExclusionDetails(account, state, form, businessDealing)
            } else if (formNumber.equals("B297") || formNumber.equals("C397")) {
                getPartnershipDetails(account, state, form, businessDealing)
            } else if (formNumber.equals("D972") || formNumber.equals("E192")) {
                getBrokerageDetails(account, state, form, businessDealing)
            } else if (formNumber.equals("F254") || formNumber.equals("F795")) {
                getLocationDetails(account, state, form, businessDealing)
            } else if (formNumber.equals("G642") || formNumber.equals("G979")) {
                getContractDetails(period, wcmJurisdiction, newForm, wcmBusiness, frm)
            }
        }