Как правильно создать класс?

Question

[Сергей]

Добрый день. Отделил логику авторизации в приложении от роутинга. Авторизацию для удобства хочу сделать классом. Посмотрите код ниже:

const {
    Router
} = require('express')
const router = Router()
const Authorized = require('../moduls/authorized')
const auth = new Authorized()

router.post('/login', async (req, res) => {
    const query = 'SELECT id FROM userslogin WHERE name = $1 AND password = $2'

    const {
        username,
        password
    } = req.body
    try {
        const data = await connectDb(query, [username.toLowerCase(), password.toString()])
        if (data.length != 0) {
            const token = auth.createSign({
                id: data[0].id,
                username: username
            }, auth.secretKey)
            res.status(200)
            res.send({
                id: data[0].id,
                token: token
            })
        } else {
            res.status(401)
            res.send('No user or password')
        }
    } catch (error) {
        console.log(error)
    }
})

Это роутер. К нему подключаю класс авторизации:

class Authorized {
    constructor() {
        const jwt = require('jsonwebtoken')
        const secretKey = 'dSjkLsSjerIfkL'
    }
    createSign(data, secretKey){
        return this.jwt.sign(data,secretKey)
    }
    isAuthorized(req, res, next){
        if (typeof req.headers.token !== "undefined") {
            const token = req.headers.token.split(' ')[1]
            this.jwt.verify(token, this.secretKey, {
                algorithms: 'HS256'
            }, (err, decoded) => {
                if (err) {
                    res.status(500).json({
                        error: 'No authorized'
                    })
                }
                next()
            })
    
        } else {
            console.log('No authorized')
        }
    }
}

module.exports = Authorized

Ошибка Cannot read property 'sign' of undefined
at Authorized.createSign js:7:25
Что то не так с методом createSign, помогите пожалуйста разобраться.

Answer 1

[Михаил]

class Authorized {
  constructor() {
    this.jwt = require('jsonwebtoken')
    this.secretKey = 'dSjkLsSjerIfkL'
  }
...

Comments

[Сергей]

Спасибо! Действительно сработало, все глаза проглядел, а такую банальную вещь упустил из вида.
Еще вопрос, при исполнении вылезает новая ошибка: ReferenceError: jwt is not defined,
про этот кусок:

if (typeof req.headers.token !== "undefined") {
            const token = req.headers.token.split(' ')[1]
            this.jwt.verify(token, this.secretKey, {
                algorithms: 'HS256'
            }

Ощущение, что проблема с контекстом исполнения, this в данном случае ссылается не на инициализированный конструктором объект. Я прав? Как правильно поправить?

[Михаил]

Сергей, насколько я понимаю ошибка
ReferenceError: jwt is not defined означает отсутствие переменной

но все же ваш метод включения этого класса странный
1 файл

class Authorized {
  ...
}

module.exports = new Authorized();

2 файл

const auth = require('../moduls/authorized')
// moduls ? может быть modules?

[Михаил]

Сергей, и + к тому вопрос:
Для чего вы jwt ставите как свойство класса? Это же просто либа для токенов, для чего конкретно она в свойстве класса?

[Сергей]

Михаил, не, тут все ок. Это nodeJs экспорт в первом файле. А во втором это путь до папки

[Сергей]

вот такую штуку написал:

isAuthorized(req, res, next){
        console.log(this)
        next()

пишет что this undefined дейтвительно контекст теряет

[Михаил]

Сергей, зависит от использования

[Сергей]

Михаил, хм, правильный вопрос. Моя идея была просто проинициализировать jwt при создании экземпляра. А как лучше рекваирить модули внутри класса?

[Михаил]

Сергей, ну вот описание класса

const jwt = require('jsonwebtoken');

class Authorized {
  constructor() {
    this.secretKey = 'dSjkLsSjerIfkL';
  }

  createSign(data, secretKey){
    return jwt.sign(data,secretKey)
  }

  isAuthorized(req, res, next){
    if (typeof req.headers.token !== "undefined") {
      const token = req.headers.token.split(' ')[1]
      jwt.verify(token, this.secretKey, {
        algorithms: 'HS256'
      }, (err, decoded) => {
        if (err) {
          res.status(500).json({
            error: 'No authorized'
          })
        }
        next()
      })
  
    } else {
      console.log('No authorized')
    }
  }
}

module.exports = new Authorized();

но как он используется?