need syntax for an array

[photoxprt1868]

Hello,

I need to write a quick java script to compare an incomming IP address to an array of IP address and if it matches one of the IPs in the array then it'll do something else.

I don't know the syntax for defining an array using javascript. Can someone help?

thank you very much

 

[kaht]

var arr = new Array()
arr[0] = "123.123.123.123";
arr[1] = "128.192.0.1";
arr[2] = "etc"

-or-

var arr = ["123.123.123.123", "128.192.0.1", "etc"];

-kaht

[small] <P> <B> <P> <B> <P> <B> <P> <B> <P> <B> <P> <B> <P> <B> <P> <B> <P> <B> <P> <B> <P> <B> <P> <B> <P> <B> <P> <.</B>[/small]
[small](He's back)[/small]

 

[photoxprt1868]

OK so here's what I got so far

<SCRIPT LANGUAGE="JavaScript">
function compareIPs(){
var user_IP = '<!--#echo var="HTTP_CLIENT_ADDR"-->';
var gotoVar = "[URL unfurl="true"]http://www.google.com";[/URL]
var IP_arr = ["155.109.28.37", "155.109.28.157"];	<!--low bandwidth only-->
for (i=0; i<=2; i++){
	if (user_IP == IP_arr[i]){
	document.gotoVar = "[URL unfurl="true"]http://www.yahoo.com";[/URL]
	break;
	}
	alert(document.gotoVar);
}
</SCRIPT>

I think I have a syntax problem because it's not doing anytning.

I'm calling this function onLoad="compareIPs();

 

[adam0101]

Looks like you're missing a closing } for the function. You should be getting an error message telling you this. Look for a little yellow icon in the lower left corner of the browser.

Adam

http://adameslinger.blogspot.com/

 

[photoxprt1868]

by the way the array now only has two values but it'll have more values later on.

 

[trollacious]

Then make your loop like this:

for (var i=0; i<IP_arr.length; i++)

Right now you have the loop counting beyond the last array element, which could cause problems. If you use the above, whatever IP addresses you add to the array will automatically extend the loop upper limit.

You also don't need to use document.gotoVar, just access the variable value with gotoVar.

Lee

 

[tsuji]

javascript comment is // or /* ... */.
[tt]<!--low bandwidth only-->[/tt]
[tt][red]//[/red]low bandwidth only[/tt]