ES6: Использование класса в качестве модуля

Question

Я пишу простое приложение с использованием шаблона репозитория, nodejs, graphql и sequelize. Во-первых вот моя основа хранилище:

import db from '../models'; 

export default class AbstractRepository { 
    constructor (model, include) { 
    if (new.target === AbstractRepository) { 
     throw new TypeError('Cannot construct AbstractRepository instances directly.'); 
    } 
    this.model = model; 
    this.include = include; 
    } 
    static findAll() { 
    return db[this.model].findAll({ include: this.include }); 
    } 
} 

Вот репозиторий, который расширяет его:

import db from '../models'; 
import AbstractRepository from './AbstractRepository'; 

class UserRepository extends AbstractRepository { 
    constructor() { 
    super('User', [db.Clickwrap]); 
    } 
} 

export default new UserRepository(); 

Наконец, здесь, где используется UserRespository (есть еще в этот файл, но это важно часть):

import UserRepository from '../../repositories/UserRepository'; 

const resolvers = { 
    Query: { 
    users (root, args, context) { 
     return UserRepository.findAll(); 
    }, 
    }, 
}; 

Когда я запускаю простой graphql запрос в graphiql так:

{ 
    users { 
    id 
    } 
} 

Я получаю эту ошибку:

_UserRepository2.default.findAll is not a function 

Не уверен, что я пропускаю, что причиной этого.

Редактировать

Я обновил свой код, чтобы он не экспортирует экземпляр больше. Все остальное не изменилось. Вот мой новый UserRepository файл:

import db from '../models'; 
import AbstractRepository from './AbstractRepository'; 

class UserRepository extends AbstractRepository { 
    constructor() { 
    super('User', [db.Clickwrap]); 
    } 
} 

export default UserRepository; 

Теперь я получаю эту ошибку:

Cannot read property 'findAll' of undefined 

Answer 1

Как я уже говорил в комментариях выше, я собирался об этом неправильном пути. Как только я сделал, как предложил, и экспортировал класс вместо экземпляра класса, я получил неопределенную ошибку, которая, как я думал, означала, что UserRepository не был определен в моем файле user.js. Однако ошибка была в моем методе findAll() в моем AbstractRepository, потому что this.model никогда не настраивался, потому что я пытался делать все со статикой, поэтому никогда не вызывал конструктор. Поэтому я отказался от статической бессмыслицы и создал экземпляр моего UserRepository. Таким образом, UserRepository выглядит так, как в части редактирования моего оригинального сообщения.Однако AbstractRepository теперь выглядит следующим образом:

import db from '../models'; 

export default class AbstractRepository { 
    constructor (model, include) { 
    if (new.target === AbstractRepository) { 
     throw new TypeError('Cannot construct AbstractRepository instances directly.'); 
    } 
    this.model = model; 
    this.include = include; 
    } 
    findAll() { 
    return db[this.model].findAll({ include: this.include }); 
    } 
} 

И мой файл user.js выглядит следующим образом:

import UserRepository from '../../repositories/UserRepository'; 

const userRepo = new UserRepository(); 
const resolvers = { 
    Query: { 
    users (root, args, context) { 
     return userRepo.findAll(); 
    }, 
    }, 
}; 

Теперь все работает отлично. Спасибо, что помогли мне в правильном направлении.

Answer 2

Статические функции не отозваны на экземпляры класса. См. Последний пример на MDN documentation on static.

В коде, вы экспортируете экземпляр UserRepository:

export default new UserRepository(); 

который вы импортировать позже с тем же именем, что и имя класса:

import UserRepository from '../../repositories/UserRepository'; 

Теперь, когда вы делаете UserRepository.findAll(), вы вызываете функцию findAll на экземпляр класса UserRepository. Вы импортировали этот экземпляр с тем же именем, что и имя класса. Вы на самом деле не ссылаетесь на статическую функцию findAll на класс. Вместо этого вы делаете это на экземпляре.

Как Берги уже предлагали, просто экспортировать класс:

export default UserRepository;