[
https://issues.apache.org/jira/browse/NIFI-13989?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Daniel Stieglitz updated NIFI-13989:
------------------------------------
Description: 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. (was: When attempting a fix for NIFI-13963 I found it very hard to
add another argument as there 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.)
> 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)
