2017-08-13 66 views
0

我在Express中用Knex构建了一个简单的TO-DO应用程序作为我的ORM。除了我的表单数据不会从我的表单发送到服务器这一事实之外,一切正常。作为一种调试手段,我确保将req.body记录到控制台。因此,我在表单中输入的信息显示在控制台中,因此此时的问题是找出它未发布到服务器的原因。我怀疑我的问题完全来自knex查询,但我只是不知道如何或为什么。这是我到目前为止...POST表单数据注销到控制台,但没有发送到服务器

第一个文件是我的app.js 第二个包含有问题的POST路由。感谢您的任何帮助!

var express = require('express'); 
 
var path = require('path'); 
 
var favicon = require('serve-favicon'); 
 
var logger = require('morgan'); 
 
var cookieParser = require('cookie-parser'); 
 
var bodyParser = require('body-parser'); 
 

 
var index = require('./routes/index'); 
 
var users = require('./routes/users'); 
 
var todo = require('./routes/todo'); 
 

 
var app = express(); 
 

 
// view engine setup 
 
app.set('views', path.join(__dirname, 'views')); 
 
app.set('view engine', 'hbs'); 
 

 
// uncomment after placing your favicon in /public 
 
//app.use(favicon(path.join(__dirname, 'public', 'favicon.ico'))); 
 
app.use(logger('dev')); 
 
app.use(bodyParser.json()); 
 
app.use(bodyParser.urlencoded({ 
 
    extended: false 
 
})); 
 
app.use(cookieParser()); 
 
app.use(express.static(path.join(__dirname, 'public'))); 
 

 
app.use('/', index); 
 
app.use('/users', users); 
 
app.use('/todo', todo); 
 

 
// catch 404 and forward to error handler 
 
app.use(function(req, res, next) { 
 
    var err = new Error('Not Found'); 
 
    err.status = 404; 
 
    next(err); 
 
}); 
 

 
// error handler 
 
app.use(function(err, req, res, next) { 
 
    // set locals, only providing error in development 
 
    res.locals.message = err.message; 
 
    res.locals.error = req.app.get('env') === 'development' ? err : {}; 
 

 
    // render the error page 
 
    res.status(err.status || 500); 
 
    res.render('error'); 
 
}); 
 

 
module.exports = app;

var express = require('express'); 
 
var router = express.Router(); 
 

 
const knex = require('../db/knex'); 
 

 
/* GET http://localhost:3000/todo page. */ 
 
router.get('/', (req, res) => { 
 
    knex('todo') 
 
    .select() 
 
    .then(todos => { 
 
     res.render('all', { 
 
     todos: todos 
 
     }); 
 
    }) 
 
}); 
 

 
router.get('/new', (req, res) => { 
 
    res.render('new'); 
 
}); 
 

 
function validTodo(todo) { 
 
    return typeof todo.title == 'string' && todo.title.trim() != '' && typeof todo.priority == 'undefined' && !isNaN(Number(todo.priority)); 
 
} 
 

 
router.post('/', (req, res) => { 
 
    console.log(req.body); 
 
    if(validTodo(req.body)) { 
 
    const todo = { 
 
     title: req.body.title, 
 
     description: req.body.description, 
 
     priority: req.body.priority, 
 
     date: new Date() 
 
    }; 
 
    // insert todo into database 
 
    knex('todo').insert(todo, 'id') 
 
     .then(ids => { 
 
     const id = ids[0]; 
 
     res.redirect(`/todo/${id}`); 
 
     }); 
 
    } else { 
 
    // respond with an error 
 
    res.status(500); 
 
    res.render('error', { 
 
     message: 'Invalid todo' 
 
    }); 
 
    } 
 
}); 
 

 
module.exports = router;

+0

你如何发布数据到服务器?尝试编写最小的应用程序,它实际上可以从您的POST:ed请求中读取数据。你似乎在读取POST数据为json。您是否将发布的数据作为json正文并设置了正确的标题?或者你只是发送普通格式编码的POST,这是不能以相同的方式阅读。 –

+0

我不确定'insert'的语法是对还是错,但你可以在这里参考https://github.com/tgriesser/knex/issues/732并检查它是否有效 –

+0

所以我想出了我的问题是。无论出于何种原因,改变我的POST路线的顺序都有诀窍。我只是将我的POST路线放在我的GET /新路线上。我真的不知道这个逻辑是如何工作的,但我很高兴我修好了。 –

回答

0

所以我想通了什么我的问题。无论出于何种原因,改变我的POST路线的顺序都有诀窍。我只是将我的POST路线放在我的GET /新路线上。我真的不知道这个逻辑是如何工作的,但我很高兴我修好了。

相关问题