Drop-down music player Code Help, I think I may have some faults... Options

[musicallyally]

I think I may have some faults in my code for the drop-down music player. I copied exact and then just filled in the Music URLS in the proper place and the name, etc. I didnt mess with anything else so if anyone could take a look at this and possibly tell me what I've done wrong I would appreciate it alot!

<!-- begin code provided by createblog.com --><br /><!-- begin code provided by createblog.com --><br /><br /><OBJECT ID="Player" height="0" width="0" CLASSID="CLSID:6BF52A52-394A-11d3-B153-00C04F79FAA6"><br /><br /><PARAM NAME="uiMode" VALUE="invisible"><br /><br /><PARAM NAME="Autostart" VALUE="True"><br /><br /></object><br /><br /><br /><br /><FORM ID="form"><br /><br /><br /><br /><SELECT ID="playlist" size="1"><br /><br /><br /><br /><!-- Add song info that appears in drop down list here --><br /><br /><option value="0">Helena -- My Chemical Romance</option><br /><br /><option value="1">Broken Angel -- Hanson</option><br /><br /><option value="2">Run -- Snow Patrol</option><br /><br /><option value="3">Cute Without the "E' -- Taking Back Sunday</option><br /><br /><option value="4">Save Me -- Unwritten Law</option><br /><br /><br /><br /></SELECT><br /><br /><br /><br /><BR><br /><br /><BUTTON ID="BackButton" onClick="Back(forms['form'].playlist);">&nbsp;<<&nbsp;</BUTTON><br /><br /><BUTTON ID="PlayPauseButton" onClick="PlayPause(forms['form'].playlist);"><FONT color="green">&nbsp;&nbsp;Play&nbsp;&nbsp;</FONT></BUTTON><br /><br /><BUTTON ID="NextButton" onClick="Next(forms['form'].playlist);">&nbsp;>>&nbsp;</BUTTON>&nbsp;<br /><br /><BUTTON ID="StopButton" onclick="Stop();"><FONT color="maroon">Stop</FONT></BUTTON><br /><br /><b><font size="3" color="black">Shuffle</font></b><INPUT TYPE=checkbox CHECKED ID="ShuffleSwitch" onclick="ToggleShuffle(this);" value="ON"><br /><br /><br /><br /></FORM><br /><br /></CENTER><br /><br /><br /><br /><br /><br /><comment ID="PlayHTML"><FONT color="green">&nbsp;&nbsp;Play&nbsp;&nbsp;</FONT></comment><br /><br /><comment ID="ShuffleOnHTML"><INPUT TYPE=checkbox CHECKED ID="ShuffleSwitch" onclick="ToggleShuffle(this);"></comment><br /><br /><comment ID="ShuffleOffHTML"><INPUT TYPE=checkbox UNCHECKED ID="ShuffleSwitch" onclick="ToggleShuffle(this);"></comment><br /><br /><br /><br /><br /><br /><script ID="Main" Language="JavaScript"><br /><br /><br /><br /> var songs = new Array();<br /><br /><br /><br /> //*******************************//<br /><br /> //****** CHANGEABLE STUFF *******//<br /><br /> //*******************************// <br /><br /> <br /><br /> var shuffle = false;  // false = shuffle off, true = shuffle on<br /><br /><br /><br /> var autoStart = true;   // false = autoStart off, true = autoStart on<br /><br /> <br /><br /> var numberTracks = true;  // true = place track number in front of list items, false = no track numbers<br /><br /> <br /><br /> // Add song URLs here (make sure it matches up with the order you have for song info, and urls need quotes):<br /><br /> songs[0]="http://a420.v8383d.c8383.g.vm.akamaistream.net/7/420/8383/3b858b51/mtvrdstr.download.akamai.com/8512/wmp/3/25831/29427_1_1_04.asf";<br /><br /> songs[1]="http://a420.v8383d.c8383.g.vm.akamaistream.net/7/420/8383/3b858b51/mtvrdstr.download.akamai.com/8512/wmp/3/435/29169_1_8_04.asf";<br /><br /> songs[2]="http://a420.v8383d.c8383.g.vm.akamaistream.net/7/420/8383/3b858b51/mtvrdstr.download.akamai.com/8512/wmp/4/16355/29212_1_7_04.asf";<br /><br /> songs[3]="http://a420.v8383d.c8383.g.vm.akamaistream.net/7/420/8383/3b858b51/mtvrdstr.download.akamai.com/8512/wmp/0/24366/27330_1_3_04.asf";<br /><br /> songs[4]="http://wm.atlrec.com/unwritten_law/saveme-wma-full.wma";<br /><br /> <br /><br /> //*******************************//<br /><br /> //*******************************//<br /><br /><br /><br />// Initializations //<br /><br />with (document){<br /><br /> var length = forms['form'].playlist.length;<br /><br /><br /><br /> if(numberTracks){<br /><br />  for (var i = 0; i < length; i++){<br /><br />   forms['form'].playlist.options[i].innerHTML = (i+1) + " - " + forms['form'].playlist.options[i].innerHTML;<br /><br />  }<br /><br /> }<br /><br /> <br /><br /> if (shuffle) {<br /><br />  var randsg = Math.floor(Math.random()*songs.length);<br /><br />  Player.url = songs[randsg];<br /><br />  forms['form'].playlist.options[randsg].selected = true;<br /><br />  forms['form'].ShuffleSwitch.outerHTML = ShuffleOnHTML.innerHTML;<br /><br /> }<br /><br /><br /><br /> else {<br /><br />  forms['form'].ShuffleSwitch.outerHTML = ShuffleOffHTML.innerHTML;<br /><br />  Player.url = songs[0];<br /><br /> }<br /><br /><br /><br /> if(autoStart){<br /><br />  var snum = forms['form'].playlist.selectedIndex;<br /><br />  if(Player.url != songs[snum]){<br /><br />   Player.url = songs[snum];<br /><br />  }<br /><br />  Player.controls.Play();<br /><br /> }<br /><br />}<br /><br /><br /><br />// Functions //<br /><br />// Discription: "PlayPause" will toggle playing and pausing if the same song is still selected,<br /><br />//              otherwise it will load the newly selected song<br /><br />function PlayPause(list) {<br /><br /> var snum = list.selectedIndex;<br /><br /> <br /><br /> if((Player.url == songs[snum] && Player.url != "") && Player.playState != 1){<br /><br />  if(Player.playState == 3){<br /><br />   Player.controls.Pause();<br /><br />  }<br /><br />  <br /><br />  else {<br /><br />   Player.controls.Play();<br /><br />  }<br /><br /> }<br /><br /><br /><br /> else {<br /><br />  Player.url = songs[snum];<br /><br />  Player.controls.Play();<br /><br /> }<br /><br />}<br /><br /><br /><br />// Discription: "Next" will move to the next music file if shuffle is off<br /><br />//              otherwise it will load a random song. Calls PlayPause to start music.<br /><br />function Next(list) {<br /><br /> var snum = list.selectedIndex;<br /><br /> if (!shuffle) {<br /><br />  if (snum == list.length-1) {<br /><br />   snum = -1;<br /><br />  }<br /><br />  snum++;<br /><br /> }<br /><br /> <br /><br /> else {<br /><br />  var temp;<br /><br />  do{<br /><br />   temp = Math.floor(Math.random()*songs.length);<br /><br />  } while(temp == snum);<br /><br />  snum = temp;<br /><br /> }<br /><br /> <br /><br /> list.options[snum].selected = true;<br /><br /> PlayPause(list);<br /><br />}<br /><br /><br /><br />// Discription: "Back" does the same thing as "Next" but moves backwads<br /><br />//              through the list. If shuffle is on then picks a random song.<br /><br />function Back(list) {<br /><br /> var snum = list.selectedIndex;<br /><br /> if (!shuffle) {<br /><br />  if (snum == 0){<br /><br />   snum = list.length;<br /><br />  }<br /><br />  snum--;<br /><br /> }<br /><br /> <br /><br /> else {<br /><br />  var temp;<br /><br />  do{<br /><br />   temp = Math.floor(Math.random()*songs.length);<br /><br />  } while(temp == snum);<br /><br />  snum = temp;<br /><br /> }<br /><br /> <br /><br /> list.options[snum].selected = true;<br /><br /> PlayPause(list);<br /><br />}<br /><br /><br /><br />// Discription: Self explanitory.<br /><br />function Stop(){<br /><br /> Player.controls.Stop();<br /><br />}<br /><br /><br /><br />// Discription: Makes the shuffle flag the same as the status of the CheckBox<br /><br />//    The status of the checkbox (true/false) indicates if the box is checked<br /><br />function ToggleShuffle(CheckBox) {<br /><br /> shuffle = CheckBox.status;<br /><br />}<br /><br /><br /><br /></SCRIPT><br /><br /><br /><br /><script ID="StateChangeHandler" Language = "JavaScript" For = "Player" Event = playStateChange(NewState)><br /><br /><br /><br />// Description: This is an interupt handler used to handle instances when the<br /><br />//              state of the player changes to play or stop for example.<br /><br /><br /><br />//STATE.innerText = NewState;<br /><br /><br /><br />switch(NewState){<br /><br /> case 8:  // Handles player after it just finishes playing through a song<br /><br />  var num = document.forms['form'].playlist.selectedIndex;<br /><br /> <br /><br />  if (!shuffle){<br /><br />   if(num == document.forms['form'].playlist.length-1){<br /><br />    num = -1;<br /><br />   }<br /><br />  <br /><br />   num++;<br /><br />  }<br /><br />  else {<br /><br />   var temp;<br /><br />   do{<br /><br />    temp = Math.floor(Math.random()*songs.length);<br /><br />   } while(temp == num);<br /><br />   num = temp;<br /><br />  }<br /><br /> <br /><br />  document.forms['form'].playlist.options[num].selected = true;<br /><br /> <br /><br />  PreviousState = NewState;<br /><br />  break;<br /><br /><br /><br /> case 1: // Handles player after it stops<br /><br />  if(PreviousState == 8){<br /><br />   var num = document.forms['form'].playlist.selectedIndex;<br /><br />   Player.settings.autoStart = true;<br /><br />   Player.url = songs[num];<br /><br />   PreviousState = NewState;<br /><br />  }<br /><br />  <br /><br />  else if (PreviousState != 3 && PreviousState != 2){<br /><br />   Player.controls.Play();<br /><br />   Player.settings.autoStart = false;<br /><br />  }<br /><br />  <br /><br />  else{<br /><br />   forms['form'].PlayPauseButton.innerHTML = PlayHTML.innerHTML;<br /><br />  }<br /><br />  break;<br /><br /><br /><br /> case 3: // Handles player after it starts to play<br /><br />  PreviousState = NewState;<br /><br />  forms['form'].PlayPauseButton.innerHTML = "Pause";<br /><br />  shuffle = document.forms['form'].ShuffleSwitch.status;<br /><br />  break;<br /><br /><br /><br /> case 2: // Handles player after being paused<br /><br />  PreviousState = NewState;<br /><br />  forms['form'].PlayPauseButton.innerHTML = PlayHTML.innerHTML;<br /><br />  break;<br /><br /><br /><br /> default:<br /><br /></SCRIPT><br /><!-- end code provided by createblog.com --><br /><!-- end code provided by createblog.com -->

This post has been edited by Sherry_Vu: Apr 18 2005, 05:13 PM

[Pulchritude]

code box next time

CODE

[CODEBOX]CODE GOES HERE[/CODEBOX]

and your main problem is this.. <br /> take all those outs you dont need those and they arent suppose to be there