CoffeeScript declare and use global funcitons

Question

Folks, given the following code... how would one re-use the returnResult and returnError functions? Is it possible to have their scope be global to all files that are required.

fs = require 'fs'

module.exports.syncJSON = (req, res)->
    returnResult = (data) ->
        res.send data

    returnError = (data) ->
        res.send data

    jsonFileContents = fs.readFileSync('sample.json', 'utf8')
    returnResult(jsonFileContents)

module.exports.asyncJSON = (req, res)->
    fs.readFile 'sample.json', (err, data) -> 
        if err
            returnError(err)
        else
            returnResult(data.toString())

    returnResult = (data) ->
        res.send data

    returnError = (data) ->
        res.send data

duplicate stackoverflow.com/questions/5447771/node-js-global-variables — Ilan Frumer
– Ilan Frumer, Commented Jan 8, 2014 at 16:19
@IlanFrumer I think he's asking about function reuse but used the word global instead — jcollum
– jcollum, Commented Jan 8, 2014 at 18:47

jcollum · Accepted Answer · 2014-01-08 18:45:44Z

1

I think what you're asking about is how to use returnError and returnResult in both of the exports (.asyncJSON and .syncJSON). Globals are generally a bad idea. What if someone else overwrites your global? You'll have a weird and intermittent failure. If so, all you need to do is move them outside their current scope and pass in the response object:

s = require 'fs'

returnResult = (res, data)->
  res.send data
returnError = (res, err) -> 
  res.send err

module.exports.syncJSON = (req, res)->
  jsonFileContents = fs.readFileSync('sample.json', 'utf8')
  returnResult(res, jsonFileContents)

module.exports.asyncJSON = (req, res)->
  fs.readFile 'sample.json', (err, data) -> 
    if err
      returnError(res, err)
    else
      returnResult(res, data.toString())

answered Jan 8, 2014 at 18:45

jcollum

47k64 gold badges212 silver badges350 bronze badges

Sign up to request clarification or add additional context in comments.

Collectives™ on Stack Overflow

CoffeeScript declare and use global funcitons

1 Answer 1

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related