2016-02-03 11 views
5

Sto solo imparando node.js e ho difficoltà a fare un semplice caricamento di file usando express e multer.Node.js TypeError: Impossibile leggere la proprietà 'file' di undefined

Qui è la forma:

Carica immagine

Nel mio configure.js ho:

app.use(express.static(path.join(__dirname, 'public'))); 
app.use(multer({dest:'../public/upload/temp'}).single('file')); 

E il image.js regolatore:

create: function(req, res) { 
     var saveImage = function() { 
      console.log(req.body); 
      var possible = 'abcdefghijklmnopqrstuvwxyz', 
       imgUrl = ''; 

      for(var i=0; i < 6; i+=1) { 
       imgUrl += possible.charAt(Math.floor(Math.random() * possible.length)); 
      } 

      var tempPath = req.files.file.path, //<line 55 error 
       ext = path.extname(req.files.file.name).toLowerCase(), 
       targetPath = path.resolve('./public/upload/' + imgUrl + ext); 

      if (ext === '.png' || ext === '.jpg' || ext === '.jpeg' || ext === '.gif') { 


       fs.rename(tempPath, targetPath, function(err) { 
        if (err) throw err; 

        res.redirect('/images/' + imgUrl); 
       }); 
      } else { 
       fs.unlink(tempPath, function() { 
        if (err) throw err; 

        res.json(500, {error: 'Only image files are allowed.'}); 
       }); 
      } 
     }; 

     saveImage(); 
    }, 

Tuttavia ottengo questo errore quando provo a caricare un'immagine:

TypeError: Cannot read property 'file' of undefined 
    at saveImage (/home/pc/node-dev/test-proj/controllers/image.js:55:37) 
    at module.exports.create (/home/pc/node-dev/test-proj/controllers/image.js:76:9) 
    at Layer.handle [as handle_request] (/home/pc/node-dev/test-proj/node_modules/express/lib/router/layer.js:95:5) 
    at next (/home/pc/node-dev/test-proj/node_modules/express/lib/router/route.js:131:13) 
    at Route.dispatch (/home/pc/node-dev/test-proj/node_modules/express/lib/router/route.js:112:3) 
    at Layer.handle [as handle_request] (/home/pc/node-dev/test-proj/node_modules/express/lib/router/layer.js:95:5) 
    at /home/pc/node-dev/test-proj/node_modules/express/lib/router/index.js:277:22 
    at Function.process_params (/home/pc/node-dev/test-proj/node_modules/express/lib/router/index.js:330:12) 
    at next (/home/pc/node-dev/test-proj/node_modules/express/lib/router/index.js:271:10) 
    at urlencodedParser (/home/pc/node-dev/test-proj/node_modules/body-parser/lib/types/urlencoded.js:95:37) 

E quando accedo l'oggetto req, file non c'è:

{ title: 'myimage', description: 'something' } 

In realtà lo snippet è solo una versione leggermente modificata di quello che ho letto in this book, che utilizza l'espresso-3 obsoleto. Quindi in pratica l'ho aggiornato con la parte multer.

Mi chiedo cosa c'è che non va qui e come risolverlo.

+0

Hai mai trovato una soluzione praticabile con multer? Ho provato la correzione elencata di seguito, e cambiandola in req.file e req.path sembra funzionare, ma ora l'esempio dal libro reindirizza a/image/route senza nominare o ricevere correttamente il file. – Dave

+0

Si var tempPath = req.file.path; var ext = path.extname (req.file.originalname) .toLowerCase(); var targetPath = path.resolve ('./ public/upload /' + imgUrl + ext); – iansari

risposta

9

Si utilizza upload.single, che è necessario utilizzare req.file non req.files. Per caricare più file, utilizzare upload.array.

Si noti che non è necessario un altro .file dopo req.file. req.file è il file caricato se si utilizza upload.single.

+0

Grazie per il suggerimento. Comunque ora che uso 'tempPath = req.file.file.path,' I causa 'TypeError: Impossibile leggere l'errore 'percorso' di undefined'. Qualche idea? – qliq

+1

@qliq [come indicato nel documento] (https://www.npmjs.com/package/multer#singlefieldname), è sufficiente utilizzare 'req.file.path', non' req.file.file.path'. . (Dal documento: il singolo file verrà memorizzato in 'req.file'.) – maowtm

+0

breve e chiaro ... grazie amico – Jana