Sergant1000
WarCraft 3

Выбор редакции

Данный ресурс был выбран командой XGM как один из лучших на сайте

Вопрос по условиям

Добавлен
Внимание вопрос)
Какую конструкцию лучше использовать, конструкцию:
function Trig_SomeFunction_Conditions takes nothing returns boolean
    return ( GetSpellAbilityId() == 'SomeSpell' )
endfunction 

unction Trig_SomeFunction_Actions takes nothing returns nothing
    call SomeAction
endfunction 
//=========
function InitTrig_SomeFunction takes nothing returns nothing
    set gg_trg_SomeFunction = CreateTrigger(  )
    call TriggerRegisterAnyUnitEventBJ( gg_trg_SomeFunction, EVENT_PLAYER_UNIT_SPELL_EFFECT )
    call TriggerAddCondition( gg_trg_SomeFunction, Condition( function Trig_SomeFunction_Conditions ) )
    call TriggerAddAction( gg_trg_SomeFunction, function Trig_SomeFunction_Actions )
endfunction
Или конструкцию:
unction Trig_SomeFunction_Actions takes nothing returns nothing
    if (GetSpellAbilityId() == 'SomeSpell') then
        call SomeAction
    endif
endfunction 
//=========
function InitTrig_SomeFunction takes nothing returns nothing
    set gg_trg_SomeFunction = CreateTrigger(  )
    call TriggerRegisterAnyUnitEventBJ( gg_trg_SomeFunction, EVENT_PLAYER_UNIT_SPELL_EFFECT )
    call TriggerAddAction( gg_trg_SomeFunction, function Trig_SomeFunction_Actions )
endfunction
Какае плюсы и минусы у них.

Принятый ответ

Сделай тогда так:
code 
function Trig_SomeFunction_Conditions takes nothing returns boolean
    if GetSpellAbilityId() == 'SomeSpell' then
        call SomeAction
    endif
    return false
endfunction 

//=========
function InitTrig_SomeFunction takes nothing returns nothing
    set gg_trg_SomeFunction = CreateTrigger(  )
    call TriggerRegisterAnyUnitEventBJ( gg_trg_SomeFunction, EVENT_PLAYER_UNIT_SPELL_EFFECT )
    call TriggerAddCondition( gg_trg_SomeFunction, Condition( function Trig_SomeFunction_Conditions ) )
endfunction
Ты сделаешь проверку в условии и там же запустишь функцию, без лишнего вызова Trig_SomeFunction_Actions . Но это почти одно и то же, что и второй вариант.
Что-то я не понял, что ты подразумеваешь под "порядком" Несколько условий?
Если у тебя несколько условий, то делай градацию, начиная с самого распространенного
code 
function Trig_SomeFunction_Conditions takes nothing returns boolean
    if GetSpellAbilityId() == 'SomeSpell' then // Самое распространенное
    if GetUnitState(GetTriggerUnit()) > 100.0 then // Менее распространенное
    if ... then // И т.д. в порядке уменьшения вероятности несрабатывания
        call SomeAction
    endif
    endif
    ....
    return false
endfunction 

//=========
function InitTrig_SomeFunction takes nothing returns nothing
    set gg_trg_SomeFunction = CreateTrigger(  )
    call TriggerRegisterAnyUnitEventBJ( gg_trg_SomeFunction, EVENT_PLAYER_UNIT_SPELL_EFFECT )
    call TriggerAddCondition( gg_trg_SomeFunction, Condition( function Trig_SomeFunction_Conditions ) )
endfunction
12 мар 2013
  • Размещен: 12.03.2013 15:21:03
  • Создан: 12.03.2013 15:21:03
  • Обновлен: 12.03.2013 15:21:03
1 607
0
15
11 лет назад
Отредактирован DaeDR
0
Тогда уж лучше вторую, она напрямую вызывает функцию SomeAction. Плюс к этому у нее лучше "читаемость" Субъективное мнение
В первом случае проверяется условие, потом вызывается функция действия, потом SomeAction.
0
17
11 лет назад
0
При использованиии 2й конструкции, не будет ли увеличение нагрузки, а в свою очередь торможения, если скажем триггеров 1000 и у всех вызов EVENT_PLAYER_UNIT_SPELL_EFFECT.
0
15
11 лет назад
0
если скажем триггеров 1000 и у всех вызов EVENT_PLAYER_UNIT_SPELL_EFFECT
Тогда стоит задуматься над общей оптимизацией и создавать триггер только тогда, когда он точно будет использоваться. Проверь. Но не должно
0
17
11 лет назад
0
Ну я и спрашиваю, если я использую 2ю конструкцию, то идет проверка всех триггеров с EVENT_PLAYER_UNIT_SPELL_EFFECT с запросом функции действия и обрубается по проверки условия. Скажем если в конструкции 1 порядок то я думаю что в первой и второй конструкции запрос одинаков, а если несколько порядков???
1
15
11 лет назад
1
Сделай тогда так:
code 
function Trig_SomeFunction_Conditions takes nothing returns boolean
    if GetSpellAbilityId() == 'SomeSpell' then
        call SomeAction
    endif
    return false
endfunction 

//=========
function InitTrig_SomeFunction takes nothing returns nothing
    set gg_trg_SomeFunction = CreateTrigger(  )
    call TriggerRegisterAnyUnitEventBJ( gg_trg_SomeFunction, EVENT_PLAYER_UNIT_SPELL_EFFECT )
    call TriggerAddCondition( gg_trg_SomeFunction, Condition( function Trig_SomeFunction_Conditions ) )
endfunction
Ты сделаешь проверку в условии и там же запустишь функцию, без лишнего вызова Trig_SomeFunction_Actions . Но это почти одно и то же, что и второй вариант.
Что-то я не понял, что ты подразумеваешь под "порядком" Несколько условий?
Если у тебя несколько условий, то делай градацию, начиная с самого распространенного
code 
function Trig_SomeFunction_Conditions takes nothing returns boolean
    if GetSpellAbilityId() == 'SomeSpell' then // Самое распространенное
    if GetUnitState(GetTriggerUnit()) > 100.0 then // Менее распространенное
    if ... then // И т.д. в порядке уменьшения вероятности несрабатывания
        call SomeAction
    endif
    endif
    ....
    return false
endfunction 

//=========
function InitTrig_SomeFunction takes nothing returns nothing
    set gg_trg_SomeFunction = CreateTrigger(  )
    call TriggerRegisterAnyUnitEventBJ( gg_trg_SomeFunction, EVENT_PLAYER_UNIT_SPELL_EFFECT )
    call TriggerAddCondition( gg_trg_SomeFunction, Condition( function Trig_SomeFunction_Conditions ) )
endfunction
Принятый ответ
Чтобы оставить комментарий, пожалуйста, войдите на сайт.