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条)

报告相同问题?

悬赏问题

  • ¥15 执行 virtuoso 命令后,界面没有,cadence 启动不起来
  • ¥50 comfyui下连接animatediff节点生成视频质量非常差的原因
  • ¥20 有关区间dp的问题求解
  • ¥15 多电路系统共用电源的串扰问题
  • ¥15 slam rangenet++配置
  • ¥15 有没有研究水声通信方面的帮我改俩matlab代码
  • ¥15 ubuntu子系统密码忘记
  • ¥15 信号傅里叶变换在matlab上遇到的小问题请求帮助
  • ¥15 保护模式-系统加载-段寄存器
  • ¥15 电脑桌面设定一个区域禁止鼠标操作