Forum libero

> Non è che oggi sia qualcosa di meglio,

Tutto cio` mi offende.
Non ci capisco ancora tutto, ma qualcosina qua e la` si`.
Sono migliorato!

> qualche appunto: definisci lo scope delle variabili con var,

Quello e` un estratto, non la funzione completa.
Le mie funzioni dichiarano tutte le locali all'inizio, mai inline.
Odio dichiararle inline.

> se m non ti serve fuori -> var m, se m ha un nome migliore di m usalo, perché non si capisce cos'è,

m sta per "matches", i matches di fetchBadges che e` questa qui:

        fetchBadges = function (nBadgesToFetch, type) {

            var matches = [ ];

                stock ({ type: type }).each (function (r) {

                    var nAvail, first, last;

                        if (nBadgesToFetch) {

                                first = new Badge (r.first, r.type),
                                last = new Badge (r.last, r.type),
                                nAvail = last.parts ().v - first.parts ().v + 1;

                                if (nAvail > nBadgesToFetch) {

                                        matches.push ({ n: nBadgesToFetch, first: first.b, last: first.offset (nBadgesToFetch - 1).b });
                                        nBadgesToFetch = 0;

                                }

                                else {

                                        matches.push ({ n: nAvail.toString (), first: first.b, last: last.b });
                                        nBadgesToFetch -= nAvail;

                                }

                        }

                });

                return nBadgesToFetch ? [ ] : matches;

        }

> usa il forEach invece di iterare con l'indice (la callback prende tre argomenti se proprio ti serve l'indice c'è),

Ah, esiste il "for each"?
Ecco, non l'avevo ancora trovato.
Da come ne parli in termini di callback, il forEach e` un metodo delle array?

> metti sempre la radice al parseInt, perché le stringhe che iniziano per 0 vengono interpretate come ottali,

Poh. Come detto, quello e` un estratto: quel campo viene validato da un altro script che lavora sul suo event handler "onkeyup", e che parsa con la radice 10 e poi riscrive, dopo altri controlli, il campo. La funzione che contiene quel codice e` legata alla validazione di tutto il form. In forma completa e` questa:

        deliveryFormSubmit = function () {

            var i, j, f, m, r, t;
            var amounts, froms, tos;

            var typesToSave = [

                        'A', 'B', 'C', 'M',
                        '820f', '821d', '821f'

                ];

            var fieldsToSave = [

                        'deliveryFormDateDay',
                        'deliveryFormDateMonth',
                        'deliveryFormDateYear',
                        'deliveryFormRecipientName',
                        'deliveryFormRecipientIdNumber',
                        'deliveryFormCounselingAgency',
                        'deliveryFormAgencyCode',
                        'deliveryFormTt951956Amount',
                        'deliveryForm1561nAmount',
                        'deliveryFormSignatureDate'

                ];

                if (!deliveryFormValidate ()) {

                        return;

                }

                r = { };

                for (i = 0; i < fieldsToSave.length; ++ i) {

                        f = fieldsToSave [i].split ('deliveryForm') [1];
                        f = f.charAt (0).toLowerCase () + f.substr (1);
                        r [f] = $(fieldsToSave [i]).value;

                }

                for (i = 0; i < typesToSave.length; ++ i) {

                        t = typesToSave [i];
                        m = fetchBadges (parseInt ($('deliveryForm' + t + 'LotAmount').value), t);

                        amounts = [ ];
                        froms = [ ];
                        tos = [ ];

                        for (j = 0; j < m.length; ++ j) {

                                amounts.push (m [j].n);
                                froms.push (m [j].first);
                                tos.push (m [j].last);

                                if (stock ({ type: t, first: m [j].first, last: m [j].last }).first ()) {

                                        stock ({ type: t, first: m [j].first, last: m [j].last }).remove ();

                                } // entire lot used

                                else {

                                        stock ({ type: t, first: m [j].first }).update ({ first: new Badge (m [j].last, t).offset (1).b });

                                } // lot partially used

                        }

                        f = t.toLowerCase ();
                        r [f + 'LotAmount'] = amounts.join (',');
                        r [f + 'LotFrom'] = froms.join (',');
                        r [f + 'LotTo'] = tos.join (',');

                }

                deliveries.insert (r);
                stockListLoad ();
                deliveriesListLoad ();
                deliveryFormClear ();

        }

> altre variabili con nome poco significativo (f è come m, se non serve fuori dal loop prefissala con var per non sporcare il namespace).

Sta per "field".

p.s. il namespace globale contiene praticamente solo tre oggetti: stock, deliveries e dbSync. I primi due sono i due DB di TAFFY con cui lavora, il terzo e` un oggetto in fase di sviluppo per le richieste CORS al database remoto, insomma questo:

        Sync = function (url, pp) {

            var q = new XMLHttpRequest (),
                f = false,
                h = false;

                this.url = url;
                this.pp = pp;

                this.post = function (data, responseHandler) { if (f == false) {

                        h = typeof (responseHandler) == 'function' ? responseHandler : false;

                        if (q) {

                                f = true;
                                q.open ('POST', this.url, true);
                                q.onreadystatechange = this.rscHandler;
                                q.send ();

                        }

                }}

                this.rscHandler = function () { if (q.readyState == 4) {

                        f = false;

                        if (h) {

                                h.call (q.status == 200 ? q.responseXML : false);

                        }

                }}

        }

che e` un abbozzo ancora da sviluppare. Non spedisce ancora dati ne` usa pp che e` intesa essere una passphrase (come una password, ma piu` una frase). Alla fine lo faro` passare da https, ammesso che le richieste CORS funzionino anche di li`. Comunque se hai altri appunti, scrivi pure.

**Modificato dall'autore il 24.03.2013 22:09**

> Perche' non usare jQuery.each() al posto del vetusto for (...) ?

Se esiste un forEach nel javascript dvro & crvdo lo uso, ma al momento vorrei tenermi lontano da jQuery. Temo mi nasconda cose che vorrei prima imparare da solo.