qq_29943797 2016-09-06 09:21 采纳率: 100%
浏览 1023
已采纳

两个很相似的方法,能否合并为一个方法

     private int GetAllMonsterEXP()
    {
        int exps = 0;
        for (int i = 0; i < monsterNum; i++)
        {
            exps += MonsterInitInfoList[roleAttMonsterIndex].exp;
        }
        return exps;
    }


    private int GetAllMonsterGold()
    {
        int gold = 0;
        for (int i = 0; i < monsterNum; i++)
        {
            gold += MonsterInitInfoList[roleAttMonsterIndex].gold;
        }
        return gold;
    }

就是上面两个方法,可以合并为一个方法吗,通过传递匿名方法之类的??

  • 写回答

5条回答

  • 迦蓝叶 2016-09-06 09:33
    关注

    如果只有这两个方法,暂时不重构也无所谓,很清晰,如果要重构,可以重构成这样

      private int GetAllMonsterEXPOrGlod(boolean isGetExp)
        {
            int data = 0;
            for (int i = 0; i < monsterNum; i++)
            {
                            if(isGetExp){
                   data += MonsterInitInfoList[roleAttMonsterIndex].exp;
                            }else{
                               data += MonsterInitInfoList[roleAttMonsterIndex].gold;
                            }
            }
            return data;
        }
    

    如果多的话,if 换个switch case ,参数换个String 或者 int

    如果需要获取的数据类型有很多,那就改策略模式就好了

    本回答被题主选为最佳回答 , 对您是否有帮助呢?
    评论
查看更多回答(4条)

报告相同问题?

悬赏问题

  • ¥20 有人能用聚类分析帮我分析一下文本内容嘛
  • ¥15 请问Lammps做复合材料拉伸模拟,应力应变曲线问题
  • ¥30 python代码,帮调试
  • ¥15 #MATLAB仿真#车辆换道路径规划
  • ¥15 java 操作 elasticsearch 8.1 实现 索引的重建
  • ¥15 数据可视化Python
  • ¥15 要给毕业设计添加扫码登录的功能!!有偿
  • ¥15 kafka 分区副本增加会导致消息丢失或者不可用吗?
  • ¥15 微信公众号自制会员卡没有收款渠道啊
  • ¥100 Jenkins自动化部署—悬赏100元