problemi con eval !!!

[citrus]

salve a tutti,
ho un problema con questo assegnamento che utilizza la funzione eval.

var allevamento = eval('document.form1.'+index+'_QuantitaAllevamento').value;
var macello= eval('document.form1.'+index+'_QuantitaMacello').value;

il browser mi dice che gli manca un ";" ma a me sembra tutto corretto...
voi sapete dirmi dov'è realmente l'errore?

grazie mille
citrus

[lake86]

metti .value dentro eval

[citrus]

non funziona lo stesso
altre opzioni?

[lake86]

sintatticamento non ci sono errori, sei sicuro che index sia valorizzato e che il risultato esista?

[citrus]

Originariamente inviato da lake86
sintatticamento non ci sono errori, sei sicuro che index sia valorizzato e che il risultato esista?

assolutamente si...
ho provato a stamapre degli alert con il contenuto dell'eval ed è tutto corretto...
quale altro potrebbe essere il problema?

[Sballerio]

var allevamento = eval('document.form1.'+index+'_QuantitaAllevamento.value');
var macello= eval('document.form1.'+index+'_QuantitaMacello.value');

Così l'eval dovrebbe funzionare, altrimenti il problema è da qualche altra parte, se non l'hai già fatto fai una prova tipo questa:

alert("document.form1.1_QuantitaAllevamento.value");

e guarda se ti stampa il valore del campo
o prova col getElementById :master:
o posta qualche riga di codice in più perchè allora l'errore non è lì

[Mega69]

E' uno spreco usare eval, basta usare le parentesi quadre:

var allevamento = document.form1[index + '_QuantitaAllevamento'].value;
var macello= document.form1[index + '_QuantitaMacello'].value;

Apparte questo segui il consiglio di Sballerio e inizia a usare getElementById e il DOM per ottenere i rifeimenti agli oggetti.

[TrueLies]

Fai come ti suggerisce mega, usa le parentesi quadre ovvero le proprietà associative della collezione form.
Eval è preferibile evitarlo finchè si può - alcuni approcci snob suggeriscono di evitarlo perchè fa apparire il codice amatoriale, ma il motivo per cui io suggerisco di evitarlo è che se si prende l'abitudine a non trovare alternative ad eval, si rischia di sviluppare una abitudine che poi si potrebbe trasportare anche sul lato del server side scripting, con conseguenze talora catastrofiche.

[Mega69]

Originariamente inviato da TrueLies
Fai come ti suggerisce mega, usa le parentesi quadre ovvero le proprietà associative della collezione form.
Eval è preferibile evitarlo finchè si può - alcuni approcci snob suggeriscono di evitarlo perchè fa apparire il codice amatoriale, ma il motivo per cui io suggerisco di evitarlo è che se si prende l'abitudine a non trovare alternative ad eval, si rischia di sviluppare una abitudine che poi si potrebbe trasportare anche sul lato del server side scripting, con conseguenze talora catastrofiche.

Beh certo, fare un eval con una stringa inserita dall'utente non è il massimo dell'intelligenza, comunque in js si evita l'utilizzo di eval semplicemente per evitare un calo di prestazioni.