[
https://issues.apache.org/jira/browse/NIFI-13989?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17896725#comment-17896725
]
David Handermann commented on NIFI-13989:
-----------------------------------------
As a first pass, I would consider reducing the number of constructors to 1,
since the others appear to be basically for convenience in testing. Having a
shared test method would obviate the need for multiple constructors, making it
easier to maintain across the board.
> Refactor JsonTreeRowRecordReader constructor to take a parameter object
> -----------------------------------------------------------------------
>
> Key: NIFI-13989
> URL: https://issues.apache.org/jira/browse/NIFI-13989
> Project: Apache NiFi
> Issue Type: Improvement
> Reporter: Daniel Stieglitz
> Assignee: Daniel Stieglitz
> Priority: Minor
>
> When attempting a fix for NIFI-13963 I found it very hard to add another
> argument as there are already 13 arguments for the current constructor. This
> is a code smell Martin Fowler identifies as a long parameter list
> [https://luzkan.github.io/smells/long-parameter-list.|https://luzkan.github.io/smells/long-parameter-list]
> One of the solutions is to create a parameter object to encompass all the
> arguments.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
