Shoutbox

Why don't you use a for statement and arrays. Rather than all those if statement you only need one and you can then use Array[i].length for the length of the string rather than specifying a number.

EDIT: I thought I might help you out by giving you an example of what I was trying to say incase you didn't understand.  I just fixed up the code that you had posted.

code:

---

*/
var hi = 'Hi +Origin (:'
var on = '1'
var Array = new Array('HI","HALLO","HI@","HELLO","SEAS","ZEAS","HUHU","MOIN,"MORGEN","SERVUS","TAG","SERVAS");
function OnEvent_ChatWndReceiveMessage(ChatWnd, Origin, Message, MessageKind){
if (Origin != Messenger.MyName){
Message = Message.toUpperCase()
for(i=0;i<Array.length;i++){
if (Message.substr(0,Array[i].lenght-1) == Array[i]){
ChatWnd.SendMessage('Hi ' + Origin + " (:" );
}
}
}
}

---

This will make it easier to extend on the script easier and fix any problem that you come accross because it is smaller and easier to read.

AhRR

Thx...
i think i should read more tuts and learn more

thanks

If you don't like looking up MSDN all the time you can download a copy of it too right here.  I hope this is as useful to you as it has been to me.

quote:

---

Originally posted by markee
If you don't like looking up MSDN all the time you can download a copy of it too right here.  I hope this is as useful to you as it has been to me.

---

It should be mandatory for anyone who script, alongside the Plus! scripting documentation tbh....

quote:

---

Originally posted by H-Des
It is working correct here.

Only remind that if both users would use this script you will get in a loop which you can't end...

---

No it won't.

MP!L has a flood protection on scripts

quote:

---

Originally posted by Mike
No it won't.

MP!L has a flood protection on scripts

---

Ok, if you say so ...

It came in a loop when I tested it by using 2 msn accounts at the same pc. But I didn't test it on another pc.

quote:

---

Originally posted by Gropfi
what is wrong in this?

code:

---

var hi = 'Hi +Origin (:'

---

---

var hi = 'Hi' +Origin + '(:' ;

Origin is a variable. The rest is text (and thus must be enclosed in parenthesis).
Also don't forget to end each line with ";".

quote:

---

Originally posted by Gropfi

code:

---

ChatWnd.SendMessage('Hi '+Origin (: );

---

---

ChatWnd.SendMessage('Hi '+ Origin + '(:');

same reason as above.

quote:

---

Originally posted by Gropfi

code:

---

if (Message.substr(0,2) "morgen") {
and similar lines

---

---

if (Message.substr(0,6) == "morgen") {

You make a comparisson, so you need a comparisson operator between the two.

substr() is a function which takes a substring out of a given string. The syntax is substr(start,end). Since the length of "morgen" is 6 characters you need to specify to get the first 6 characters in the substr function which is done by substr(0, 6).

quote:

---

Originally posted by H-Des
It came in a loop when I tested it by using 2 msn accounts at the same pc. But I didn't test it on another pc.

---

It should have exactly the same outcome.

To avoid looping use regular expressions like RacePROuk showed. And with regular expressions you can also do what markee showed using an array btw.

quote:

---

Originally posted by Silentdragon
If you like I could add this as an option in Answer Machine Plus.

---

Psttt... it's already possible with your script....people forget about the Plus! tags like (!N)

Received: hi
Answer: hi (!N)
This outputs: hi <contact's friendly name>