Merge pull request #848 from Humanizr/full-long-range-ToWords-ru-uk

Support full long range for Russian and Ukraininan ToWords
Humanizr · Sep 6, 2019 · e4a6929 · e4a6929
2 parents b34fdab + 29a6f8d
commit e4a6929
Show file tree

Hide file tree

Showing 4 changed files with 68 additions and 31 deletions.
diff --git a/src/Humanizer.Tests.Shared/Localisation/ru-RU/NumberToWordsTests.cs b/src/Humanizer.Tests.Shared/Localisation/ru-RU/NumberToWordsTests.cs
@@ -5,7 +5,6 @@ namespace Humanizer.Tests.Localisation.ruRU
     [UseCulture("ru-RU")]
     public class NumberToWordsTests
     {
-
         [Theory]
         [InlineData(0, "ноль")]
         [InlineData(1, "один")]
@@ -64,6 +63,28 @@ public class NumberToWordsTests
         [InlineData(4111111118, "четыре миллиарда сто одиннадцать миллионов сто одиннадцать тысяч сто восемнадцать")]
         [InlineData(-751633617, "минус семьсот пятьдесят один миллион шестьсот тридцать три тысячи шестьсот семнадцать")]
         [InlineData(999999999999, "девятьсот девяносто девять миллиардов девятьсот девяносто девять миллионов девятьсот девяносто девять тысяч девятьсот девяносто девять")]
+        [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.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/RussianNumberToWordsConverter.cs b/src/Humanizer/Localisation/NumberToWords/RussianNumberToWordsConverter.cs
@@ -16,11 +16,6 @@ internal class RussianNumberToWordsConverter : GenderedNumberToWordsConverter
 
         public override string Convert(long input, GrammaticalGender gender)
         {
-            if (input > 999999999999 || input < -999999999999)
-            {
-                throw new NotImplementedException();
-            }
-
             if (input == 0)
             { 
                 return "ноль";
@@ -31,9 +26,11 @@ public override string Convert(long input, GrammaticalGender gender)
             if (input < 0)
             {
                 parts.Add("минус");
-                input = -input;
             }
 
+            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, "тысяча", "тысячи", "тысяч");
@@ -174,15 +171,15 @@ private static string GetPrefix(long number)
             return string.Join("", parts);
         }
 
-        private static void CollectParts(ICollection<string> parts, ref long 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));

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))];
         }