Be careful about chaining conditions
This may dangerous, because it's harder to debug.
For example:
return
expected.key === actual.key &&
expected.value === actual.value &&
expected.concern === actual.concern;
If we broke it down to multiple
if conditions it makes it easier to debug.if (expected.key === actual.key) {
return false;
}
if (expected.value === actual.value) {
return false;
}
if (expected.concern === actual.concern) {
return false;
}
return true;
Pros for using multiple conditions:
- If you use
Exceptions instead ofreturnstatements, you can easily get a stack trace which is extremly useful. - Easier usage of
printstyle debugging.
Cons for using multiple
if conditions:- Longer function statements.