PostIdのソート順の仕様を変更

diff --git a/OpenTween.Tests/Models/PostIdTest.cs b/OpenTween.Tests/Models/PostIdTest.cs
index 33f6702..e4a9060 100644 (file)
--- a/OpenTween.Tests/Models/PostIdTest.cs
+++ b/OpenTween.Tests/Models/PostIdTest.cs
@@ -60,6 +60,15 @@ namespace OpenTween.Models
         }
 
         [Fact]
+        public void CompareTo_IdLengthTest()
+        {
+            var a = this.CreatePostId("twitter", "200");
+            var b = this.CreatePostId("twitter", "1000");
+            Assert.True(a.CompareTo(b) < 0);
+            Assert.True(b.CompareTo(a) > 0);
+        }
+
+        [Fact]
         public void OperatorGreaterThan_Test()
         {
             var a = this.CreatePostId("twitter", "100");

diff --git a/OpenTween/Models/PostId.cs b/OpenTween/Models/PostId.cs
index da60a81..3cd81d7 100644 (file)
--- a/OpenTween/Models/PostId.cs
+++ b/OpenTween/Models/PostId.cs
@@ -44,6 +44,11 @@ namespace OpenTween.Models
             if (compareByIdType != 0)
                 return compareByIdType;
 
+            // 辞書順による比較のみだと "20" > "100" となってしまうため文字数による比較も加える
+            var compareByIdLength = this.Id.Length.CompareTo(other.Id.Length);
+            if (compareByIdLength != 0)
+                return compareByIdLength;
+
             return this.Id.CompareTo(other.Id);
         }