sstrickx / yahoofinance-api

Java Client API for Yahoo Finance
MIT License
498 stars 219 forks source link

Refactor/excessive params list #217

Open VictorM-Coder opened 9 months ago

VictorM-Coder commented 9 months ago

Resolution of Issue #216

Taking the issue for code quality improvement of the system as a reference, focusing on addressing code smells, we performed code refactoring for the code smell Excessive Parameter List in HistoricalQuote and StockDividend classes using the builder pattern as refactoring method

Code Analysis

After performing the necessary refactoring, we ran PMD once again following the same directives indicated in the issue and found the following results (indicating that we were able to address the aforementioned code smells):]

Notion