Hola Epa,
la solución me parece bien, sólo que cambiaría esto
document.DESCR.aficiones.value.replace(/\s/g, '')
por esto
document.DESCR.aficiones.value.replace(/ /g, '')
dado que \s te quitará además de los espacios los saltos de línea también (en el caso de que lo apliques a un textarea)
Si lo estás aplicando a un campo tipo text entonces está bien
