Originariamente inviato da Lathspell
Innanzitutto stai ciclando un insieme di oggetti (jQuery) che ripeti 3 volte, e per 3 volte chiedi a jQuery di attraversare il dom cercando quello che ti serve. Sarebbe buona cosa metterseli in cache e poi usarli ogni volta che ti servono.

Scendendo ancora più nel dettaglio, in quella porzione di codice, jQuery nemmeno servirebbe, perchè potresti ottenere tutti gli elementi con il tag "input" del DOM e popolarti un array con solo quelli a cui sei veramente interessato. Usando poi i metodi nativi di Javascript per controllarli per i tuoi scopi.

Da ultimo, i cicli forEach sono sempre sconsigliati, anche se con pochi elementi forse sono più comodi e non intaccano molto le performance.
per il primo punto hai perfettemente ragione.
devo fare un po di refactoring e compattare.

per il secondo punto, il discorso è che a me concettualmente servono tutti.
cioè io sono interessato a tutti, devo controllarne uno a uno e verificare che siano ceccati o meno, e in caso fare tutte le operazioni, ecc........
quello che non mi viene bene è mettere tutto insieme compattando il tutto.
-controllo iniziale
-controll ad ogni click
-eseguire le operazioni a seconda del risultato
per come ho fatto funziona, anche se come dici è scritto un pò male.