Shoutbox

so guys... newb is back ... i got the same code as yesterday but today a bit modified. so i got that idea its only for scripting training and meuh... dont talk about it, isnt very  important... so now my problem

code:

---

function OnEvent_MenuClicked(IdClicked)
{
if (IdClicked == "MnuAbout")
var Wnd = MsgPlus.CreateWnd("About.xml", "About");
else if (IdClicked == "MnuToast")
{
var Message == "Hello " + Messenger.MyName + "";
MsgPlus.DisplayToast("", Message, "notify.wav");
}
else if (IdClicked == "MnuSound")
var Wnd = MsgPlus.CreateWnd("Sound.xml", "Sound");
}
else if (IdClicked == "MnuGoogle")
var Wnd = MsgPlus.CreateWnd("Google.xml", "Google");
}

---

I got error on line 7...

quote:

---

Originally posted by Scriptdebugger
Fehler: ';' erwartet.
       Zeile: 7. Code: -2146827284.

---

I don't mean to sound rude, but do you ever read your own code? You've missed a brace yet again! These are things that reading your own code will pick up!
Forget it, the brace was ghiding behind the code tag you didn't close properly.

what do you mean by that? where's the fault? make it bold please.
so where is the problem in that piece of code?

the problem is here:

var Message == "Hello " + Messenger.MyName + "(Smilie)";
MsgPlus.DisplayToast("", Message, "notify.wav");

it needs to be:

var Message = "Hello " + Messenger.MyName + "(Smilie)";
MsgPlus.DisplayToast("", Message, "notify.wav");

"==" is used to see if two things are already equal.  "=" is used to make one thing equal to another.

some times your request for help gets lost in the sea of other requests    hope that helps!

oh dear... i will never get that... "==" is not "="... thx anyway

EDIT: still an error... says in line 4 must be a ";" added...

ok, show what you have so far.

sorry... youre abit too late. markee helped me on WLM... he said "=" for variables and "==" when i want to say that something is equal another thing . everything works nice now.

Please READ your code... There are soo much errors

code:

---

function OnEvent_MenuClicked(MnuId)
{
  switch(MnuId)
  {
    case "MnuAbout":
      var Wnd = MsgPlus.CreateWnd("About.xml", "About");
      break;
    case "MnuToast":
      var Message = "Hello " + Messenger.MyName + "";
      MsgPlus.DisplayToast("Title", Message, "notify.wav");
      break;
    case "MnuSound":
      var Wnd = MsgPlus.CreateWnd("Sound.xml", "Sound");
      break;
    case "MnuGoogle":
      var Wnd = MsgPlus.CreateWnd("Google.xml", "Google");
      break;
  }
}

---

Also it's more usefull to use a switch instead of lots of else if.

And there is no real use in using so much xml files, just use one and put all the windows in there.

ok. if you explain me how to do that i will