SA-MP Forums Archive
Help - Printable Version

+- SA-MP Forums Archive (https://sampforum.blast.hk)
+-- Forum: SA-MP Scripting and Plugins (https://sampforum.blast.hk/forumdisplay.php?fid=8)
+--- Forum: Scripting Help (https://sampforum.blast.hk/forumdisplay.php?fid=12)
+--- Thread: Help (/showthread.php?tid=631918)


Help - astanalol - 06.04.2017

Hi Guys
i have small problems my Code no effect in my Server
PHP код:
public OnPlayerText(playerid, text[])
{
    if(text[0] == '1')
    {
        if(FirstWeaponText[playerid] == 1)
        {
           new Amount;
           if( sscanf ( text[1], "i",Amount))  return SendClientMessage(playerid,-1, "[amount]");// amount he wants.
           if(Amount < 1 || Amount > 10000) return SendClientMessage(playerid,-1,"You can only buy from 1 - 10000 bullets");// check if player has writen more than required
           {
               GivePlayerMoney(playerid,-Amount*100); // this makes 1 bullet for 100$
               GivePlayerWeapon(playerid, WEAPON_M4, Amount);
           }
        }
    } 
i tired to Type 1 in chat after that's i Type Amount Ammo i needed not working why ?

Re: Help - khRamin78 - 06.04.2017

as i know you should remov brackers

PHP код:
 if(Amount < 1 || Amount > 10000) return SendClientMessage(playerid,-1,"You can only buy from 1 - 10000 bullets");// check if player has writen more than required 
           { // This one 
               GivePlayerMoney(playerid,-Amount*100); // this makes 1 bullet for 100$ 
               GivePlayerWeapon(playerid, WEAPON_M4, Amount); 
           } // and this one 
i realy dont know why u put them there :S cus of that last if statement u used a return value

Sorry for my bad eng

Re: Help - astanalol - 06.04.2017

[QUOTE=khRamin78;3864803]as i know you should remov brackers

PHP код:
public OnPlayerText(playerid, text[])
{
    if(text[0] == '1')
    {
        if(FirstWeaponText[playerid] == 1)
        {
           new Amount;
           if( sscanf ( text[1], "i",Amount))  return SendClientMessage(playerid,-1, "[amount]");// amount he wants.
           { // This one
               GivePlayerMoney(playerid,-Amount*100); // this makes 1 bullet for 100$
               GivePlayerWeapon(playerid, WEAPON_M4, Amount);
           } // and this one
         }
    } 
its working only when i write in chat 1 100
give me 100 ammo
i want making it when i'm write 1 in chat after that's write my amount ammo in chat

Re: Help - astanalol - 07.04.2017

[QUOTE=astanalol;3864817]
Quote:
Originally Posted by khRamin78
Посмотреть сообщение
as i know you should remov brackers

PHP код:
public OnPlayerText(playerid, text[])
{
    if(text[0] == '1')
    {
        if(FirstWeaponText[playerid] == 1)
        {
           new Amount;
           if( sscanf ( text[1], "i",Amount))  return SendClientMessage(playerid,-1, "[amount]");// amount he wants.
           { // This one
               GivePlayerMoney(playerid,-Amount*100); // this makes 1 bullet for 100$
               GivePlayerWeapon(playerid, WEAPON_M4, Amount);
           } // and this one
         }
    } 
its working only when i write in chat 1 100
give me 100 ammo
i want making it when i'm write 1 in chat after that's write my amount ammo in chat
Bump!!

Re: Help - XpDeviL - 07.04.2017

I think you want something like that:

PHP код:
public OnPlayerText(playerid, text[]) 
{  
    if(FirstWeaponText[playerid] == 1) 
    { 
        if(!isNumeric(text))
        {
            SendClientMessage(playerid,-1, "[Enter only digits]");// amount he wants.
            return 0;
        }
         
        if(strval(text) < 1 || strval(text) > 10000)
        {
            SendClientMessage(playerid,-1,"You can only buy from 1 - 10000 bullets");// check if player has writen more than required 
            return 0;
        }
        GivePlayerMoney(playerid,-strval(text)*100); // this makes 1 bullet for 100$ 
        GivePlayerWeapon(playerid, WEAPON_M4, strval(text)); 
        FirstWeaponText[playerid] = 0;
        return 0;
    }
    else
    {
        if(!strcmp(text, "1")) 
        {
            FirstWeaponText[playerid] = 1;
            SendClientMessage(playerid, -1, "Enter the amount of ammo:");
            return 0;
        }
    }
    return 1;
} 


Re: Help - astanalol - 07.04.2017

Quote:
Originally Posted by XpDeviL
Посмотреть сообщение
I think you want something like that:

PHP код:
public OnPlayerText(playerid, text[]) 
{  
    if(FirstWeaponText[playerid] == 1) 
    { 
        if(!isNumeric(text))
        {
            SendClientMessage(playerid,-1, "[Enter only digits]");// amount he wants.
            return 0;
        }
         
        if(strval(text) < 1 || strval(text) > 10000)
        {
            SendClientMessage(playerid,-1,"You can only buy from 1 - 10000 bullets");// check if player has writen more than required 
            return 0;
        }
        GivePlayerMoney(playerid,-strval(text)*100); // this makes 1 bullet for 100$ 
        GivePlayerWeapon(playerid, WEAPON_M4, strval(text)); 
        FirstWeaponText[playerid] = 0;
        return 0;
    }
    else
    {
        if(!strcmp(text, "1")) 
        {
            FirstWeaponText[playerid] = 1;
            SendClientMessage(playerid, -1, "Enter the amount of ammo:");
            return 0;
        }
    }
    return 1;
} 
error
Quote:

(389) : error 017: undefined symbol "isNumeric"
PHP код:
if(!isNumeric(text)) 


Re: Help - XpDeviL - 07.04.2017

PHP код:
stock isNumeric(const string[])
{
 new length=strlen(string);
 if (length==0) return false;
 for (new i = 0; i < length; i++)
  {
   if (
      (string[i] > '9' || string[i] < '0' && string[i]!='-' && string[i]!='+') // Not a number,'+' or '-'
       || (string[i]=='-' && i!=0)                       // A '-' but not at first.
       || (string[i]=='+' && i!=0)                       // A '+' but not at first.
     ) return false;
  }
 if (length==1 && (string[0]=='-' || string[0]=='+')) return false;
 return true;
} 


Re: Help - astanalol - 07.04.2017

Quote:
Originally Posted by XpDeviL
Посмотреть сообщение
PHP код:
stock isNumeric(const string[])
{
 new length=strlen(string);
 if (length==0) return false;
 for (new i = 0; i < length; i++)
  {
   if (
      (string[i] > '9' || string[i] < '0' && string[i]!='-' && string[i]!='+') // Not a number,'+' or '-'
       || (string[i]=='-' && i!=0)                       // A '-' but not at first.
       || (string[i]=='+' && i!=0)                       // A '+' but not at first.
     ) return false;
  }
 if (length==1 && (string[0]=='-' || string[0]=='+')) return false;
 return true;
} 
fucking awesome thanks man working prefct +rep