记一个不写重复代码的技巧

上周tute实现组合模式的时候,被tutor指出写了重复的代码,经过指导后,改进了自己编写代码的习惯,在此记录一下。
具体情景如下,在一个游戏中,不同轮次具有不一样的计分规则,最后取当前轮所有可用的规则所计算出的分值的最大值为最终分数,因此需要一个组合规则去管理规则,一开始我通过实例化三个rule对象去实现功能,被指出这样比较浪费空间而且冗余,因此通过一个ArrayList来实现,当到达对应轮次时再实例化当前轮可用规则的对象去计算分数,然后最后通过一个循环去找出最大值,避免了冗余比较和重复代码的问题。

public class CompositeRule implements iStrategy{
	
	private ArrayList<Integer> r;
	public CompositeRule(){
		r = new ArrayList<Integer>();
	}
	
	@Override
	public int calculator(int turn, BoardGame board, Player currentPlayer,ArrayList<Die> dices) {
		// TODO Auto-generated method stub
		r.add(new Rule1().calculator(turn, board, currentPlayer, dices));
		if(turn%2 == 0) {
			r.add(new Rule2().calculator(turn, board, currentPlayer, dices));
		}
		else if(turn%3 == 0) {
			r.add(new Rule3().calculator(turn, board, currentPlayer, dices));
		}		
		int tempScore = 0;
		for(int i:r) {
			if(i > tempScore) {
				tempScore = i;
			}
		}
		currentPlayer.addScore(tempScore);	
		return 0;	
	}

}
  • 0
    点赞
  • 0
    收藏
    觉得还不错? 一键收藏
  • 0
    评论

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

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

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

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

抵扣说明:

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

余额充值