#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.