java 重构 设计模式_java-设计模式以嵌套if或switch语句重构代码

最新推荐文章于 2021-07-21 19:14:26 发布
最新推荐文章于 2021-07-21 19:14:26 发布
阅读量331 收藏
点赞数
文章标签： java 重构 设计模式
版权声明：本文为博主原创文章，遵循 CC 4.0 BY-SA 版权协议，转载请附上原文出处链接和本声明。
本文链接：https://blog.csdn.net/weixin_32001433/article/details/114231820
版权

我必须重构庞大的现有代码.在SO和其他网站上解决了许多类似的问题,但仍然感到困惑.如果有人可以提出一些建议或任何想法,将大有帮助.

有5个下拉菜单,我需要根据下拉菜单中的选定值更新视图.

首先下拉菜单有以下选项：

"TOP DOWN BUDGET"

"TEMPLATE BUDGET"

"ORIGINAL BUDGET"

"REVISED BUDGET"

第二个下拉菜单具有以下选项：

"Day"

"Week"

"Month"

"Quarter"

"Year"

第三个下拉列表具有以下选项：

"Details"

"Summary"

第四个下拉列表具有以下选项：

"Hours"

"Dollars"

第五下拉菜单具有以下选项：

"StartDate"

"EndDate"

现在,代码具有以下情形：

public List XYZ(...){//opening of some method XYZ....

List workPlanReportList=null;

switch(first_Drop_Down_Value){

case "TOP DOWN BUDGET":

List timeLine=getTimeLine("firstDropDownA", second_drop_down_val, third_drop_down_val, fourth_drop_down_val);

workPlanReportList=setWorkPlanByTimeLine(timeLine, "firstDropDownA", second_drop_down_val, third_drop_down_val, fourth_drop_down_val);

break;

case "TEMPLATE BUDGET":

List timeLine=getTimeLine("firstDropDownB", second_drop_down_val, third_drop_down_val, fourth_drop_down_val, fifth_dd_val);

workPlanReportList=setWorkPlanByTimeLine(timeLine, "firstDropDownA", second_drop_down_val, third_drop_down_val, fourth_drop_down_val);

break;

case "ORIGINAL BUDGET":

List timeLine=getTimeLine("firstDropDownC", second_drop_down_val, third_drop_down_val, fourth_drop_down_val, fifth_dd_val);

workPlanReportList=setWorkPlanByTimeLine(timeLine, "firstDropDownA", second_drop_down_val, third_drop_down_val, fourth_drop_down_val);

break;

case "REVISED BUDGET":

List timeLine=getTimeLine("firstDropDownD", second_drop_down_val, third_drop_down_val, fourth_drop_down_val, fifth_dd_val);

workPlanReportList=setWorkPlanByTimeLine(timeLine, "firstDropDownA", second_drop_down_val, third_drop_down_val, fourth_drop_down_val);

break;

}

return workPlanReportList;

}// Closing of some method XYZ....

private List getTimeLine(String first_Drop_Down_Value, String second_dd_val, third_dd_val, fourth_dd_val, String fifth_dd_val){

switch(second_Drop_Down_Value){

case "Day":

if(third_dd_val.equals("Details")){

if(fourth_dd_val.equals("Hours")){

if(fifth_dd_val.equals("startDate"){

//prepare query & call DB for fetching days timeline for hours details of TOP DOWN BUDGET filtered by start date...

}

else// means endDate{

//prepare query & call DB for fetching days timeline for hours details of TOP DOWN BUDGET filtered by end date...

}

}

else{//means Dollars

if(fifth_dd_val.equals("startDate"){

//prepare query & call DB for fetching days timeline for dollars details of TOP DOWN BUDGET filtered by start date...

}

else// means endDate{

//prepare query & call DB for fetching days timeline for dollars details of TOP DOWN BUDGET filtered by end date...

}

}

}

else// means summary...

{

if(fourth_dd_val.equals("Hours")){

if(fifth_dd_val.equals("startDate"){

//prepare query & call DB for fetching days timeline for 'hours' "summary" of TOP DOWN BUDGET filtered by start date...

}

else// means endDate{

//prepare query & call DB for fetching days timeline for hours summary of TOP DOWN BUDGET filtered by end date...

}

}

else{//means Dollars

if(fifth_dd_val.equals("startDate"){

//prepare query & call DB for fetching days timeline for dollars details of TOP DOWN BUDGET filtered by start date...

}

else// means endDate{

//prepare query & call DB for fetching days timeline for dollars details of TOP DOWN BUDGET filtered by end date...

}

}

}

break;

case "Week":

//....similar code as in case "Day" just here we need to fetch data week-wise

break;

case "Month":

//....similar code as in case "Day" just here we need to fetch data month-wise

break;

case "Quarter":

//....similar code as in case "Day" just here we need to fetch data quarter-wise

break;

case "Year":

//....similar code as in case "Day" just here we need to fetch data year-wise

break;

}

}

private List setWorkPlanByTimeLine(List timeLine, String "firstDropDownA", String second_drop_down_val, String third_drop_down_val, String fourth_drop_down_val){

WorkPlanReport wpr=new WorkPlanReport();

// Here I have real mess..., Iterating the timeLine list and have switch case and inside switch case multilevel nesting of if else to decide which setter we need to use to set the value.

for example:

If it is "TOP DOWN BUDGET" in first drop-down, "Day" in second drop-down, "Details" in third drop down, "Hours" in fourth drop-down & "StartDate" in fifth drop-down, I have to call follwing code:

wpr.setTDBDetailsHoursByStartDate(taskDetails.getTDBDetailsByHourStartDayView());

If it is "TOP DOWN BUDGET" in first drop-down, "Day" in second drop-down, "Details" in third drop down, "Hours" in fourth drop-down & "EndDate" in fifth drop-down, I have to call follwing code:

wpr.setTDBDetailsHoursByEndDate(taskDetails.getTDBDetailsByHourEndDayView());

}

这段代码超过1000行,我不愿使用一些合适的设计模式来对其进行重构.

工作计划报告和TaskDetails DTO共有一些相似类型的属性,并且还有许多其他不同属性.

我不允许更改那些DTO,因为它们在某些相关的通用代码库中使用.

编辑：这是此代码中使用的一种方法.我尽力使级别变得简单,但无法提出任何可行的想法.

private void setSummaryColumns(String DolOrHr, String columnFilter,

Map filterBy, Object[] obj,

WorkplanReport workplanReport, TemplateDump td) {

switch(filterBy.get(columnFilter)){

case "TOP DOWN":

td.setTopDownBudgetStartDate(obj[1] != null ? (Date)obj[1]:null);

td.setTopDownBudgetEndDate(obj[2] != null ? (Date)obj[2]:null);

workplanReport.setStartDatebyMajorMeasure(obj[1] != null ? obj[1].toString():"-");

workplanReport.setEndDatebyMajorMeasure(obj[2] != null ? obj[2].toString():"-");

workplanReport.setDolorHrsbyMajorMeasure(obj[3] != null ? obj[3].toString():"-");

if(DolOrHr.equals("BudgetDollars"))

td.setTopDownBudgetDollars(obj[3] != null ? (BigDecimal)obj[3]:null);

else

td.setTopDownBudgetHours(obj[3] != null ? (BigDecimal)obj[3]:null);

break;

case "TEMPLATE":

td.setTemplateBudgetStartDate(obj[1] != null ? (Date)obj[1]:null);

td.setTemplateBudgetEndDate(obj[2] != null ? (Date)obj[2]:null);

workplanReport.setStartDatebyTemplateBudget(obj[1] != null ? obj[1].toString():"-");

workplanReport.setEndDatebyTemplateBudget(obj[2] != null ? obj[2].toString():"-");

workplanReport.setDolorHrsbyTemplateBudget(obj[3] != null ? obj[3].toString():"-");

if(DolOrHr.equals("BudgetDollars"))

td.setTemplateBudgetDollars(obj[3] != null ? (BigDecimal)obj[3]:null);

else

td.setTemplateBudgetHours(obj[3] != null ? (BigDecimal)obj[3]:null);

break;

case "ORIGINAL":

td.setOriginalBudgetStartDate(obj[1] != null ? (Date)obj[1]:null);

td.setOriginalBudgetEndDate(obj[2] != null ? (Date)obj[2]:null);

workplanReport.setOrgStartDate(obj[1] != null ? obj[1].toString():"-");

workplanReport.setOrgEndDate(obj[2] != null ? obj[2].toString():"-");

workplanReport.setOrgBudgetedDollarsHours(obj[3] != null ? obj[3].toString():"-");

if(DolOrHr.equals("BudgetDollars"))

td.setOriginalBudgetDollars(obj[3] != null ? (BigDecimal)obj[3]:null);

else

td.setOriginalBudgetHours(obj[3] != null ? (BigDecimal)obj[3]:null);

break;

case "REVISED":

td.setRevisedBudgetStartDate(obj[1] != null ? (Date)obj[1]:null);

td.setRevisedBudgetEndDate(obj[2] != null ? (Date)obj[2]:null);

workplanReport.setRevStartDate(obj[1] != null ? obj[1].toString():"-");

workplanReport.setRevEndDate(obj[2] != null ? obj[2].toString():"-");

workplanReport.setRevBudgetedDollarsHours(obj[3] != null ? obj[3].toString():"-");

if(DolOrHr.equals("BudgetDollars"))

td.setRevisedBudgetDollars(obj[3] != null ? (BigDecimal)obj[3]:null);

else

td.setRevisedBudgetHours(obj[3] != null ? (BigDecimal)obj[3]:null);

break;

case "MANAGER":

td.setManagerBudgetStartDate(obj[1] != null ? (Date)obj[1]:null);

td.setManagerBudgetEndDate(obj[2] != null ? (Date)obj[2]:null);

workplanReport.setStartDatebyManagersRevised(obj[1] != null ? obj[1].toString():"-");

workplanReport.setEndDatebyManagersRevised(obj[2] != null ? obj[2].toString():"-");

workplanReport.setDolorHrsbyManagersRevised(obj[3] != null ? obj[3].toString():"-");

if(DolOrHr.equals("BudgetDollars"))

td.setManagerBudgetDollars(obj[3] != null ? (BigDecimal)obj[3]:null);

else

td.setManagerBudgetHours(obj[3] != null ? (BigDecimal)obj[3]:null);

break;

case "ACTUAL":

td.setActualBudgetStartDate(obj[1] != null ? (Date)obj[1]:null);

td.setActualBudgetEndDate(obj[2] != null ? (Date)obj[2]:null);

workplanReport.setStartDatebyActual(obj[1] != null ? obj[1].toString():"-");

workplanReport.setEndDatebyActual(obj[2] != null ? obj[2].toString():"-");

workplanReport.setDolorHrsbyActual(obj[3] != null ? obj[3].toString():"-");

if(DolOrHr.equals("BudgetDollars"))

td.setActualBudgetDollars(obj[3] != null ? (BigDecimal)obj[3]:null);

else

td.setActualBudgetHours(obj[3] != null ? (BigDecimal)obj[3]:null);

break;

}

}

一个雪莉
关注
  • 0
    点赞
  • 踩
  • 0
    收藏
    觉得还不错? 一键收藏
  • 0
    评论
【设计模式Java语言】-- 实战：使用策略模式重构switch case语句
Badme
02-25 1922
【设计模式Java语言】-- 实战：使用策略模式重构switch case语句
java写一个swatch case_用java lambdas重构一个switch case
weixin_39890633的博客
02-25 621
我试图重新考虑遗留代码,在这种情况下,我有一个巨大的switch case,它决定了要执行的命令switch(operation)case addition : return add(int a, String b, String c);case multiply : return multiply(int a, int b);case substract : return substract...
重构-改善既有代码的设计（一）switch statements
virusswb的专栏
03-19 141
　　《重构-改善既有代码的设计》是一本好书，值得大家去反复看几遍，对后面学习设计模式有很好的帮助。设计模式猛一看，会很晕，云里雾里的，所以可以从重构开始，一点一点改善代码的设计，然后重构到模式，这样可以理解的更透侧，更容易消化。 　　书中会列举出很多的坏味道bad smell，然后重构这些bad smell，增加复用性。其中一个就是当你使用switch的时候，尤其是case的东西定义了枚举类型的话...
if-else嵌套太深？教你一个设计模式搞定
qq_39235296的博客
08-18 1290
多层嵌套if写法 我们正常多层嵌套if的写法 ```java /** * @author saier * @date 2020/3/31 18:03 */ public class Order { public Message interrupt1(){ return null; } public Message interrupt2(){ return null; } public Message interrupt3(){
java状态机设计模式switch，上岸蚂蚁金服！
weixin_57992300的博客
07-21 268
Java代码是怎么运行的？ Java的基本类型 Java虚拟机是如何加载Java类的 JVM是如何执行方法调用的？（上） JVM是如何执行方法调用的？（下） JVM是如何处理异常的？ JVM是如何实现反射的？ JVM是怎么实现invokedynamic的？（上） JVM是怎么实现invokedynamic的？（下） Java对象的内存布局 垃圾回收（上） 垃圾回收（下） Java内存模型 Java虚拟机是怎么实现synchronized的？ Java语法糖与Java编译器 16
If-Else-Block-Refactoring:使用嵌套的ifelse代码块重构业务逻辑
05-14
1. **提炼函数（Extract Method）**：当`if`语句或`else`块包含大量代码时，可以考虑将这部分代码提取为独立的函数，以提高代码的模块化程度。这样每个函数都有一个明确的目的，使得整体代码更易理解。 2. **策略...
重构-改善既有代码的设计 中文版.pdf
10-22
9.5 Replace Nested Conditional with Guard Clauses （以卫语句取代嵌套条件式） 9.6 Replace Conditional with Polymorphism（以多态取代条件式） 9.7 Introduce Null Object（引入Null对象） 9.8 Introduce ...
重构－改善既有代码的设计（中文版）
08-10
　Switch Statements switch语句　 　Parallel Inheritance Hierarchies 平行继承体系　 　Lazy Class 冗余类　 　Speculative Generality 理论上的一般性　 　Temporary Field 临时字段　 　Message Chains 消息链...
重构：改善既有代码的设计.[美]Martin Fowler.epub【文字版手机格式】
04-23
本书也因此成为与《设计模式》齐名的经典著作，被译为中、德、俄、日等众多语言，在世界范围内畅销不衰。 本书凝聚了软件开发社区专家多年摸索而获得的宝贵经验，拥有不因时光流逝而磨灭的价值。今天，无论是重构...
提升java代码简洁度的几个小技巧
07-07
2. 避免冗余代码：如果一段代码在多个地方重复出现，考虑将其封装为方法或使用设计模式。例如，可以创建一个通用的方法处理异常，而不是在每个可能出现异常的地方都进行相同的处理。 3. 使用Java 8及更高版本的新...
java 代码重构-第一章(使用策略模式,把恶心的switch代码去掉...) 二
HK学习
01-07 673
  上一篇文章:java 代码重构-第一章(使用策略模式,把恶心的 switch代码去掉...) 一 下一篇文章:java 代码重构-第一章(结语...目录,源代码下载)   上一篇文章,没有把switch代码去掉,现在再继续去做..这里正式运用策略模式 我的作法是一次取出一个case 分支，在相应的class内建立一个覆写函数（overriding method）。先从 Regular...
重构：运用Java反射加多态 “干掉” switch
热门推荐
IT小小鸟～～
10-21 1万+
前言：本篇文章主要描述我是如何通过Java的反射加多态干掉 swtich这个代码的坏味道 目录 代码的坏味道 《重构》曰 遭遇switch 利剑：多态加反射 结束战斗 代码的坏味道 有这么一句话：普通的代码是写给机器看的，优秀的代码是写给人看的，在软件开发的过程中，不知道大家有没有遇到过if-else或switch满天飞，函数长的看都不想看这种情况，先不说软件的扩展性怎么样，这样的代码就算是我们
java代码重构
knclin的专栏
03-28 1016
最近工作中会用到java代码重构。于是在查找这方面的资料。涉及到几点记录下来，方便日后查阅。 重构重点 1.Duplicated Code(重复代码) 同一个类的两个函数含有相同的表达式，需要将重复的这段代码提出来，让这两个函数都调用这段代码,两个互为兄弟的子类内含相同表达式，需要将代码提炼出来放入父类中; 如果两个毫不相关的类出现重复代码需要将其提炼到一个独立类中，然后在另一个类内使用这个新...
java中switch语句常量_Java枚举：重构switch语句的常量表达式需要'编译错误？
weixin_40002336的博客
02-25 346
我有一个类为我的应用程序声明常量public class GroupConstants {..public static final int INTEGER_VALUE = 1;public static final int LONG_VALUE = 2;public static final int STRING_VALUE = 3;..}在代码中有一组switch语句private stati...
java什么是重构 何时使用重构_java代码重构的方法学习
weixin_42169971的博客
02-16 691
2,8阅读第一章 第一个案例在此案例中使用了，方法的提取(extract method),方法的移动(move method)，临时变量的清除，变量及函数名重构，switch类型重构(replace type code with state/strategy)等。重构使得代码功能更加好的被复用，偏于维护和修改。注意：switch语句的属性尽量不要是另一个对象的数据，最好在自己的数据基础上进行第二...
switch方法重构案例
I'm Joker
09-19 556
面向对象语言开发过程中对于switch语句是很敏感的，大多数switch基本都可以通过多态方式进行重构，从而使程序获得较好的拓展能力，最近项目开发中遇到这样一个案例，在此分享一下。   重构前程序 public void update(Request newReq){ Request existReq=dao.getExist(); ...
Java8之重构代码
qimowei的博客
12-23 357
Java8之重构代码 第7章 并行数据处理与性能 1、将顺序流转换为并行流 1)parallel方法： public static long parallelSum(long n) { return Stream.iterate(1L, i -> i + 1) .limit(n) .pa...
JAVA代码重构--改善既有代码的设计(一)
qq_38658567的博客
08-21 7855
JAVA代码重构–改善既有代码的设计(一) 本文主要借鉴了<重构 改善既有代码的设计>与<设计模式>这两本书; 主要解释重构的原理（principles）和最佳实践方式（best practices），并指出何时何地你应该开始挖掘你的代码以求改善。 <重构 改善既有代码的设计>核心是一份完整的重构名录（catalog of refactoring），其中每...
重构 改善既有代码的设计 java,重构——改善既有代码的设计 源代码
weixin_36486217的博客
03-27 311
package cn.com;import java.util.Enumeration;import java.util.Vector;public class Customer{private String _name;// 姓名private Vector _rentals = new Vector();// 租借记录public Customer(String name){this._nam...
java重构代码经典设计模式
最新发布
01-03
Java重构代码经典设计模式是指在进行代码重构过程中，使用经典的设计模式来改善代码结构和提高代码的可维护性、可读性和可扩展性。其中，常用的设计模式包括： 1. 单例模式：确保一个类只有一个实例，并提供全局...

“相关推荐”对你有帮助么？

  • 非常没帮助
  • 没帮助
  • 一般
  • 有帮助
  • 非常有帮助
提交
评论
添加红包

请填写红包祝福语或标题

个

红包个数最小为10个

元

红包金额最低5元

当前余额3.43元 前往充值 >
需支付：10.00元
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付元
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明：

1.余额是钱包充值的虚拟货币，按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载，可以购买VIP、付费专栏及课程。

余额充值