I have a module that I'm exporting. I need one function to call another function. Here's a simplified version of what I'm trying to do.

module.exports = {
  isEven: (number) => {
    return (number%2 == 0)
  },
  isTenEven: () => {
    return isEven(10)
  }
}

The code above throws isEven is not defined when moduleName.isTenEven() is called.

It makes sense why it fails. But how would you rewrite it? (While maintaining the singleton pattern)

I have a module that I'm exporting. I need one function to call another function. Here's a simplified version of what I'm trying to do.

module.exports = {
  isEven: (number) => {
    return (number%2 == 0)
  },
  isTenEven: () => {
    return isEven(10)
  }
}

The code above throws isEven is not defined when moduleName.isTenEven() is called.

It makes sense why it fails. But how would you rewrite it? (While maintaining the singleton pattern)

• javascript
• node.js
• node-modules

6 Answers 6

Define the functions first, then export them:

const isEven = (number) => number % 2 === 0
const isTenEven = () => isEven(10)

module.exports = {
  isEven,
  isTenEven
}

The object is only used to group the functions together. There's nothing really OO about it so define the functions separately. Construct the object at the end.

const isEven = number => number % 2 === 0;
const isTenEven = () => isEven(10);
module.exports = { isEven, isTenEven };

Maybe just do this? Define then export.

const isEven = number => number % 2 === 0;

module.exports = {
  isEven,
  isTenEven: () => isEven(10)
};

Just to add one more solution to the mix. You don't have to define the function elsewhere. Since the object declaration is plete before the function gets called, you can refer to it via module.exports or via exports like this:

module.exports = exports = {
  isEven: (number) => {
    return (number%2 === 0)
  },
  isTenEven: () => {
    return exports.isEven(10)
  }
}

If you were doing this in a lot of methods, you could define a shorter variable name for the exported object and refer to it.

If you can afford Babel or a version of Node.js that supports import/export statements you could also do:

export const isEven = num => num % 2 === 0;
export const isTenEven = () => isEven(10);

Inside JS object literal using this refers to the object itself so you can have:

module.exports = {
  isEven: (number) => {
    return (number%2 == 0)
  },
  isTenEven: function () {
    return this.isEven(10)
  }
}