Expongo lo que pretendo hacer:
Por ahora solo estoy con el backend
es decir, con NodeJS
, mis dos cuestiones principales son:
- Estoy intentando modificar el nombre del fichero que quiero subir con la librería
multer
.
Mi routes\user.js
es el siguiente:
const express = require('express');
const router = express.Router();
const path = require('path');
const usuario_controller = require('../controllers/user');
//subida Ficheros
var multer = require('multer');
const multer_storage = multer.diskStorage({
destination:path.join(__dirname,'../imagenes'),
filename:(req,file,cb)=>{
cb(null,file.originalname);
}
});
const upload = multer({
dest:path.join(__dirname,'src/imagenes'),
storage:multer_storage
}).single('img_perfil');
router.post('/add',usuario_controller.añadir);
router.get('/all',usuario_controller.obtener_todos);
router.delete('/:id',usuario_controller.borrar);
router.post('/uploadImage/:id',upload,(req,res)=>{
console.log(req.file);
var id = req.params.id;
var nombre_img = id+'_imgPerfil';
req.file.originalname = nombre_img;
console.log('NOMBRE IMAGEN => '+nombre_img)
res.send('Imagen subida');
})
router.get('/:id',usuario_controller.obtener_por_id);
En este fragmento :
const multer_storage = multer.diskStorage({
destination:path.join(__dirname,'../imagenes'),
filename:(req,file,cb)=>{
cb(null,file.originalname);
}
});
Configuro los ficheros que subo para que tenga el nombre del fichero y no nombres tales como : 876876sad768ad6876876ads876asd768
, una vez configurado se guardan con su nombre y extension correctos.
Pero lo que yo pretendo es lo siguiente:
Quiero que el usuario que se registre y suba su foto de perfil guarde dicha foto con el siguiente patron : usuario._id
+ _imgPerfil
para que quede almacenado la imagen con un nombre parecido a : 5cae69fc61429c3674737ffe_imgPerfil
y asi tener un orden de las fotos y poder acceder a ellas cuando quiera.
- A mi parecer lo veo un poco desordenado como tengo organizado la libreria
multer
, es decir, la tengo en el fichero usuario.route.ts que anque si es verdad que esta relacionado, no esta modularizado. ¿Sería posible tenerlo en una clase que exporte la funcion o algo así?
¿Como lo haríais vosotros? Es mas que nada por modularizar y abstraer n código de otro.
Me gustaría tener algo como sale en mi route
de Usuario
, asi : usuario_controller.añadir
pero en este caso usuario_controller.añadirFoto
Finalmente dejo mis ficheros:
/model/user.js
module.exports = function () {
var db = require('../libs/db-connection')();
var Schema = require('mongoose').Schema;
var User = new Schema({
nombre_real:String,
apellidos:String,
pais:String,
ciudad:String,
email:{type:String,unique:true},
nombre_usuario:{type:String, unique:true},
password:String
});
return db.model('usuarios',User);
}
/controller/user.js :
const User = require('../models/user')();
exports.añadir = function(req,res){
const {nombre_real, apellidos, email, nombre_usuario, password} = req.body;
var newUsuario = {
nombre_real,
apellidos,
email,
nombre_usuario,
password
};
User.create(newUsuario)
.then(usuario=>{
console.log("USUARIO PARA GUARDAR = "+usuario)
if (usuario) {
res.send({
status:200,
message:'Usuario añadido correctamente'
})
}
if (!usuario) {
res.send({
status:400,
message:'No se pudo crear el usuario'
})
}
})
.catch(err=>{
if (err["code"] == 11000) {
res.send({
status:400,
message:'Usuario o Email ya han sido registrados',
})
}
else{
res.send({
status:400,
message:'Error al guardar el usuario'
});
}
});
}
exports.obtener_por_id = function(req,res){
var id = req.params.id;
User.findOne({_id:id})
.then(usuario=>{
if (usuario) {
res.send({
status:200,
message:'Usuario encontrado',
usuario:usuario
});
}
if (!usuario) {
res.sed({
status:404,
message:'Usuario NO encontrado'
})
}
})
.catch(err=>{
res.send({
status:400,
message:'Error al buscar el usuario por ID',
error:err
})
})
}
exports.obtener_todos = function(req,res){
User.find()
.then(personajes=>{
if (personajes) {
res.send({
status:200,
personajes:personajes
})
}
else{
res.send({
status:400,
message:'No se pudieron obtener los usuarios'
})
}
})
.catch(err=>{
res.send({
status:400,
message:'Error al obtener a los usuarios',
error:err
})
})
}
exports.borrar = function(req,res){
var id = req.params.id;
console.log('ID = '+id)
User.deleteOne({_id:id})
.then(usuario=>{
if (usuario) {
res.send({
status:200,
message:'El usuario se ha borrado correctamente'
})
}
else{
res.send({
status:400,
message:'No se ha podido borrar el usuario'
})
}
})
.catch(err=>{
res.send({
status:400,
message:'Error al borrar el usuario',
error:err
})
})
}
routes
y el otro en un directoriomodel
y simplemente se llamanuser.js
pero pense que llamandolesnombre+loqueEs
seria mas entendible.Es de forma aclarativa, mis ficheros se llama ambosuser.js
pero seria un lio referirse a ellos aqui asi. Un saludo @MauricioContreras