Half-Life и Adrenaline Gamer форум

Вопросы касаются твоей оптимизации и оптимизации в общем. 1й - почему ты переправил "static" на "new" в обозначениях "trail" (если не ошибаюсь для оптимизации кода, для переменных имеющих по сути не изменяемое значение, лучше использовать "static"). 2й какой смысл "return PLUGIN_CONTINUE" ? "return HAM_IGNORED" ? если все работало норм )). 3й Зачем обьявлять cvars через new ? если define пашет норм ( меньше переменных в памяти ) 4й тут есть замечание по поводу работы плагина - на стрелу арбалета действует rendering хотя не должен ( нет в этом смысла стрела летит быстро и этого практически не заметно, за стрелой нету следа а так же нет следа за гранатами ) ... А насчет функции - set_rendering 5 баллов и еще насчет конечного размера: 4.88кб мой, 5.13кб твой )

1.

static satchel_trail, grenade_trail, snark_trail, bolt_trail

new g_satchel_trail, g_grenade_trail, g_snark_trail, g_bolt_trail

Абсолютно ничем не различаются, т.к. объявление происходит на уровне файла (не в функции). Поэтому лучше использовать обычное "new".

Q: Что такое "static"?
A: Это глобальная переменная (т.е. время её жизни равняется времени жизни плагина), но имеющая область видимости равную блоку в котором она объявлена.
Примеры:

new g_i1;   // Глобальная переменная: время жизни = время жизни плагина; область видимости = весь файл.
static g_i2; // Глобальная переменная: время жизни = время жизни плагина; область видимости = весь файл.

Function()
{
   new i1; // Локальная переменная:
              // время жизни = время исполнения функции (создается каждый раз при входе в функцию);
              // область видимости = функция Function.
   static i2; // Локальная переменная:
                // время жизни = время жизни плагина (создается один раз раз при старте плагина);
                // область видимости = функция Function.
}

Т.е. разница, в том, что static создается один раз, а new каждый раз при входе в блок. И надо выбирать по критерию: иcпользуемая память / нагрузка на процессор от частоты создания. Если переменная часто используется (функция часто вызывается), то имеет смысл объявить её static.
Второе различие, вытекающее из разового создания: значение переменной сохраняется между вызовами функции. Т.е. по сути получается глобальная переменная с локальной областью видимости.
Имеет смысл изменить объявление

      new classname[32], sprite, lifetime, skip

на

      static classname[32], sprite, lifetime, skip

так как функция обработчик spawn будет использоваться часто.

2.
У каждой функции есть значение возврата по умолчанию - 0. Т.е. если не написать

return <value>;

или написать просто

return;

, то функция вернет ноль.
Если посмотреть инклюдники, то увидим:

#define PLUGIN_CONTINUE      0   /* Results returned by public functions */

Т.е. возврат равнозначен возвращению 0, использованию return без значения, или просто опусканию оператора return.

/**
 * Ham return types.
 * -
 * Return these from hooks to disable calling the target function.
 * Numbers match up with fakemeta's FMRES_* for clarity.  They are interchangable.
 * 0 (or no return) is also interpretted as HAM_IGNORED.
 */
#define HAM_IGNORED      1   /**< Calls target function, returns normal value */

Тоже самое и в случае Ham Sandwich. В его случае и 0 и 1 равнозначны. Т.е. их можно не писать.

public plugin_init() 
{
...
   return PLUGIN_CONTINUE   
}

Тут Корд развоевался просто.

Q: Бывают ли случаи когда надо в конце функции писать "return PLUGIN_CONTINUE"?
A: Ага:

public Function(id)
{
   if (id == 1)
      return PLUGIN_HANDLED;
   client_print(id, print_chat, "Your Id = %i", id);
   return PLUGIN_CONTINUE;
}

Если не написать в конце return со значением, то АМХХ не даст скомпилять плагин.

3.

#define trails_cvar      get_cvar_num("amx_HL_trails")
   register_cvar("amx_HL_trails", "1")
   if(trails_cvar) {trails(ent);}

В данном случае использовался доступ с получением адреса квара по названию квара (операции со списками строк в движке).

new g_cvarTrails;
   g_cvarTrails = register_cvar("amx_HL_trails", "1")
   if(get_pcvar_num(g_cvarTrails))

В этом случае, при регистрации квара, получается адрес значения квара в памяти, и, в последуещем, неспоредственное получение значения по адресу - в разы быстрее. Почитай про pcvars.

4.
Не понял что там работает и что нет. По коду, вроде бы, нет разницы между твоей и версией Корда.

Насчет размера плагина: лучше сохранять легко читаемое форматирование, чем экономить символы. Я ведь могу этот плагин ужать в 1 кб, но читать его будет, мягко говоря, нелегко. хД

PS А, и забыл: я же не Корд, но, вобщем, удалять уж не буду...

По 1, 2 и 3 согласен во всем со Львом

4. My bad
Класснейм у стрелы арбалета не тот.

equali(classname, "crossbow_bolt") 

--->

equali(classname, "bolt")

И еще нужен post хук спауна.

RegisterHam(Ham_Spawn, g_classes[i], "fw_EntSpawn")

--->

RegisterHam(Ham_Spawn, g_classes[i], "fw_EntSpawn", 1)

_________________

Vi Veri Veniversum Vivus Vici

Огромное спасибо за развёрнутый ответ , просто горы нереально полезной информации. Меня сильно заинтересовали твои слова Lev:
Если появится возможность продемонстрировать - буду очень рад .
KORD_12.7 - теперь всё работает норм ))

Используй тыкание на имя пользователя над аватаром - вставляет ник пользователя в текст сообщения в более привычном виде: Turanga_Leela

Никогда так не делайте.

Не будем

_________________

Vi Veri Veniversum Vivus Vici

Про сжатие в 1кб я думал речь идет о amxx, но все же зачет