Comments on: Substituindo parâmetros booleanos por enumeração

By: Erick Sasse

Erick Sasse — Mon, 18 Sep 2006 18:58:31 +0000

José, “assima” é demais mesmo! Hahaha.
É o tipico erro sem explicação, se fosse uma palavra difícil ainda vá lá.

Valeu cara, já corrigi!

By: Jose Kraze

Jose Kraze — Mon, 18 Sep 2006 18:26:15 +0000

Erick sasse wrote:
“No exemplo assima usei …”

Caramba, “assima” é lenha, heim…

By: Erick Sasse

Erick Sasse — Sun, 17 Sep 2006 13:33:55 +0000

Júlio, use o bom senso, não leve ao extremo.

Não é para criar novas procedures para procedures já existentes, a dica é para quando for criar suas procedures que usam booleanos usar enumerações.

Lógicamente não vamos ficar traduzindo funções já prontas, isso não se justificaria.

By: Júlio Cascalles

Júlio Cascalles — Sun, 17 Sep 2006 12:48:55 +0000

O código do Daniel é bem melhor mesmo:

…Senão teríamos que criar uma nova procedure com enumeração para toda função que usa booleanos – só para ficar mais legível.

Declarar constantes booleanas ajuda muito na legibilidade sem perder tempo inchando o código com novas procedures desnecessárias.

By: Erick Sasse

Erick Sasse — Tue, 12 Sep 2006 18:39:52 +0000

Boa observação, porém no meu ponto de vista, a utilização de expressões booleanas é muito menor se comparada ao uso dos valores diretos, e o código continua mais difícil de ler, pois você tem que traduzir a expressão para entender como a função está sendo chamada.

No caso do seu exemplo com a variável UsuarioEscolheuSobrescrever nem tanto, mas nesse caso dependemos do desenvolvedor usar bons nomes de variáveis, o que muitas vezes não acontece.

Por isso eu ainda acho que é mais interessante as enumerações do que os booleanos.

By: Daniel Maltarolli

Daniel Maltarolli — Tue, 12 Sep 2006 12:48:26 +0000

Essa é uma boa dica. Só é chato porque impede o uso de expressões boleanas para passar os parâmetros, como por exemplo:

CopyFile(‘arq1.txt’, ‘arq2.txt’, (ExtractFileExt(‘arq2.txt’) = ‘.xyz’) or UsuarioEscolheuSobrescrever);

Um alternativa é criar constantes de valor boleano no escopo público da função:

const
cmFailIfExists = False;
cmOverwrite = True;

Assim você tem o melhor dos dois mundos: flexibilidade e legibilidade!