SA-MP Forums Archive
Variables 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: Variables Help! (/showthread.php?tid=312777)


Variables Help! - Nuke547 - 22.01.2012

So I've been working on my script, but the problem is, the variables are applying for everybody. I want everybody to go by the same variable, but each individual has there own value. What it does right now, is if somebody does /radiooff, it'll turn the radio variable to 0 for everybody.

All I'm doing is putting the variable at the top of the script like this:

PHP код:
new radio = 1 
Then, the script for it is:

PHP код:
    if (strcmp("/radiooff", cmdtext, true, 10) == 0)
    {
        if(radio == 1)
        {
            StopAudioStreamForPlayer(playerid);
            SendClientMessage(playerid, white, "To turn the radio back on, do /radioon");
              radio--;
        }
        else if(radio == 0)
        {
            SendClientMessage(playerid, white, "Your radio is already off!");
        }
    } 
Then, the radio on is:
PHP код:
    if (strcmp("/radioon", cmdtext, true, 10) == 0)
    {
        if(radio == 0)
        {
            PlayAudioStreamForPlayer(playerid, "http://yp.shoutcast.com/sbin/tunein-station.pls?id=1283896");
            SendClientMessage(playerid, white, "Your radio has been turned on. To turn it off, type /radiooff");
            radio++;
        }
        else if(radio == 1)
        {
            SendClientMessage(playerid, white, "Your radio is already on, to turn it off, do /radiooff!");
        }
    } 
The radio-- and radio++ are taking affect for everybody's variables.
Help Please!

Re: Variables Help! - HuSs3n - 22.01.2012

try radio=0 & radio = 1, instead of -- & ++

edit :

Код:
    new radio[MAX_PLAYERS];
    if (strcmp("/radiooff", cmdtext, true, 10) == 0) 
    { 
        if(radio[playerid] == 1) 
        { 
            StopAudioStreamForPlayer(playerid); 

            SendClientMessage(playerid, white, "To turn the radio back on, do /radioon"); 
              radio[playerid]=0;
        } 
        else if(radio[playerid] == 0) 
        { 
            SendClientMessage(playerid, white, "Your radio is already off!"); 
        } 
    }  

   if (strcmp("/radioon", cmdtext, true, 10) == 0) 
    { 
        if(radio[playerid] == 0) 
        { 
            PlayAudioStreamForPlayer(playerid, "http://yp.shoutcast.com/sbin/tunein-station.pls?id=1283896"); 
            SendClientMessage(playerid, white, "Your radio has been turned on. To turn it off, type /radiooff"); 
            radio[playerid]=1; 
        } 
        else if(radio[playerid] == 1) 
        { 
            SendClientMessage(playerid, white, "Your radio is already on, to turn it off, do /radiooff!"); 
        } 
    }


Re: Variables Help! - Steven82 - 22.01.2012

You need to do this. I'll rewrite the code for you.

pawn Код:
new radio[MAX_PLAYERS] = 0;

if (strcmp("/radiooff", cmdtext, true, 10) == 0) 
    { 
        if(radio[playerid] == 1) 
        { 
            StopAudioStreamForPlayer(playerid); 

            SendClientMessage(playerid, white, "To turn the radio back on, do /radioon"); 
              radio[playerid] = 0; 
        } 
        else if(radio[playerid] == 0) 
        { 
            SendClientMessage(playerid, white, "Your radio is already off!"); 
        } 
    } 

if (strcmp("/radioon", cmdtext, true, 10) == 0) 
    { 
        if(radio[playerid] == 0) 
        { 
            PlayAudioStreamForPlayer(playerid, "http://yp.shoutcast.com/sbin/tunein-station.pls?id=1283896"); 
            SendClientMessage(playerid, white, "Your radio has been turned on. To turn it off, type /radiooff"); 
            radio[playerid] = 1; 
        } 
        else if(radio[playerid] == 1) 
        { 
            SendClientMessage(playerid, white, "Your radio is already on, to turn it off, do /radiooff!"); 
        } 
    }
You were making a global variable. That was your issue. Try using my code and tell me if it works or not.

Re: Variables Help! - Bogdan1992 - 22.01.2012

With one cmd you can turn it on or off. Much simple
PHP код:
if (strcmp("/radio", cmdtext, true, 7) == 0) 
{
    if(radio[playerid] == 0)
    {
        PlayAudioStreamForPlayer(playerid, "http://yp.shoutcast.com/sbin/tunein-station.pls?id=1283896");
        SendClientMessage(playerid, white, "You are listening to radio ....");
        radio[playerid] = 1;
    }
    else
    {
        StopAudioStreamForPlayer(playerid);
        SendClientMessage(playerid, white, "Thank you for listeing the radio, tune again soon.");
        radio[playerid] = 0;
    }
    return 1;
} 


Re: Variables Help! - Nuke547 - 22.01.2012

Quote:
Originally Posted by Steven82
Посмотреть сообщение
You need to do this. I'll rewrite the code for you.

pawn Код:
new radio[MAX_PLAYERS] = 0;

if (strcmp("/radiooff", cmdtext, true, 10) == 0) 
    { 
        if(radio[playerid] == 1) 
        { 
            StopAudioStreamForPlayer(playerid); 

            SendClientMessage(playerid, white, "To turn the radio back on, do /radioon"); 
              radio[playerid] = 0; 
        } 
        else if(radio[playerid] == 0) 
        { 
            SendClientMessage(playerid, white, "Your radio is already off!"); 
        } 
    } 

if (strcmp("/radioon", cmdtext, true, 10) == 0) 
    { 
        if(radio[playerid] == 0) 
        { 
            PlayAudioStreamForPlayer(playerid, "http://yp.shoutcast.com/sbin/tunein-station.pls?id=1283896"); 
            SendClientMessage(playerid, white, "Your radio has been turned on. To turn it off, type /radiooff"); 
            radio[playerid] = 1; 
        } 
        else if(radio[playerid] == 1) 
        { 
            SendClientMessage(playerid, white, "Your radio is already on, to turn it off, do /radiooff!"); 
        } 
    }
You were making a global variable. That was your issue. Try using my code and tell me if it works or not.
Thanks alot! It worked!