Гибкий шаблон декоратора?

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

Теперь, если оружие давало персонажу дополнительную силу/выносливость/хп или что-то еще, то, возможно, стоит подумать об этом. Но не похоже, что вы собираетесь изменять (или украшать) свойства персонажа с помощью всего этого.

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

С учетом сказанного, я предполагаю, что вы захотите снять оружие с помощью какого-то уникального идентификатора, скажем:

Character.unequip(LIGHTSABER);

Если вы попытаетесь вписать это в шаблон декоратора, вам придется отслеживать экипированные в данный момент предметы, а затем, после удаления оружия, вам придется обновить ссылку на объект, украшающий СВЕТОВОЙ МЕЧ, на тот, который СВЕТОВОЙ САБЕР украшает. Это много работы.

Вместо этого, возможно, стоит рассмотреть идею @Mitch и позволить оружию персонажа помочь в сумке с имуществом. Помните, что у персонажа ЕСТЬ набор оружия. Мне кажется, что композиция может быть выходом.

Есть три ответа на реализацию распространяемой статистики в видеоигре:

(1) это удовлетворит любую любительскую игру, которую вы когда-либо будете делать (и практически любую профессиональную игру):

character.GetStrength() {
  foreach(item in character.items)
    strFromItems += item.GetStrengthBonusForItems();
       foreach(buff in character.buffs)
    strFromBuffs += buff.GetStrengthBonusForBuffs();
  ...

  return character.baseStrength + strFromItems + ...;
}

(обратите внимание, что разные функции GetStrength*() не имеют ничего общего друг с другом)

(2) это удовлетворит все игры, в названии которых нет слова «diablo»:

 character.GetStr() { ... // same as above, strength is rarely queried }
 character.GetMaxHP() { 
   if (character._maxHPDirty) RecalcMaxHP();
   return character.cachedMaxHP;
 }
 // repeat for damage, and your probably done, but profile to figure out
 // exactly which stats are important to your game

(3) еще

 // changes in diablo happen very infrequently compared to queries, 
 // so up propegate to optimize queries.  Moreover, 10 people edit 
 // the stat calculation formulas so having the up propegation match 
 // the caculation w/o writing code is pretty important for robustness.

 character.OnEquip(item) {
     statList.merge(item.statlist);
 }

 character.GetStrength() {
     statList.getStat(STRENGTH);
 }

 statlist.getStat(id) {
     if (IS_FAST_STAT(id)) return cachedFastStats[id];
     return cachedStats.lookup(id);
 }

 statlist.merge(statlist) {
      // left for an exercise for the reader
 }

И, честно говоря, (3) было, вероятно, излишним.

Хммм.. Я думаю, что, возможно, шаблон команды был бы хорошим решением этой проблемы. Вот что я имею в виду:

Это ваш класс персонажа:

Public class Character {

 //various character related variables and methods here...

 Command[] equipCommands;
 Command[] unequipCommands;

 public Character(Command[] p_equipCommands, Command[] p_unequipCommands) {

  equipCommands = p_equipCommands;
  unequipCommands = p_unEquipCommands;
 }

 public void itemEquiped(int itemID) {

  equipCommands[itemID].execute(this);
 }

 public void itemUnequiped(int itemID) {

  unequipCommands[itemID].execute(this);
 }
}

Вот несколько примеров команд:

public class SwordOfDragonSlayingEquipCommand implements ItemCommand{

 public void execute(Character p_character) {

  //There's probably a better way of doing this, but of the top of my head...
  p_character.addItemToInventory(Weapons.getIteM(Weapons.SWORD_OF_DRAGON_SLAYING));

  //other methods that raise stats, give bonuses etc. here...
 }
}

public class SwordOfDragonSlayingUnequipCommand implements ItemCommand{

 public void execute(Character p_character) {

  //There's probably a better way of doing this, but of the top of my head...
  p_character.removeItemFromInventory(Weapons.getIteM(Weapons.SWORD_OF_DRAGON_SLAYING));

  //other methods that lower stats, remove bonuses etc. here...
 }
}

Конечно, это всего лишь предложение и определенно открыт для обсуждения, я не говорю, что это лучший или единственный способ сделать это...

Просто сохраните 2 набора характеристик, базовую статистику и эффективную статистику. Когда вы экипируете или снимаете предмет, добавьте или вычтите из эффективных характеристик, где это уместно. Тогда вам не придется просматривать список своего снаряжения каждый раз, когда вы хотите узнать, какова ваша статистика.

Я знаю, что этот вопрос старый, но это может помочь кому-то еще, если не ОП. Прочтите эту статью, чтобы понять, как на самом деле нужно делать подобные вещи в играх (от одного из разработчиков, работавших над играми Tony Hawk):

http://cowboyprogramming.com/2007/01/05/evolve-your-heirachy/

Составьте свои сущности/игровые объекты. Для построения поведения сущностей в играх никогда, НИКОГДА не полагайтесь на наследование или на что-либо, что само по себе каким-либо образом зависит от наследования - это включает шаблон декоратора, как предлагалось в ОП. Вы будете связывать себе руки. Композиция — это путь.

Вы ищете шаблон стратегии?

Почему бы не закодировать оружие следующим образом:

1 = бензопила 2 = дробовик 4 = рельсовая пушка

Так что теперь сумма 6 может означать только то, что у персонажа есть дробовик и рельсотрон. Это краткое изложение, поэтому вам не нужно повторять список словаря оружия. Вам по-прежнему нужна какая-то конструкция для хранения оружия, но, по крайней мере, с таким подходом вы получите скорость. Это предполагает, что у вас может быть только одно оружие каждой категории, но много категорий одновременно.