2016-02-03 17 views
5

Po prostu uczę się pliku node.js i mam trudności z przesłaniem prostego pliku za pomocą express i multer.Node.js TypeError: Nie można odczytać właściwości 'file' z undefined

Oto forma:

obraz Załaduj

W moim configure.js mam:

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

A image.js kontroler:

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(); 
    }, 

jednak uzyskać ten błąd, gdy próbuję przesłać obraz:

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) 

A gdy loguję obiekt req, file tam nie ma:

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

Właściwie ten fragment to po prostu nieco zmodyfikowana wersja, którą przeczytałem w this book, która używa przestarzałego express-3. Więc po prostu zaktualizowałem go o część multer.

Zastanawiam się, co jest nie tak i jak to naprawić.

+0

Czy kiedykolwiek znalazłeś na to rozwiązanie z multer? Spróbowałem poprawki wymienionej poniżej, a zmiana jej na req.file i req.path wydaje się działać, ale teraz przykład z książki przekierowuje do/image/route bez pomyślnego nadania nazwy lub otrzymania pliku. – Dave

+0

You var tempPath = req.file.path; var ext = ścieżka.tekst (req.file.originalname) .toLowerCase(); var targetPath = ścieżka.resolve ("./ public/upload /" + imgUrl + ext); – iansari

Odpowiedz

9

Używasz upload.single, którego powinieneś użyć req.file nie req.files. Aby przesłać wiele plików, użyj upload.array.

Zauważ, że nie potrzebujesz kolejnego .file po req.file. req.file to plik przesłany, jeśli używasz upload.single.

+0

Dzięki za cynk. Jednak teraz, kiedy używam 'tempPath = req.file.file.path,' I powoduje 'TypeError: Can not read property 'path' of undefined' error. Jakieś pomysły? – qliq

+1

@qliq [zgodnie z dokumentem] (https://www.npmjs.com/package/multer#singlefieldname), wystarczy użyć 'req.file.path', a nie' req.file.file.path' . (Z dokumentu: Pojedynczy plik będzie przechowywany w 'req.file'.) – maowtm

+0

krótki i wyraźny .. dzięki znajomemu – Jana

Powiązane problemy