SA-MP Forums Archive
stock - 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: stock (/showthread.php?tid=632864)


stock - Man43 - 20.04.2017

Hi again, what's going on?...

There's no errors/warrings but after i connecting to server i got Random Colors... i think its not work..
PHP код:
stock SetPlayerColors(playerid, level)
{
  if(GetPlayerWantedLevel(playerid) >= 0)
  {
     SetPlayerColor(playerid, COLOR_WHITE);
  }
  else if(GetPlayerWantedLevel(playerid) >= 1 && GetPlayerWantedLevel(playerid) <= 3)
  {
     SetPlayerColor(playerid, COLOR_YELLOW);
  }
  else if(GetPlayerWantedLevel(playerid) >= 4 && GetPlayerWantedLevel(playerid) <= 10)
  {
     SetPlayerColor(playerid, COLOR_ORANGE);
  }
  else if(GetPlayerWantedLevel(playerid) >= 11 && GetPlayerWantedLevel(playerid) <= 50)
  {
     SetPlayerColor(playerid, COLOR_DARKORANGE);
  }
  return 1;
} 


Re: stock - AndreiWow - 20.04.2017

As I know it should end with an else, not sure.

Re: stock - Vince - 20.04.2017

You mean "function". There's no such thing as "a stock". It's not random, it's just flawed logic. Unless you have negative levels the color will always be white. Consider using a switch.

PHP код:
switch(GetPlayerWantedLevel(playerid))
{
    case 0: SetPlayerColor(playerid, COLOR_WHITE);
    case 1 .. 3: SetPlayerColor(playerid, COLOR_YELLOW);
    case 4 .. 10: SetPlayerColor(playerid, COLOR_ORANGE)
    default: SetPlayerColor(playerid, COLOR_DARKORANGE);
} 


Re: stock - Man43 - 20.04.2017

Quote:
Originally Posted by Vince
Посмотреть сообщение
You mean "function". There's no such thing as "a stock". It's not random, it's just flawed logic. Unless you have negative levels the color will always be white. Consider using a switch.

PHP код:
switch(GetPlayerWantedLevel(playerid))
{
    case 0: SetPlayerColor(playerid, COLOR_WHITE);
    case 1 .. 3: SetPlayerColor(playerid, COLOR_YELLOW);
    case 4 .. 10: SetPlayerColor(playerid, COLOR_ORANGE)
    default: SetPlayerColor(playerid, COLOR_DARKORANGE);
} 
Hello vince, i make it like this

and still not work...
PHP код:
stock SetPlayerColors(playerid, level)
{
  switch(GetPlayerWantedLevel(playerid))
{
    case 0: SetPlayerColor(playerid, COLOR_WHITE);
    case 1 .. 3: SetPlayerColor(playerid, COLOR_YELLOW);
    case 4 .. 10: SetPlayerColor(playerid, COLOR_ORANGE)
    default: SetPlayerColor(playerid, COLOR_DARKORANGE);
}
  return 1;
} 


Re: stock - AndreiWow - 20.04.2017

Quote:
Originally Posted by Man43
Посмотреть сообщение
Hello vince, i make it like this

and still not work...
PHP код:
stock SetPlayerColors(playerid, level)
{
  switch(GetPlayerWantedLevel(playerid))
{
    case 0: SetPlayerColor(playerid, COLOR_WHITE);
    case 1 .. 3: SetPlayerColor(playerid, COLOR_YELLOW);
    case 4 .. 10: SetPlayerColor(playerid, COLOR_ORANGE)
    default: SetPlayerColor(playerid, COLOR_DARKORANGE);
}
  return 1;
} 
Remove 'stock', also what are you trying to achieve? Do you want the color to update regarding his wanted level?

Re: stock - Man43 - 20.04.2017

its should be like this?

Код:
SetPlayerColors(playerid, level)
{
  switch(GetPlayerWantedLevel(playerid))
{
    case 0: SetPlayerColor(playerid, COLOR_WHITE);
    case 1 .. 3: SetPlayerColor(playerid, COLOR_YELLOW);
    case 4 .. 10: SetPlayerColor(playerid, COLOR_ORANGE)
    default: SetPlayerColor(playerid, COLOR_DARKORANGE);
}
  return 1;
}


Re: stock - AndreiWow - 20.04.2017

Yea, and call it somewhere, for example when the wanted level updates use

SetPlayerColors(playerid, level);

What is that level actually? I don't see it being used?

Re: stock - Man43 - 20.04.2017

i got warring

Код:
warning 203: symbol is never used: "SetPlayerColors"


Re: stock - Bolex_ - 20.04.2017

SetPlayerColor

Re: stock - DRIFT_HUNTER - 20.04.2017

Quote:
Originally Posted by Man43
Посмотреть сообщение
its should be like this?

Код:
SetPlayerColors(playerid, level)
{
  switch(GetPlayerWantedLevel(playerid))
{
    case 0: SetPlayerColor(playerid, COLOR_WHITE);
    case 1 .. 3: SetPlayerColor(playerid, COLOR_YELLOW);
    case 4 .. 10: SetPlayerColor(playerid, COLOR_ORANGE)
    default: SetPlayerColor(playerid, COLOR_DARKORANGE);
}
  return 1;
}
You have level as argument but you dont use it, instead you use GetPlayerWantedLevel. My guess is that you dont set that wanted level in first place. Anyway try replacing GetPlayerWantedLevel(playerid) with level