mongoose findOne is not a function - node.js

Question

I just tried to create simple login and registration form with mongodb version 3.2.21 because I've got Windows 7 with 32bit CPU so here is the code
var express = require('express');
var app = express();
var mongoose = require('mongoose');
var consolidate = require('consolidate');
var http = require('http').Server(app);
var io = require('socket.io')(http);
var multer = require('multer')
var upp = multer()
var bodyParser = require('body-parser')
var router = express.Router()
app.use(bodyParser.json());
app.use(bodyParser.urlencoded({extended: true}));
app.use(upp.array());
app.use(express.static('public'));
app.engine('html', consolidate.mustache);
app.set('view engine', 'html');
app.set('views', __dirname + '/views');
var mongoDB = 'mongodb://localhost/my_database';
mongoose.connect(mongoDB, {useNewUrlParser: true});
// Get Mongoose to use the global promise library
mongoose.Promise = global.Promise;
//Get the default connection
var db = mongoose.connection;
if(db){
console.log('connection')
}
//Bind connection to error event (to get notification of connection errors)
db.on('error', console.error.bind(console, 'MongoDB connection error:'));
var Schema = mongoose.Schema;
var SomeModelSchema = new Schema({
username: String,
password: String
});
var SomeModel = mongoose.model('SomeModel', SomeModelSchema );
var newuser = new SomeModel();
app.get('/', function(req, res){
res.render('mainpage')
})
app.get('/reg', function(req, res){
res.render('reg')
})
app.post('/reeg', function(req,res){
newuser.username = req.body.username;
newuser.password = req.body.password;
newuser.save(function (err, savedObject){
if (err){
console.log(err)
}else{
console.log(savedObject);
res.sendFile(__dirname + '\\views\\mainpage.html')
}})
});
app.post('/login', function(req,res){
var userr = req.body.usernamee
var pass = req.body.passwordd
newuser.findOne({username: userr, password: pass}).exec(),
function(err,obj){
if(err) {return res.status(404)
}else if(!obj){
res.send('notregistered')
}if(obj){
res.send('hey' + userr)
}
}})
http.listen('8080')
and this is giving me this error
So what is problem here? I am not using express router yet and this code is in one file

Answer 1

newuser is not a model, it's an instance of SomeModel - list of methods here
findOne is method of model - SomeModel list of methods here
so replace: newuser.findOne to SomeModel.findOne
I've rewritten overall code for better readability, async/await features, with object destructuring examples and etc best practices:
'use strict';
const path = require('path');
const http = require('http');
const express = require('express');
const socketIO = require('socket.io');
const bodyParser = require('body-parser');
const consolidate = require('consolidate');
// DB connection
const mongoose = require('mongoose');
const connectionString = 'mongodb://localhost/my_database';
mongoose.Promise = global.Promise;
mongoose
.connect(connectionString, {useNewUrlParser: true})
.then(() => {
console.log('Connection to database established');
})
.catch(error => {
console.error('MongoDB connection error:', error.message);
process.exit(-1);
});
// End of: DB connection
// Defining DB schemas and models
const Schema = mongoose.Schema;
const UserSchema = new Schema({
username: String,
password: String
});
const User = mongoose.model('User', UserSchema);
// End of: Defining DB schemas and models
// Initial app configuration
const app = express();
const server = http.Server(app);
const io = socketIO(server);
app.engine('html', consolidate.mustache);
app.set('view engine', 'html');
app.set('views', path.join(__dirname, 'views'));
app.use(bodyParser.json());
app.use(bodyParser.urlencoded({extended: true}));
app.use(express.static('public'));
// End of: Initial app configuration
// App route handlers
app.get(
'/',
(req, res) => {
res.render('mainpage');
});
app.get(
'/reg',
(req, res) => {
res.render('reg');
});
app.post(
'/reg',
async (req, res) => {
try {
const {username, password} = req.body;
const user = await User.create({username, password});
console.log('Created user:', user);
res.sendFile(path.join(__dirname, 'views', 'mainpage.html'));
}
catch (error) {
res.status(500).send(error.message);
}
});
app.post(
'/login',
async (req, res) => {
try {
const {username, password} = req.body;
const user = await User.findOne({username, password}).select('-password').lean();
if(!user) {
res.send('User: ' + username +' not registered');
return;
}
res.send('hey ' + username);
}
catch (error) {
res.status(500).send(error.message);
}
});
// End of: App route handlers
// Binding http listener to port
server.listen('8080');
Notice: lean is method of query manual here
Fix Your login form:
<form action="/login" method="POST">
<input type="text" placeholder="username" name="username">
<input type="password" placeholder="pass" name="password">
<button type="submit">Login</button> <br/>
or <strong>Sign Up</strong>
</form>
P.S. Write in comments if something will go wrong.