Lucker
WarCraft 3

Есть ли утечка?

Добавлен
Крайне не силён в jass, поэтому сильно не бейте. Нужно ли прописывать на случай не выполнения условия call DestroyTimer(t) или утечки нет?
P.S Пример чисто теоретический.
....
function Example1 takes nothing returns nothing
    local timer t=CreateTimer()
    if  udg_example == 1 then
    call  TimerStart(t, 1, false, function Example2)
    endif
    set t = null
endfunction
...

Никто не догодался?
if udg_example == 1 then
	call TimerStart( CreateTimer(), 1, false, function Example2)
endif
Нет ну рили, нахрена тут переменная?
5 сен 2017
2 321
`
ОЖИДАНИЕ РЕКЛАМЫ...
Этот комментарий удален
0
Показать
27
0
Показать
В данный момент нет утечек. В памяти остаётся таймер. Но удалять его сейчас не нужно, удаляют после завершения работы таймера, например в функции Example2 (короче как у тебя написано через 1 минуту). Если мы удалили бы сейчас, то запуск таймера не пройзошел бы (ибо таймера больше нет).
0
Показать
2
0
Показать
я имею ввиду, что если udg_example не равен 1 то получается что таймер создан, но не запущен, нужно ли его удалять DestroyTimer(t) или t = null достаточно?
1
Показать
27
1
Показать
Можно его и не создавать
local timer t //объявляем локалку
If udg_example == 1 then //если условие прокатывает
set t = CreateTimer() //создаём таймер
call TimerStart(t,1.0,false, function Example2) //запускаем его
endif
set t = null //обнуляем
0
Показать
25
0
Показать
Steal nerves, тогда уж и обнуление в тот же иф загонять. Зачем выполнять лишние действия, если таймер не создан?
local timer t
If udg_example == 1 then
set t = CreateTimer()
call TimerStart(t,1.0,false, function Example2)
set t = null
endif
0
Показать
27
0
Показать
когда уже можно удалить (код) 
//в функции Example1 запускаем таймер. прошла 1 секунда, и запускается Example2

function Example2 takes nothing returns nothing
local timer t=GetExpiredTimer() // Истекший ( запущенный ) таймер 
.....//делаем что-то, может быть по номеру хэндла таймера достаем из хэша что-то
call DestroyTimer(t) //тогда можно уже удалить
set t = null
endfunction

function Example1 takes nothing returns nothing
local timer t
If udg_example == 1 then
set t = CreateTimer()
call TimerStart(t,1.0,false, function Example2)
set t = null
endif
endfunction
У тебя должна быть библиотека в JNGP. Мне пришлось вспоминать нативку GetExpiredTimer() jass.wikia.com/wiki/Functions
avuremybe, ну ошибся, не существенно
0
Показать
2
0
Показать
Steal nerves:
Можно его и не создавать
local timer t //объявляем локалку
If udg_example == 1 then //если условие прокатывает
set t = CreateTimer() //создаём таймер
call TimerStart(t,1.0,false, function Example2) //запускаем его
endif
set t = null //обнуляем
ок, спасибо
2
Показать
32
2
Показать
Никто не догодался?
if udg_example == 1 then
	call TimerStart( CreateTimer(), 1, false, function Example2)
endif
Нет ну рили, нахрена тут переменная?
Принятый ответ
0
Показать
25
0
Показать
quq_CCCP, человек спросил как вести дела с переменной. И уточнил, что эта функция - чисто пример.
Может ему надо с этим таймером еще какие-то действия выполнять (вязать что-то в хэш, например), для которых ему и нужна эта переменная.
0
Показать
32
0
Показать
avuremybe, юзаем глобалку для этих целей.
0
Показать
25
0
Показать
quq_CCCP, это чем же глобалка в этой ситуации лучше локалки? о.О
0
Показать
32
0
Показать
avuremybe, не создавтаь локалки там где это не всегда нужно.
0
Показать
25
0
Показать
quq_CCCP, а преимущество то в чем?
0
Показать
32
0
Показать
avuremybe, ну а типо вы не знали что создание локальной переменной это действие, да и внезапно обнуление тоже действие...
Поэтому лепить 1005000 локалок, особенно в таких примитивных функциях не стоит, лучше по возможности использовать 1 темповую глобалку на много функций.
0
Показать
15
0
Показать
avuremybe:
а преимущество то в чем?
Преимущество ничтожно. Если использование локалок повышает удобочитаемость программы - используйте.
Чтобы оставить комментарий, пожалуйста, войдите на сайт.