Обработка коллекций объектов с помощью переключателя заявление

Question

мне нужно иметь игрока оборудовать пункт, как показано здесь:

IEquiptable является интерфейсом.

и этот метод находится в моем классе player.

public void equip(IEquiptable equipable) 
    { 
     switch (equipable.GetType()) 
     { 
      case equipable is Weapons: 
       this.weapon = equipable; 
       break; 

      case equipable is Shield: 
       this.shield = equipable 
       break; 

       //etc etc.. 
     } 
    } 

Я получаю сообщение об ошибке, что переключатель experession должен быть логическим значение, символ, строка, интеграл, перечисление или соответствующим обнуляемого типа.

Я мог бы справиться с этим, используя метод equit в каждом из классов моего оружия/щита и передать класс player в качестве параметра. Но я чувствую, что это немного нелогично, так как игрок должен экипировать предмет, а не предмет, который экипирует его на игроке.

Answer 1

У вас уже есть интерфейс, так почему бы вам не воспользоваться им?

public interface IEquipable 
{ 
    void EquipOn(Player player); 
} 

public class Shield : IEquipable 
{ 
    public void EquipOn(Player player) 
    { 
     player.shield = this; 
    } 
} 

public class Weapons : IEquipable 
{ 
    public void EquipOn(Player player) 
    { 
     player.weapon = this; 
    } 
} 

и ваш код становится

public void equip(IEquiptable equipable) 
{ 
    equipable.EquipOn(this); 
} 

В самом деле, вам даже не нужен этот метод, поскольку он сводится лишь к пересылке его работу собственного класса. Каждый раз, когда у вас есть свой IEquipable, вы просто называете его EquipOn.

Answer 2

public void equip(IEquiptable equipable) 
{ 
    var type = equipable.GetType(); 
    if(type is Weapons) 
    { 
    this.weapon = equipable; 
    return; 
    } 
    if(type is Shield) 
    { 
    this.shield = equipable; 
    return; 
    } 
    //etc 
} 

Answer 3

Из значений случаев должны быть константами, обычное решение было бы создать перечисления ..

public enum EquippableTypes { 
    Weapons, 
    Shield, 
    Antenna, 
    Underwear 
} 

public partial class Player { 
    public void Equip(IEquippable equippable) { 
     switch(equippable.Type) { 
      case EquippableTypes.Weapons: 
       this.weapon=equippable; 
       break; 

      case EquippableTypes.Shield: 
       this.shield=equippable; 
       break; 

      case EquippableTypes.Antenna: 
       this.antenna=equippable; 
       break; 

      case EquippableTypes.Underwear: 
       this.underwear=equippable; 
       break; 

      // ... 
     } 
    } 

    IEquippable weapon, shield, antenna, underwear; 
} 

Так что ваш Одеваемый интерфейс/классы будут выглядеть ..

public partial interface IEquippable { 
    EquippableTypes Type { 
     get; 
    } 
} 

public partial class Weapons: IEquippable { 
    public EquippableTypes Type { 
     get { 
      return EquippableTypes.Weapons; 
     } 
    } 
} 

public partial class Shield: IEquippable { 
    public EquippableTypes Type { 
     get { 
      return EquippableTypes.Shield; 
     } 
    } 
} 

public partial class Antenna: IEquippable { 
    public EquippableTypes Type { 
     get { 
      return EquippableTypes.Antenna; 
     } 
    } 
} 

public partial class Underwear: IEquippable { 
    public EquippableTypes Type { 
     get { 
      return EquippableTypes.Underwear; 
     } 
    } 
}