Дополнение метода у конкретного объекта

мамин программист · 29 Май 2017

Как переопределить конкретный метод для конкретного объекта какого-нибудь класса, с вызовом в этом же методе изначального состояния этого метода?

Так как ни один нормальный человек скорее всего не поймет тот бред, который я изложил выше, задам свой вопрос на таком примере:
Задача: При направлении способности на базовое дерево ~~Третий подряд вопрос про деревья ДА СКОЛЬКО МОЖНО!?7~~, в точке местоположения вашего героя создается новое дерево, в то время как старое (то, на которое была направлена способность) срубается на неограниченный срок. При ломке нового дерева старое тут же вырастает. (Вообще цель скила - передвинуть дерево, а после его ломки вернуть обратно. Но двигать его через SetOrigin - так себе идея.)
Никому не нужные рассуждения: Если не писать свое дерево с нуля (чего мне бы не хотелось делать), то единственный способ создать дерево в точке - это CreateTempTree( pos, 2283221337 ). А единственный способ убрать такое дерево раньше срока - Kill(). Поэтому я уверен, что все дотовские предметы и скилы расправляются со временным деревом именно так. Соответственно мне нужно переопределить метод Kill() для этого дерева так, чтобы в нем сначала выращивалось старое дерево, а потом уже удалялось новое. Но как мне удалить новое дерево, если единственный способ сделать это - Kill(), а именно его я сейчас и переопределяю?
Я пытался сделать так:

Код:

--[[
Вырванный кусок кода из тела функции OnSpellStart(). hTarget - старое дерево.
CreateMyTree - созданный мною метод. Делает то же, что и CreateTempTree, но еще и возвращает только что созданное дерево
]]
local model = hTarget:GetModelName()
hTarget:CutDownRegrowAfter( 99999, self:GetCaster():GetTeamNumber() )

local hTree = CreateMyTree( self:GetCaster():GetOrigin(), 99999)
hTree:SetModel( model )
hTree.assigned_tree = hTarget
function hTree:Kill()
	if self.assigned_tree then
		self.assigned_tree:GrowBack()
	end

	self.BaseClass.Kill()
end

Но это все равно вызвало рекурсивный вызов, заставивший Доту вылететь. Если честно, я вообще не знаю, как работает ".BaseClass"; я случайно нашел его в шаблоне abilities_lua (или как он там называется) и решил попробовать. Был бы очень рад, если бы еще и объяснили, как пользоваться сим изобретением.
Вопрос: Как выполнить задачу? Абстрактный вопрос описан первым предложением темы.

CryDeS · 29 Май 2017

Для начала, в lua нет классов, методов и объектов, не упарывайся настолько.
А так, нахрен тебе переопределять его? Сделай вместо переопределение новый метод CustomKill() к примеру, и там уже нормально убивай дерево.

мамин программист · 30 Май 2017

CryDeS написал(а):
в lua нет классов, методов и объектов, не упарывайся настолько.

А как же тогда их называть? ;D

мамин программист · 30 Май 2017

Только что нашел слушателя (событие) tree_cut и подумал "Вот оно!", но нет. Он передает только координаты сломанного дерева, а не само дерево. И вычислить сломанное дерево по координатам будет нельзя, так как оно уже сломано. Нет его.

CryDeS · 30 Май 2017

мамин написал(а):
Мне нужно, чтобы старое дерево восстанавливалось не зависимо от того, как было сломано новое, будь то топорик, или третий скилл энигмы. Как я уже написал, все эти способности скорее всего убирают временные деревья через Kill(). А если делать CustomKill(), то придется с нуля переписывать все существующие абилки, ломающие деревья и менять в них kill на CustomKill. А это долго... (Мне нужны уже существующие абилки)

А как же тогда их называть? ;D

Почему в своей задаче ты так не используешь SetOrigin?
И все дотовские предметы и абилки сделаны не на lua, соответственно просто переопределив lua функцию нихрена не поменяется(скорее всего). И ты уверен что оно уходит в рекурсию(проверил принтами?), а не просто крашит на чем нибудь еще.

MahouShoujo · 30 Май 2017

Методы можно переопределять только для своего кода. Все С++ классы в луа представлены как прокси.

мамин программист · 30 Май 2017

[quote author=CryDeS link=topic=1465.msg9462#msg9462 date=1496121257]
Почему в своей задаче ты так не используешь SetOrigin?
И все дотовские предметы и абилки сделаны не на lua, соответственно просто переопределив lua функцию нихрена не поменяется(скорее всего). И ты уверен что оно уходит в рекурсию(проверил принтами?), а не просто крашит на чем нибудь еще.
[/quote]

SetOrigin для деревьев работает крайне неадекватно. Лучше его вообще не использовать.
Про то, что абилки сделаны не на луа, и переопредение кила ничего не даст, верно подмечено. Я об этом не подумал. А тут еще и пишут, что встроенные функции переопределить вообще нельзя.
И нет, рекурсивный вызов я не проверял. Но когда дота виснет и через пару секунд крашится - это признак, чаще всего свойственный именно неограниченной рекурсии.

Короче понял. Мою задачу надо решать другим путем. У меня есть одна кривая, но единственная идея. Когда реализую - скину код.

Спасибо всем за ответы!

[size=8pt](Вопрос про .BaseClass еще актуален)[/size]

MahouShoujo · 30 Май 2017

Код:

local oldKill = hTree.Kill
function hTree:Kill(...)
	if self.assigned_tree then
		self.assigned_tree:GrowBack()
	end

	oldKill(...)
end

Про бейзкласс в этом случае конкретно не знаю, по-моему он работает только в абилках.

CryDeS · 30 Май 2017

[quote author=MahouShoujo link=topic=1465.msg9468#msg9468 date=1496131765]

Код:

local oldKill = hTree.Kill
function hTree:Kill(...)
	if self.assigned_tree then
		self.assigned_tree:GrowBack()
	end

	oldKill(...)
end

Про бейзкласс в этом случае конкретно не знаю, по-моему он работает только в абилках.
[/quote]
Кстати, очень вероятно, так что можно самому допилить этот BaseClass ко всем деревьям ссылаясь, к примеру, на CBaseEntity в данном случае.

MahouShoujo · 30 Май 2017

Вопрос был для конкретного инстанса, но да, можно. Просто редко имеет смысл.

CryDeS · 30 Май 2017

MahouShoujo написал(а):
Вопрос был для конкретного инстанса, но да, можно. Просто редко имеет смысл.

Или hTree.BaseClass = CBaseEntity, что даст эффект для конкретного инстанса. Но если нужен ток Kill, то твой способ пизже.

мамин программист · 30 Май 2017

В общем, я ошибся везде, где только можно, и даже там, где нельзя.

Дота крашилась не из-за "рекурсивного" вызова через ".BaseClass", а из-за хер знает чего. А именно, из-за этой строчки:

Код:

local model = hTarget:GetModelName()

Причем проблема не в GetModelName, а в том, что я применяю его к дереву. С другими юнитами эта функция нормально работает.

Касательно ".BaseClass", когда я все-таки сделал через него, мне написали нил валуе. Видимо, он действительно есть только для способностей

Большое спасибо за этот код:
[quote author=MahouShoujo link=topic=1465.msg9468#msg9468 date=1496131765]

Код:

local oldKill = hTree.Kill
function hTree:Kill(...)
	if self.assigned_tree then
		self.assigned_tree:GrowBack()
	end

	oldKill(...)
end

[/quote]
Теперь я хотя бы примерно понимаю, как работать с функциями в луа. Но его надо было немножко изменить, чтобы он начал работать:

Код:

hTree.oldKill = hTree.Kill
function hTree:Kill()
	if self.assigned_tree then
		self.assigned_tree:GrowBack()
	end

	self.oldKill()
end

И все-таки, как и было предположено, переопределение этого несчастного кила мне ничего не дало. Он делает то, что нужно, когда я вызываю Kill в своем коде. Но валвавские абилки просто ломают дерево, не вызывая Kill.

И внезапно появляется новый вопрос: "Как получить модель выбранного дерева, если GetModelName для него не работает?". А без ответа на этот вопрос, весь этот аутизм, которым я страдал вместе с моими деревьями пропадет напрасным трудом, так как все-таки цель скила - переместить дерево.

мамин программист · 30 Май 2017

Задача решена, если не учитывать проблему с заменой модели. Если коротко, то при использовании скила на кастера вешается модификатор, в который передаются старое и новое деревья, а в его функции OnIntervalThink выполняется проверка на существование нового дерева.

Код:

function vengefulspirit_nether_swap_sc_lua:OnSpellStart()
	local hCaster = self:GetCaster()
	local hTarget = self:GetCursorTarget()

	local vPos1 = hCaster:GetOrigin()

	hTarget:CutDownRegrowAfter( 99999, hCaster:GetTeamNumber() )

	hTree = CreateMyTree( vPos1, 99999 )
	local info = { hOldTree = hTarget, hNewTree = hTree }
	local tree_checker = hCaster:AddNewModifier( hCaster, self, "tree_checker", { duration = -1 } )
	tree_checker:SetInfo( info )
	function tree_checker:OnIntervalThink()
		if self.info.hNewTree:IsNull() then
			self.info.hOldTree:GrowBack()
			self:Destroy()
		end
	end
	hTree.tree_checker = tree_checker
			
			
	ResolveNPCPositions( vPos1, 64.0 )
end

Код:

tree_checker = class( {} )

function tree_checker:GetAttributes()
	return MODIFIER_ATTRIBUTE_MULTIPLE
end

function tree_checker:IsHidden()
	return true
end

function tree_checker:OnCreated( kv )
	self:StartIntervalThink( 0.2 )
end

function tree_checker:OnIntervalThink()
end

function tree_checker:SetInfo( info )
	self.info = info
end

Это не будет работать для временных деревьев, но это уже не по теме.

Илья · 30 Май 2017

По моему, тебе действительно проще было бы создать свои деревья и раскидать их по карте. Т.к. двигаться они не будут, то и на производительности это не отразится.

I_GRIN_I · 1 Июн 2017

А на деревья модификаторы нельзя кидать, это же декорации, да? А как работает сова у трента тогда?

мамин программист · 1 Июн 2017

I_GRIN_I написал(а):
А на деревья модификаторы нельзя кидать, это же декорации, да? А как работает сова у трента тогда?

Как именно она работает - одному лишь Габену известно. Да и вообще задаваться таким вопросом не совсем корректно, если все базовые способности написаны на плюсах. Другой вопрос - "Как сделать такую же способность, чтобы она работала?". А тут уже очень много способов существует. Как видишь, я у себя сделал имитацию модификатора для дерева: модификатор накладывается на героя, но хранит в себе инфу о дереве.

И да, деревья - не декорации. Это полноценные энтити от класса CBaseEntity, но еще не NPC, поэтому прямо на них модифаер наложить нельзя. Хотя что из себя представляет базовое дерево я даже представить не могу. В списке API написано, что деревья наследуются от CBaseEntity, но некоторых методов CBaseEntity у них просто нет (например SetModel).

I_GRIN_I · 18 Окт 2017

мамин программист,

Код:

CDOTA_MapTree:CreateTreeOnAbsHeroAndCutTargetTree(hero)
self:CutDownRegrowAfter(999999,hero:GetTeamNumber())
local newtree = CreateTempTree(hero:GetAbsOrigin(),999999)
newtree.formertree = self
end

Подключаешь эвент tree_cut, ищешь при этом эвенте вокруг понта ентити дерева, вытаскиваешь переменную formertree, если она есть и formertree:GrowBack(), главное отчистить переменную newtree.formertree, но хз ентити дерева наверное успеет исчезнуть

мамин программист · 19 Окт 2017

[quote author=I_GRIN_I link=topic=1465.msg11921#msg11921 date=1508341596]
мамин программист,

Код:

CDOTA_MapTree:CreateTreeOnAbsHeroAndCutTargetTree(hero)
self:CutDownRegrowAfter(999999,hero:GetTeamNumber())
local newtree = CreateTempTree(hero:GetAbsOrigin(),999999)
newtree.formertree = self
end

Подключаешь эвент tree_cut, ищешь при этом эвенте вокруг понта ентити дерева, вытаскиваешь переменную formertree, если она есть и formertree:GrowBack(), главное отчистить переменную newtree.formertree, но хз ентити дерева наверное успеет исчезнуть
[/quote]

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

I_GRIN_I · 19 Окт 2017

мамин программист, тогда ты можешь при создании дерева заносить его в массив и проверять каждую секунду исстендинг этого дерева и если нет, то возводить старое дерево

Дополнение метода у конкретного объекта

мамин программист

Друзья CG

CryDeS

Друзья CG

мамин программист

Друзья CG

мамин программист

Друзья CG

CryDeS

Друзья CG

MahouShoujo

Продвинутый

мамин программист

Друзья CG

MahouShoujo

Продвинутый

CryDeS

Друзья CG

MahouShoujo

Продвинутый

CryDeS

Друзья CG

мамин программист

Друзья CG

мамин программист

Друзья CG

Илья

Друзья CG

I_GRIN_I

Друзья CG

мамин программист

Друзья CG

I_GRIN_I

Друзья CG

мамин программист

Друзья CG

I_GRIN_I

Друзья CG