Permitir "\n" como separador además de las comas

12meses12katas · Feb 6, 2012 · 271f571 · 271f571
1 parent 2fa8ae4
commit 271f571
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 3 deletions.
diff --git a/diegoguemes/EneroStringCalculator/StringCalculator.cs b/diegoguemes/EneroStringCalculator/StringCalculator.cs
@@ -5,21 +5,23 @@ namespace EneroStringCalculator
 {
     public class StringCalculator
     {
-        private const string SEPARATOR = ",";
+        private const string COMMA_SEPARATOR = ",";
+        private const string NEW_LINE_SEPARATOR = "\n";
 
         public int Add(string numbers)
         {
             if(string.IsNullOrEmpty(numbers))
                 return 0;
             if(!ContainsSeparator(numbers))
                 return int.Parse(numbers);
-            var parsedNumbers = numbers.Split(SEPARATOR.ToCharArray(), StringSplitOptions.RemoveEmptyEntries);
+            var separators = new string[] { COMMA_SEPARATOR, NEW_LINE_SEPARATOR };
+            var parsedNumbers = numbers.Split(separators, StringSplitOptions.RemoveEmptyEntries);
             return parsedNumbers.Sum(n => int.Parse(n));
         }
 
         private bool ContainsSeparator(string numbers)
         {
-            return numbers.Contains(SEPARATOR);
+            return numbers.Contains(COMMA_SEPARATOR) || numbers.Contains(NEW_LINE_SEPARATOR);
         }
     }
 }
diff --git a/diegoguemes/EneroStringCalculator/StringCalculatorTests.cs b/diegoguemes/EneroStringCalculator/StringCalculatorTests.cs
@@ -35,5 +35,11 @@ public void AddWithMultipleNumbers()
         {
             Assert.AreEqual(50, calculator.Add("2,3,5,10,30"));
         }
+
+        [Test]
+        public void AllowAddWithNewLineSeparator()
+        {
+            Assert.AreEqual(10, calculator.Add("2\n3,5"));
+        }
     }
 }