Clayton Alves

Aparentemente como zip vai OK. ACBrTEFAPIElginComum.zip

A implementação do TEF Elgin possui alguns vazamentos de memória. A unit corrigida pode ser obtida nesse link. Não anexei a unit pois o fórum está me retornando erro quando eu tento anexar:

@Daniel Simoes @WINDEL Por acaso vocês tiveram retorno da Setis sobre esta questão da imagem no Pinpad ?

@Cleber Ferreira sim, pode fechar... achei que marcando a resposta como solucionada já fechava automaticamente.

Pergunta meio boba mas não achei em lugar nenhum. Qual o significado de "CD" no nome do componente ACBrPIXCD ?

Boa tarde @Pedro Frayman, Muitos dos meus clientes utilizam a Verifone C680. Alguns com a Stone. Outros com a cooperativa Sicredi. Então pensei que era possível integrar meu sistema com essas maquininhas utilizando o ACBrPOSTEF. Sabe me dizer se existe documentação disponível para esses outros equipamentos? sem ser SmartPOS. Fiz uma busca na internet mas não achei nada.

Existem outras "maquininhas" que suportam integração POSTEF além da PAX S920 ? Vi algumas empresas utilizando integração do sistema comercial com a Verifone C680 e também com a Q92. Essas máquinas são compatíveis com o ACBrPOSTEF ? Existem outros modelos compatíveis ?

Especificamente no caso do TMedCollection não teriam memory leak já que o TMedCollection é uma TObjectList que é dona de seus itens, ou seja, os itens serão liberados junto com a coleção no momento da liberação. Por fim o trecho abaixo resolveu meu problema de AV, mas é um "workaround" e acredito que a não duplicação desses itens possa ocasionar problemas no futuro. FProd.Assign(ItemXML.Prod); // Ajuste técnico FProd.med.OwnsObjects := False; FProd.med.Clear; FProd.med.OwnsObjects := True; Agradeço o tempo de vocês.

Mas só pra deixar claro, meu problema nem é com a TDetCollection (apesar de para mim estar claro que existe um problema na cópia) e sim com a classe TMedCollection que não implementa o Assign de modo a criar cópias do TMedCollectionItem.

@BigWings sim, o TDetCollection dispara o AV da mesma forma. Veja o projeto de exemplo em anexo. Na minha interpretação, a semântica do método Assign é de Copiar e não de Mover. Conforme consta na documentação do Freepascal: "Assign copies the contents of Source to Self, if the classes of the destination and source classes are compatible." E da embarcadero: "Copies the contents of another, similar object." Sim, as documentações falam do método Assign da classe TPersistent mas não estou me referindo a implementação e sim a semântica. Exemplo que dispara o AV: program Project1; {$APPTYPE CONSOLE} uses //FastMM4, SysUtils, pcnNFe; var Det1, Det2: TDetCollection; begin Det1 := TDetCollection.Create; Det2 := TDetCollection.Create; Det1.New; Det2.Assign(Det1); // Move algumas referências, quando na verdade deveria copiar. WriteLn('Liberando Det1'); Det1.Free; WriteLn('Liberando Det1'); Det2.Free; // <- Access Violation WriteLn('Fim'); end. Project1.dpr

O que estou sugerindo é que o método Assign seja implementado na classe TMedCollection assim como já foi implementado na classe TpagCollection que aliás segue a mesma hierarquia de herança (TACBrObjectList -> TObjectList -> TList).

Entendo. Mas o exemplo que dei foi bem simplificado propositalmente para deixar mais claro o problema. No caso real, eu não instancio um TMedCollection. Eu tenho uma instancia de TDetCollectionItem, que por sua vez tem uma instância de TProd que por sua vez tem uma instancia de TMedCollection. Neste cenário, utilizar o Assign de TDetCollectionItem com outro TDetCollectionItem e posteriormente realizar o Free irá disparar o AV da mesma forma.

var MedicamentosA: TMedCollection; MedicamentosB: TMedCollection; begin MedicamentosA := TMedCollection.Create; MedicamentosB := TMedCollection.Create; MedicamentosA.New; MedicamentosB.Assign(MedicamentosA); MedicamentosA.Free; MedicamentosB.Free; // <-- Access Violation end;

Sim, TList possui um método Assign que não cria cópias das Instâncias de TMedCollectionItem, causando o problema. Em outras palavras, o Assign do TList não chama o Assign de cada TMedCollectionItem. Realmente não estou com os fontes atualizados, mas basicamente o que está faltando é esta implementação: procedure TMedCollection.Assign(Source: TMedCollection); var I: Integer; begin Self.Clear; for I := 0 to Source.Count - 1 do Self.New.Assign(Source.Items[I]); end;

Ao trabalhar com cópias de TMedCollection utilizando o método Assign da forma que está atualmente obtenho uma Violação de Acesso ao liberar minha cópia. Isso acontece porque não existe implementado um método Assign na classe TMedCollection, o que faz com que a cópia utilize a referência de TMedCollectionItem e não uma cópia da instância. Segue anexo código corrigido. pcnNFe.pas