How about use toString() to print log ?

[HJ1AN]

For example:

logEngineParameters() in class DefaultRulesEngine

private void logEngineParameters() {
    LOGGER.info("Rule priority threshold: {}",
        parameters.getPriorityThreshold());
    LOGGER.info("Skip on first applied rule: {}",
        parameters.isSkipOnFirstAppliedRule());
    LOGGER.info("Skip on first non triggered rule: {}",
        parameters.isSkipOnFirstNonTriggeredRule());
    LOGGER.info("Skip on first failed rule: {}",
        parameters.isSkipOnFirstFailedRule());
    LOGGER.info("Skip on missing fact: {}",
        parameters.isSkipOnMissingFact());
}

It just print all properties of RulesEngineParameters, so maybe override toString() is briefer, like:

@Override
public String toString() {
    return "RulesEngineParameters{" +
            "skipOnFirstAppliedRule=" + skipOnFirstAppliedRule +
            ", skipOnFirstNonTriggeredRule=" + skipOnFirstNonTriggeredRule +
            ", skipOnFirstFailedRule=" + skipOnFirstFailedRule +
            ", skipOnMissingFact=" + skipOnMissingFact +
            ", priorityThreshold=" + priorityThreshold +
            '}';
}

and use it as:

LOGGER.info(parameters.toString());

the toString() above is create by IDEA, we can set another format. The same way to Rules and Facts.

What do you think about @benas ?

[fmbenhassine]

Yes, good idea.

You can do it on the master branch without a PR.

Thank you!

[HJ1AN]

Ok, I will finish it later.

[fmbenhassine]

@wg1j This is done. Thanks!