Рекомендованные сообщения



Не найдено.

2 posts in this topic

Можно ли как-то сделать этот код более нормальным? А то мне кажется что написано как-то странно.

Спойлер

 


    case DIALOG_MASK:
	{
		if(response)
		{
			if(Player[playerid][pMaskuse] == 0 || Player[playerid][pMaskuse] == 1)
			{
				switch(listitem)
				{
				case 0:
					{
						foreach(Player,i)
						{
							if(IsPlayerConnected(i))
							{
								ShowPlayerNameTagForPlayer(i,playerid,0);
							}
						}
						if(Player[playerid][pSex] == 1 || Player[playerid][pSex] == 0) SetPlayerSkin(playerid,29);
						else SetPlayerSkin(playerid,93);
						if(Player[playerid][pMaskuse] == 0)
						{
							f(string,"* %s одевает маскировку.",Player[playerid][pName]);
							ProxDetector(15.0,playerid,string,COLOR_PURPLE);
							SetPlayerColor(playerid,0xFFFFFF00);
							Player[playerid][pMaskuse] = 1;
						}
					}
  				case 1:
					{
						foreach(Player,i)
						{
							if(IsPlayerConnected(i))
							{
								ShowPlayerNameTagForPlayer(i,playerid,0);
							}
						}
						if(Player[playerid][pSex] == 1 || Player[playerid][pSex] == 0) SetPlayerSkin(playerid,33);
						else SetPlayerSkin(playerid,93);
						if(Player[playerid][pMaskuse] == 0)
						{
					 		f(string,"* %s одевает маскировку.",Player[playerid][pName]);
					 		ProxDetector(15.0,playerid,string,COLOR_PURPLE);
						  	SetPlayerColor(playerid,0xFFFFFF00);
						   	Player[playerid][pMaskuse] = 1;
						}
					}
				case 2:
					{
						foreach(Player,i)
						{
							if(IsPlayerConnected(i))
							{
								ShowPlayerNameTagForPlayer(i,playerid,0);
							}
						}
						if(Player[playerid][pSex] == 1 || Player[playerid][pSex] == 0) SetPlayerSkin(playerid,101);
						else SetPlayerSkin(playerid,93);
						if(Player[playerid][pMaskuse] == 0)
						{
					 	f(string,"* %s одевает маскировку.",Player[playerid][pName]);
					 	ProxDetector(15.0,playerid,string,COLOR_PURPLE);
				  		SetPlayerColor(playerid,0xFFFFFF00);
			   			Player[playerid][pMaskuse] = 1;
						}
					}
				case 3:
					{
						foreach(Player,i)
						{
							if(IsPlayerConnected(i))
							{
								ShowPlayerNameTagForPlayer(i,playerid,0);
							}
						}
						if(Player[playerid][pSex] == 1 || Player[playerid][pSex] == 0) SetPlayerSkin(playerid,230);
						else SetPlayerSkin(playerid,93);
					 	if(Player[playerid][pMaskuse] == 0)
						{
					 	f(string,"* %s одевает маскировку.",Player[playerid][pName]);
					 	ProxDetector(15.0,playerid,string,COLOR_PURPLE);
					 	SetPlayerColor(playerid,0xFFFFFF00);
					 	Player[playerid][pMaskuse] = 1;
						}
					}
				case 4:
				 	{
						foreach(Player,i)
						{
							if(IsPlayerConnected(i))
							{
								ShowPlayerNameTagForPlayer(i,playerid,0);
							}
						}
						if(Player[playerid][pSex] == 1 || Player[playerid][pSex] == 0) SetPlayerSkin(playerid,93);
						else SetPlayerSkin(playerid,93);
						if(Player[playerid][pMaskuse] == 0)
						{
					 	f(string,"* %s одевает маскировку.",Player[playerid][pName]);
					 	ProxDetector(15.0,playerid,string,COLOR_PURPLE);
					 	SetPlayerColor(playerid,0xFFFFFF00);
					 	Player[playerid][pMaskuse] = 1;
						}
					}
				case 5:
					{
						if(Player[playerid][pMaskuse] == 0)
						{
							switch(Player[playerid][pSex])
							{
							case 1:send(playerid,0xFF0000AA,"[ОШИБКА] {FFFFFF}Ты не одевал маскировку.");
							case 2:send(playerid,0xFF0000AA,"[ОШИБКА] {FFFFFF}Ты не одевала маскировку.");
							}
							return 1;
						}
						foreach(Player,i)
						{
							if(IsPlayerConnected(i))
							{
								ShowPlayerNameTagForPlayer(i,playerid,1);
							}
						}
				 		if(Player[playerid][pMember] == 0) SetPlayerSkin(playerid,Player[playerid][pChar]);
					 	else SetPlayerSkin(playerid,Player[playerid][pChar]);
					 	Player[playerid][pMaskuse] = 0;
						f(string,"* %s снимает маскировку.",Player[playerid][pName]);
						ProxDetector(15.0,playerid,string,COLOR_PURPLE);
						SetPlayerColor(playerid,0xFFFFFF20);
					}
				}
			}
		}
		return 1;
	}

 

 

Edited by odosenok
Причина: занес код под спойлер

Share this post


Link to post
Share on other sites

@kala4ik, можно лишь убрать switch (в приведенных здесь примерах они скорее потеряют в производительности, потому в них нет смысла), поправить табуляцию, создать выход из функции в начале, если response равно нулю, а также убрал внутри foreach проверку на подключение игрока (это не требуется, если перебираете игроков с помощью foreach):

 

Спойлер

case DIALOG_MASK:
{
	if(!response)
		return 1;
	
	if(!(Player[playerid][pMaskuse] == 0 || Player[playerid][pMaskuse] == 1))
		return 1;

	switch(listitem)
	{
		case 0:
		{
			foreach(Player,i)
			{
				ShowPlayerNameTagForPlayer(i,playerid,0);
			}
			
			if(Player[playerid][pSex] == 1 || Player[playerid][pSex] == 0) SetPlayerSkin(playerid,29);
			else SetPlayerSkin(playerid,93);
			
			if(Player[playerid][pMaskuse] == 0)
			{
				f(string,"* %s одевает маскировку.",Player[playerid][pName]);
				ProxDetector(15.0,playerid,string,COLOR_PURPLE);
				SetPlayerColor(playerid,0xFFFFFF00);
				Player[playerid][pMaskuse] = 1;
			}
		}	
		case 1:
		{
			foreach(Player,i)
			{
				ShowPlayerNameTagForPlayer(i,playerid,0);
			}
			
			if(Player[playerid][pSex] == 1 || Player[playerid][pSex] == 0) SetPlayerSkin(playerid,33);
			else SetPlayerSkin(playerid,93);
			
			if(Player[playerid][pMaskuse] == 0)
			{
				f(string,"* %s одевает маскировку.",Player[playerid][pName]);
				ProxDetector(15.0,playerid,string,COLOR_PURPLE);
				SetPlayerColor(playerid,0xFFFFFF00);
				Player[playerid][pMaskuse] = 1;
			}
		}
		case 2:
		{
			foreach(Player,i)
			{
				ShowPlayerNameTagForPlayer(i,playerid,0);
			}
			
			if(Player[playerid][pSex] == 1 || Player[playerid][pSex] == 0) SetPlayerSkin(playerid,101);
			else SetPlayerSkin(playerid,93);
			
			if(Player[playerid][pMaskuse] == 0)
			{
				f(string,"* %s одевает маскировку.",Player[playerid][pName]);
				ProxDetector(15.0,playerid,string,COLOR_PURPLE);
				SetPlayerColor(playerid,0xFFFFFF00);
				Player[playerid][pMaskuse] = 1;
			}
		}
		case 3:
		{
			foreach(Player,i)
			{
				ShowPlayerNameTagForPlayer(i,playerid,0);
			}
			
			if(Player[playerid][pSex] == 1 || Player[playerid][pSex] == 0) SetPlayerSkin(playerid,230);
			else SetPlayerSkin(playerid,93);
			
			if(Player[playerid][pMaskuse] == 0)
			{
				f(string,"* %s одевает маскировку.",Player[playerid][pName]);
				ProxDetector(15.0,playerid,string,COLOR_PURPLE);
				SetPlayerColor(playerid,0xFFFFFF00);
				Player[playerid][pMaskuse] = 1;
			}
		}
		case 4:
		{
			foreach(Player,i)
			{
				ShowPlayerNameTagForPlayer(i,playerid,0);
			}
			
			if(Player[playerid][pSex] == 1 || Player[playerid][pSex] == 0) SetPlayerSkin(playerid,93);
			else SetPlayerSkin(playerid,93);
			
			if(Player[playerid][pMaskuse] == 0)
			{
				f(string,"* %s одевает маскировку.",Player[playerid][pName]);
				ProxDetector(15.0,playerid,string,COLOR_PURPLE);
				SetPlayerColor(playerid,0xFFFFFF00);
				Player[playerid][pMaskuse] = 1;
			}
		}
		case 5:
		{
			if(Player[playerid][pMaskuse] == 0)
			{
				new sex = Player[playerid][pSex];
				
				if(sex == 1)
					send(playerid,0xFF0000AA,"[ОШИБКА] {FFFFFF}Ты не одевал маскировку.");
				else
					send(playerid,0xFF0000AA,"[ОШИБКА] {FFFFFF}Ты не одевала маскировку.");

				return 1;
			}
			
			foreach(Player,i)
			{
				ShowPlayerNameTagForPlayer(i,playerid,1);
			}
			
			if(Player[playerid][pMember] == 0) SetPlayerSkin(playerid,Player[playerid][pChar]);
			else SetPlayerSkin(playerid,Player[playerid][pChar]);
			
			Player[playerid][pMaskuse] = 0;
			f(string,"* %s снимает маскировку.",Player[playerid][pName]);
			ProxDetector(15.0,playerid,string,COLOR_PURPLE);
			SetPlayerColor(playerid,0xFFFFFF20);
		}
	}
	return 1;
}

 

 

Если хотите менее раздутый код в OnDialogResponse, воспользуйтесь диалоговыми процессорами.

Share this post


Link to post
Share on other sites
Guest
This topic is now closed to further replies.
Sign in to follow this  
Followers 0

  • Recently Browsing   0 members

    No registered users viewing this page.

  • Similar Content

    • Дима Николаев
      By Дима Николаев
      Нужен человек кто сделает html шаблон для сайта. https://vk.com/9at0z
    • KAIF
      By KAIF
      Здравствуйте, хотел бы продать переделанный игровой мод всем знакомого проекта GTA RolePlay
      Фракции:
      1)Правительство
      2)ГИБДД
      3)Полиция Южного
      4)ФСБ
      5)ЦГБ-А
      6)Автошкола
      7)ВВС
      8)СМИ
      9)ОМОН
      10)опг.Тукаевские
      11)опг.Тяп-Ляп
      12)Чеченская Мафия
      13)Американская Мафия
      14)Якудза
      15)Русская Мафия
      16)Итальянская Мафия
      Добавлено/исправлено:
      1)Добавлен ВЕСЬ ОПТИМИЗИРОВАННЫЙ маппинг GTA ROLEPLAY.(2018-2019г)
      2)Добавлены организации мафий.
      3)Перенесены все чекпоинты как на GTA ROLEPLAY.
      4)Добавил в донат покупку военного билета.
      5)Изменил название рангов во фракциях как на GTA ROLEPLAY/
      6)Изменен логотип GTA ROLEPLAY
      7)Переделан весь GPS, изменены все координаты.
      В моде - 6 уровней администрации.(если хотите можете увелить кол-во админок)
      Имеется фуллдоступ, выдаётся в pawno.
      Так же в комплекте будет идти мод-пак к серверу.
      После покупки вы получите архив с модом, а в самом архиве будет текстовый файл с ссылкой на скачивание модпака.
    • KAIF
      By KAIF


      Просмотр файла [CRMP 0.3e] GTA RolePlay - КОПИЯ */СКИДКА/*
      Здравствуйте, хотел бы продать переделанный игровой мод всем знакомого проекта GTA RolePlay
      Фракции:
      1)Правительство
      2)ГИБДД
      3)Полиция Южного
      4)ФСБ
      5)ЦГБ-А
      6)Автошкола
      7)ВВС
      8)СМИ
      9)ОМОН
      10)опг.Тукаевские
      11)опг.Тяп-Ляп
      12)Чеченская Мафия
      13)Американская Мафия
      14)Якудза
      15)Русская Мафия
      16)Итальянская Мафия
      Добавлено/исправлено:
      1)Добавлен ВЕСЬ ОПТИМИЗИРОВАННЫЙ маппинг GTA ROLEPLAY.(2018-2019г)
      2)Добавлены организации мафий.
      3)Перенесены все чекпоинты как на GTA ROLEPLAY.
      4)Добавил в донат покупку военного билета.
      5)Изменил название рангов во фракциях как на GTA ROLEPLAY/
      6)Изменен логотип GTA ROLEPLAY
      7)Переделан весь GPS, изменены все координаты.
      В моде - 6 уровней администрации.(если хотите можете увелить кол-во админок)
      Имеется фуллдоступ, выдаётся в pawno.
      Так же в комплекте будет идти мод-пак к серверу.
      После покупки вы получите архив с модом, а в самом архиве будет текстовый файл с ссылкой на скачивание модпака.
      Добавил KAIF Добавлено 27.12.2020 Категория Моды Автор Kytip  
    • glvde.
      By glvde.
      Хранение данных (версия MySQL): R41-4.
      Командный процессор: Pawn.CMD.
      Количество строк в моде (без маппинга): 44006.
      Анти-чит: Nexius-AC.
      Автор данного мода: -.

      Скриншоты: 
       
       
       
       
       

      На данный момент, мод продаётся за 3699₽ на соседнем борде Pawno-Info (не реклама, скрины взял из темы с продажей).

      Не исключаю того, что в моде имеются баги.
      По поводу дальнейшей доработки узнать можно будет в моей группе - vk.com/way_dev
      Так же есть доработанная версия данного мода, за покупкой обращаться в ВК - vk.com/way.developer