Função "FormaPagamentoToDescricao" - Possibilidade de tornar novo parâmetro 'xPag' como opcional

[EliasCesar]

Foi incluído um novo parâmetro chamado 'xPag' na função "FormaPagamentoToDescricao" da pcnConversao.pas. 

A declaração dessa função poderia ter esse parâmetro vazio como default ?  Pois pode quebrar códigos que já usavam ela.

A sugestão seria algo como:

function FormaPagamentoToDescricao(const t: TpcnFormaPagamento; const xPag: String = ''): string;

Verificado que a alteração foi devido ao caso:

Realmente no problema descrito no tópico, o parâmetro se torna necessário, porém podem existir outros momentos em que a função é usada sem a necessidade de indicação do xPag.

[BigWings]

Eu não considero o novo parâmetro opcional, caso se informe o tPag = tpOutros, a descrição é a tag xPag, conforme a NT 2020.006.

Possíveis cenários onde não há essa necessidade pode ser facilmente ajustada na aplicação, informando uma string vazia no parâmetro.

[Daniel Simoes]

Eu já sou mais a favor, de manter a compatibilidade, evitando quebras na compilação...

O que me dizem @moderadores ?

 

[Victor H. Gonzales - Panda]

Boa noite,

é uma faca de 2 gumes isso.

A partir da entrada da NT, você não poderá mais usar o literal "Outros" ao menos não deveria, se não quebrar a aplicação, passará isso despercebido e só descobrirá que houve o problema quando tiver reclamação.

Por outro lado, quebrando a build, você já se prepara para o novo cenário, então quando houver a mudança você já está preparado, sendo um parâmetro obrigatório na NT.

Ou fazer um overload do método atual, colocar ele como @deprecated, ao menos vai aparecer no console da build que aquele metodo foi sobreescrito por um novo, e o programador deveria se atualizar para o novo padrão da função.

Acredito que um overload e depreciando (se possível) agradará a todos, e se o programador não ler os log de compilação, ai já é outro cenário, ao menos, foi avisado na criação da build.

function FormaPagamentoToDescricao(const t: string): string; overload; deprecated 'Use FormaPagamentoToDescricao(Indice;Descricao)';
function FormaPagamentoToDescricao(const t: string; const xPag: String): string; overload;

Abraços

[EMBarbosa]

Gostei da ideia do @Victor H. Gonzales - Panda

[BigWings]

Fiz o ajuste conforme a sugestão do @Victor H. Gonzales - Panda, rev 21892.