亚洲在线久爱草,狠狠天天香蕉网,天天搞日日干久草,伊人亚洲日本欧美

為了賬號安全,請及時綁定郵箱和手機立即綁定
已解決430363個問題,去搜搜看,總會有你想問的

代碼重復求改進方法

代碼重復求改進方法

郎朗坤 2019-05-25 15:28:02
@OverridepublicOptiongetTrafficChartOption(Stringtype,ReportTypereportType,IntegerdeviceId,Integerdirection){Integerdevice=deviceId+1010000;Listdata=chartDao.getTrafficChartData(reportType,device,direction);Stringtitle=Titlehelper.getChartTitle(reportType);Stringsubtitle=Titlehelper.gettrafficSubText(reportType.getReportTime(),deviceId,direction);Optionoption=newOption();switch(type){case"bar":option=BarOption.BarOptionBuiler(title,subtitle,data);break;case"line":option=LineOption.OptionBuilerhelp(title,subtitle,data);break;case"pie":option=PieOption.PieOptionbuilder(title,subtitle,data);break;}returnoption;}@OverridepublicOptiongetAmmeterChartOption(Stringtype,ReportTypereportType,IntegerdeviceId){Listdata=chartDao.getAmmeterDataChartData(reportType,deviceId);Stringtitle=Titlehelper.getChartTitle(reportType);Stringsubtitle=Titlehelper.gettrafficSubText(reportType.getReportTime(),deviceId,1);Optionoption=newOption();switch(type){case"bar":option=BarOption.BarOptionBuiler(title,subtitle,data);break;case"line":option=LineOption.OptionBuilerhelp(title,subtitle,data);break;case"pie":option=PieOption.PieOptionbuilder(title,subtitle,data);break;}returnoption;}代碼結構非常相似,只是dao層取數據不一樣,另外這個switch有沒有改進空間,我知道使用eumn來枚舉,沒寫以減少無關代碼
查看完整描述

2 回答

?
一只萌萌小番薯

TA貢獻1795條經驗 獲得超7個贊

你的代碼是要創建Option類,適合使用工廠方法重構。
//方法參數盡量少于3個,3個以上時建議使用組合對象
classOptionParam{
privateStringtype;
privateReportTypereportType;
privateIntegerdiviceId;
privateIntegerdirection;
//getterandsetter
}
//Option抽象工廠,每個具體工廠都必須繼承自抽象工廠
publicabstractclassAbstractOptionFactory{
@Autowired
ChartDaochartDao;
//希望具體工廠實現的方法,交于具體工廠實現
abstractListgetData(OptionParamoptionParam);
abstractStringgetSubtitle(OptionParamoptionParam);
//公共邏輯代碼,創建對象調用該方法
publicOptioncreate(OptionParamoptionParam){
assertoptionParam!=null;
assertoptionParam.getReportType()!=null;
Stringtype=optionParam.getType();
Listdata=getData(optionParam);
Stringtitle=Titlehelper.getChartTitle(optionParam.getReportType());
Stringsubtitle=getSubtitle(optionParam);
Optionoption=newOption();
switch(type){
case"bar":
option=BarOption.BarOptionBuiler(title,subtitle,data);
break;
case"line":
option=LineOption.OptionBuilerhelp(title,subtitle,data);
break;
case"pie":
option=PieOption.PieOptionbuilder(title,subtitle,data);
break;
}
returnoption;
}
}
@Component
classTrafficChartOptionFactoryextendsAbstractOptionFactory{
@Override
ListgetData(OptionParamoptionParam){
returnchartDao.getTrafficChartData(optionParam.getReportType(),
optionParam.getDiviceId(),
optionParam.getDirection());
}
@Override
StringgetSubtitle(OptionParamoptionParam){
returnTitlehelper.gettrafficSubText(
optionParam.getReportType().getReportTime(),
optionParam.getDiviceId(),
optionParam.getDirection());
}
}
@Component
classAmmeterChartOptionFactoryextendsAbstractOptionFactory{
@Override
ListgetData(OptionParamoptionParam){
returnchartDao.getAmmeterDataChartData(optionParam.getReportType(),optionParam.getDiviceId());
}
@Override
StringgetSubtitle(OptionParamoptionParam){
returnTitlehelper.gettrafficSubText(optionParam.getReportType().getReportTime(),
optionParam.getDiviceId(),
1);
}
}
使用工廠方法的好處就不多說了,具體可以閱讀我的博客
                            
查看完整回答
反對 回復 2019-05-25
?
手掌心

TA貢獻1942條經驗 獲得超3個贊

Stringtitle=Titlehelper.getChartTitle(reportType);
Stringsubtitle=Titlehelper.gettrafficSubText(reportType.getReportTime(),deviceId,1);
Optionoption=newOption();
switch(type){
case"bar":
option=BarOption.BarOptionBuiler(title,subtitle,data);
break;
case"line":
option=LineOption.OptionBuilerhelp(title,subtitle,data);
break;
case"pie":
option=PieOption.PieOptionbuilder(title,subtitle,data);
break;
}
returnoption;
這幾行提取出來放在一個方法里調用不就行了
                            
查看完整回答
反對 回復 2019-05-25
  • 2 回答
  • 0 關注
  • 411 瀏覽
慕課專欄
更多

添加回答

舉報

0/150
提交
取消
微信客服

購課補貼
聯系客服咨詢優惠詳情

幫助反饋 APP下載

慕課網APP
您的移動學習伙伴

公眾號

掃描二維碼
關注慕課網微信公眾號