Делегирование расчётов "статическому" классу

[Appleman]

Сообщение от caseyryan

Как-то не айс, когда значения для объекта расситываются в классе, который к нему никакого отношения не имеет. Это создает дополнительную связанность, которая тут ни к чему. Присоединяюсь к ораторам выше) Плюсую за наследование

На самом деле наследуется. То есть метод override-ится в наследниках, но в каждом - отсылка на свой вариант расчёта. Всё понял, отстал. Оставлю пока как временную отладочную фишку, потом внесу в тело класса, когда будет рабочий вариант.

[caseyryan]

То есть там и перезапись и в каждом перезаписанно еще и вызов статического? Это конечно вообще дичь, как по мне) Лучше действительно прямо в этих методах написать рассчет.
п.с. А можно глянуть что там за расчет?

[Godwarlock]

Есть расчет жизней в методе, относительно выносливости персонажа. Расчет применяется в инвентаря и в бое. Смысл наследовать класс, в котором прописан этот метод и инвентарю, и бою? Почему бы его не сделать статичным?

[Appleman]

Сообщение от caseyryan

То есть там и перезапись и в каждом перезаписанно еще и вызов статического? Это конечно вообще дичь, как по мне) Лучше действительно прямо в этих методах написать рассчет.
п.с. А можно глянуть что там за расчет?

Да нечего там особенно смотреть. Если совсем минималистично, то выглядит примерно так (сорри если отступы не получатся, писал прямо здесь):

public class Hero extends Character
{
   override public function get attackRating()
   {
      return AttackRating.calculate(this);
   }
}
 
public class AttackRating
{
static private const POW: uint = 3; // степень
static private const BASE: Nubmer = 1.5; // база расчёта
 
static public function calculate(ch: Character) : Number
{
   // Всякие расчёты
   return result;
}

Я думаю, что вопрос закрыт. Позицию сообщества я понял.

Добавлено через 27 часов 7 минут
Ребята! Давайте забудем про статические классы, с ними всё понятно. Я тут пока обдумывал ситуацию, мне такая крамольная мысль в голову пришла. А верно ли я вообще втыкаю приведённый выше расчёт в наследника класса Character. Что Вы думаете с точки зрения философии ООП, принципа одной ответственности класса и т.п.?

Смотрите. Сейчас у меня метод execute() класса Action ломится в наследника Character за показателем attackRating, который рассчитывается на основе свойств персонажа, хранящихся в Character. Корректно ли в данном случае помещать подобный метод в Character? Или правильнее запросить из Character значения необходимых свойств, и посчитать непосредственно в классе Action, создав там отдельный приватный метод?

[Zebestov]

Правильно было бы иметь какой-то XML со всеми св-вами, грузить его в какую-то модель и таким образом вынести в нее подсчет результата тех или иных событий: кто-то поднял аптечку — насколько поправилось здоровье, кто-то треснулся в препятствие — насколько покалечился, кто-то с кем-то обменялся ударами — чо-там-кто-кого. Это еще более-менее норм было бы.

[Wolsh]

Что Вы думаете

Я думаю, тебя понесло в какие-то немыслимые дебри.
Изначально ты говорил, что этот пресловутый метод — всего-лишь геттер свойства Характера.
Не имеет никакого значения, насколько сложно там это свойство вычисляется: тем, кто обращается за ним к персонажу, нужно всего-лишь получить значение этого свойства. Почему расчет должен происходить где-то еще, совершенно не понятно. Тогда вообще не надо обращаться к персонажу, а сразу идти к сионским мудрецам AttackRating.calculate(_hero); Но если это свойство ПЕРСОНАЖА, присущее ЕМУ, то немозголомно было бы _hero.attackRating;
Не? Почему ты считаешь, что его свойство — это не его ответственность?

Или правильнее запросить из Character значения необходимых свойств, и посчитать непосредственно в классе Action

Если это не свойство, присущее Характеру, а какая-то переменная величина для дальнейших расчетов, необходимая только локально в функции, то можно.

[Appleman]

Сообщение от Wolsh

Я думаю, тебя понесло в какие-то немыслимые дебри.

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