Автор: ADOLF.

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

Вот отрывок кода, его надо несколько раз перечитать, чтобы понять, к какому блоку что относиться. Мне сначала показалось, что endloop - окончание функции, далее идет объявление другой. Логику расстановки отступов понять я так и не смог.

((код jass

local real StartLife = 0.
local real EndLife = 0.
if Time < 1 and GetWidgetLife(Dummy) > 0.405 then
call GroupEnumUnitsInRange(Group,X,Y,120,null)
loop
set Picked = FirstOfGroup(Group)
exitwhen Secret_Pirouette_Filter(Caster,Picked) == true or Picked == null
call GroupRemoveUnit(Group,Picked)
endloop

if Picked != null then
if LoadBoolean(udg_Hashtable,Handle,7) == true then
set StartLife = GetUnitState(Picked,UNIT_STATE_LIFE)
endif
call UnitDamageTarget(Caster,Picked,Damage, true, false, ATTACK_TYPE_MAGIC, DAMAGE_TYPE_MAGIC, null)
call Secret_Pirouette_DPS(Caster,Picked,25,LoadBoolean(udg_Hashtable,Handle,7))
call DummyCastTarget(Caster,Picked,Level,'A001',"thunderbolt")//Raw-code
if LoadBoolean(udg_Hashtable,Handle,7) == true then
set EndLife = GetUnitState(Picked,UNIT_STATE_LIFE)
call TextTag(Picked,udg_PlayerColors[GetPlayerId(GetOwningPlayer(Caster))+1]+"-"+I2S(R2I(StartL​ife-EndLife))+"|r",8)
endif
call DestroyEffect(AddSpecialEffectTarget("Abilities\\Weapons\\WingedSerpentMissile\\WingedSerpentMi​ssile.mdl",Picked,"origin"))
call DestroyEffect(AddSpecialEffectTarget("Abilities\\Spells\\Other\\Stampede\\StampedeMissileDeath.​mdl",Picked,"origin"))
call KillUnit(Dummy)
call SaveReal(udg_Hashtable,Handle,9,10)
endif
=====
call SetUnitX(Dummy,X+Speed*Cos(Angle-25))
call SetUnitY(Dummy,Y+Speed*Sin(Angle-25))
))

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

((код jass

*********************\\-------------------------------------------------------\\
=====================\\ *** * ***** **** **** ***** * * \\
Timer Library \\ * * * * * * * * * * * * * \\
by \\ ***** * **** ** **** **** **** *** \\
ALEXPREY © \\ * * * * * * * * * * * * \\
=====================\\ * ***** ***** * * ***** ** \\
*********************\\-------------------------------------------------------\\
))

Даешь осмысленные комментарии! ФИЛЬТР! Я СКАЗАЛ ГОРБА ФИЛЬТР! Главное, без комментария догадаться, что эта функция что-то фильтрует, нельзя. Точно также, как и понять из комментария, что же именно фильтрует эта функция.

((код jass

ФИЛЬТР!
===============================
function Secret_Pirouette_Filter takes unit Caster,unit Picked returns boolean
return IsUnitEnemy(Picked,GetOwningPlayer(Caster)) and GetWidgetLife(Picked) > 0.405 and not IsUnitType(Picked,UNIT_TYPE_STRUCTURE) and not IsUnitType(Picked,UNIT_TYPE_MAGIC_IMMUNE) and not IsUnitType(Picked,UNIT_TYPE_MECHANICAL)
endfunction
===============================
))

На десерт.

((код jass

function Chat_Space takes nothing returns nothing
_________________________________________________
call ClearTextMessages()
_________________________________________________
endfunction
))

Хорош тот код, который не нуждается в комментариях. Назовите функцию CreateTextTag*** (OnUnit например) - и будет вам счастье.

((код jass

Проверка на неуязвимость. Автор - DioD (from XGM forum)
...
Вывод текста. Автора нет.

function TextTag takes unit u, string text, real height returns texttag
))

"Милый, заберешь сегодня нашего малого из детского садика."
Милый идет в садик, забирает оттуда домой всех детей, приводит их домой и начинает по одному проверять, не его ли это сын, если нет - отправляет ребенка назад в садик.
Также обратите внимание на b == true. Это неправильно. Для надежности надо писать (b == true and b != false), этому нас еще в свое время MPI3 учил. На самом деле надо писать просто b, например if b then...

((код jass

local group Group = CreateGroup()
...
call GroupEnumUnitsInRange(Group,X,Y,120,null)
loop
set Picked = FirstOfGroup(Group)
exitwhen Secret_Pirouette_Filter(Caster,Picked) == true or Picked == null
call GroupRemoveUnit(Group,Picked)
endloop
...
call DestroyGroup(Group)
set Group = null
))

Правильно:

((код jass

c_temp и t_temp - глобальные переменные
gr_temp - Глобальная, всегда пустая группа, создання для переборов юнитов

JASS
function foo takes nothing returns boolean
if t_temp == null and SecretPirouetteFilter (c_temp, GetFilterUnit()) then
set t_temp = GetFilterUnit()
endif
return false
endfunction
...
set c_temp = Caster
set t_temp = null
call GroupEnumUnitsInRange (gr_temp, x, y, 120., Condition(function Foo))

cJass
c_temp = Caster
t_temp = null
GroupEnumUnitsInRange (gr_temp, x, y, 120., Condition(lambda boolean () {
if (t_temp == null && SecretPirouetteFilter (c_temp, GetFilterUnit()) {
t_temp = GetFilterUnit()
}
return false
}))

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

Что это за ключ 1? Юнит-пустышка? Юнит, применивший способность? Цель? А вы наверное и переменные называете u1, u2 и т.д.?

((код jass

call SaveUnitHandle(udg_Hash,GetHandleId(t),1,u)
))

Неправильно решение проблемы - высока возможность опечатки, да и вообще, криво это все.

((код jass

call RemoveSavedHandle(udg_Hashtable,GetHandleId(Caster),StringHash("Omnislash Caster Trigger"))
))

Внимание, правильный ответ:

((код jass

JASS
globals
constant integer HASHKEY_OMNISLASH_CASTER = 1
constant integer HASHKEY_OMNISLASH_TARGET = 2
endglobals
...
call RemoveSavedHandle (udg_Hashtable, GetHandleId(Caster), HASHKEY_OMNISLASH_CASTER)

cJass
enum (HASHTABLENAME_KEYS) {
HASHKEY_OMNISLASH_CASTER
HASHKEY_OMNISLASH_TARGET
}
...
RemoveSavedHandle (udg_Hashtable, GetHandleId(Caster), HASHKEY_OMNISLASH_CASTER)

))

Чем больше всего обнулить, тем быстрее будет работать.

((код jass

local real StartLife = 0.
local real EndLife = 0.
...
set StartLife = 0.
set EndLife = 0.
))

Самые продвинутые обнуляют даже boolean.

((код jass

local boolean TextMessages = true
...
set TextMessages = false
endfunction
))

Скалярные локальные переменные обнулять не нужно. Не нужно! Ещё раз - не нужно! integer, boolean, real не нужно обнулять. Вообще, сам термин обнуления некорректен.
Утечки могут быть вызваны удаленными объектами, на которые остались ссылки. Читайте мое добавление к статье Сергея "Осваиваем jass", там об этом подробно.

Даеш еще!

Алсо не очень понял что там про комментарии к функции TextTag.

Установил статус важной :3

Шик. Неужели кто-то подумал, что перебор группы лупом хорошо оценят?

Вкусно :3
какое счастье, за мной ничего этого не водится

круто, адик умеет "ненавязчиво ткнуть рожей" в ошибки=)

Да, я краб. Ну что ж.

Rampage добавил:
Я ни одной статьи не прочитал про Jass, и обнулял реальные. Кстати, это спелл я из-за крабовства и отменил. В Omnislash'e максимально аккуратно делал.

call RemoveSavedHandle(udg_Hashtable,GetHandleId(Caster),StringHash("Omnislash Caster Trigger"))

Чем плохо? Нормально.

call SaveUnitHandle(udg_Hash,GetHandleId(t),1,u)

действительно ужасно =(

Spy_ добавил:
и, кстати, может я чего то не понимаю:

        if lvl == 1 then
            return 15.00
        elseif lvl == 2 then
            return 30.00
        elseif lvl == 3 then
            return 45.00
        endif

не проще ли

return 15.*lvl

?

Spy_, ну там могут быть кастом цифры, не связанные напрямую с уровнем

поддержим грамматического нациста выше

незнание, казалось бы, банальных
чтобы понять, к какому блоку
ASCII-графики
догадаться, что эта функция что-то фильтрует, нельзя
приводит их домой и начинает (надо без запятой)
Это неправильно.

вот уж действительно

переменными, используемыми многими функциями, важно
Что это за ключ 1? Юнит-пустышка?
да и вообще, криво это все
Внимание, правильный ответ

ShadoW_DaemoN:

обратите внимание на b == true. Это не правильно. Для надежности надо писать (b == true and b != false)

плакал =)
ShadoW_DaemoN:

Самые продвинутые обнуляют даже boolean.

уже даже не плакал - рыдал! =DDDD

Написано под впечатлением от проверки Spell Contest. пораньше бы Т_Т

не успеют

ошибка номер 0 - Они считают себя программистами.

Они считают себя программистами.

кстати да, это всё равно что web-дизайнеры себя программистами посчитают, т.к. по факту jass - это скрипт

Faion, зачем функции лол? дефайны на что?
алсо, насчет скобочек, я везде так пишу, screenshot.su/show.php?img=939863ae17352e866cf356cdef844637.jpg , т.к. имхо это правильнее, хотя это личное дело каждого.