Estoy siguiendo los ejemplos de un libro sobre nodejs y tengo dos fragmentos de código idénticos, uno para un set
y el otro para un get
. Como me parece que esto viola el principio DRY, intenté reescribir el código en una función, pero no funcionó.
Este es el código que funciona, repetido, en un esquema para Usuarios en mongoose:
var UserSchema = new Schema({
// otros campos...
website: {
type: String,
get: function(url) {
if(!url) {
return url
} else {
if (url.indexOf('http://') !== 0 && url.indexOf('https://') !== 0) {
url = 'http://' + url;
}
return url;
}
}, // ./get
set: function(url){
if(!url){
return url;
} else {
if (url.indexOf('http://') !== 0 && url.indexOf('https://') !== 0){
url = 'http://' + url;
}
return url;
}
} // ./set
} // ./website
} // ./UserSchema
Entiendo que si uso el
get
ya no necesitaría elset
, pero la comprensión y resolución del problema es parte de mi proceso de aprendizaje.
Trato de evitar la repetición del código, haciendo una función que haga este trabajo:
function dameURI(url) {
if(!url) {
return url
} else {
if (url.indexOf('http://') !== 0 && url.indexOf('https://') !== 0) {
url = 'http://' + url;
}
return url;
}
}
Y reemplazo la función anónima de get
por esta función nueva:
// ...
website: {
// ...
get: dameURI(url),
set: function(url) {
// la función anómina que si funciona
}
}
Al ejecutar el servidor, nodejs se queja porque dice que url
no está definida:
$ node server
/la/ruta/a/mi/proyecto/app/models/user.server.model.js:34
get: dameURI(url),
^
ReferenceError: url is not defined
También intenté usar esta forma (que no sé como se llama):
var dameURI = function(url) {
// ...
}
con idénticos resultados negativos.
Pregunta
¿Cómo puedo pasar el contenido del campo website
a mi función dameURI
tanto en el get
como en el set
para no tener código repetido?
Extra points: ¿Cuál es mi error?
website : { get : dameUri, set : dameUri }
set
porget
pero al intentar usar la función, no funcionó y no entendía por qué.