Centyrion:

мне известно что каждый объект который создается занимает в памяти определенное кол-во байтов

Uint32 это четыре байта. Ну и строки остаются в памяти до конца игры.

Только чтоб это сказалось на производительности, это нужно специально постараться.

Выше всё правильно сказали, integer по определению не может вызывать утечек, также
Centyrion:

прочел тут

https://warcraft3ft.clan.su/forum/60-48648-1

-Данная функция абсолютно не криптостойкая, она генерирует 32-битный ключ и её исходники есть в интернете

https://www.hiveworkshop.com/threads/bits-of-interest.213272

Поэтому ничего не стоит сделать брутфорсер и узнать, что за герой будет у тебя создаваться

http://forum.wc3edit.net/deprotection-cheating-f64/hidden-activa...

Где-то на гитхабе видел готовый брутфорсер на java.

Centyrion,

я тебе в 10 раз говорю НИКОГДА не используй вейты в Циклах

Ну не знаю, я использовал на практике специальную функцию, где есть таймер и цикл, а в цикле вейт(!) на 1 сек, цикл работает, пока не истекает таймер. Так вот, с ней никогда никаких ошибок и крашей не было, всё стабильно. Даже на паузе. Версия 1.26а

function BackEnd takes nothing returns nothing
local timer t=GetExpiredTimer()
local integer id=GetHandleId(t)
local unit c=LoadUnitHandle(udg_HT,id,0)
call UnitRemoveAbility(c,'A06I') // здесь убираем абилку и очищаем следы
call DestroyTimer(t)
call FlushChildHashtable(udg_HT,id)
set t=null
set c=null
endfunction
function Back takes nothing returns nothing
local unit u=GetSpellAbilityUnit()
local timer t=CreateTimer()
call SaveUnitHandle(udg_HT,GetHandleId(t),0,u)
call UnitAddAbility(u,'A06I') // это пассивная абилка
call TimerStart(t,0.5,false,function BackEnd) //запускаем таймер чтобы делать чтото
set u=null
set t=null
endfunction

Centyrion, это обыкновенный хэш. Там нечему утекать и ломаться в мультиплеере. Его часто используют для именования ключей хэш-таблиц.

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

Centyrion, это обыкновенный хэш. Там нечему утекать и ломаться в мультиплеере. Его часто используют для именования ключей хэш-таблиц.

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

((цитата
создает ли эта нативка утечки или чтото такое

и в мултиплеере как она себя поведет не вызовет десинх или все нормально будет

native StringHash takes string s returns integer

native I2S  takes integer i returns string

Centyrion:

Borodach:

Centyrion, Может способность не для героя, либо юнит не герой. Краш бывает, если проводить манипуляции с характеристиками на простых юнитах

может быть, я даю способность вампиризм когда юнит выучил способность, триггер проверяет уровень способности и если уровень абилки больше 0 то даю герою пасивку на основе вампиризма, у него тут рекурсивная абилка в 2 Скрине которая стопорит поток, в цикле никогда не надо использовать Wait, дал все что нужно герою и запустил таймер далее достаешь значения из переменных и работаешь с ними.

В моем случае мне нужно было чтобы тригер работал с другими игроками, но при этом на "нужном" игроке, счётчик не увеличивался. Поэтому просто выключить его я не могу. И потому, я решил таким образом попробовать реализовать. Цикл я ещё даже не тестил, можно не обращать на него внимание.
В принципе если конкретно то в чем проблема не найдем, я попробую завтра снести полностью этот тригер, и переделать его как только смогу. Мне просто интересно что конкретно может вызывать эту проблему.

Centyrion, Может способность не для героя, либо юнит не герой. Краш бывает, если проводить манипуляции с характеристиками на простых юнитах

function BackEnd takes nothing returns nothing
local timer t=GetExpiredTimer()
local integer id=GetHandleId(t)
local unit c=LoadUnitHandle(udg_HT,id,0)
call UnitRemoveAbility(c,'A06I')
call DestroyTimer(t)
call FlushChildHashtable(udg_HT,id)
set t=null
set c=null
endfunction
function Back takes nothing returns nothing
local unit u=GetSpellAbilityUnit()
local timer t=CreateTimer()
call SaveUnitHandle(udg_HT,GetHandleId(t),0,u)
call UnitAddAbility(u,'A06I')
call TimerStart(t,0.5,false,function BackEnd)
set u=null
set t=null
endfunction

Centyrion:

конечно русская локализация радует глаз но с первым триггером Скрин 1 все нормально а со 2 Скрином, бред, юзай таймеры вместо вейтов, они утечные и не универсальные, и фатал вызывает что в цикле от 1 до 30 ты стопоришь поток, конечно он выкинет игрока который кастовал способность

Если бы. Цикл я добавил позже, и знаю что он скорее всего нерабочий. Оно и без цикла крашит или не даёт запустить карту. =(
UPD: Сейчас даже удалил цикл, на всякий, как и говорил - та же история. Карту не запускает, сразу в меню варика выкидывает.
UPD1: Если удалить первый тригер, то игра крашается. Если его не удалять, то не крашается, но не запускается. =/

Больше никаких оптимизаций в современных версиях компилятора vJass нет?

function F_Item_Stack_Cond takes nothing returns boolean
return (GetItemCharges(GetManipulatedItem()) > 0) and (GetItemType(GetManipulatedItem()) == ITEM_TYPE_CHARGED) and (GetManipulatedItem()!=GetItemOfTypeFromUnitBJ(GetManipulatingUnit(),GetItemTypeId(GetManipulatedItem()))) and (UnitHasItemOfTypeBJ(GetManipulatingUnit(),GetItemTypeId(GetManipulatedItem()))==true)
endfunction

local item i=GetManipulatedItem()
local item gi
local unit u=GetTriggerUnit()
local integer id=GetItemTypeId(i)
call UnitRemoveItem(u,i)
if((GetInventoryIndexOfItemTypeBJ(u,id)>0))then
call SetItemCharges(gi,(GetItemCharges(gi)+GetItemCharges(i)))
call RemoveItem(i)
else
call UnitAddItem(u,i)
endif

GetInventoryIndexOfItemTypeBJ

function UnitHasItemOfTypeBJ takes unit whichUnit, integer itemId returns boolean
    return GetInventoryIndexOfItemTypeBJ(whichUnit, itemId) > 0
endfunction

GetInventoryIndexOfItemTypeBJ

function GetInventoryIndexOfItemTypeBJ takes unit whichUnit, integer itemId returns integer
    local integer index
    local item    indexItem

    set index = 0
    loop
        set indexItem = UnitItemInSlot(whichUnit, index)
        if (indexItem != null) and (GetItemTypeId(indexItem) == itemId) then
            return index + 1
        endif

        set index = index + 1
        exitwhen index >= bj_MAX_INVENTORY
    endloop
    return 0
endfunction

> native TriggerSleepAction takes real timeout returns nothing

или
function PolledWait takes real duration returns nothing

ни то ни другое, лол. ты спросил, тебе ответили, а ты все равно на стенку лезешь?

Centyrion, каким образом протестировали? Вейт не точен, эти неточности накапливаются за время игры, и в один прекрасный момент чувака десинхает.

quq_CCCP:

Вейт не точен, эти неточности накапливаются за время игры, и в один прекрасный момент чувака десинхает.

Так вейты синхронизируются, но десинки это таки как-то вызывает, судя по словам мудрых людей.... Лучше юзать таймер.

Centyrion, каким образом протестировали? Вейт не точен, эти неточности накапливаются за время игры, и в один прекрасный момент чувака десинхает.

> чем чреват этот wait в определенных местах

падением потока. в худшем случаем десинхронизации в мультиплеере