Making a variable that it changes for the player only and not for the script

[Samuzero15tlh]

What if i want to make a variable that is changed by the player and it don't change the value in the script? I mean, In the script keeps with a same value but in the player, it changes.

Oh... Net Clientside

[SwordGrunt]

You can make an array and have each player use one value (usually equal to the number of that player)

What are you trying to do? Why would you want a variable to be read differently by the player and the server? I can only imagine you want something that can be done with an array, but perhaps that's not what you're looking for.

In case you don't know, this is what it'd look like

Code: Select all

int myVar[8]; // supports up to 8 players. Raise it up to whatever amount you deem reasonable for your mod.
[...]

myVar[PlayerNumber()] = CheckInventory("Shell"); // stores the amount of shells the player has

[...]

Print(s:"You have \cd ",i:myVar[PlayerNumber()],s:" \cfshells left!"); // prints to the player activating the script the amount of shells he has

Edit: I wouldn't recommend using net clientside scripts that will affect ANYTHING other than the player activating them, OR that have any interaction at all with anything in a server script. This includes modifying variables set by a server script. Try arrays and see if it helps you out, otherwise I'm sure there's another way to get what you want done without the use of clientside scripts.

[Samuzero15tlh]

Net because it's executed with a bind puke (always), what im trying to do is a cyclic menu, but it needs a variable that it changes by the player only and not the server. Also im using the custom inventory.

I'd just noticed that Clientside sucks and its broken.

[ibm5155]

Clientside is doesn't sucks, it's just clientside, and it's not broken.
Think of it as a separated server that only the cliente can access the data.
So just take a look at this example:

Code: Select all

int i=0;

script 1 open{
	i = 30;
	print(d:i);
}

script 2 open clientside{
	delay(35);
	print(d:i);
}

the print inside script 1 will show 30, and the print inside script 2 will show zero, why? because when you set the value 30 to i, you're changing the server i value, so, the clientside i is not going to be changed, the only way the server can change it is calling a clientside script, like this one

Code: Select all

int i=0;

script 1 open{
	i = 30;
	acs_execute(3,0,30);
	print(d:i);
}

script 2 open clientside{
	delay(35);
	print(d:i);
}

script 3 (int a) clientside{
	i = a;
}

that way, both prints will show 30...
Clientside is only used for scripts that MUST not change the gameplay, just the graphics.

EDIT:
there's also a way to make a clientside script call a serverside script, so the cliente can pass a value to the serve (like the cliente pass 30 to server store in i, and then server prints 30), but, idk the specific acs call for this case

EDIT2: also, when you call an clientside script, all players will start the script, so you must check if the activator has the same tid as the consoleplayernumber, if not, call terminate(); (this way, only the player who called the script will execute it till the end)

[Samuzero15tlh]

It's strange, but why it gives me a wrong answer?...

Code: Select all

#include "zcommon.acs"

int MenuOptions[6] = {0,1,2,3,4,5};

Int MenuPlayer[8];

Script 808 (Int Direction) Net
{
MenuPlayer[playernumber()] = CheckActorInventory(1000 + PlayerNumber(),"MenuItem");
If(Direction == 1)
 {
 TakeInventory("MenuItem",1);
 HudMessage(d:MenuPlayer[playernumber()]; HUDMSG_FADEOUT, 1,0,0.5,0.65,3.0);
 }
 
Else If(Direction == 2)
 {
 GiveInventory("MenuItem",1);
 HudMessage(d:MenuPlayer[playernumber()]; HUDMSG_FADEOUT, 1,0,0.5,0.65,3.0);
 }
}

script 1 enter
{
    Thing_ChangeTID(0, 1000 + PlayerNumber());
}

The script 808 1 will lower the value and script 808 2 will raise it, but when you want to change the direction, it gives at the first execute the opposite operation and the second gives the correct operation. -.-

[SwordGrunt]

I don't know if I understand the way you're trying to do the menu, but the issue you're reporting sounds like a typo in script execution. I don't see how it'd execute the opposite operation if you're puking it with the correct argument.

[Catastrophe]

It's strange, but why it gives me a wrong answer?...

Code: Select all

#include "zcommon.acs"

int MenuOptions[6] = {0,1,2,3,4,5};

Int MenuPlayer[8];

Script 808 (Int Direction) Net
{
MenuPlayer[playernumber()] = CheckActorInventory(1000 + PlayerNumber(),"MenuItem");
If(Direction == 1)
 {
 TakeInventory("MenuItem",1);
 HudMessage(d:MenuPlayer[playernumber()]; HUDMSG_FADEOUT, 1,0,0.5,0.65,3.0);
 }
 
Else If(Direction == 2)
 {
 GiveInventory("MenuItem",1);
 HudMessage(d:MenuPlayer[playernumber()]; HUDMSG_FADEOUT, 1,0,0.5,0.65,3.0);
 }
}

script 1 enter
{
    Thing_ChangeTID(0, 1000 + PlayerNumber());
}

The script 808 1 will lower the value and script 808 2 will raise it, but when you want to change the direction, it gives at the first execute the opposite operation and the second gives the correct operation. -.-

Have you tried waiting an entire minute before puking 808? There might be a completely different script you haven't posted that's still setting up some values which might mess up the logic of this script.

Also when you say it gives you an "opposite operation" do you mean it gives you "MenuItem" when you do puke 808 1 the first time? Because that's impossible from the code I'm seeing.

[Vincent(PDP)]

Try this:

Code: Select all

Script 808 (Int Direction) Net
{
If(Direction == 1)
{
TakeInventory("MenuItem",1);
MenuPlayer[playernumber()] = CheckActorInventory(1000 + PlayerNumber(),"MenuItem");
HudMessage(d:MenuPlayer[playernumber()]; HUDMSG_FADEOUT, 1,0,0.5,0.65,3.0);
}

Else If(Direction == 2)
{
GiveInventory("MenuItem",1);
MenuPlayer[playernumber()] = CheckActorInventory(1000 + PlayerNumber(),"MenuItem");
HudMessage(d:MenuPlayer[playernumber()]; HUDMSG_FADEOUT, 1,0,0.5,0.65,3.0);
}
}

You can't do this only once:

Code: Select all

MenuPlayer[playernumber()] = CheckActorInventory(1000 + PlayerNumber(),"MenuItem");

You need to check the player's inventory EVERYTIME you make a change to it (adding, taking, etc.), or else your MenuPlayer array won't update until you execute the script the next time.

---

The reason why it goes in the "opposite" direction is because you change the player's inventory, but you don't notify your MenuPlayer array about it until next time you execute the script.

This is what your script does at the moment:

Direction 1, first puke:
Set player menu position (CheckInventory)
Decrease inventory
View the previous position (since the array is not notified about the decrease)

Direction 2, second puke:
Set player menu position to the decreased value
Increase inventory
View the previous position (which is the decreased value due to no update)

Direction 2, third puke:
Set player menu position to the increased value
Increase once more
View the previous position (the previously increased value)

So try my script above, it should solve it. :)

[Samuzero15tlh]

Im taking off my hat in front of you!

Code: Select all

Str MenuOptions[6] = {"A","B","C","D","E","F"};

Int MenuPlayer[8];

Script 808 (Int Direction) Net
{
MenuPlayer[playernumber()] = CheckActorInventory(1000 + PlayerNumber(),"MenuItem");
If(Direction == 1)
{
TakeInventory("MenuItem",1);
If(MenuPlayer[PlayerNumber()] < 1) Giveinventory("MenuItem",5);
MenuPlayer[playernumber()] = CheckActorInventory(1000 + PlayerNumber(),"MenuItem");
HudMessage(s:MenuOptions[MenuPlayer[playernumber()]]; HUDMSG_FADEOUT, 1,0,0.5,0.65,3.0);
}

Else If(Direction == 2)
{
GiveInventory("MenuItem",1);
If(MenuPlayer[PlayerNumber()] > 5) Takeinventory("MenuItem",6);
MenuPlayer[playernumber()] = CheckActorInventory(1000 + PlayerNumber(),"MenuItem");
HudMessage(s:MenuOptions[MenuPlayer[playernumber()]]; HUDMSG_FADEOUT, 1,0,0.5,0.65,3.0);
}
}

script 1 enter
{
Thing_ChangeTID(0, 1000 + PlayerNumber());
}

That worked so good! Thanks everyone!
PD:I put the code just for educational pruposes.