[rules-users] "collection contains" problem

[Mike Dougherty]

Hello Edson,

Thanks for your reply. Yes, I believe I am implementing equals and hashCode property in the StatusProperty class:

    @Override
    public boolean equals(Object o) {
        if (this == o) return true;
        if (o == null || getClass() != o.getClass()) return false;

        StatusProperty that = (StatusProperty) o;

        if (!name.equals(that.name)) return false;
        if (!value.equals(that.value)) return false;

        return true;
    }

    @Override
    public int hashCode() {
        int result = name.hashCode();
        result ^= value.hashCode();
        return result;
    }

Yes, I am adding to the properties list before inserting the TrainingStatus object into the session. 

Also, I tried changing from update to modify and it broke the rule. I'm using Drool 5.0.1, has that method been deprecated?


Thanks again for your help,
Mike



On Jan 24, 2010, at 6:36 AM, Edson Tirelli wrote:

> 
>    The rule looks fine, so the problem is somewhere else: 
> 
> * Are you properly implementing the equals()/hashcode() methods in the StatusProperty class? or are you checking for the exact same instance (in which case the defaul equals()/hashcode() would work)
> 
> * are you adding the properties to the TrainingStatus.properties collection before inserting TrainingStatus into the session? or are you calling update/modify after inserting it?
> 
>     Also, not related to the problem at all, but we recommend you to use modify() instead of update() in your rules consequence:
> 
> modify( t ) {
>     setNextSection( "section_b" ),
>     setPreviousSection( "section_overview" )
> }
>   
>    []s
>    Edson
> 
> 
> 2010/1/24 Mike Dougherty <mjdougherty at gmail.com>
> Hello all,
> 
> I've been getting more and more into using Drools Expert, but I certainly still have a lot to learn. I have almost a complete rules file ready to go. But I am having an issue with the LHS checking that a collection contains a certain object value. The rule currently looks like this:
> 
> rule "Rule Name"
> no-loop true
>        when
>            property : StatusProperty( name == "propertyName", value == "valueA" )
>            t : TrainingStatus( currentSection == "section_a", properties contains property )
>        then
>            t.setNextSection( "section_b" );
>            t.setPreviousSection( "section_overview" );
>            update( t );
> end
> 
> 
> The TrainingStatus object has the following fields:
> 
> public class TrainingStatus implements Serializable {
>        private String currentSection;
>        private String nextSection;
>        private String previousSection;
>        private Set<StatusProperty> properties = new TreeSet<StatusProperty>();
> }
> 
> And the StatusProperty object has the following:
> 
> public class StatusProperty implements Serializable {
>        private String name;
>        private String value;
> }
> 
> 
> I'm inserting a TrainingStatus object that should match this rule. But the resulting object does not have the nextSection and previousSection fields set to the values they should.
> 
> Can anyone advise on how to get this rule working as expected?
> 
> Thanks for your help!
> 
> Mike
> 
> 
> _______________________________________________
> rules-users mailing list
> rules-users at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/rules-users
> 
> 
> 
> -- 
>  Edson Tirelli
>  JBoss Drools Core Development
>  JBoss by Red Hat @ www.jboss.com
> _______________________________________________
> rules-users mailing list
> rules-users at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/rules-users

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.jboss.org/pipermail/rules-users/attachments/20100125/df5f4f0b/attachment.html