[jQuery] Ottimizzare controlli

[ANTAGONIA]

Buon giorno.
Ho una funzione che gestisce l'invio dei dati ad una seconda pagina che esegue l'INSERT.
Soltanto che vorrei ottimizzarla ma non so come fare.
Tutte quelle if, per esempio non posso essere ottimizzate?

function subInUppIstr(tip,id_istr,id_dett){
            var morDal        = $("#morDal").val();
            var morAl        = $("#morAl").val();
            var salMor        = $("#salMor").val();
            var intDal        = $("#intDal").val();
            var intAl        = $("#intAl").val();
            var salInt        = $("#salInt").val();
            var impCre        = $("#impCre").val();
            var impDeb        = $("#impDeb").val();
            var impRim        = $("#impRim").val();
            var saldTot        = $("#saldTot").val();
            var morUtePrec    = $("#morUtePrec").val();
            var ImpCont        = $("#ImpCont").val();
            var numRat        = $("#numRat").val();
            if(morDal==undefined){
                morDal = '';
            }
            if(morAl==undefined){
                morAl = '';
            }
            if(salMor==undefined){
                salMor = '';
            }
            if(intDal==undefined){
                intDal = '';
            }
            if(intAl==undefined){
                intAl = '';
            }
            if(salInt==undefined){
                salInt = '';
            }
            if(impCre==undefined){
                impCre = '';
            }
            if(impDeb==undefined){
                impDeb = '';
            }
            if(impRim==undefined){
                impRim = '';
            }
            if(saldTot==undefined){
                saldTot = '';
            }
            if(morUtePrec==undefined){
                morUtePrec = '';
            }
            if(ImpCont==undefined){
                ImpCont = '';
            }
            if(numRat==undefined){
                numRat = '';
            }
           
                      if(confirm("Aggiornare?")){/*ecc ecc*/}
}

Mi aiutate per favore?

[KillerWorm]

Ciao, puoi fare diverse cose e in vari modi.

Intanto puoi eliminare la ripetizione dell'istruzione var e usare la virgola.
Poi una prima soluzione per eliminare tutti quegli if può essere l'uso dell'operatore || (OR logico) direttamente sull'assegnazione del valore (che in questo caso, se non sbaglio, viene chiamata "valutazione a corto circuito"):

var morDal =  $("#morDal").val()||"",
    morAl = $("#morAl").val()||"",
    salMor =  $("#salMor").val()||"",
    intDal = $("#intDal").val()||"",
    intAl =  $("#intAl").val()||"",
    salInt = $("#salInt").val()||"",
    impCre =  $("#impCre").val()||"",
    impDeb = $("#impDeb").val()||"",
    impRim =  $("#impRim").val()||"",
    saldTot = $("#saldTot").val()||"",
     morUtePrec = $("#morUtePrec").val()||"",
    ImpCont =  $("#ImpCont").val()||"",
    numRat =  $("#numRat").val()||"";

[KillerWorm]

Io personalmente ottimizzerei anche tutte le assegnazioni usando un oggetto con le relative proprietà al posto delle variabili, ed un semplice ciclo per assegnare i rispettivi valori:

var  i,o = {},a = ["morDal", "morAl", "salMor", "intDal", "intAl", "salInt",  "impCre", "impDeb", "impRim", "saldTot", "morUtePrec", "ImpCont",  "numRat"];
while (i = a.shift()){o[i] = $("#"+i).val() ||  ""}

[KillerWorm]

Chiaramente nel caso precedente dovrai usare le proprietà dell'oggetto per ottenere i relativi valori.

[KillerWorm]

In alternativa, utilizzando un sistema analogo, puoi creare delle variabili al volo attraverso la funzione eval().
Ad ogni modo l'uso di eval() è una prassi sconsigliata da molti.

var i,a = ["morDal", "morAl", "salMor", "intDal", "intAl",  "salInt", "impCre", "impDeb", "impRim", "saldTot", "morUtePrec", "ImpCont",  "numRat"];
while (i = a.shift()){eval("var "+i+' = $("#"+i).val() ||  ""')}

EDIT: Scusate il post frammentato, non mi faceva postare diversamente.. soliti problemi

[ANTAGONIA]

Io personalmente ottimizzerei anche tutte le assegnazioni usando un oggetto con le relative proprietà al posto delle variabili, ed un semplice ciclo per assegnare i rispettivi valori:

var  i,o = {},a = ["morDal", "morAl", "salMor", "intDal", "intAl", "salInt",  "impCre", "impDeb", "impRim", "saldTot", "morUtePrec", "ImpCont",  "numRat"];
while (i = a.shift()){o[i] = $("#"+i).val() ||  ""}

Buon giorno, ti ringrazio per la prima parte dell'ottimizzazione.
Ma l'ottimizzazione per l'assegnazione dei valori, non l'ho capita.
Grazie.

[KillerWorm]

Ciao, cosa non hai capito esattamente?
Cerco di spiegare. Inizializzo un oggetto vuoto e un array che contiene i "nomi". Questo array lo do in pasto al ciclo che crea dinamicamente le proprietà dell'oggetto e il selettore JQuery in base ai singoli valori dell'array. Quindi assegno, ad ogni proprietà, il valore restituito dal relativo selettore JQuery o una stringa vuota (se il selettore restituisce il valore "undefined").

Posso quindi accedere alle proprietà dell'oggetto, utilizzando la notazione a punto, per ottenere i relativi valori.
Ad esempio:

alert(o.morDal)

E' chiaro che in questo caso si dovrà sempre fare riferimento all'oggetto per accedere alle varie proprietà.

Giusto per conoscenza, esiste la funzione with() che permetterebbe di usare direttamente le proprietà dell'oggetto specificato senza fare riferimento all'oggetto stesso, ma questo metodo è sconsigliato/pericoloso/deprecato, quindi da non usare.