Re: [pmd-devel] Extracting violation details from Listener

[Caleb K. <cal...@en...>]

Awesome, thanks!

On Jul 12, 2017 3:47 PM, "Juan Martín Sotuyo Dodero" <jua...@gm...>
wrote:

> Yeap, you got it.
>
> On Wed, Jul 12, 2017 at 5:27 PM, Caleb Knox <cal...@en...>
> wrote:
>
>> So, I should add my variables into the ruleViolationAdded method, and/or
>> add the RuleViolation instances to a list? Right now, my variables are
>> lists to which I add a value every time the method is called.
>>
>> public void ruleViolationAdded(RuleViolation ruleViolation) {
>>                     violations.incrementAndGet();
>>                     l_rule.add(ruleViolation.getRule().getName());
>>                     l_message.add(ruleViolation.getDescription());
>>                     l_beginLine.add(ruleViolation.getBeginLine());
>>                     l_endLine.add(ruleViolation.getEndLine());
>>
>>                     //l_ruleset.add("Custom");
>>                     //l_priority.add(3);
>>
>>                     //j++;
>>                 }
>>
>> On Wed, Jul 12, 2017 at 3:20 PM, Juan Martín Sotuyo Dodero <
>> jua...@gm...> wrote:
>>
>>> You should use directly the ruleViolationAdded method.
>>>
>>> Keep in mind that method will be called once for each violation found on
>>> any file. Your local variables l_rule, l_message, etc. will be
>>> rewritten, and you will only keep the last one. I’d suggest you add the
>>> RuleViolations to a list and process them after analysis.
>>>
>>> Regards
>>> ​
>>>
>>> On Wed, Jul 12, 2017 at 4:43 PM, Caleb Knox <cal...@en...>
>>> wrote:
>>>
>>>> Hello again,
>>>>
>>>> Here is some code I have for extracting the details of the PMD reports.
>>>>
>>>> RuleContext l_ctx = new RuleContext();
>>>>             final AtomicInteger violations = new AtomicInteger(0);
>>>>             l_ctx.getReport().addListener(new
>>>> ThreadSafeReportListener() {
>>>>                 @Override
>>>>                 public void ruleViolationAdded(RuleViolation
>>>> ruleViolation) {
>>>>                     violations.incrementAndGet();
>>>>                 }
>>>>
>>>>                 // here is where you can extract violation info
>>>>                 // to update bean
>>>>                 public void ruleViolationDetailExtractor(RuleViolation
>>>> ruleViolation) {
>>>>                     l_rule = ruleViolation.getRule().getName();
>>>>                     l_message = ruleViolation.getDescription();
>>>>                     l_beginLine = ruleViolation.getBeginLine();
>>>>                     l_endLine = ruleViolation.getEndLine();
>>>>
>>>>                 }
>>>>
>>>>                 @Override
>>>>                 public void metricAdded(Metric metric) {
>>>>                 }
>>>>             });
>>>>
>>>> Does anyone know how I use the listener or my
>>>> ruleViolationDetailExtractor method outside of this context such that I can
>>>> populate a bean with the specified details (l_rule, l_message, etc.)?
>>>>
>>>> Should I return the variables? Put my bean updating in the listener
>>>> (that seems very wrong)?
>>>>
>>>> Thank you.
>>>>
>>>> --
>>>> Caleb Knox
>>>>
>>>> Endeveran
>>>>
>>>> ------------------------------------------------------------
>>>> ------------------
>>>> Check out the vibrant tech community on one of the world's most
>>>> engaging tech sites, Slashdot.org! http://sdm.link/slashdot
>>>> _______________________________________________
>>>> Pmd-devel mailing list
>>>> Pmd...@li...
>>>> https://lists.sourceforge.net/lists/listinfo/pmd-devel
>>>>
>>>>
>>>
>>
>>
>> --
>> Caleb Knox
>>
>> Endeveran
>>
>
>