You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Flag arguments are ugly. Passing a boolean into a function is a truly terrible practice. It
immediately complicates the signature of the method, loudly proclaiming that this function
does more than one thing. It does one thing if the flag is true and another if the flag is false!
Method call render(true) is just plain confusing to a poor reader
Method
render(booleanisSuite)
Should have split the function into two:
renderForSuite()
and
renderForSingleTest().
The text was updated successfully, but these errors were encountered:
immediately complicates the signature of the method, loudly proclaiming that this function
does more than one thing. It does one thing if the flag is true and another if the flag is false!
Should have split the function into two:
renderForSuite()
and
renderForSingleTest().
The text was updated successfully, but these errors were encountered: