1
I am making some improvements in a method, which today takes on average 19 to 22s to bring the information to VIEW and saw the possibility of replacing the foreach for Parallel.Foreach.
I changed and ran the code locally and it worked, even dropped to 5.68s the return to VIEW, only when running on the test server the following error:
( That doesn’t happen locally.)
Follow the code with the foreach and without.
Code using foreach
foreach (DataRow dr in myDs.Tables[0].Rows)
{
PedidoFollowup myPedido = new PedidoFollowup()
{
Id = int.Parse(dr["iPedidoCompra"].ToString()),
DataImplantacao = Convert.ToDateTime(dr["dDataPedido"].ToString()),
Ordem = new Ordem()
{
Id = int.Parse(dr["iNumeroOrdem"].ToString()),
DataEntrega = Convert.ToDateTime(dr["dPrevisaoEntrega"].ToString()),
UnidadeDestino = new SEW.Model.UnidadeDestino() { Id = int.Parse(dr["cDestino"].ToString()) },
Pim = new Pim.Model.Pim() { Id = int.Parse(dr["iPim"].ToString()) },
QuantidadeSolicitada = Convert.ToDecimal(dr["dQtde"].ToString()),
Narrativa = dr["cNarrativaOc"].ToString(),
PrecoTotal = Convert.ToDecimal(dr["dPrecoTotal"].ToString()),
PrecoUnitario = Convert.ToDecimal(dr["dPrecoUnitario"].ToString()),
Saldo = Convert.ToDecimal(dr["dSaldo"].ToString()),
Requisitante = new Requisitante() { Id = dr["cRequisitante"].ToString() },
Item = new Item()
{
Id = dr["cItem"].ToString(),
Descricao = dr["cItemDescricao"].ToString()
},
Natureza = int.Parse(dr["iNaturezaOrdem"].ToString()),
VisualizadoFornecedor = (Boolean)dr["lVisualizadoForn"],
DataVisualizadoForn = dr["cDataVisualizacaoForn"].ToString() == "" ? (DateTime?)null : Convert.ToDateTime(dr["cDataVisualizacaoForn"].ToString())
},
Fornecedor = new Fornecedor()
{
Id = int.Parse(dr["iCodEmitente"].ToString()),
NomeAbrev = dr["cNomeAbreEmitente"].ToString(),
Cidade = dr["cCidade"].ToString()
},
Comentario = dr["cComentario"].ToString(),
Prioridade = int.Parse(dr["iPrioridade"].ToString()),
NotasFiscais = new intranetRepository.MFT.Repository.NotaFiscalRepository().ListarNotasFiscaisPorOrdemCompra(new Ordem() { Id = int.Parse(dr["iNumeroOrdem"].ToString()) }),
DivergenciaValor = false,
DivergenciaQuantidade = false,
QuantidadeFaturada = 0,
TotalFaturado = 0,
Situacao = dr["cSituacao"].ToString() != "" ? Convert.ToString(dr["cSituacao"].ToString()) : " ",
OcorrenciaSemVisualizacao = Convert.ToBoolean(dr["lOcorrenciaSemVisualiz"].ToString()),
Coleta = dr["cDataColeta"].ToString() == "" ? null : new ColetaFollowup() { DataColeta = Convert.ToDateTime(dr["cDataColeta"].ToString()) },
TipoEntrega = new TipoEntrega() { Id = int.Parse(dr["iTipoEntrega"].ToString()) }
};
myQtdeTotal = 0;
myValorTotal = 0;
if (myPedido.NotasFiscais != null)
{
foreach (MFT.Model.NotaFiscal myNota in myPedido.NotasFiscais)
{
foreach (MFT.Model.ItemNotaFiscal myItem in myNota.Itens)
{
myValorTotal += myItem.ValorTotal;
myQtdeTotal += myItem.Quantidade;
}
}
myPedido.QuantidadeFaturada = myQtdeTotal;
myPedido.TotalFaturado = myValorTotal;
if (myPedido.Situacao != "A")
{
if (myPedido.Ordem.Natureza == 2)
{
myPedido.DivergenciaValor = myValorTotal != myPedido.Ordem.PrecoTotal;
}
else
{
myPedido.DivergenciaValor = (myValorTotal != myPedido.Ordem.PrecoTotal);
myPedido.DivergenciaQuantidade = myQtdeTotal != myPedido.Ordem.QuantidadeSolicitada;
}
if ((myPedido.DivergenciaValor) || (myPedido.DivergenciaQuantidade))
{
myPedido.Situacao = "DN";
}
else
{
if(myPedido.TipoEntrega.Id != 3)
{
ColetaFollowup myColeta = new ColetaFollowup()
{
Pedido = myPedido,
Usuario = pUsuario,
DataColeta = DateTime.Now,
Situacao = "P"
};
new ColetaFollowupRepository().SolicitarColeta(myColeta);
myPedido.Situacao = "A";
}
}
}
pedidos.Add(myPedido);
}
else
{
if ((myPedido.Ordem.DataEntrega.AddDays(-10) <= DateTime.Now) && (myPedido.OcorrenciaSemVisualizacao == false) && (myPedido.Coleta == null))
{
myPedido.Situacao = "P";
}
pedidos.Add(myPedido);
}
}
Code using Parallel.Foreach
Parallel.ForEach(myDs.Tables[0].Rows.Cast<DataRow>(), dr =>
{
PedidoFollowup myPedido = new PedidoFollowup()
{
Id = int.Parse(dr["iPedidoCompra"].ToString()),
DataImplantacao = Convert.ToDateTime(dr["dDataPedido"].ToString(), CultureInfo.InvariantCulture),
Ordem = new Ordem()
{
Id = int.Parse(dr["iNumeroOrdem"].ToString()),
DataEntrega = Convert.ToDateTime(dr["dPrevisaoEntrega"].ToString(), CultureInfo.InvariantCulture),
UnidadeDestino = new SEW.Model.UnidadeDestino() { Id = int.Parse(dr["cDestino"].ToString()) },
Pim = new Pim.Model.Pim() { Id = int.Parse(dr["iPim"].ToString()) },
QuantidadeSolicitada = Convert.ToDecimal(dr["dQtde"].ToString()),
Narrativa = dr["cNarrativaOc"].ToString(),
PrecoTotal = Convert.ToDecimal(dr["dPrecoTotal"].ToString()),
PrecoUnitario = Convert.ToDecimal(dr["dPrecoUnitario"].ToString()),
Saldo = Convert.ToDecimal(dr["dSaldo"].ToString()),
Requisitante = new Requisitante() { Id = dr["cRequisitante"].ToString() },
Item = new Item()
{
Id = dr["cItem"].ToString(),
Descricao = dr["cItemDescricao"].ToString()
},
Natureza = int.Parse(dr["iNaturezaOrdem"].ToString()),
VisualizadoFornecedor = (Boolean)dr["lVisualizadoForn"],
DataVisualizadoForn = dr["cDataVisualizacaoForn"].ToString() == "" ? (DateTime?)null : Convert.ToDateTime(dr["cDataVisualizacaoForn"].ToString(), CultureInfo.InvariantCulture)
},
Fornecedor = new Fornecedor()
{
Id = int.Parse(dr["iCodEmitente"].ToString()),
NomeAbrev = dr["cNomeAbreEmitente"].ToString(),
Cidade = dr["cCidade"].ToString()
},
Comentario = dr["cComentario"].ToString(),
Prioridade = int.Parse(dr["iPrioridade"].ToString()),
NotasFiscais = new intranetRepository.MFT.Repository.NotaFiscalRepository().ListarNotasFiscaisPorOrdemCompra(new Ordem() { Id = int.Parse(dr["iNumeroOrdem"].ToString()) }),
DivergenciaValor = false,
DivergenciaQuantidade = false,
QuantidadeFaturada = 0,
TotalFaturado = 0,
Situacao = dr["cSituacao"].ToString() != "" ? Convert.ToString(dr["cSituacao"].ToString()) : " ",
OcorrenciaSemVisualizacao = Convert.ToBoolean(dr["lOcorrenciaSemVisualiz"].ToString()),
Coleta = dr["cDataColeta"].ToString() == "" ? null : new ColetaFollowup() { DataColeta = Convert.ToDateTime(dr["cDataColeta"].ToString(), CultureInfo.InvariantCulture) },
TipoEntrega = new TipoEntrega() { Id = int.Parse(dr["iTipoEntrega"].ToString()) }
};
myQtdeTotal = 0;
myValorTotal = 0;
if (myPedido.NotasFiscais != null)
{
foreach (MFT.Model.NotaFiscal myNota in myPedido.NotasFiscais)
{
foreach (MFT.Model.ItemNotaFiscal myItem in myNota.Itens)
{
myValorTotal += myItem.ValorTotal;
myQtdeTotal += myItem.Quantidade;
}
}
myPedido.QuantidadeFaturada = myQtdeTotal;
myPedido.TotalFaturado = myValorTotal;
if (myPedido.Situacao != "A")
{
if (myPedido.Ordem.Natureza == 2)
{
myPedido.DivergenciaValor = myValorTotal != myPedido.Ordem.PrecoTotal;
}
else
{
myPedido.DivergenciaValor = (myValorTotal != myPedido.Ordem.PrecoTotal);
myPedido.DivergenciaQuantidade = myQtdeTotal != myPedido.Ordem.QuantidadeSolicitada;
}
if ((myPedido.DivergenciaValor) || (myPedido.DivergenciaQuantidade))
{
myPedido.Situacao = "DN";
}
else
{
if (myPedido.TipoEntrega.Id != 3)
{
ColetaFollowup myColeta = new ColetaFollowup()
{
Pedido = myPedido,
Usuario = pUsuario,
DataColeta = DateTime.Now,
Situacao = "P"
};
new ColetaFollowupRepository().SolicitarColeta(myColeta);
myPedido.Situacao = "A";
}
}
}
pedidos.Add(myPedido);
}
else
{
if ((myPedido.Ordem.DataEntrega.AddDays(-10) <= DateTime.Now) && (myPedido.OcorrenciaSemVisualizacao == false) && (myPedido.Coleta == null))
{
myPedido.Situacao = "P";
}
pedidos.Add(myPedido);
}
});
Some of your data is incorrectly formatted when converting to Datetime, the database may be different, and the testing one is inconsistent, since the error message accuses the exception in some Convert.Todatetime
– Gustavo Santos
Probably has a string that is getting the date in a formatting, for example:
2018-07-23, and when it goes to the server this string is receiving the date in another format, such as23-07-2018. And due to this different formatting, it fails to convert the string to a date as it does not recognize as a valid date format.– Thiago Magalhães
But locally I point to the same database that the server is pointing to.
– Anderson Apdo de Souza
As @Gustavosantos and Thiago pointed out, this could be the format problem, and this is related to the server. Locally you are using your machine’s configuration format, which may be for example dd/mm/yyyy and the server may be in English and be mm/dd/yyyyyy which may be causing error
– Ricardo Pontual
Locally I am with en and the server, in web.config is with the following configuration : <globalization Culture="en-BR" uiCulture="en" />
– Anderson Apdo de Souza