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 为什么使用javacv转封装rtsp为rtmp时出现如下问题:[h264 @ 000000004faf7500]no frame?
  • ¥15 乘性高斯噪声在深度学习网络中的应用
  • ¥15 运筹学排序问题中的在线排序
  • ¥15 关于docker部署flink集成hadoop的yarn,请教个问题 flink启动yarn-session.sh连不上hadoop,这个整了好几天一直不行,求帮忙看一下怎么解决
  • ¥30 求一段fortran代码用IVF编译运行的结果
  • ¥15 深度学习根据CNN网络模型,搭建BP模型并训练MNIST数据集
  • ¥15 C++ 头文件/宏冲突问题解决
  • ¥15 用comsol模拟大气湍流通过底部加热(温度不同)的腔体
  • ¥50 安卓adb backup备份子用户应用数据失败
  • ¥20 有人能用聚类分析帮我分析一下文本内容嘛