Script not detecting chat {resolved}

[Johnny Blockhead]

Hi, I was coding myself a little script today. I got a peice of code off of this thread so my Chatbot could recognize chat. This is how it turned out: (merged with a tiny bit of my code)

Code: [Select]

package bootup 
{
	function NMH_Type::Send(%this)
	{
		%word = %this.getValue();
        	%words = getWordCount(%word);
        	for(%i=0;%i<%words;%i++)
       		{
         	  %word = getWord(%message, %i);
           	  if(%word $= "Bootup, JSys" || %word $= "bootup, jsys" || %word $= "Bootup, jsys" || %word $= "Bootup, Jsys" || %word $= "Boot!")
           	  {
                  	activatepackage("SystemMain");
					commandToServer('messagesent',"JSys²: Booting up.");
					schedule(2000, 0, "commandtoServer", 'messageSent', "Beware: This is for use only with Johnny Blockhead.");
           	  }
		}
        }
	Parent::Send(%this);
}
activatepackage("bootup");

I'd think everything would be fine, except on

Code: [Select]

	Parent::Send(%this);

It said there where errors like so:

Code: [Select]

	Parent:##:##Send(%this);

I know its syntax, but because it is not my code, I don't really know what is wrong with it.
Please help :/

[Ad Bot]

[$trinick]

Move the } immediately prior to the parent::send call to right after it. It closes the function before Parent::Send is called. Also, you need a semicolon after a package declaration.

[Johnny Blockhead]

Move the } immediately prior to the parent::send call to right after it. It closes the function before Parent::Send is called. Also, you need a semicolon after a package declaration.

I got syntax errors when I put the semicolon there, let me test without the semicolon

[$trinick]

Packages must be declared with a semicolon.

[Johnny Blockhead]

Now I have ## error marks like so:

Code: [Select]

activatepackage(##"##Bootup");

[Johnny Blockhead]

Updated Code:

Code: [Select]

package bootup;
{
	function NMH_Type::Send(%this)
	{
		%word = %this.getValue();
        	%words = getWordCount(%word);
        	for(%i=0;%i<%words;%i++)
       		{
         	  %word = getWord(%message, %i);
           	  if(%word $= "Bootup, JSys" || %word $= "bootup, jsys" || %word $= "Bootup, jsys" || %word $= "Bootup, Jsys" || %word $= "Boot!")
           	  {
                  	activatepackage("SystemMain");
					commandToServer('messagesent',"JSys²: Booting up.");
					schedule(2000, 0, "commandtoServer", 'messageSent', "Beware: This is for use only with Johnny Blockhead.");
           	  }
		}
        
	Parent::Send(%this);
	}
}
activatepackage("bootup");

[$trinick]

No, haha. The semicolon goes after the closing } for the package.

Here's a properly formatted, corrected version.

Code: [Select]

package bootup 
{
    function NMH_Type::Send(%this)
    {
        %word = %this.getValue();
        %words = getWordCount(%word);
        for(%i=0;%i<%words;%i++)
        {
            %word = getWord(%message, %i);
            if(%word $= "Bootup, JSys" || %word $= "bootup, jsys" || %word $= "Bootup, jsys" || %word $= "Bootup, Jsys" || %word $= "Boot!")
            {
                activatepackage("SystemMain");
                commandToServer('messagesent',"JSys²: Booting up.");
                schedule(2000, 0, "commandtoServer", 'messageSent', "Beware: This is for use only with Johnny Blockhead.");
            }
        }
        Parent::Send(%this);
    }
};
activatepackage("bootup");

[Johnny Blockhead]

Thanks, no syntax errors, but now when I say Bootup, JSys it doesn't respond with either of the said commands.

[Zeblote]

        %word = %this.getValue();

should be

        %message = %this.getValue();

[Johnny Blockhead]

        %word = %this.getValue();

should be

        %message = %this.getValue();

Still doesn't work :/

Is the code just borked?

[$trinick]

Code: [Select]

%message = %this.getValue();
%words = getWordCount(%message);

[ultimamax]

%message doesn't appear to have been defined before getWord.

[Johnny Blockhead]

%message doesn't appear to have been defined before getWord.

Switching the two

[Johnny Blockhead]

Still doesn't work :/

Code: [Select]

package bootup 
{
    function NMH_Type::Send(%this)
    {
		%words = getWordCount(%word);
		%message = %this.getValue();
        
        for(%i=0;%i<%words;%i++)
        {
            %word = getWord(%message, %i);
            if(%word $= "Bootup, JSys" || %word $= "bootup, jsys" || %word $= "Bootup, jsys" || %word $= "Bootup, Jsys" || %word $= "Boot!")
            {
                activatepackage("SystemMain");
                commandToServer('messagesent',"JSys²: Booting up.");
                schedule(2000, 0, "commandtoServer", 'messageSent', "Beware: This is for use only with Johnny Blockhead.");
            }
        }
        Parent::Send(%this);
    }
};
activatepackage("bootup");

[$trinick]

No. You've mixed the order of the declaration of %message and %words.

Code: [Select]

package bootup 
{
    function NMH_Type::Send(%this)
    {
        %message = %this.getValue();
        %words = getWordCount(%message);
        for(%i=0;%i<%words;%i++)
        {
            %word = getWord(%message, %i);
            if(%word $= "Bootup, JSys" || %word $= "bootup, jsys" || %word $= "Bootup, jsys" || %word $= "Bootup, Jsys" || %word $= "Boot!")
            {
                activatepackage("SystemMain");
                commandToServer('messagesent',"JSys²: Booting up.");
                schedule(2000, 0, "commandtoServer", 'messageSent', "Beware: This is for use only with Johnny Blockhead.");
            }
        }
        Parent::Send(%this);
    }
};
activatepackage("bootup");