Server Help

Hakaku · Posted: Thu Aug 21, 2008 1:19 am Post subject: hud/counter

In Devastation, we utilize a credits system that's displayed on the top left corner of a player's screen. It was originally ported by Hallowed from a bot plugin, but Hallowed's ceased development and I'm not sure what exactly is wrong code-wise after comparing both sources.

One of the problems is that the hud doesn't match the player's credits, meaning that the math part of the code is probablematic somewhere. Say if I have "8000000 credits", the hud will display "8000800". Here's an example screenshot of what it displays, and what it should be :

The second problem is displaying zeroes. I can edit some lines to make all zeroes appear (left and right - like in the above screenshot), but otherwise it doesn't display them at all.

I would really appreciate if someone could revise the code and explain why some of the numbers are not what they're suppose to be. I've attached the source here, and if it helps, the debuilt counter ini.

Samapico · Posted: Thu Aug 21, 2008 8:00 am Post subject:

well, the leading zeros ( 00000000999 ) are not supposed to appear, unless 'showZeroes' is true

There might be a small glitch if the jackpot is 0 and a player enters arena, the 0 probably won't be shown (not a big deal)
oh wait, if there are any zeros in the jackpot, they won't be displayed on EnterArena, because you make it refresh the jackpot using an 'old' jackpot of 0, and any 0's will be considered 'the same as before', so it won't update it.

adding an 'init' parameter would work.

Code: Show/Hide

local void sendCreds(Player *p, int old, int init)
{
Target target;
target.type = T_PLAYER;
target.u.p = p;

int credits = creds->GetCredits(p);
int i = 0, num, oldnum, zeroes, pastZero = 0;

//cleanUp(p,old);

//Put the jp on the screen.
obj->Toggle(&target, 2001, 1);
for (; i < 10; i++)
{
   zeroes = 9 - i;
   num = (int)((credits / (int)pow(10, zeroes)) % 10);
   oldnum = (int)((old / (int)pow(10, zeroes)) % 10);

if (((num == 0) && (!showZeroes) && (i != 9) && (!pastZero)) || (num == oldnum && !init))
{
//Do nothing.
}
else
{
   obj->Toggle(&target, calculateID(num, i), 1);
      obj->Toggle(&target, calculateID(oldnum, i), 0);
   pastZero = 1;
}
}
}

local void sendCreds(Player *p, int old)
{
sendCreds(p, old, 0);
}

local void cPlayerAction(Player *p, int action, Arena *arena)
{
if (action == PA_ENTERARENA)
{
sendCreds(p, 0, 1);
}
...

_________________
(Insert a bunch of dead links here)

Hakaku · Posted: Thu Aug 21, 2008 5:09 pm Post subject:

Well the problem is that whether or not ShowZeroes = 1 (or 0), it won't show them at all. So 18M credits will display just the 18.

If I re-order the following lines, it'll make all zeroes display (so 14M shows up as 0014000400 -- where the second four came from I don't know; but it's being read as a zero, so it won't refresh if I use credits [so 13M will look like 0013000400]).

Code: Show/Hide

obj->Toggle(&target, calculateID(oldnum, i), 0);
obj->Toggle(&target, calculateID(num, i), 1);

With Sama's suggestion, it seems to display the proper zeroes (and I'm guessing the second 'sendCreds' was meant to be cUpdate, in which case it should say "sendCreds(p, old, 1);" to avoid refresh issues). But I'm still getting weird things like '11000100' for 11M, and this is my primary concern.

Samapico · Posted: Thu Aug 21, 2008 5:28 pm Post subject:

the 3rd digit from the right seems to really be a problem lol... let's see...

Samapico · Posted: Thu Aug 21, 2008 5:37 pm Post subject:

with the init, you'd also need to add an 'if':

Code: Show/Hide

...
else
{
obj->Toggle(&target, calculateID(num, i), 1);
if (oldnum != num) //Don't hide it if you just turned it on
obj->Toggle(&target, calculateID(oldnum, i), 0);
pastZero = 1;
}

Samapico · Posted: Thu Aug 21, 2008 5:40 pm Post subject:

Make it print i, num, and oldnum for each iteration of the 'for', that might help to narrow down the bug with that wrong digit

oh, and print 'calculateID(num, i)' and 'calculateID(oldnum, i)' too

Mine GO BOOM · Posted: Thu Aug 21, 2008 6:08 pm Post subject:

The pow() function looks bad in there. Without running it, it is either that or you are sending the wrong old value.

Attached is a simple little file that will calculate the values a bit nicer. I left it in the printf() format, as you can easily add in the toggling abilities.

Hakaku · Posted: Thu Aug 21, 2008 10:57 pm Post subject:

Dr Brain · Posted: Fri Aug 22, 2008 6:49 am Post subject:

Make sure the lvz is correct, I guess.
_________________
Hyperspace Owner

Smong> so long as 99% deaths feel lame it will always be hyperspace to me

tcsoccerman · Posted: Fri Aug 22, 2008 2:49 pm Post subject:

It's starting to occur to me that a general number displaying module would be a good idea.

Expect one from me in the near future.

Samapico · Posted: Fri Aug 22, 2008 3:25 pm Post subject:

I have something like that for mervbot and c++ though

As MGB did, the approach of using /= 10 every iteration is better than the pow(). Even if it doesn't make much sense, it's really the only place I could see that something goes wrong ?.o