[FizzyWalshy]

Remove -Fixed.

[Sew_Sumi]

What are you doing?

Код:

GivePlayerMoney(randomNumbers[0] > randomNumbers[1] ? inviter : playerid, amount);
GivePlayerMoney(randomNumbers[0] > randomNumbers[1] ? playerid : inviter, -amount);

This seems advanced, yet for what?



What I'm saying is, why so complicated for such a simple task? It looks like you should redo the entire bottom of this script (From the else down first off, but could extend to the rest of the script needing to be redone somehow)

[SyS]

Quote:

Originally Posted by Sew_Sumi What are you doing? Код: GivePlayerMoney(randomNumbers[0] > randomNumbers[1] ? inviter : playerid, amount); GivePlayerMoney(randomNumbers[0] > randomNumbers[1] ? playerid : inviter, -amount); This seems advanced, yet for what? What I'm saying is, why so complicated for such a simple task? It looks like you should redo the entire bottom of this script (From the else down first off, but could extend to the rest of the script needing to be redone somehow)

Simple he is using dice throwing minigame. Two players will throw dice whomever throws greater number will get money from the other guy

Debug the targeteid on that command.Better one should be using zcmd and scan params instead of that strcmp and cmd text.Also using pvars in this case is not a good idea. Use player enum holding data of each player. As sew said the code can be simplified more and need to rewritten.

[Sew_Sumi]

I asked the OP to explain so he could be more clear in what he's aiming for, as that alone is quite often a good path towards debugging.

You know, when you run it through yourself when someone asks about it and you go "Oh damn, what the heck was I thinking?!"


I was also hinting at the old "Keep it simple stupid" technique. Making things overly complicated can bite you on the ass at times.

[FizzyWalshy]

Can someone fixed this to me please!

[Sew_Sumi]

That's not what this is for... We don't fix things for you, you can learn to do it...


How did you make the code in the first place?

[FizzyWalshy]

nah i just get that script

[Sew_Sumi]

Should almost learn to make a new one. Main things to take note of would be where it does do the randomizing and the commands and messages for the script itself.