GitHub user santiago-a-vidal opened a pull request:
https://github.com/apache/jena/pull/353
Refactoring complex methods of classes FBRuleInfGraph and LPRuleStore
Related to no particular issue. This is a behavior-preserving refactoring.
Summary of this pull request:
⢠We are evaluating a research prototype called Bandago that assists in
the refactoring of complex methods. Bandago is an Eclipse plugin that
automatically identifies and refactors a type of code smell called Brain
Method. A Brain Method centralizes the intelligence of a class and manifests
itself as a long and complex method that is difficult to understand and
maintain We have applied Bandago to 2 complex methods of your project, and we
would like to receive feedback.
⢠Bandago is very conservative and you should not observe many source
code changes (only in the affected class).
⢠The source code (after the refactoring) should behave equivalently to
the original one.
⢠As a sanity check, we have run tests before and after Bandago performed
the refactoring(s) on the project. All tests passed.
⢠The goal of the refactorings applied is to improve the legibility of
the refactored method.
⢠In this case, Bandago refactored the method FBRuleInfGraph.prepare
extracting fragments of its code into the new methods initializeDeductions,
initializeTGCCache, and callPreprocessingHook. Also, the method
LPRuleStore.compileAll() was refactored extracting fragments of its code into
the new methods addWildCardRules and buildTwoLevelIndices.
Thanks in advance for your help in this evaluation!
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/santiago-a-vidal/jena BandagoRefactorings
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/jena/pull/353.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #353
----
commit 5eacd1bf704122dd086dd9e939186de3d134c23f
Author: Santiago Vidal <santiagovidal@...>
Date: 2018-02-05T16:26:50Z
Brain methods of classes FBRuleInfGraph and LPRuleStore automatically
refactored with Bandago
----
---
