前提・実現したいこと
Node.Js, Expressでログイン機能の実装をしております。メールアドレスとパスワードをいれてログインボタンを押すと、ターミナル上でCannot read property 'length' of undefinedというエラーメッセージが表示されますので、これを解決したいです。 新規登録画面は実装済みです。自分としては、login.jsのrowsがうまく定義されてない?と思ってるのですが、ご教示頂けますと幸いです。
login.js
var express = require('express'); var router = express.Router(); var connection = require('../mysqlConnection'); router.get('/', function(req, res, next) { if (req.session.user_id) { res.redirect('/'); } else { res.render('login', { title: 'ログイン' }); } }); router.post('/', function(req, res, next) { var email = req.body.email; var password = req.body.password; var query = 'SELECT user_id FROM users WHERE email = "' + email + '" AND password = "' + password + '" LIMIT 1'; connection.query(query, function(err, rows) { var userId = rows.length? rows[0].user_id: false; if (userId) { req.session.user_id = userId; res.redirect('/'); } else { res.render('login', { title: 'ログイン', noUser: 'メールアドレスとパスワードが一致するユーザーはいません' }); } }); }); module.exports = router;
app.js
var createError = require('http-errors'); var express = require('express'); var path = require('path'); var cookieParser = require('cookie-parser'); var logger = require('morgan'); var session = require('express-session'); var indexRouter = require('./routes/index'); var usersRouter = require('./routes/users'); var register = require('./routes/register'); var login = require('./routes/login'); var message = require('./routes/message'); var logout = require('./routes/logout'); var app = express(); // view engine setup app.set('views', path.join(__dirname, 'views')); app.set('view engine', 'ejs'); app.use(logger('dev')); app.use(express.json()); app.use(express.urlencoded({ extended: false })); app.use(cookieParser()); app.use(express.static(path.join(__dirname, 'public'))); app.use(session({ secret: 'keyboard cat', resave: false, saveUninitialized: true })); app.use('/', indexRouter); app.use('/users', usersRouter); app.use('/register', register); app.use('/login', login); app.use('/message', message); app.use('/logout', logout); // catch 404 and forward to error handler app.use(function(req, res, next) { next(createError(404)); }); // 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;
login.ejs
<!DOCTYPE html> <html> <head> <title><% title %></title> <meta http-equiv="X-UA-Compatible" content="IE=edge,chrome=1"> <meta name="viewport" content="width=device-width, initial-scale=1.0, maximum-scale=1.0, user-scalable=no"> <meta name="format-detection" content="telephone=no"> <link rel='stylesheet' href='/stylesheets/style.css'> </head> <body> <div class="wrapper"> <p class="main-title"><%= title %></p> <form action="/login" method="post" class="board-form"> <span class="label">Eメール</span><input type="email" name="email" class="input" required><br> <br> <span class="label">パスワード</span><input type="password" name="password" class="input" required><br> <br> <button type="submit" class="submit">ログイン</button> </form> <% if (typeof noUser !== 'undefined') { %> <p class="error"><%= noUser %></p> <% } %> <a href="/" class="btn">トップへもどる</a> </div> </body> </html>
※ログイン機能実装に当たって参考にしたURL→ https://github.com/osamu38/node-express-curriculum/wiki/Node.js%E3%81%A7%E4%BC%9A%E5%93%A1%E7%99%BB%E9%8C%B2%E3%82%B7%E3%82%B9%E3%83%86%E3%83%A0%E3%82%92%E5%B0%8E%E5%85%A5%E3%81%97%E3%82%88%E3%81%86
あなたの回答
tips
プレビュー