chi mi risolve un bug in questo script?

[diego78]

La funzione è la seguente, serve per ricavare la querystring, il problema è che quando nella querystring non vi è alcun parametro restituisce un errore: n.length non è un oggetto.

if(self.location.search.substring(0) != "");
{

var flag = 0;
var strQS = self.location.search.substring(1);
n = strQS.match(/=/gi);
arrValues = new Array(n.length);
for (i = 0; i < n.length; i++)
{
if (i == n.length - 1)
{
StartValue = strQS.indexOf("=", flag);
arrValues[i] = strQS.substring(StartValue + 1, strQS.length);
}
else
{
StartValue = strQS.indexOf("=", flag);
EndValue = strQS.indexOf("&", flag);
arrValues[i] = strQS.substring(StartValue + 1, EndValue);
flag = EndValue + 1;
}
}

if (arrValues[0] == 1) {
document.form1.submit();
}
}

[Mich_]

il primo if dovrebbe essere:

if(self.location.search != "") {

altrimenti il tuo programma va ad applicare metodi di stringa a qualcosa che una stringa non e` (e ti da` errore).

Comunque io userei document.location.search , altrimenti in caso di frame non sai cosa ti prende.

Ciao
Michele

[willybit]

Ciao diego78,

un bug è il punto e virgola ( nella prima if che non ci deve essere

per leggere i parametri passati in querystring potresti usare un metodo più semplice... io mi sono fatto questa funzioncina

function objQueryString(qs){ 
	dic = new Array()
	if(!qs)	qs = location.search
	if(qs!=''){
		qs = qs.qs.replace(/^\?/,'')
		aQs = qs.split('&')
		txt = ''
		for(i=0;i<aQs.length;i++){
			aPV = aQs[i].split('=')
			dic[aPV[0]]=unescape(aPV[1])
		}
	}
	return dic
}

da usare ad esempio così

qs = objQueryString()
for(x in qs) 
	document.write(x+': '+qs[x]+'
')

o se vuoi vedere se esiste il parametro pippo

qs = objQueryString()
if(qs['pippo'])
	document.write('qs[\'pippo\']='+qs['pippo'])

se ti possono essere utili
Aloha

[willybit]

Originariamente inviato da Mich_
il primo if dovrebbe essere:

if(self.location.search != "") {

altrimenti il tuo programma va ad applicare metodi di stringa a qualcosa che una stringa non e` (e ti da` errore).

Comunque io userei document.location.search , altrimenti in caso di frame non sai cosa ti prende.

Ciao
Michele

per la if mi ha preceduto (confermando) Mich_
Nel caso di frame con la mia funzioncina si può fare così per prendere la querystring del frameset

qs = objQueryString(top.location.search)
for(x in qs) 
	document.write(x+': '+qs[x]+'
')