Eliminare le virgole che vengono inserite per dividere gli elementi di un array

[brutusin]

ragazzi ho trovato e modificato leggermente questo script grazie al forum

function stampa(aSelect) {
var a = new Array();
//
l = aSelect.length;
j = 0;
for (i=0;i<l;i++) {
curOpt = aSelect.options(i);
if (curOpt.selected) {
a[j] = curOpt.value + '\n';
j++;
}
}
s = a.toString();
s.replace(",","");
document.getElementById('testo').value += s;
}

lo script prende dei valori dalla select a selezione multipla e le stampa nella TextArea denominata testo
ho notato un problema che quando seleziono più valori mi viene inserita una virgola tra un elemento e l'altro
con le righe di codice
s = a.toString();
s.replace(",","");

ho tentato invano di sostituirle ma non c'è stato nulla da fare
qualche idea ?

[br1]

Il separatore non e' la sola virgola... e' seguita da uno spazio

s = s.replace(", ","");

ciao

[brutusin]

ok

ne approfitto scusa un secondo ma per quale motivo su firefox ottengo questo errore

Errore: aSelect.options is not a function

alla riga

curOpt = aSelect.options(i);

????

[br1]

La sintassi corretta prevede l'uso delle parentesi quadre.

ciao

[brutusin]

ho provato e l'errore in firefox scompare però

Errore: document.getElementById("testo") has no properties

in corrispondenza di questa riga di codice

document.getElementById('testo').value += s.replace(/,/g , "");

ho provato a sostituire le parentesi tonde con le quadre ma non funziona con IE

[br1]

Originariamente inviato da brutusin
ho provato e l'errore in firefox scompare però

Errore: document.getElementById("testo") has no properties

in corrispondenza di questa riga di codice

document.getElementById('testo').value += s.replace(/,/g , "");

ho provato a sostituire le parentesi tonde con le quadre ma non funziona con IE

... ma cos'e' realmente l'oggetto con id="testo" ? :master: magari e' pure vero che non ha proprieta'...

Per le quadre: le devi usare quando stai trattando un array o una collezione... per metodi e funzioni continua ad usare le tonde

ciao

[brutusin]

è una textarea cosi dichiarata

<textarea name="testo" id="testo" rows="10" cols="70"></textarea>

[brutusin]

risolto ho svuotato la cache e ha cominciato a funzionare
grazie