fparse icon indicating copy to clipboard operation
fparse copied to clipboard

Variable names cannot include spaces

Open wjeffery95 opened this issue 2 years ago • 2 comments

It would be useful to be able to have spaces in variable to aid readability.

Example

const squareMyVariable = new Formula('[My Variable]^2');
fObj.evaluate({"My Variable": 3}); // Throws: Uncaught Error: Cannot evaluate MyVariable: No value given

The error is thrown because when the function is cleaned we remove all spaces meaning the variable no longer matches, and would instead have to be MyVariable.

wjeffery95 avatar May 07 '23 11:05 wjeffery95

I'm happy to contribute a solution but don't see anything detailing how you'd like contributions to be submitted. Thanks

wjeffery95 avatar May 07 '23 11:05 wjeffery95

I clean the formula intentionally (removing whitespaces), to make the parsing simpler. But this should not be a big problem to have it changed, I don't see any drawbacks from this feature. If you want to contribute, just create a pull request.

bylexus avatar May 07 '23 14:05 bylexus