转自 ：http://www.cnblogs.com/zhangchenliang/archive/2013/01/08/2850726.html
1 IGame游戏公司的故事
1.1 讨论会
话说有一个叫IGame的游戏公司，正在开发一款ARPG游戏（动作&角色扮演类游戏，如魔兽世界、梦幻西游这一类的游戏）。一般这类游戏都有一个基本的功能，就是打怪（玩家攻击怪物，借此获得经验、虚拟货币和虚拟装备），并且根据玩家角色所装备的武器不同，攻击效果也不同。这天，IGame公司的开发小组正在开会对打怪功能中的某一个功能点如何实现进行讨论，他们面前的大屏幕上是这样一份需求描述的ppt：
图1.1 需求描述ppt
各个开发人员，面对这份需求，展开了热烈的讨论，下面我们看看讨论会上都发生了什么。
1.2 实习生小李的实现方式
在经过一番讨论后，项目组长Peter觉得有必要整理一下各方的意见，他首先询问小李的看法。小李是某学校计算机系大三学生，对游戏开发特别感兴趣，目前是IGame公司的一名实习生。
经过短暂的思考，小李阐述了自己的意见：
“我认为，这个需求可以这么实现。HP当然是怪物的一个属性成员，而武器是角色的一个属性成员，类型可以使字符串，用于描述目前角色所装备的武器。角色类有一个攻击方法，以被攻击怪物为参数，当实施一次攻击时，攻击方法被调用，而这个方法首先判断当前角色装备了什么武器，然后据此对被攻击怪物的HP进行操作，以产生不同效果。”
而在阐述完后，小李也飞快的在自己的电脑上写了一个Demo，来演示他的想法，Demo代码如下。
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
namespace IGameLi
{
/// <summary>
/// 怪物
/// </summary>
internal sealed class Monster
{
/// <summary>
/// 怪物的名字
/// </summary>
public String Name { get; set; }
/// <summary>
/// 怪物的生命值
/// </summary>
public Int32 HP { get; set; }
public Monster(String name,Int32 hp)
{
this.Name = name;
this.HP = hp;
}
}
}
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
namespace IGameLi
{
/// <summary>
/// 角色
/// </summary>
internal sealed class Role
{
private Random _random = new Random();
/// <summary>
/// 表示角色目前所持武器的字符串
/// </summary>
public String WeaponTag { get; set; }
/// <summary>
/// 攻击怪物
/// </summary>
/// <param name="monster">被攻击的怪物</param>
public void Attack(Monster monster)
{
if (monster.HP <= 0)
{
Console.WriteLine("此怪物已死");
return;
}
if ("WoodSword" == this.WeaponTag)
{
monster.HP -= 20;
if (monster.HP <= 0)
{
Console.WriteLine("攻击成功！怪物" + monster.Name + "已死亡");
}
else
{
Console.WriteLine("攻击成功！怪物" + monster.Name + "损失20HP");
}
}
else if ("IronSword" == this.WeaponTag)
{
monster.HP -= 50;
if (monster.HP <= 0)
{
Console.WriteLine("攻击成功！怪物" + monster.Name + "已死亡");
}
else
{
Console.WriteLine("攻击成功！怪物" + monster.Name + "损失50HP");
}
}
else if ("MagicSword" == this.WeaponTag)
{
Int32 loss = (_random.NextDouble() < 0.5) ? 100 : 200;
monster.HP -= loss;
if (200 == loss)
{
Console.WriteLine("出现暴击！！！");
}
if (monster.HP <= 0)
{
Console.WriteLine("攻击成功！怪物" + monster.Name + "已死亡");
}
else
{
Console.WriteLine("攻击成功！怪物" + monster.Name + "损失" + loss + "HP");
}
}
else
{
Console.WriteLine("角色手里没有武器，无法攻击！");
}
}
}
}
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
namespace IGameLi
{
class Program
{
static void Main(string[] args)
{
//生成怪物
Monster monster1 = new Monster("小怪A", 50);
Monster monster2 = new Monster("小怪B", 50);
Monster monster3 = new Monster("关主", 200);
Monster monster4 = new Monster("最终Boss", 1000);
//生成角色
Role role = new Role();
//木剑攻击
role.WeaponTag = "WoodSword";
role.Attack(monster1);
//铁剑攻击
role.WeaponTag = "IronSword";
role.Attack(monster2);
role.Attack(monster3);
//魔剑攻击
role.WeaponTag = "MagicSword";
role.Attack(monster3);
role.Attack(monster4);
role.Attack(monster4);
role.Attack(monster4);
role.Attack(monster4);
role.Attack(monster4);
Console.ReadLine();
}
}
}
程序运行结果如下：
图1.2 小李程序的运行结果
1.3 架构师的建议
小李阐述完自己的想法并演示了Demo后，项目组长Peter首先肯定了小李的思考能力、编程能力以及初步的面向对象分析与设计的思想，并承认小李的程序正确完成了需求中的功能。但同时，Peter也指出小李的设计存在一些问题，他请小于讲一下自己的看法。
小于是一名有五年软件架构经验的架构师，对软件架构、设计模式和面向对象思想有较深入的认识。他向Peter点了点头，发表了自己的看法：
“小李的思考能力是不错的，有着基本的面向对象分析设计能力，并且程序正确完成了所需要的功能。不过，这里我想从架构角度，简要说一下我认为这个设计中存在的问题。
首先，小李设计的Role类的Attack方法很长，并且方法中有一个冗长的if…else结构，且每个分支的代码的业务逻辑很相似，只是很少的地方不同。
再者，我认为这个设计比较大的一个问题是，违反了OCP原则。在这个设计中，如果以后我们增加一个新的武器，如倚天剑，每次攻击损失500HP，那么，我们就要打开Role，修改Attack方法。而我们的代码应该是对修改关闭的，当有新武器加入的时候，应该使用扩展完成，避免修改已有代码。
一般来说，当一个方法里面出现冗长的if…else或switch…case结构，且每个分支代码业务相似时，往往预示这里应该引入多态性来解决问题。而这里，如果把不同武器攻击看成一个策略，那么引入策略模式（Strategy Pattern）是明智的选择。
最后说一个小的问题，被攻击后，减HP、死亡判断等都是怪物的职责，这里放在Role中有些不当。”
Tip：OCP原则，即开放关闭原则，指设计应该对扩展开放，对修改关闭。
Tip：策略模式，英文名Strategy Pattern，指定义算法族，分别封装起来，让他们之间可以相互替换，此模式使得算法的变化独立于客户。
小于边说，边画了一幅UML类图，用于直观表示他的思想。
图1.3 小于的设计
Peter让小李按照小于的设计重构Demo，小李看了看小于的设计图，很快完成。相关代码如下：
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
namespace IGameLiAdv
{
internal interface IAttackStrategy
{
void AttackTarget(Monster monster);
}
}
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
namespace IGameLiAdv
{
internal sealed class WoodSword : IAttackStrategy
{
public void AttackTarget(Monster monster)
{
monster.Notify(20);
}
}
}
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
namespace IGameLiAdv
{
internal sealed class IronSword : IAttackStrategy
{
public void AttackTarget(Monster monster)
{
monster.Notify(50);
}
}
}
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
namespace IGameLiAdv
{
internal sealed class MagicSword : IAttackStrategy
{
private Random _random = new Random();
public void AttackTarget(Monster monster)
{
Int32 loss = (_random.NextDouble() < 0.5) ? 100 : 200;
if (200 == loss)
{
Console.WriteLine("出现暴击！！！");
}
monster.Notify(loss);
}
}
}
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
namespace IGameLiAdv
{
/// <summary>
/// 怪物
/// </summary>
internal sealed class Monster
{
/// <summary>
/// 怪物的名字
/// </summary>
public String Name { get; set; }
/// <summary>
/// 怪物的生命值
/// </summary>
private Int32 HP { get; set; }
public Monster(String name,Int32 hp)
{
this.Name = name;
this.HP = hp;
}
/// <summary>
/// 怪物被攻击时，被调用的方法，用来处理被攻击后的状态更改
/// </summary>
/// <param name="loss">此次攻击损失的HP</param>
public void Notify(Int32 loss)
{
if (this.HP <= 0)
{
Console.WriteLine("此怪物已死");
return;
}
this.HP -= loss;
if (this.HP <= 0)
{
Console.WriteLine("怪物" + this.Name + "被打死");
}
else
{
Console.WriteLine("怪物" + this.Name + "损失" + loss + "HP");
}
}
}
}
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
namespace IGameLiAdv
{
/// <summary>
/// 角色
/// </summary>
internal sealed class Role
{
/// <summary>
/// 表示角色目前所持武器
/// </summary>
public IAttackStrategy Weapon { get; set; }
/// <summary>
/// 攻击怪物
/// </summary>
/// <param name="monster">被攻击的怪物</param>
public void Attack(Monster monster)
{
this.Weapon.AttackTarget(monster);
}
}
}
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
namespace IGameLiAdv
{
class Program
{
static void Main(string[] args)
{
//生成怪物
Monster monster1 = new Monster("小怪A", 50);
Monster monster2 = new Monster("小怪B", 50);
Monster monster3 = new Monster("关主", 200);
Monster monster4 = new Monster("最终Boss", 1000);
//生成角色
Role role = new Role();
//木剑攻击
role.Weapon = new WoodSword();
role.Attack(monster1);
//铁剑攻击
role.Weapon = new IronSword();
role.Attack(monster2);
role.Attack(monster3);
//魔剑攻击
role.Weapon = new MagicSword();
role.Attack(monster3);
role.Attack(monster4);
role.Attack(monster4);
role.Attack(monster4);
role.Attack(monster4);
role.Attack(monster4);
Console.ReadLine();
}
}
}
编译运行以上代码，得到的运行结果与上一版本代码基本一致。
1.4 小李的小结
Peter显然对改进后的代码比较满意，他让小李对照两份设计和代码，进行一个小结。小李简略思考了一下，并结合小于对一次设计指出的不足，说道：
“我认为，改进后的代码有如下优点：
第一，虽然类的数量增加了，但是每个类中方法的代码都非常短，没有了以前Attack方法那种很长的方法，也没有了冗长的if…else，代码结构变得很清晰。
第二，类的职责更明确了。在第一个设计中，Role不但负责攻击，还负责给怪物减少HP和判断怪物是否已死。这明显不应该是Role的职责，改进后的代码将这两个职责移入Monster内，使得职责明确，提高了类的内聚性。
第三，引入Strategy模式后，不但消除了重复性代码，更重要的是，使得设计符合了OCP。如果以后要加一个新武器，只要新建一个类，实现IAttackStrategy接口，当角色需要装备这个新武器时，客户代码只要实例化一个新武器类，并赋给Role的Weapon成员就可以了，已有的Role和Monster代码都不用改动。这样就实现了对扩展开发，对修改关闭。”
Peter和小于听后都很满意，认为小李总结的非常出色。
IGame公司的讨论会还在进行着，内容是非常精彩，不过我们先听到这里，因为，接下来，我们要对其中某些问题进行一点探讨。别忘了，本文的主题可是依赖注入，这个主角还没登场呢！让主角等太久可不好。
扫一扫
1107
被折叠的 条评论
为什么被折叠?
到【灌水乐园】发言
