JavaScript Help?

Menthu_Rae · 17th August 2002, 10:43 PM

Edit: Fixed the site... removed the "Javascript" - works perfectly now...

Chazza · 17th August 2002, 10:55 PM

First of all, learn how to indent code.

Secondly, it helps if you explain what you need the code to do.

Bangers · 18th August 2002, 3:08 AM

Code:

<script language="JavaScript">
<!-- 

function scrollit(seed) {
	var msg="..Drama†ic †o †he b o n e..";
	var out = " "; 
	var c=1; 

	if(seed>100) {
		seed; 
		cmd="scrollit("+seed+")"; 
		timerTwo=window.setTimeout(cmd,100);
	} else if (seed <= 100 && seed > 0) {
		for(c=0 ; c < seed ; c++) {
			(out+=" "); out+="msg;seed--;
			window.status=out;cmd="scrollit("+seed+");
			timerTwo=window.setTimeout(cmd,100); 
		}
	} else if (seed <= 0) {
		if (-seed < msg) {
			}
	}
} //
//-->

The whole thing doesnt make sense.

I suggest you rethink the structure of this function.

I tried to clean it up as much as possible, but soem areas I have nfi, and I also think you don't either. These areas are in bold

Chazza · 18th August 2002, 12:05 PM

I'll rewrite it from scratch if you just tell me what it's supposed to do.

17th August 2002, 10:43 PM	#1
Menthu_Rae Member Join Date: Mar 2002 Location: Northern Beaches, Sydney Posts: 6,137	JavaScript Help? Edit: Fixed the site... removed the "Javascript" - works perfectly now... Last edited by Menthu_Rae; 18th August 2002 at 7:09 AM.

17th August 2002, 10:55 PM	#2
Chazza Member Join Date: Jan 2002 Location: Sydney Posts: 1,112	First of all, learn how to indent code. Secondly, it helps if you explain what you need the code to do. __________________ Chazza

18th August 2002, 3:08 AM	#3
Bangers Member Join Date: Dec 2001 Location: Silicon Valley Posts: 7,281	Code: <script language="JavaScript"> <!-- function scrollit(seed) { var msg="..Drama†ic †o †he b o n e.."; var out = " "; var c=1; if(seed>100) { seed; cmd="scrollit("+seed+")"; timerTwo=window.setTimeout(cmd,100); } else if (seed <= 100 && seed > 0) { for(c=0 ; c < seed ; c++) { (out+=" "); out+="msg;seed--; window.status=out;cmd="scrollit("+seed+"); timerTwo=window.setTimeout(cmd,100); } } else if (seed <= 0) { if (-seed < msg) { } } } // //--> The whole thing doesnt make sense. I suggest you rethink the structure of this function. I tried to clean it up as much as possible, but soem areas I have nfi, and I also think you don't either. These areas are in bold __________________ There's a story about a golfer who sinks a 30-meter putt and someone says: 'Gee, that was lucky' and the golfer says, 'Yes, amazing how lucky you get when you practice 8 hours a day for 20 years'.

18th August 2002, 12:05 PM	#4
Chazza Member Join Date: Jan 2002 Location: Sydney Posts: 1,112	I'll rewrite it from scratch if you just tell me what it's supposed to do. __________________ Chazza