別のファイルに記述したclass/methodを双方がお互いを参照しようとすると、undefinedエラーとなる

Question

双方を参照するような構造をしようとすると、xxxx is not a function　というエラーが出ます。

子要素を生成することができる親要素を用意したいと思っています。
ここでいう要素とは、たとえばclass Person{}としています。
で、new Person()生成する関数factory()を用意しています。
Personにはchild()関数を用意しており、
factory()を実行して、子要素を作っています。

この仕組みを1ファイルで記述すると成功するのですが、
ファイルを分割して、双方の関数をrequireして使うようにしていると、
次のようなエラーが出てしまいます。

TypeError: factory is not a function

どのように解決すればよいでしょうか？？

コード例です。

factory.js

const {Person} = require("./Person");

function factory(){
    return new Person()
}
module.exports = {
    factory
};

Person.js

const {factory} = require("./factory");

class Person{
    constructor() {}
    say(){
        return "hello";
    }
    child(){
        return factory() // ←このfactory()がundefinedだと怒られます。
    }
}
module.exports = {
    Person
};

メイン処理
main.js

const {factory} = require("./factory");
const parent = factory()
parent.say() // "hello"
parent.child().say() // "helloとでてほしい"
parent.child().child().say() // "helloとでてほしい"

Answer 1

循環参照と呼ばれるダメなパターンですね。
相互のrequireを行うと、空のオブジェクトになるようです。
参考記事: node.jsにおける循環参照に対処するための3つの方法 - Qiita

クラスは起点となる事が多いので、
何かに依存させるとすぐ循環参照になって不便です。

メソッド内で実行した時に初めてrequireする手法とかありますが、
ESModulesやWebpackで動いたり動かなかったりであまりオススメじゃないですね。

ここはDIを前提とした設計にすると良いですよ。
例えばchildrenを作るfactory関数を外部から注入する感じですかね。
依存のフローが確実に上から下になるので循環参照を解消できます。

class Person{
    constructor(factory) {
        this.factory = factory
    }
    say(){
        return "hello"
    }
    child(){
        return this.factory()
    }
}

function factory () {
    return new Person(factory)
}

factory().child().child().say()
// "hello"

Comments

[eternalbeginne]

この発想を持っていませんでした。ありがとうございます！

function factory () {
return new Person(factory)
}