#beSomething is quite common pattern (even recommended in Kent's Best Practice Patterns), maybe it should check for name 'be'<Uppercase> before applying the critic?
This is a good proposal. Would be nice to hear other opinions. Maybe we can open an issue on the bug tracker so people can comment.

However I do not really want to ban the critic, because there may other legitimate applications in the method/class; and in Roassal-heavy code it's a nuisance.
So it would require context-aware checks (probably user-defined?) like "if receiver is subclass of RTShape then don't apply this (yourself) rule���.
��
Yes, it���s a very good point. It is in my plans to see how we can allow frameworks to provide their own critics and interact/override the default ones. For now all I can suggest is to ban the critic on the level of the class, or if we will reach the common sense there can be a ban on the level of package tag, so you can keep all the roassal-related classes in one tag and ban the rule for them.

I created issues for both so it doesn't get lost here.

Peter��


On Tue, Jul 28, 2015 at 3:24 PM, stepharo <stepharo@free.fr> wrote:


Le 28/7/15 11:43, Alexandre Bergel a ��crit :
But, what does the useless button do? Does it remove from the rules quality�� assistant uses forever? Or is it per class? the rule "Too many methods" is useless for the class RTexample, but it is useful for other classes

Alex
I was confused too :)
I guess that it does nothing. Just report to the server of yuriy that you think that this rule is useless.
So there is a clear problem at the UI level. The user cannot disntinguish what he is doing:
�� �� marking false positive
�� �� or giving feedback in general.

Stef

Alexandre



Le 27 juil. 2015 �� 12:53, Yuriy Tymchuk <yuriy.tymchuk@me.com> a ��crit :

This is sad, and shows that I completely failed with that button. Because ideally you had to press it so I can know that this rule had a useful case for somebody.