Support full long range for Ukraininan ToWords

Humanizr · Sep 5, 2019 · 29a6f8d · 29a6f8d
1 parent 465f885
commit 29a6f8d
Show file tree

Hide file tree

Showing 2 changed files with 39 additions and 20 deletions.
diff --git a/src/Humanizer.Tests.Shared/Localisation/uk-UA/NumberToWordsTests.cs b/src/Humanizer.Tests.Shared/Localisation/uk-UA/NumberToWordsTests.cs
@@ -59,7 +59,29 @@ public class NumberToWordsTests
         [InlineData(751633617, "сімсот п'ятдесят один мільйон шістсот тридцять три тисячі шістсот сімнадцять")]
         [InlineData(1111111118, "один мільярд сто одинадцять мільйонів сто одинадцять тисяч сто вісімнадцять")]
         [InlineData(-751633617, "мінус сімсот п'ятдесят один мільйон шістсот тридцять три тисячі шістсот сімнадцять")]
-        public void ToWords(int number, string expected)
+        [InlineData(1_000_000_000_000, "один трильйон")]
+        [InlineData(3_000_000_000_000, "три трильйона")]
+        [InlineData(5_000_000_000_000, "п'ять трильйонів")]
+        [InlineData(999_000_000_000_000, "дев'ятсот дев'яносто дев'ять трильйонів")]
+        [InlineData(
+            long.MaxValue,
+            "дев'ять квінтильйонів " +
+            "двісті двадцять три квадрильйона " +
+            "триста сімдесят два трильйона " +
+            "тридцять шість мільярдів " +
+            "вісімсот п'ятдесят чотири мільйона " +
+            "сімсот сімдесят п'ять тисяч " +
+            "вісімсот сім")]
+        [InlineData(
+            long.MinValue,
+            "мінус дев'ять квінтильйонів " +
+            "двісті двадцять три квадрильйона " +
+            "триста сімдесят два трильйона " +
+            "тридцять шість мільярдів " +
+            "вісімсот п'ятдесят чотири мільйона " +
+            "сімсот сімдесят п'ять тисяч " +
+            "вісімсот вісім")]
+        public void ToWords(long number, string expected)
         {
             Assert.Equal(expected, number.ToWords());
         }

diff --git a/src/Humanizer/Localisation/NumberToWords/UkrainianNumberToWordsConverter.cs b/src/Humanizer/Localisation/NumberToWords/UkrainianNumberToWordsConverter.cs
@@ -16,31 +16,28 @@ internal class UkrainianNumberToWordsConverter : GenderedNumberToWordsConverter
 
         public override string Convert(long input, GrammaticalGender gender)
         {
-            if (input > Int32.MaxValue || input < Int32.MinValue)
-            {
-                throw new NotImplementedException();
-            }
-            var number = (int)input;
-            if (number == 0)
+            if (input == 0)
             {
                 return "нуль";
             }
 
             var parts = new List<string>();
 
-            if (number < 0)
+            if (input < 0)
             {
                 parts.Add("мінус");
-                number = -number;
             }
 
-            CollectParts(parts, ref number, 1000000000, GrammaticalGender.Masculine, "мільярд", "мільярда", "мільярдів");
-            CollectParts(parts, ref number, 1000000, GrammaticalGender.Masculine, "мільйон", "мільйона", "мільйонів");
-            CollectParts(parts, ref number, 1000, GrammaticalGender.Feminine, "тисяча", "тисячі", "тисяч");
+            CollectParts(parts, ref input, 1000000000000000000, GrammaticalGender.Masculine, "квінтильйон", "квінтильйона", "квінтильйонів");
+            CollectParts(parts, ref input, 1000000000000000, GrammaticalGender.Masculine, "квадрильйон", "квадрильйона", "квадрильйонів");
+            CollectParts(parts, ref input, 1000000000000, GrammaticalGender.Masculine, "трильйон", "трильйона", "трильйонів");
+            CollectParts(parts, ref input, 1000000000, GrammaticalGender.Masculine, "мільярд", "мільярда", "мільярдів");
+            CollectParts(parts, ref input, 1000000, GrammaticalGender.Masculine, "мільйон", "мільйона", "мільйонів");
+            CollectParts(parts, ref input, 1000, GrammaticalGender.Feminine, "тисяча", "тисячі", "тисяч");
 
-            if (number > 0)
+            if (input > 0)
             {
-                CollectPartsUnderOneThousand(parts, number, gender);
+                CollectPartsUnderOneThousand(parts, input, gender);
             }
 
             return string.Join(" ", parts);
@@ -103,7 +100,7 @@ public override string ConvertToOrdinal(int number, GrammaticalGender gender)
             return string.Join(" ", parts);
         }
 
-        private static void CollectPartsUnderOneThousand(ICollection<string> parts, int number, GrammaticalGender gender)
+        private static void CollectPartsUnderOneThousand(ICollection<string> parts, long number, GrammaticalGender gender)
         {
             if (number >= 100)
             {
@@ -173,15 +170,15 @@ private static string GetPrefix(int number)
             return string.Join("", parts);
         }
 
-        private static void CollectParts(ICollection<string> parts, ref int number, int divisor, GrammaticalGender gender, params string[] forms)
+        private static void CollectParts(ICollection<string> parts, ref long number, long divisor, GrammaticalGender gender, params string[] forms)
         {
-            if (number < divisor)
+            var result = Math.Abs(number / divisor);
+            if (result == 0)
             {
                 return;
             }
 
-            var result = number / divisor;
-            number %= divisor;
+            number = Math.Abs(number % divisor);
 
             CollectPartsUnderOneThousand(parts, result, gender);
             parts.Add(ChooseOneForGrammaticalNumber(result, forms));
@@ -229,7 +226,7 @@ private static int GetIndex(RussianGrammaticalNumber number)
             return 2;
         }
 
-        private static string ChooseOneForGrammaticalNumber(int number, string[] forms)
+        private static string ChooseOneForGrammaticalNumber(long number, string[] forms)
         {
             return forms[GetIndex(RussianGrammaticalNumberDetector.Detect(number))];
         }