Forse (e forse è un grosso problema) in alcuni casi (ancora una volta) che potrebbero (e potrebbero essere grandi) aumentare le prestazioni dell'app, anche se non posso dire perché.
Ciò che sicuramente fa è rendere il codice più difficile da leggere, dato che dovresti entrare nel callback per scoprire a quale elemento l'evento è realmente associato; e potenzialmente viola anche il principio DRY, dal momento che non è possibile riutilizzare la callback in un altro evento di elemento. Puoi certamente aggiungere un'altra clausola al tuo event.target
se istruzione, ma questo potrebbe offuscare ulteriormente il codice.
Il che mi riporta alla mia prima affermazione, qualcuno deve aver pensato che potrebbe aumentare le prestazioni dell'app, poiché l'unica ragione per offuscare il codice che ha senso, è migliorare le prestazioni. Purtroppo, la maggior parte delle persone tende ad attaccare i problemi di performance laddove pensano che il problema sia, a differenza del profiling del codice e della determinazione esatta di ciò che è underperforming.
Ancora una volta, non sono sicuro che ciò migliorerebbe le prestazioni, ma sono abbastanza sicuro che offusca il codice in modo inutile, e questo è di per sé negativo, specialmente un paio di mesi dopo aver iniziato a scriverlo e tornare a modificare esso. Probabilmente il codice avrà più eventi legati a documenti con if multipli (alcuni con una clausola e alcuni con clausole n) da verificare quando si tenta di eseguire il debug di un evento.
solo i miei 2 centesimi
Modifica: Non riesco davvero a trovare una situazione in cui legare un evento al documento lo renderebbe migliore. Forse il codice che hai visto richiede l'acquisizione di un clic in qualsiasi punto del documento e l'esecuzione dello stesso codice indipendentemente da dove ha origine il clic? è l'unica ragione che trovo potrebbe giustificare fare una cosa del genere