вопрос по валидности и оптимизации кода

Danyas

Danyas

Участник
Сообщения
2,173
Реакции
1,072
PHP: 
#include <sdktools>
public Plugin:myinfo = {name    = "WeaponSpawn",author    = "Danyas"};
new g_SteamSprite;
public OnPluginStart()
{	decl String:map[16];GetCurrentMap(map,16);
	if (StrEqual(map, "35hp_2_remake"))	{HookEvent("round_start", round_start);}
	else {SetFailState("Invalid Map. Unloading."); decl String:fn[256]; GetPluginFilename(INVALID_HANDLE, fn, sizeof(fn)); ServerCommand("sm plugins unload %s", fn);}
}

public OnMapStart()
{
	g_SteamSprite = PrecacheModel("sprites/steam1.vmt");
}

public round_start(Handle:event, const String:name[], bool:dontBroadcast)
{
new weapon,randomint = GetRandomInt(0,6);
if (randomint == 0) weapon = CreateEntityByName("weapon_ak47");
else if (randomint == 1) weapon = CreateEntityByName("weapon_m4a1");
else if (randomint == 2) weapon = CreateEntityByName("weapon_deagle");
else if (randomint == 3) weapon = CreateEntityByName("weapon_awp");
else if (randomint == 4) weapon = CreateEntityByName("weapon_scout");
else if (randomint == 5) weapon = CreateEntityByName("weapon_m249");
else weapon = CreateEntityByName("weapon_m3");
new Float:pos[3]; pos[0] = -300.86; pos[1] = 545.63; pos[2] = 276.22;
TeleportEntity(weapon, pos, NULL_VECTOR, NULL_VECTOR);
DispatchSpawn(weapon);SetEntityMoveType(weapon, MOVETYPE_NONE);
SetEntityRenderColor(weapon, GetRandomInt(0,250), GetRandomInt(0,250), GetRandomInt(0,250), GetRandomInt(128,250));
SetEntityRenderFx(weapon, RENDERFX_DISTORT);
CreateTimer(GetRandomFloat(0.1, 10.0), weaponfx, _, TIMER_REPEAT);
}
public Action:weaponfx (Handle:timer)
{
new Float:pos[3]; pos[0] = -300.86; pos[1] = 545.63; pos[2] = GetRandomFloat(270.0,300.0);
TE_SetupSparks(pos, pos, GetRandomInt(10,100), GetRandomInt(100,1000))
TE_SetupSmoke(pos, g_SteamSprite, GetRandomFloat(3.0,7.0), GetRandomInt(5,15));
TE_SetupDust(pos,pos,GetRandomFloat(50.0,500.0),GetRandomFloat(1.0,50.0));
TE_SetupEnergySplash(pos, pos, true);
TE_SendToAll();
}

Интересует больше всего часть с рандомным оружием
 
Саша Шеин

Саша Шеин

Кому костылей?
Сообщения
1,697
Реакции
621
Danyas написал(а):
if (randomint == 0) weapon = CreateEntityByName("weapon_ak47");
else if (randomint == 1) weapon = CreateEntityByName("weapon_m4a1");
else if (randomint == 2) weapon = CreateEntityByName("weapon_deagle");
else if (randomint == 3) weapon = CreateEntityByName("weapon_awp");
else if (randomint == 4) weapon = CreateEntityByName("weapon_scout");
else if (randomint == 5) weapon = CreateEntityByName("weapon_m249");
else weapon = CreateEntityByName("weapon_m3");
Нажмите, чтобы раскрыть...
Может лучше через switch?
 
A

artyrka94

Участник
Сообщения
181
Реакции
38
я бы вот это
PHP: 
public round_start(Handle:event, const String:name[], bool:dontBroadcast) 
{ 
new weapon,randomint = GetRandomInt(0,6); 
if (randomint == 0) weapon = CreateEntityByName("weapon_ak47"); 
else if (randomint == 1) weapon = CreateEntityByName("weapon_m4a1"); 
else if (randomint == 2) weapon = CreateEntityByName("weapon_deagle"); 
else if (randomint == 3) weapon = CreateEntityByName("weapon_awp"); 
else if (randomint == 4) weapon = CreateEntityByName("weapon_scout"); 
else if (randomint == 5) weapon = CreateEntityByName("weapon_m249"); 
else weapon = CreateEntityByName("weapon_m3");

заменил на
PHP: 
new String:wid[7][] = {"weapon_ak47","weapon_m4a1","weapon_deagle","weapon_awp","weapon_scout","weapon_m249", "weapon_m3"};
public round_start(Handle:event, const String:name[], bool:dontBroadcast) 
{ 
	new weapon = CreateEntityByName(wid[GetRandomInt(0,6)]);
 
AlmazON

AlmazON

Не путать с самим yand3xmail
Сообщения
5,099
Реакции
2,755
Danyas написал(а):
SetFailState("Invalid Map. Unloading."); decl String:fn[256]; GetPluginFilename(INVALID_HANDLE, fn, sizeof(fn)); ServerCommand("sm plugins unload %s", fn);
Нажмите, чтобы раскрыть...
Бесполезный код. После SetFailState плагин ставится на паузу и весь код ниже не выполняется.
Danyas написал(а):
new Float:pos[3]; pos[0] = -300.86; pos[1] = 545.63; pos[2] = GetRandomFloat(270.0,300.0);
Нажмите, чтобы раскрыть...
Тогда бы сразу:
PHP: 
new Float:pos[3] = {-300.86, 545.63, GetRandomFloat(270.0,300.0)};
Но лучше через decl.
artyrka94 написал(а):
new String:wid[7][] = {"weapon_ak47","weapon_m4a1","weapon_deagle","weapon_awp","weapon_scout","weapon_m249", "weapon_m3"};
Нажмите, чтобы раскрыть...
Я бы немного не так сделал. Слишком много памяти требует. Особенно, если используются почти все виды оружия (и даже хранятся глобально).
 
G

gibs

Фитиль народного волненья
Сообщения
722
Реакции
407
Мошенник
Фейл стейт не ставь, если ты ставишь фейл стейт то плагин автоматом выгружается с ошибкой, и твой "sm plugins unload" все-равно не сработает. Проверку названия карты делай при старте карты, это как-то логичней. Используй свич. Если при объявлении не присваиваешь переменной значение, то используй decl. Оператор decl создает переменную в уже выделенной памяти, а не выделяет её для новой переменной в отличии от оператора new.
А так в принципе делай ударение на читаемость, чтобы всё было красивенько. Делай отступы.
 
Danyas

Danyas

Участник
Сообщения
2,173
Реакции
1,072
PHP: 
#include <sdktools>
public Plugin:myinfo = {name    = "WeaponSpawn",author    = "Danyas"};
new g_SteamSprite;

public OnMapStart()
{
	g_SteamSprite = PrecacheModel("sprites/steam1.vmt");
	decl String:map[16];GetCurrentMap(map,16);
	if (StrEqual(map, "35hp_2_remake"))
	{
		HookEvent("round_start", round_start);
	}
	else
	{
		decl String:fn[16];
		GetPluginFilename(INVALID_HANDLE, fn, sizeof(fn));
		ServerCommand("sm plugins unload %s", fn);
	}

}

public round_start(Handle:event, const String:name[], bool:dontBroadcast)
{
	decl Float:pos[3],weapon; 
	new randomint = GetRandomInt(0,6);
	
	switch (randomint)
	{
		case 0:{weapon = CreateEntityByName("weapon_ak47");}
		case 1:{weapon = CreateEntityByName("weapon_m4a1");}
		case 2:{weapon = CreateEntityByName("weapon_deagle");}
		case 3:{weapon = CreateEntityByName("weapon_awp");}
		case 4:{weapon = CreateEntityByName("weapon_scout");}
		case 5:{weapon = CreateEntityByName("weapon_m249");}
		case 6:{weapon = CreateEntityByName("weapon_m3");}
	}

	pos[0] = -300.86; pos[1] = 545.63; pos[2] = 276.22;

	TeleportEntity(weapon, pos, NULL_VECTOR, NULL_VECTOR);
	DispatchSpawn(weapon);
	SetEntityMoveType(weapon, MOVETYPE_NONE);
	SetEntityRenderColor(weapon, GetRandomInt(0,250), GetRandomInt(0,250), GetRandomInt(0,250), GetRandomInt(128,250));
	SetEntityRenderFx(weapon, RENDERFX_DISTORT);
	CreateTimer(GetRandomFloat(0.1, 10.0), weaponfx, _, TIMER_REPEAT);
}
public Action:weaponfx (Handle:timer)
{
	decl Float:pos[3];
	pos[0] = -300.86; pos[1] = 545.63; pos[2] = GetRandomFloat(270.0,300.0);
	TE_SetupSparks(pos, pos, GetRandomInt(10,100), GetRandomInt(100,1000))
	TE_SetupSmoke(pos, g_SteamSprite, GetRandomFloat(3.0,7.0), GetRandomInt(5,15));
	TE_SetupDust(pos,pos,GetRandomFloat(50.0,500.0),GetRandomFloat(1.0,50.0));
	TE_SetupEnergySplash(pos, pos, true);
	TE_SendToAll();
}
Тогда как-то так.

А как с 45-й строки (таймер) передать pos[3] в таймер?
Можно как-то без data-pack'a сделать?
 
AlmazON

AlmazON

Не путать с самим yand3xmail
Сообщения
5,099
Реакции
2,755
Danyas, как полностью отключить плагин без выдачи ошибки SetFailState, использования лишь старые переменные (полная экономия памяти):
PHP: 
public OnMapStart() 
{ 
	decl String:map[15];GetCurrentMap(map,15);
	if (strcmp(map, "35hp_2_remake"))
	{
		if (g_SteamSprite)
		{
			UnhookEvent("round_start", round_start, EventHookMode_PostNoCopy);
			g_SteamSprite = 0;
		}
	}
	else
	{
		if (g_SteamSprite == 0) HookEvent("round_start", round_start, EventHookMode_PostNoCopy);
		g_SteamSprite = PrecacheModel("sprites/steam1.vmt");
	}
}
Danyas написал(а):
CreateTimer(GetRandomFloat(0.1, 10.0), weaponfx, _, TIMER_REPEAT);
Нажмите, чтобы раскрыть...
Таймер будет бесконечный в данном случае (и с каждой картой - ещё один). Нужно хотя бы добавить флаг TIMER_FLAG_NO_MAPCHANGE.
Danyas написал(а):
как с 45-й строки (таймер) передать pos[3] в таймер?
Можно как-то без data-pack'a сделать?
Нажмите, чтобы раскрыть...
Хранить массив глобально. Для возможно столь частого таймера (0.1 сек в повторении) - это даже оптимальнее.
 
R1KO

R1KO

fuck society
Сообщения
9,457
Реакции
7,786
  • Команда форума
  • #10
насколько я помню юзать нужно будет так:
PHP: 
Float:TIMER_POSITION
или
PHP: 
#define TIMER_POSITION Float:{-300.86, 545.63, 276.22}
 
Danyas

Danyas

Участник
Сообщения
2,173
Реакции
1,072
R1KO написал(а):
насколько я помню юзать нужно будет так:
PHP: 
Float:TIMER_POSITION
или
PHP: 
#define TIMER_POSITION Float:{-300.86, 545.63, 276.22}
Нажмите, чтобы раскрыть...

PHP: 
#define pos Float:{-300.86, 545.63, 276.22}

На данный момент код выглядит так
PHP: 
#include <sdktools>
public Plugin:myinfo = {name    = "WeaponSpawn",author    = "Danyas"};
new g_SteamSprite;
#define pos Float:{-300.86, 545.63, 276.22}  
public OnMapStart()
{
	g_SteamSprite = PrecacheModel("sprites/steam1.vmt");
	decl String:map[16];GetCurrentMap(map,16);
	if (StrEqual(map, "35hp_2_remake"))
	{
		HookEvent("round_start", round_start);
	}
	else
	{
		decl String:fn[16];
		GetPluginFilename(INVALID_HANDLE, fn, sizeof(fn));
		ServerCommand("sm plugins unload %s", fn);
	}

}

public round_start(Handle:event, const String:name[], bool:dontBroadcast)
{
	decl weapon; 
	new randomint = GetRandomInt(0,6);
	
	switch (randomint)
	{
		case 0:{weapon = CreateEntityByName("weapon_ak47");}
		case 1:{weapon = CreateEntityByName("weapon_m4a1");}
		case 2:{weapon = CreateEntityByName("weapon_deagle");}
		case 3:{weapon = CreateEntityByName("weapon_awp");}
		case 4:{weapon = CreateEntityByName("weapon_scout");}
		case 5:{weapon = CreateEntityByName("weapon_m249");}
		case 6:{weapon = CreateEntityByName("weapon_m3");}
	}


	TeleportEntity(weapon, pos, NULL_VECTOR, NULL_VECTOR);
	DispatchSpawn(weapon);
	SetEntityMoveType(weapon, MOVETYPE_NONE);
	SetEntityRenderColor(weapon, GetRandomInt(0,250), GetRandomInt(0,250), GetRandomInt(0,250), GetRandomInt(128,250));
	SetEntityRenderFx(weapon, RENDERFX_DISTORT);
	CreateTimer(GetRandomFloat(0.1, 10.0), weaponfx, _, TIMER_REPEAT|TIMER_FLAG_NO_MAPCHANGE);
}
public Action:weaponfx (Handle:timer)
{
	TE_SetupSparks(pos, pos, GetRandomInt(10,100), GetRandomInt(100,1000))
	TE_SetupSmoke(pos, g_SteamSprite, GetRandomFloat(3.0,7.0), GetRandomInt(5,15));
	TE_SetupDust(pos,pos,GetRandomFloat(50.0,500.0),GetRandomFloat(1.0,50.0));
	TE_SetupEnergySplash(pos, pos, true);
	TE_SendToAll();
}

AlmazON написал(а):
Насколько я помню, компилятор сам определяет тип константы в зависимости от написания. Ты же не ставишь String:, когда пишешь текст в #define, верно? Также и тут. Есть дробная часть - используется Float.
Нажмите, чтобы раскрыть...

Неа, не определил почему-то.
 
R1KO

R1KO

fuck society
Сообщения
9,457
Реакции
7,786
  • Команда форума
  • #12
AlmazON написал(а):
Ты же не ставишь String:, когда пишешь текст в #define, верно? Также и тут. Есть дробная часть - используется Float.
Нажмите, чтобы раскрыть...
А ты проверь))
String: мы не ставим потому что компилятор уже подразумевает что то что в кавычках это строка:
StringToInt("145");
Тут же String: не пишем.
А у массива тип не флоат, хотя внутри него как раз так. Поэтому нужно на сам массив присваивать тип.
 
R1KO

R1KO

fuck society
Сообщения
9,457
Реакции
7,786
  • Команда форума
  • #13
Danyas написал(а):
PHP: 
decl weapon;     new randomint = GetRandomInt(0,6);        switch (randomint)    {        case 0:{weapon = CreateEntityByName("weapon_ak47");}        case 1:{weapon = CreateEntityByName("weapon_m4a1");}        case 2:{weapon = CreateEntityByName("weapon_deagle");}        case 3:{weapon = CreateEntityByName("weapon_awp");}        case 4:{weapon = CreateEntityByName("weapon_scout");}        case 5:{weapon = CreateEntityByName("weapon_m249");}        case 6:{weapon = CreateEntityByName("weapon_m3");}    }
Нажмите, чтобы раскрыть...

artyrka94 написал(а):
я бы вот это
PHP: 
public round_start(Handle:event, const String:name[], bool:dontBroadcast) 
{ 
new weapon,randomint = GetRandomInt(0,6); 
if (randomint == 0) weapon = CreateEntityByName("weapon_ak47"); 
else if (randomint == 1) weapon = CreateEntityByName("weapon_m4a1"); 
else if (randomint == 2) weapon = CreateEntityByName("weapon_deagle"); 
else if (randomint == 3) weapon = CreateEntityByName("weapon_awp"); 
else if (randomint == 4) weapon = CreateEntityByName("weapon_scout"); 
else if (randomint == 5) weapon = CreateEntityByName("weapon_m249"); 
else weapon = CreateEntityByName("weapon_m3");

заменил на
PHP: 
new String:wid[7][] = {"weapon_ak47","weapon_m4a1","weapon_deagle","weapon_awp","weapon_scout","weapon_m249", "weapon_m3"};
public round_start(Handle:event, const String:name[], bool:dontBroadcast) 
{ 
	new weapon = CreateEntityByName(wid[GetRandomInt(0,6)]);
Нажмите, чтобы раскрыть...

так сделай
 
R1KO

R1KO

fuck society
Сообщения
9,457
Реакции
7,786
  • Команда форума
  • #15
AlmazON, но при компиляции выдает несоответстиве тегов
 
AlmazON

AlmazON

Не путать с самим yand3xmail
Сообщения
5,099
Реакции
2,755
Danyas написал(а):
специально в свич сделал, как посоветовали
Нажмите, чтобы раскрыть...
Слишком много повторения кода, его нативов - это тоже плохо. Если жалко глобальную память, сделай хотя бы в decl:
PHP: 
decl String:wid[][] = {"weapon_ak47","weapon_m4a1","weapon_deagle", "weapon_awp","weapon_scout","weapon_m249","weapon_m3"};
Оффтоп
 
Danyas

Danyas

Участник
Сообщения
2,173
Реакции
1,072
AlmazON написал(а):
Слишком много повторения кода, его нативов - это тоже плохо. Если жалко глобальную память, сделай хотя бы в decl:
PHP: 
decl String:wid[][] = {"weapon_ak47","weapon_m4a1","weapon_deagle", "weapon_awp","weapon_scout","weapon_m249","weapon_m3"};
Оффтоп
Нажмите, чтобы раскрыть...

Сделал глобально.

Оффтоп
 
AlmazON

AlmazON

Не путать с самим yand3xmail
Сообщения
5,099
Реакции
2,755
Danyas написал(а):
ее же сразу нельзя присвоить, не так ли?
Нажмите, чтобы раскрыть...
Можно в массивах. Если значение явное, известно заранее (как бы константа):
PHP: 
decl i[2] = {0, 4}; //ВЕРНО.
Присваивать переменную - нельзя:
PHP: 
new a, b;
decl i[2] = {a, b}; //НЕПРАВИЛЬНО.
 
Последнее редактирование:
Войдите или зарегистрируйтесь для ответа.

Похожие темы

B
Простите за спам.. вопрос удален (был не внимателен в написании кода)
Ответы
0
Просмотры
202
Banana
B
Mr_panica
  • Вопрос
Pterodactyl mount для сервера - вопрос и реализация
Ответы
1
Просмотры
287
Mr_panica
Mr_panica
Mr_panica
Какие способы по отлову мульти (смурф) аккаунтов существуют?
Ответы
3
Просмотры
367
DeathScore13
DeathScore13
Nekro
Дискриминация и позиция администрации 2
2
Ответы
23
Просмотры
2К
Vanooo123
Vanooo123
Nekro
  • Закрыта
Дискриминация и позиция администрации
Ответы
1
Просмотры
266
Virus
Virus
Сверху Снизу