Hi,

What I'm trying to do, is this:

[you type] /countdown
[send msg]5
[send msg]4
[send msg]3
[send msg]2
[send msg]1
[send msg]"Happy Newyear"

This is what I have so far:

code:

---

var counter = 6;
var currentCW;
var counting = false;

function OnEvent_Initialize(MessengerStart)
{
    Debug.trace(" ");
    Debug.trace("------------------ My Script initialised ------------------");
    Debug.trace(" ");
}

function OnEvent_Uninitialize(MessengerExit)
{
}

function OnEvent_ChatWndSendMessage(chatWnd, msg)
{
    if(counting)
    {
        return "";
    }
    var isCommand = checkCommand(msg);
    if(isCommand)
    {
        var command = getCommand(msg);
        switch (command)
        {           
            case "countdown":
                counting = true;
                countDown(chatWnd);
                return "";
                break;
               
            default:
                break;
        }
    }
    else
    {
        return msg;
    }
}

function countDown(chatWnd)
{
    currentCW = chatWnd;
    MsgPlus.AddTimer("commandCountDown", 500);
}

function onEvent_Timer()
{
    counter --;
    Debug.trace("counter = "+counter);
    if(counter == 0)
    {
        counting = false;
        sendMessage("Happy New Year!");
    }
    else
    {
        Debug.trace("Else");
        sendMessage(counter);
    }
}

function sendMessage(msg)
{
    currentCW.sendMessage(msg)
}

function getCommand(msg)
{
    var msgArray = msg.split(" ");
    var command = msgArray[0].substr(1);
    return command;
}

function checkCommand(msg)
{
    if(msg.charAt(0) == "/")
    {
        if(msg.charAt(1) == "/")
        {
            return false;
        }
        else
        {
            return true;
        }
    }
    else
    {
        return false;
    }

}

---

1. Jscript automatically converts between types.
2. Stuff:

code:

---

if(counting)
{
return "";
}

---

code:

---

MsgPlus.AddTimer("commandCountDown", 1000);

---

code:

---

else
{
Debug.trace("Else");
sendMessage(counter);

---

code:

---

MsgPlus.AddTimer("commandCountDown", 1000);

---

quote:

---

Originally posted by Silentdragon

code:

---

if(counting)
{
return "";
}

---

That returns nothing making your script not work. Return msg.

---

quote:

---

code:

---

MsgPlus.AddTimer("commandCountDown", 1000);

---

One second timer.

code:

---

else
{
Debug.trace("Else");
sendMessage(counter);

---

Add

code:

---

MsgPlus.AddTimer("commandCountDown", 1000);

---

Timers run once then go away, you need to make it run each time you need it to.

---

Your code looks needlessly complex, and the fact that OnEvent_ChatWndSendMessage is called evertime a message is sent isn't a problem... And your check command function isn't really needed either. you could just do a simple

code:

---

if (message == "!countdown") {
countDown();
}

---

quote:

---

Originally posted by Dauntless

quote:

---

Originally posted by Silentdragon

code:

---

if(counting)
{
return "";

---

}

---

That returns nothing making your script not work. Return msg.

If I would return msg, you would see "/countdown"... And I dont want it to display the command!

---

code:

---

var counter = 6;
var curWnd;

function OnEvent_ChatWndSendMessage(chatWnd, msg)
{
    if(msg == "/countdown") {
        curWnd = chatWnd;
        MsgPlus.AddTimer("commandCountDown", 1000);
        return "";
    }
}

function onEvent_Timer()
{
    counter--;
    Debug.trace("counter = "+counter);
    if(counter == 0)
        curWnd.sendMessage("Happy New Year!");
    else
    {
        curWnd.sendMessage(counter);
        MsgPlus.AddTimer("commandCountDown", 1000);
    }
}

---

quote:

---

If you read your code you'd understand that because you don't do that, it causes your whole code to fail. Also doing that won't make the command show, as it returns the message if it is counting.

---

Yes, it converts automatically. Although adding strings to integers and such is somewhat weird, but its understandable.

The bottom has more info.
http://pages-perso.esil.univ-mrs.fr/~tourai/JavaScript/ms/5.htm

quote:

---

Originally posted by Dauntless
Now, it's really not that complex... I mean, how complex can 100 lines of basic JScript be

---

quote:

---

Originally posted by Dauntless
Since JScript doesnt support propper OO, I can't put this in nice, structured classes

---

quote:

---

Originally posted by Dauntless
And from your code, i take it that type conversion is done automaticly ?

---

quote:

---

The only thing that JScript doesn't support is inheritance, which also removes polymorphism and abstraction. Both of which aren't necessary in scripting really.
There are plenty of other OO features though: a function declaration can actually define a class, and using the 'prototype' property, objects both built-in and custom can be extended.

---

you can typecast stuff with

code:

---

var xxx = 1;
var yyy = new String(xxx);
Debug.Trace( typeof(yyy));

---

code:

---

var xxx = "1";
var yyy = parseInt(xxx);
Debug.Trace(typeof(yyy));

---

Thx dt .

also Flash uses SpiderMonkey to provide its javascript (which they call ActionScript)
so i would love to know what real "classes" you are talking about since im sure only the prototype one exists eg

code:

---

function moo(xxx){
this.xxx = xxx;
}

moo.prototype = {
"boo" : function(){
Debug.Trace(this.xxx);
}
}

var test = new moo();
moo.boo();

---

You are right that it actually only supports prototype based classes, BUT: In AS 2.0, you CAN write classes. But when you compile your movie, the compiler transforms your AS 2.0 class into AS 1.0 prototype based OO.

In AS 2.0, this is possible:

code:

---

class Foo extends SomeThing
{
   private static var myVartring = "Hi";

   public function Foo()
   {
      trace("constructor called");
    }

   public static var saySomeThing()tring
   {
       return myVar;
    }
}

---