From 7b4e653723fe8cab56b40c87e738f558ed71436b Mon Sep 17 00:00:00 2001 From: Zafar Khaja Date: Tue, 27 Nov 2012 22:00:08 +0400 Subject: [PATCH] Refactor previous commit; remove duplications --- .../com/github/zafarkhaja/semver/Version.java | 27 +++++++------------ 1 file changed, 9 insertions(+), 18 deletions(-) diff --git a/src/main/java/com/github/zafarkhaja/semver/Version.java b/src/main/java/com/github/zafarkhaja/semver/Version.java index 06a998d..54da22e 100644 --- a/src/main/java/com/github/zafarkhaja/semver/Version.java +++ b/src/main/java/com/github/zafarkhaja/semver/Version.java @@ -173,40 +173,31 @@ public class Version implements Comparable { } private int comparePreReleaseVersions(Version other) { - int result; - if (preReleaseVersion == null && - other.getPreReleaseVersion() == null) { - result = 0; - } else if (preReleaseVersion == null || - other.getPreReleaseVersion() == null) { - result = preReleaseVersion == null ? 1 : -1; + if (preReleaseVersion == null ^ other.getPreReleaseVersion() == null) { + return preReleaseVersion == null ? 1 : -1; } else { - result = compareAlphaNumericVersions( + return compareAlphaNumericVersions( preReleaseVersion, other.getPreReleaseVersion() ); } - return result; } private int compareBuildVersions(Version other) { - int result; - if (buildVersion == null && - other.getBuildVersion()== null) { - result = 0; - } else if (buildVersion == null || - other.getBuildVersion()== null) { - result = buildVersion == null ? -1 : 1; + if (buildVersion == null ^ other.getBuildVersion() == null) { + return buildVersion == null ? -1 : 1; } else { - result = compareAlphaNumericVersions( + return compareAlphaNumericVersions( buildVersion, other.getBuildVersion() ); } - return result; } private int compareAlphaNumericVersions(String thisOp, String otherOp) { + if (thisOp == null && otherOp == null) { + return 0; + } String[] thisIdents = thisOp.split("\\."); String[] otherIdents = otherOp.split("\\.");