Chatbot syntax errors.

[slimabob]

I was following this tutorial and found it kind of confusing. Nonetheless, I tried my best to do what it said and got this

Code: [Select]

package chatbot {
function clientCmdChatMessage(%a,%b,%c,%fmsg,%cp,%name,%cs,%msg) {
parent::clientCmdChatMessage(%a,%b,%c,%msg,%cp,%name,%cs,%msg)
if (%msg $= "Testing") {
commandToServer('messageSent',"This was a triumph!");
}
};
activatePackage(chatbot);
I don't know a ton about scripting yet and I'm wondering what is causing the syntax errors.

[Ad Bot]

[Munkey]

You need a semicolon(;) after you parent.

Code: [Select]

parent::clientCmdChatMessage(%a,%b,%c,%msg,%cp,%name,%cs,%msg);

[slimabob]

Thanks. Now it says line 7 contains syntax errors though.

Fixed. Now it is saying line 8 is incorrect and contains errors.

[Wizzeh]

You've got the wrong number of brackets. The number of open and closed brackets in your script should be the same.

[slimabob]

You've got the wrong number of brackets. The number of open and closed brackets in your script should be the same.

But if i have 5 brackets total, what do i do? Just add another one? And if so, does it matter where?

[Orthone]

Code: [Select]

package chatbot {
  function clientCmdChatMessage(%a,%b,%c,%fmsg,%cp,%name,%cs,%msg)
    {
    parent::clientCmdChatMessage(%a,%b,%c,%msg,%cp,%name,%cs,%msg);
    if (%msg $= "Testing") {
      commandToServer('messageSent',"This was a triumph!");
    }
  }
};
activatePackage(chatbot);

[slimabob]

Awesome. I get it now, thanks a lot.

[slimabob]

Okay I have another problem, now all my chat is red and it doesnt say who sent what message.

(Oh crap, double post)

[otto-san]

Code: [Select]

package chatbot {
  function clientCmdChatMessage(%a,%b,%c,%fmsg,%cp,%name,%cs,%msg)
    {
    parent::clientCmdChatMessage(%a,%b,%c,%fmsg,%cp,%name,%cs,%msg);
    if (%msg $= "Testing") {
      commandToServer('messageSent',"This was a triumph!");
    }
  }
};
activatePackage(chatbot);
the fmsg arg was wrong in the parent

[slimabob]

So if I wanted to add more commands, I think i would just do this (but im not sure):

Code: [Select]

package chatbot {
  function clientCmdChatMessage(%a,%b,%c,%fmsg,%cp,%name,%cs,%msg)
    {
    parent::clientCmdChatMessage(%a,%b,%c,%fmsg,%cp,%name,%cs,%msg);
    if (%msg $= "Testing") {
      commandToServer('messageSent',"This was a triumph!");
    if (%msg $= "Hi") {
      commandToServer('messageSent',"<color:e5e500>Mimic<color:ffffff>: Yo.");
      }
    }
  }
};
activatePackage(chatbot);
It's giving me syntax errors and I made sure that the brackets were equal.

[otto-san]

the brackets are wrong

you misplaced the closing bracket on the first statement, i fixed it up a bit.

if it's only one line in the if statement, you can do without brackets.

i personally think else if is better in this case.
"if it isn't the last case, is it this?" is basically what it does

Code: [Select]

package chatbot
{
function clientCmdChatMessage(%a,%b,%c,%fmsg,%cp,%name,%cs,%msg)
{
parent::clientCmdChatMessage(%a,%b,%c,%fmsg,%cp,%name,%cs,%msg);
if(%msg $= "Testing")
commandToServer('messageSent',"This was a triumph!");
else if(%msg $= "Hi")
commandToServer('messageSent',"Mimic: Yo.");
}

};
activatePackage(chatbot);

[slimabob]

Alright, that worked but at line 25 i get a syntax error.

[otto-san]

Alright, that worked but at line 25 i get a syntax error.

restart blockland


there is no line 25

[slimabob]

restart blockland


there is no line 25

Yes there is. I sent you the code since I don't want people taking this script (however simple it may be).

UPDATE: it gives my a syntax error at line 25 but everything works until line 34.

[Headcrab Zombie]

If you're going to add a bunch of commands, you'd be better of using a switch:

Code: [Select]

package chatbot
{
function clientCmdChatMessage(%a,%b,%c,%fmsg,%cp,%name,%cs,%msg)
{
parent::clientCmdChatMessage(%a,%b,%c,%fmsg,%cp,%name,%cs,%msg);
                switch$(%msg)
                {
                        case "Testing":
                               commandToServer('messageSent',"This was a triumph!");
                        case "Hi":
                              commandToServer('messageSent',"Mimic: Yo.");
                        case "Stuff":
                                commandtoserver('messagesent',"stuff");
                        case "More Stuff":
                                commandtoserver('messagesent',"More stuff");
                }
}

};
activatePackage(chatbot);