Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Return validation result instead of throwing errors #4

Open
KrishnaPG opened this issue Sep 17, 2018 · 0 comments
Open

Return validation result instead of throwing errors #4

KrishnaPG opened this issue Sep 17, 2018 · 0 comments

Comments

@KrishnaPG
Copy link

It would be great if the error throwing could be replaced with normal function returns (true or false), and let the caller decide to throw or not. in their wrappers if needed.

Its impossible to write below kind of simple code, very useful for pedagogic reasons.

console.log("validation Result for AAA: ", cc.validateFulfillment(fulfillment1, condition));
console.log("validation Result for BBB: ", cc.validateFulfillment(fulfillment2, condition));

Not to mention, trying to validate an array of fulfillments in for loop or Array map or piping with lodash primitives!!

 results = fulfillmentsArray.map(f => cc.validateFulfillment(f, condition);
 _.some(results,  result => { return result == true });

The returned result value could be object (instead of plain boolean value), if required.

   _.some(results,  result => { return result.fulfilled == true });

These kind of things are very very tricky to achieve with throws.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant