Sugestão para TACBrTXTClass.LFill

[Renan Eustaquio]

Essa função de function TACBrTXTClass.LFill(const Value: String;
                             Size: Integer = 0;
                             Nulo: Boolean = false;
                             Caracter: Char = '0'): String;

poderia ter um incremento   Value := StringReplace(Result, FDelimitador,'', [rfReplaceAll])   , pois acaba que se o value passado possuir o delimitador gera erro no arquivo.
exemplo spedfiscal se tiver no nome do produto um "|" ele gera um erro grotesco no sped, para eu corrigir isso em todos os lugares q são texto tenho que usar o StringReplace.

isso poderia ser feito ?

 

function TACBrTXTClass.LFill(const Value: String;
                             Size: Integer = 0;
                             Nulo: Boolean = false;
                             Caracter: Char = '0'): String;
begin
  if (Nulo) and (Length(Value) = 0) then
  begin
     Result := FDelimitador;
     Exit;
  end;

  /// Se a propriedade TrimString = true, Result retorna sem espaços em branco
  /// iniciais e finais.
  if FTrimString then
     Result := Trim(Result);

  Value := StringReplace(Value, FDelimitador,'', [rfReplaceAll])     
     
  if (Size > 0) and (Length(Value) > Size) then
     Result := Copy(Value, 1, Size)
  else
     Result := StringOfChar(Caracter, Size - length(Value)) + Value;

  Result := FDelimitador + Result;
end;

Editado 11 Setembro, 2020 por Renan Eustaquio

[EMBarbosa]

  Em 11/09/2020 at 16:40, Renan Eustaquio disse:

Essa função de function TACBrTXTClass.LFill(const Value: String;
                             Size: Integer = 0;
                             Nulo: Boolean = false;
                             Caracter: Char = '0'): String;

poderia ter um incremento   Value := StringReplace(Result, FDelimitador,'', [rfReplaceAll])   , pois acaba que se o value passado possuir o delimitador gera erro no arquivo.
exemplo spedfiscal se tiver no nome do produto um "|" ele gera um erro grotesco no sped, para eu corrigir isso em todos os lugares q são texto tenho que usar o StringReplace.

 

Expandir  

Olá

Obrigado pela análise. Realmente existe essa limitação na função.

  Em 11/09/2020 at 16:40, Renan Eustaquio disse:

isso poderia ser feito ?

 

function TACBrTXTClass.LFill(const Value: String;
                             Size: Integer = 0;
                             Nulo: Boolean = false;
                             Caracter: Char = '0'): String;
begin
  if (Nulo) and (Length(Value) = 0) then
  begin
     Result := FDelimitador;
     Exit;
  end;

  /// Se a propriedade TrimString = true, Result retorna sem espaços em branco
  /// iniciais e finais.
  if FTrimString then
     Result := Trim(Result);

  Value := StringReplace(Value, FDelimitador,'', [rfReplaceAll])     
     
  if (Size > 0) and (Length(Value) > Size) then
     Result := Copy(Value, 1, Size)
  else
     Result := StringOfChar(Caracter, Size - length(Value)) + Value;

  Result := FDelimitador + Result;
end;

Expandir  

Na minha opinião, não seria interessante, porque vai incluir uma responsabilidade que não pertence a essa função. O certo seria tratar o nome do produto antes de enviar ao componente.

Mas vamos aguardar outros desenvolvedores do ACBr pra ver se concordam com isso.

[Juliomar Marchetti]

O componente serve para gerar o arquivo os dados devem ser tratados pelo desenvolvedor antes de alimentar o componente

[Daniel Simoes]

Sempre achei uma má escolha do Sped, usar Pipes como delimitadores de campo...

Mas realmente isso deve ser tratado do lado da aplicação... afinal poderíamos maquiar erros ou modificar textos, sem a permissao do usuario...

[BigWings]

O ideal seria o validador aceitar o caractere "|", talvez com um caractere de escape...

Eu até acho que pode ser feita a implementação, mas deve haver uma forma de permitir que o desenvolvedor decida o que fazer com o caractere, e não apenas remover, pra isso seria preciso nova ou novas propriedades.

Seria um adendo à TrimString que já está sanitizando a entrada.

PS: Dividi o tópico do anterior.

[Renan Eustaquio]

  Em 18/09/2020 at 12:54, BigWings disse:

O ideal seria o validador aceitar o caractere "|", talvez com um caractere de escape...

Eu até acho que pode ser feita a implementação, mas deve haver uma forma de permitir que o desenvolvedor decida o que fazer com o caractere, e não apenas remover, pra isso seria preciso nova ou novas propriedades.

Seria um adendo à TrimString que já está sanitizando a entrada.

PS: Dividi o tópico do anterior.

Expandir  

Concordo com o bigwiggs , Imagina vc ter que tratar tudo em todos lugares pra remover uma coisa que vai gerar erro, mesmo que a responsabilidade seja do desenvolvedor que opção ele vai ter alem de remover o caracter ??

[EMBarbosa]

  Em 21/09/2020 at 13:11, Renan Eustaquio disse:

Concordo com o bigwiggs

Expandir  

Conforme o que o BigWings disse:

  Em 18/09/2020 at 12:54, BigWings disse:

Eu até acho que pode ser feita a implementação, mas deve haver uma forma de permitir que o desenvolvedor decida o que fazer com o caractere, e não apenas remover, pra isso seria preciso nova ou novas propriedades.

Expandir  

Nesse caso, seria necessário implementar essa nova ou novas propriedades na classe para poder tratar essa situação, lembrando de manter o comportamento atual como padrão.

[Renan Eustaquio]

posso fazer ?

Editado 28 Setembro, 2020 por Renan Eustaquio

[EMBarbosa]

  Em 28/09/2020 at 13:05, Renan Eustaquio disse:

posso fazer ?

Expandir  

Pode sim.

[Renan Eustaquio]

 

Segue anexo alteração , fiz algumas alterações na criação do SpedFiscal no set dos campos , pois os mesmos da forma que estavam não aplicavam para os registros filhos (me embasei no spedPisConfins)

ACBrSpedPisCofins.pasFetching info... ACBrSpedFiscal.pasFetching info... ACBrTXTClass.pasFetching info...

[Juliomar Marchetti]

Já está no backlog para avaliação

[Juliomar Marchetti]

No svn. validei com lazarus e delphi, também rodei os exemplos

só falta uma coisa analisar os demais que usam e propagar essa mudança pois só tem para fiscal e pis cofins

[Renan Eustaquio]

vc mesmo vai fazer isso ou precisa que eu faça algo?

[Juliomar Marchetti]

  Em 17/11/2020 at 14:32, Renan Eustaquio disse:

vc mesmo vai fazer isso ou precisa que eu faça algo?

Expandir  

Se quiser ajudar a fazer pois é bastante componentes que fazem uso. será de uma grande ajuda

[Renan Eustaquio]

Realizado a implementações nas outras classes, acredito ter feito em todas.
Aproveitei e acertei no create o default de algumas variáveis que não estavam sendo passadas de forma correta.

Espero que esteja tudo correto , caso necessário é só da o grito ai que revejo o erro.

ACBrLFD.pasFetching info... ACBrPonto.pasFetching info... ACBrSEF2.pasFetching info... ACBrSpedContabil.pasFetching info... ACBrSpedECF.pasFetching info... ACBrSpedFiscal.pasFetching info... ACBrSpedPisCofins.pasFetching info... ACBrTXTClass.pasFetching info... UACBrLCDPR.pasFetching info... ACBrADRCST.pasFetching info... ACBrDeSTDA.pasFetching info...

[Renan Eustaquio]

Desculpe incomodar , tem previsão de por no svn?

[Juliomar Marchetti]

Feito no svn

vou fechar o tópico . obrigado