[jQuery] problema con $.each

[techno]

ciao a tutti sto avendo un problema con $.each vi posto il codice

HTML:

<input id="insert_data" type="text" name="number_card" value="" placeholder="Numero Carta" maxlength="5">
<input id="insert_data" class="input-disabled cnt-data" type="text" name="amount_spending" value="" placeholder="Importo spesa &euro;" disabled>

Javascript:

$(document).ready(function() {
var active=$("input[name=active]").val();

// inserimento numeri
$(".send-number").click(function(){
if(active==1){
var number=$(this).attr("rel");
$("#insert_data").each(function(index) {
if($(this).focus()) var indata=index;
});
alert(indata);
var tag;			
}
});
});

il problema e che sulla variabile "indata" invece di darmi l'index mi da l'oggetto.

[cavicchiandrea]

ID insert_data deve essere univoco. A cosa ti serve eseguire un ciclo? Essendo univoco hai solo quell'oggetto pertanto corretto che restituisca l'oggetto e non il numero

[techno]

facvendo cosi eseguo un ciclo su tutti gli input con quell'id e mi serve per sapere in quale input vi è il focus

[KillerWorm]

Ciao, come già ti ha indicato cavicchiandrea, gli id dentro uno stesso documento html devono necessariamente essere univoci. A prescindere dal problema che descrivi, non puoi avere id uguali sui vari elementi, dovresti usare piuttosto una classe. Si tratta di una nozione di base che vale per tutti i linguaggi di programmazione web.

Altro errore, la variabile indata non puoi dichiararla dentro la funzione each() ed aspettarti che sia vista in quell'alert fuori dalla funzione. Infatti quell'alert genera un errore perché gli viene passata una variabile non definita.

Ad ogni modo la variabile index , argomento della funzione each, a me personalmente restituisce in modo corretto l'indice degli elementi come ci si aspetta che faccia.

Bisogna poi capire a cosa ti serve recuperare l'index in quel modo. Qual'è l'obiettivo che vuoi raggiungere?

Se intendi avere un riferimento all'elemento che ha il focus, quello non è il modo corretto.

if($(this).focus())

In questo modo non stai verificando se l'elemento in questione ha il focus ma, anzi, lo stai proprio impostando.

Piuttosto dovresti avere una condizione del genere:

if($(this).is(':focus'))

Ma anche in questo caso l'operazione fallirebbe perché nel momento in cui clicchi su .send-number il focus si porterebbe su quest'ultimo elemento.

Non è chiaro ciò che vuoi ottenere. Prova a spiegarti meglio.

[techno]

si sei stato molto chiaro hai perfettamente ragione ho combinato un casino