2013-05-14 17 views
14

Ich versuche, eine neue Route in meiner Express-App hinzuzufügen, aber ich bekomme Fehler beim Versuch, den Server zu starten. Der Fehler istHinzufügen einer neuen Route zum Knoten express

C:\development\node\express_app\node_modules\express\lib\router\index.js:252 
    throw new Error(msg); 
     ^
Error: .get() requires callback functions but got a [object Undefined] 

hier sind meine Dateien, ich bin neu, so lassen Sie mich wissen Knoten, wenn ich aus links eine wichtige Datei

Routen/furniture.js

exports.furniture = function(req, res){ 
    res.render('furniture', { title: '4\267pli' }); 
}; 

Routen/index.js

/* 
* GET home page. 
*/ 

exports.index = function(req, res){ 
    res.render('index', { title: '4\267pli' }); 
}; 

views/furniture.ejs

<!DOCTYPE html> 
<html> 
<head> 
    <title>4&middot;pli -- architecture</title> 
    <link rel='stylesheet' href='/stylesheets/style.css'/> 
    <link href='http://fonts.googleapis.com/css?family=Didact+Gothic' rel='stylesheet' type='text/css'> 
</head> 
<body> 
<div class="wrapper"> 
    <h1 class="logo"><%= title %></h1> 
</div> 
</body> 
</html> 

app.js

/** 
* Module dependencies. 
*/ 

var express = require('express') 
    , routes = require('./routes') 
    , user = require('./routes/user') 
    , furniture = require('./routes/furniture') 
    , http = require('http') 
    , path = require('path'); 

var app = express(); 

// all environments 
app.set('port', process.env.PORT || 3000); 
app.set('views', __dirname + '/views'); 
app.set('view engine', 'ejs'); 
app.use(express.favicon()); 
app.use(express.logger('dev')); 
app.use(express.bodyParser()); 
app.use(express.methodOverride()); 
app.use(app.router); 
    app.use(require('stylus').middleware(__dirname + '/public')); 
app.use(express.static(path.join(__dirname, 'public'))); 

// development only 
if ('development' == app.get('env')) { 
    app.use(express.errorHandler()); 
} 

app.get('/', routes.index); 
app.get('/users', user.list); 
app.get('/furniture', routes.furniture); 

http.createServer(app).listen(app.get('port'), function(){ 
    console.log('Express server listening on port ' + app.get('port')); 
}); 

Antwort

24

Das Problem ist:

routes = require('./routes'), 
user = require('./routes/user'), 
furniture = require('./routes/furniture'), 

Diese 3 Ihre Routen Ordner setzen, nicht eine bestimmte Datei, zum Ausdruck für eine index.js aussehen wird (nicht gefunden , dann -> Fehler)

In diesen Ordnern befinden, sollten Sie einen index.js mit Ihrem setzen:

exports.xxxx = function(req, res){ 
    res.render('xx', { foo: foo}); 
}; 

Dann Projektordnerstruktur sollte wie folgt aussehen:

routes/ 
    ├── index.js 
    │ 
    ├── user/ 
    │  └── index.js (with a exports.user inside) 
    │ 
    └── fourniture/ 
     └── index.js (with a exports.furniture inside) 

Sie Funktionen mehrere Export wie diese einer Route hinzufügen:

app.js

// a folder called routes with the index.js file inside 
routes = require('./routes') 

. 
. 
. 

app.get('/', routes.main_function); 
app.get('/sec_route', routes.sec_function); 
app.post('/other_route', routes.other_function); 

/routen/index.js

exports.main_function = function(req, res){ 
    res.render('template1', { foo: foo }); 
}; 

exports.sec_function = function(req, res){ 
    res.render('template2', { bar: bar }); 
}; 

exports.other_function = function(req, res){ 
    res.render('template1', { baz: baz }); 
}; 
+0

Dank funktioniert super –

+0

@ 3boll was, wenn Sie mehrere Funktionen unter Benutzer haben wollte. .. index.js admin.js und foo.js ... wie würdest du das schreiben? – Cmag

+1

@Clustermagnet überprüfen Sie den Code der Antwort hinzugefügt. – jmingov

6

Wenn Ihre Website ist manchmal so groß, ich ziehe so etwas wie zu tun:

routes/furniture.js:

module.exports = function(app) 
{ 
    app.get("/furniture/", function(req, res) { 
     res.render('furniture', { title: '4\267plieee' }); 
    }); 
} 

Und dann in app.js:

require("./routes/furniture")(app); 

Es ist in erster Linie das gleiche, aber App. js wird sauberer sein.

+0

gehalten wird, ich finde, dass dies nützlich genug sein wird. –

+1

Dies ist definitiv eine nette Methode, da es die HTTP-Routen selbst innerhalb des routes-Ordners hält. –

2

Obwohl dies etwas alt ist, obwohl teilen die Art, wie ich das tue. Hier ist ein weiterer Ansatz, der Code cleaner und einfacher macht, Routen hinzuzufügen.

app.js

const app = express(); 
const routes = require('./routes'); 
app.use('/api', routes); //Main entry point 

/Routen/index.js

const router = require('express').Router(); 
const user = require('./user'); 
const admin = require('./admin'); 

//This is a simple route 
router.get('/health-check', (req, res) => 
    res.send('OK') 
); 

router.route('/users') 
     .post(validate, user.createUser); 

router.route('/users/:userId') 
     .get(validateUser, user.getUser) 
     .patch(validateUser, user.updateUser) 
     .delete(validateUser, user.deleteUser); 

router.route('/admins/:adminId/dashboard') 
     .get(validateAdmin,admin.getDashboard); 

module.exports = router; 

‚ValidateUser‘ und ‚validateAdmin‘ sind benutzerdefinierte Mittelwaren, die verwendet werden, Anforderungsparameter validiert oder eine Vorverarbeitung zu tun, bevor die eigentliche Anforderung Request-Handler erreichen. Dies ist optional und Sie können auch mehrere Middleware (durch Komma getrennt) haben.

/routes/user.js

module.exports = { 
    createUser:function(req,res,next){ 

    }, 
    updateUser:function(req,res,next){ 

    }, 
    deleteUser:function(req,res,next){ 

    } 
} 

/routes/admin.js

module.exports = { 
    getDashboard:function(req,res,next){ 

    } 
} 
Verwandte Themen