Merge pull request #1572 from AgnaldoSilva0/Branch_21158

Padronizado métodos de arredondamento nas classes DFe e NFe
ZeusAutomacao · Jan 10, 2025 · 290b9d0 · 290b9d0
2 parents 8881250 + dbaf397
commit 290b9d0
Show file tree

Hide file tree

Showing 4 changed files with 51 additions and 7 deletions.
diff --git a/DFe.Classes/Valor.cs b/DFe.Classes/Valor.cs
@@ -1,14 +1,15 @@
-using System.Globalization;
+using System;
+using System.Globalization;
 
 namespace DFe.Classes
 {
     public static class Valor
     {
         public static decimal Arredondar(this decimal valor, int casasDecimais)
         {
-            var valorNovo = decimal.Round(valor, casasDecimais);
-            var valorNovoStr = valorNovo.ToString("F" + casasDecimais, CultureInfo.CurrentCulture);
-            return decimal.Parse(valorNovoStr);
+            var valorArredondado = decimal.Round(valor, casasDecimais, MidpointRounding.ToEven);
+            var valorArredondadoFormatado = valorArredondado.ToString("F" + casasDecimais, CultureInfo.CurrentCulture);
+            return decimal.Parse(valorArredondadoFormatado);
         }
 
         public static decimal? Arredondar(this decimal? valor, int casasDecimais)

diff --git a/DFe.Testes/Valores/DadosDeTeste/ValorDadosDeTeste.cs b/DFe.Testes/Valores/DadosDeTeste/ValorDadosDeTeste.cs
@@ -0,0 +1,23 @@
+using System.Collections.Generic;
+
+namespace DFe.Testes.Valores.DadosDeTeste
+{
+    public class ValorDadosDeTeste
+    {
+        public static IEnumerable<object[]> ObterValoresDecimaisParaArredondar()
+        {
+            return new List<object[]>
+            {
+                new object[] { 20.35m * 15.90m, 2 },
+                new object[] { 0.35m * 15.90m, 2 },
+                new object[] { 3.665m, 2 },
+                new object[] { 4.775m, 2 },
+
+                new object[] { 20.35m * 15.90m, 3 },
+                new object[] { 0.35m * 15.90m, 3 },
+                new object[] { 4.77575m, 4 },
+                new object[] { 5.445545m, 5 },
+            };
+        }
+    }
+}
diff --git a/DFe.Testes/Valores/ValorTesteUnitario.cs b/DFe.Testes/Valores/ValorTesteUnitario.cs
@@ -0,0 +1,20 @@
+using DFe.Testes.Valores.DadosDeTeste;
+using Microsoft.VisualStudio.TestTools.UnitTesting;
+using NFe.Classes;
+
+namespace DFe.Testes.Valores
+{
+    [TestClass]
+    public class ValorTesteUnitario
+    {
+        [TestMethod(displayName: "Dado um valor e uma quantidade de casas decimais, quando o arredondamento for feito utilizando os métodos da DFe e NFe, então o valor arredondado deve ser igual em ambos os casos")]
+        [DynamicData(nameof(ValorDadosDeTeste.ObterValoresDecimaisParaArredondar), typeof(ValorDadosDeTeste), DynamicDataSourceType.Method)]
+        public void DadoUmValorEUmaQuantidadeDeCasasDecimaisQuandoOArredondamentoForFeitoUtilizandoOsMetodosDaDfeENfeEntaoOValorArredondadoDeveSerIgualEmAmbosOsCasos(decimal valor, int casasDecimais)
+        {
+            var valorArredondadoDfe = Classes.Valor.Arredondar(valor, casasDecimais);
+            var valorArredondadoNfe = Valor.Arredondar(valor, casasDecimais);
+
+            Assert.AreEqual(valorArredondadoDfe, valorArredondadoNfe);
+        }
+    }
+}
diff --git a/NFe.Classes/Valor.cs b/NFe.Classes/Valor.cs
@@ -7,9 +7,9 @@ public static class Valor
     {
         public static decimal Arredondar(this decimal valor, int casasDecimais)
         {
-            var valorNovo = decimal.Round(valor, casasDecimais, MidpointRounding.AwayFromZero);
-            var valorNovoStr = valorNovo.ToString("F" + casasDecimais, CultureInfo.CurrentCulture);
-            return decimal.Parse(valorNovoStr);
+            var valorArredondado = decimal.Round(valor, casasDecimais, MidpointRounding.ToEven);
+            var valorArredondadoFormatado = valorArredondado.ToString("F" + casasDecimais, CultureInfo.CurrentCulture);
+            return decimal.Parse(valorArredondadoFormatado);
         }
 
         public static decimal? Arredondar(this decimal? valor, int casasDecimais)