Advertisement
Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- final CsvMapper csvMapper, final boolean includeHeader, final boolean yankColumnRequired, final boolean includePreferredSiteToggle) {
- Boolean parameters are a bad idea.
- You could end up with functions like:
- public void someFunc(true, true, false, false, true)
- where there's no context into what the parameter represents.
- It would be better if these booleans could be wrapped in something like a csvOptions object, which has attributes like:
- csvOptions.includeHeaders
- csvOptions.yankColumnRequired
- csvOptions.includePreferredSiteToggle
Advertisement
Add Comment
Please, Sign In to add comment
Advertisement