Gabriel Roldan ( https://osgeo-org.atlassian.net/secure/ViewProfile.jspa?accountId=557058%3A54db8b09-1e64-436a-adac-248049585cee ) *created* an issue
GeoServer ( https://osgeo-org.atlassian.net/browse/GEOS?atlOrigin=eyJpIjoiYzI2ZjViN2VlZGY4NGQ4MWIyNDI3YjRjYTIzYzEyYzAiLCJwIjoiaiJ9 ) / Improvement ( https://osgeo-org.atlassian.net/browse/GEOS-10109?atlOrigin=eyJpIjoiYzI2ZjViN2VlZGY4NGQ4MWIyNDI3YjRjYTIzYzEyYzAiLCJwIjoiaiJ9 ) GEOS-10109 ( https://osgeo-org.atlassian.net/browse/GEOS-10109?atlOrigin=eyJpIjoiYzI2ZjViN2VlZGY4NGQ4MWIyNDI3YjRjYTIzYzEyYzAiLCJwIjoiaiJ9 ) Excessive (500+) method lengths. Add a PMD rule to enforce it. ( https://osgeo-org.atlassian.net/browse/GEOS-10109?atlOrigin=eyJpIjoiYzI2ZjViN2VlZGY4NGQ4MWIyNDI3YjRjYTIzYzEyYzAiLCJwIjoiaiJ9 ) Issue Type: Improvement Assignee: Unassigned Created: 12/Jun/21 3:20 PM Priority: Medium Reporter: Gabriel Roldan ( https://osgeo-org.atlassian.net/secure/ViewProfile.jspa?accountId=557058%3A54db8b09-1e64-436a-adac-248049585cee ) https://pmd.github.io/latest/pmd_rules_plsql_design.html#excessivemethodlength This QA metric is primarily related to the difficulty to follow code paths on too large methods. Both for the original author and others. Excessive method length speaks also of a bad internal design for a class. Ideally, a method should do one single thing, at a given abstraction level, and defer lowe level implementation details to other, properly named methods, and so on, in a way that at a glance, a reader can figure out what a method does, and drill down to more and more implementation detail only if needed. The PMD rule's default threshold is 100 lines of code. We have way too many methods over that count to tackle it in a single shot. Setting an initial value of *500* leaves us with the 4 biggest offenders. Please check this assessment for more information: https://github.com/groldan/geoserver/wiki/ExcessiveMethodLength-PMD-assessment---GeoServer ( https://osgeo-org.atlassian.net/browse/GEOS-10109#add-comment?atlOrigin=eyJpIjoiYzI2ZjViN2VlZGY4NGQ4MWIyNDI3YjRjYTIzYzEyYzAiLCJwIjoiaiJ9 ) Add Comment ( https://osgeo-org.atlassian.net/browse/GEOS-10109#add-comment?atlOrigin=eyJpIjoiYzI2ZjViN2VlZGY4NGQ4MWIyNDI3YjRjYTIzYzEyYzAiLCJwIjoiaiJ9 ) Get Jira notifications on your phone! Download the Jira Cloud app for Android ( https://play.google.com/store/apps/details?id=com.atlassian.android.jira.core&referrer=utm_source%3DNotificationLink%26utm_medium%3DEmail ) or iOS ( https://itunes.apple.com/app/apple-store/id1006972087?pt=696495&ct=EmailNotificationLink&mt=8 ) This message was sent by Atlassian Jira (v1001.0.0-SNAPSHOT#100166- sha1:25236ce )
_______________________________________________ Geoserver-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/geoserver-devel
