[Risolto][JS] Classe definita in (function); renderla glob

[TommyB1992]

Ciao ragazzi,

ho scritto una classe di cui riporto il pezzo di codice incriminato:
http://pastebin.com/HLgEcvSB

Se potete notare la riga: div.setAttribute("onClick", "Keyboard.insert_char(this.innerHTML)"); non funzionerà mai perchè ovviamente la classe Keyboard la definisco dentro a una funzione, ed anche se definisco var Keyboard = {}; al di fuori di essa non funziona.

[Zoff]

Stai usando lo strict-mode è normale che sia così.

Scrivere il codice js come stringa in un attributo è decisamente sconsigliabile. Basta usare addEventListener:

Codice: Seleziona tutto

div.addEventListener('click', function(){ Keyboard.insert_char(this.innerHTML); })

In questo modo eviti la necessità che Keyboard sia visto globalmente.


Consiglio di guardare lo standard AMD e RequireJS su come gestire le dipendenze in javascript.
Per il futuro sono utili anche i moduli di ECMAScript 6 (ovvero la nuova versione di JS): https://developer.mozilla.org/en-US/doc ... nts/import

[TommyB1992]

Stai usando lo strict-mode è normale che sia così.

Scrivere il codice js come stringa in un attributo è decisamente sconsigliabile. Basta usare addEventListener:

Codice: Seleziona tutto

div.addEventListener('click', function(){ Keyboard.insert_char(this.innerHTML); })

In questo modo eviti la necessità che Keyboard sia visto globalmente.


Consiglio di guardare lo standard AMD e RequireJS su come gestire le dipendenze in javascript.
Per il futuro sono utili anche i moduli di ECMAScript 6 (ovvero la nuova versione di JS): https://developer.mozilla.org/en-US/doc ... nts/import

Ciao Zoff, inserire l'elemento nell'addEventListener è quello che ho fatto a prescindere, ma avendo provato questa versione precedentemente ero curioso di capire il perchè non funzionasse.

Comunque grazie per le guide, gli darò una letta.

[TommyB1992]

Vorrei comunque incollare anche la soluzione di un altro utente (non si sa mai che sia utile a qualcuno):

una soluzione è quella, l'altra è quella di rendere globale Keyboard, per farlo basta togliere il var.

[Zoff]

Considera però che l'utilizzo delle globals è una Bad practice.