javascript using non-inline function

Question

I'm using the 'binary-parser' NPM module in node.js and am trying to use my own 'assert' function instead of supplying it inline. I don't know if the problem is that of 'binary-parser' or is that of my ignorance of calling functions and/or function references in javascript.

Here's the code that works but not what I want:

var Parser = require('binary-parser').Parser;

var data = new Buffer([0x23, 0x36, 0x74, 0x0e, 0x01, 0xff, 0xff]);

function range (min, val, max) {
    return (min <= val && val <= max);
}
var dataParser = new Parser()
.endianess('little')
.uint8('inside_humidity', {assert: function(x){return(x <= 100);}})
.uint16('barometer', {assert: function(x){return(20000 <= x && x <= 32500);},
     formatter: function(x) {return x/1000;}})
.uint16('wind_direction', {assert: function(x){return(x <= 360);}})
.uint16('crc');

console.log(dataParser.parse(data));

I'd like to call my own 'range' function like this:

    .uint16('barometer',
        {assert: range(20000, x, 32500),
     formatter: function(x) {return x/1000;}})

But I get:

ReferenceError: x is not defined

So I tried:

assert: function(x){range(20000, x, 325000)}

and

{assert: function(x){range.bind(this, 20000, x, 32500)},

and get:

ReferenceError: range is not defined

Is it possible to do what I want?

Maybe stackoverflow.com/questions/12930272/… is an interesting and helpful read? — Dilettant
– Dilettant, Commented Jun 2, 2016 at 17:06
Thank you and I found this to be even more helpful (at least to me): stackoverflow.com/questions/111102/… — kevdev
– kevdev, Commented Jun 2, 2016 at 18:06

Bergi · Accepted Answer · 2016-06-02 17:10:39Z

2

You will want to use a closure:

function range(min, max) {
    return function(x) {
        return min <= val && val <= max;
    };
}

With that you can write

…
.uint8('inside_humidity', {assert: range(-Infinity, 100)})
.uint16('barometer', {assert: range(20000, 32500), formatter: function(x) {return x/1000;}})
.uint16('wind_direction', {assert: range(-Infinity, 360)})
…

answered Jun 2, 2016 at 17:10

Bergi

671k162 gold badges1k silver badges1.5k bronze badges

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

4 Comments

kevdev Over a year ago

That comes back 'min is not defined'. It recognizes 'min' and 'max' outside of 'function(x)' but not inside of it. I tried:

function range (someMin, someMax) {     var min = someMin;     var max = someMax;     return function(x) { 	return min <= x && x <= max;     }; }

and it still did not recognize 'min' inside function(x).

Bergi Over a year ago

Oh my, what a horrible library. It stringifies the function which makes closures impossible. You should file a bug.

Bergi Over a year ago

@kevdev: As a workaround,

function range(min, max) { return new Function("x", "return "+JSON.stringify(min)+" <= x && x <= "+JSON.stringify(max)+";"); }

should do it.

kevdev Over a year ago

Wow, I never would've come up with that but it works. Thanks!

Dilettant · Accepted Answer · 2016-06-02 17:04:37Z

0

Did you try:

.uint16('barometer', {assert: function(x){return range(20000, x, 32500);},

HTH

answered Jun 2, 2016 at 17:04

Dilettant

3,3853 gold badges33 silver badges30 bronze badges

1 Comment

kevdev Over a year ago

It still comes back 'range is not defined'.

Collectives™ on Stack Overflow

javascript using non-inline function

2 Answers 2

4 Comments

1 Comment

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

4 Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Linked

Related