BpmnAutoLayout时提示报错Comparison method violates its general contract!

问题描述

基于activiti开发的工作流服务，在处理自动布局时，偶尔出现Comparison method violates its general contract!的报错信息。

在 JDK7 版本以上，Comparator 要满足自反性，传递性，对称性，不然 Arrays.sort，Collections.sort会报异常。具体可以搜索相关文档了解，不在此展开了。

定位问题

项目使用的版本如下

<groupId>org.activiti</groupId>
<artifactId>activiti-spring-boot-starter-actuator</artifactId>
<version>5.22.0</version>

//入口处
new BpmnAutoLayout(bpmnModel).execute();

根据报错信息找到类mxCoordinateAssignment

继续看WeightedCellSorter这个内部类，它实现了 Comparable，但是重写compareTo中没有返回相等的情况，所以会导致违反了上面提到的自反性，传递性，对称性。

public int compareTo(Object arg0)
		{
			if (arg0 instanceof WeightedCellSorter)
			{
				if (weightedValue > ((WeightedCellSorter) arg0).weightedValue)
				{
					return -1;
				}
				else if (weightedValue < ((WeightedCellSorter) arg0).weightedValue)
				{
					return 1;
				}
				else
				{
					if (nudge)
					{
						return -1;
					}
					else
					{
						return 1;
					}
				}
			}
			else
			{
				return 0;
			}
		}

解决问题

方案一

大部分网上的答案是针对Comparison method violates its general contract!这类报错，设置 java.util.Arrays.useLegacyMergeSort = true。
但是源码中是一个静态变量，无法通过一般配置来改变。

只能在加在启动脚本中，-Djava.util.Arrays.useLegacyMergeSort=true
这样就涉及改各个环境的服务启动服务脚本，不太方便，也不清楚会不会因为选择新排序方式导致其他问题。

方案二

从根本解决该问题。根本问题就是WeightedCellSorter的重写compareTo不完善，这个问题我觉得作者应该后续升级相关jar处理过。
从：Maven Repository官网 查询jgraphx的相关版本。

        <dependency>
            <groupId>org.tinyjee.jgraphx</groupId>
            <artifactId>jgraphx</artifactId>
            <version>2.3.0.5</version>
        </dependency>

升级后确实解决了排序问题，但是又出现mxEdgeStyle.orthConnector首字母大小写问题，不知道为啥会出现。。。无奈下只能自己定义个BpmnAutoLayout类，将其中问题代码mxEdgeStyle.orthConnector替换为mxEdgeStyle.OrthConnector。

最后

如果大家有更好的建议，可以告诉我哈！