[ 
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 already 10 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]] 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 10 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]] 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
>    Affects Versions: 2.0.0
>            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 already 10 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]] 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)

Reply via email to