Server Help

JaqMs14 · Guest Offline

How do I code a !setfreq command? Here is my curret code but it is doesn't work at all.

else if(c->check("setfreq"))
      {
         if (isNumeric(c->final))
         {
            String setship="*setfreq ";
            setfreq+=getInteger(c->final,10);
            for(_listnode <Player> *pl=playerlist->head;pl;pl=pl->next)
            {
               if(pl->item->freq!=8)
               {sendPrivate(pl->item,setfreq);}
            }
            sendPrivate(p,"Freq changes Complete");
         }
         else
         {sendPrivate(p,"You must specify a freq number.");}

      }

Then the error is:

c:\documents and settings\....\desktop\src\bot\command.cpp(125) : error C2065: 'setfreq' : undeclared identifier

CypherJF · Posted: Mon Jul 05, 2004 1:44 pm Post subject:

You never defined setfreq

unsigned int setfreq = 0; // change.
setfreq+=getInteger(c->final,10);
_________________
Performance is often the art of cheating carefully. - James Gosling

Solo Ace · Posted: Mon Jul 05, 2004 2:14 pm Post subject:

Ew.

The coding style in that piece of code is even more ugly than Dustpup's style.

And Cypher, what you gave him as solution wouldn't work either, he could've used "String setfreq = "*setfreq ";" instead of what he used on that line.

Me thinks you should use the following.

Code: Show/Hide

else if (c->check("setfreq"))
{
if (isNumeric(c->final))
{
   String setfreq = "*setfreq ";
   setfreq += getInteger(c->final,10);

   _listnode <Player> *parse = playerlist->head;

   while (parse)
   {
      Player *p = parse->item;

      if (p->freq != 8) // Why are you doing this?!? Btw, when I worked with MERV p->team was used for the frequency number, did it get changed!?
         sendPrivate(p, setfreq);

      parse = parse->next;
   }
   sendPrivate(p,"Frequency changes complete.");
}
else
   sendPrivate(p, "You must specify a valid frequency number.");
}

JaqMs14 · Guest Offline

Yes I know, it's messy. I copied it from a !setship command and I thought it would do the same thing.

Solo Ace · Posted: Mon Jul 05, 2004 2:32 pm Post subject:

Of course checking if the parameter of the command is a valid freq number (0 <= number && number <= 9999) or something, and making sure the number is not the spectator frequency number could both be useful too.

But that's up to you, heh.

CypherJF · Posted: Mon Jul 05, 2004 4:11 pm Post subject:

Oh true lol.. i didnt really pay much attention.. oops :/

JaqMs14 · Guest Offline

One more question, how do I make it so that the users cannot switch frequencies/ships? I know you can do it by putting code in under case EVENT_PlayerTeam but I want to know if I can just lock it or something.

JaqMs14 · Guest Offline

Sorry, never mind about that top post. Apperently, I didn't know all the things that *lock did.

50% Packetloss · Posted: Mon Jul 05, 2004 9:25 pm Post subject:

Uint16 Freq= getInteger(c->final,10);//you could use this or do it by hand
if(Freq >= 0 && Freq <=9999)
{
char SetMsg[14];
sprintf(SetMsg,"*setfreq %d",Freq);

//then whatever else you want
}

Thing with Strings is that they allocate/deallocate memory everytime they perform a task and they run a bunch of crap code. I used them so i can get bots done quickly, but something small such as the *setfreq command should be put together with the sprintf function

Cyan~Fire · Posted: Tue Jul 06, 2004 10:21 am Post subject:

Actually, for even more speedy-quickness, you don't have to do a negative check (unsigned).
_________________
This help is informational only. No representation is made or warranty given as to its content. User assumes all risk of use. Cyan~Fire assumes no responsibility for any loss or delay resulting from such use.
Wise men STILL seek Him.

Mine GO BOOM · Posted: Tue Jul 06, 2004 11:40 am Post subject: