Variáveis globais, locais e arquivos de Log
comentada em 15/03/2020 22:35

Seria interessante saber em qual linguagem vc está programando e como vc está implementando as suas funções. Vc comentou:  "criei um arquivo de log como uma variável global que passo para essas funções para registrar tudo o que o programa está fazendo".

Se vc passa essa variável como parâmetro dessas funções ela não precisaria ser global.

Geralmente eu tento evitar de usar variáveis globais. Tem várias maneiras de fazer isso e as soluções dependem do paradigma que vc está trabalhando. Se vc trabalha com Orientação à Objetos pode usar um padrão de projeto chamado Singleton, pode usar inversão de dependências, etc.

Só ver quem usa! Var local é usada ali e pronto, estando inclusive protegida de ser alterada por outros modulos/funcoes. Agora se mais de um modulo/funcao a utiliza ela é global.

@Mark.Valley, estou usando Visual Basic e não é orientado a objeto. É só uma aplicação de console que eu pretendo agendar para executar uma vez por dia. O que me interessa é o resultado que está dentro do log.
Vou ver como consigo melhorar meu código. Obrigado pela resposta.

@gcaldeir  Pelo que entendi a sua instância do arquivo de LOG está nessa variável global, é isso? E quando precisa você utiliza esse objeto para escrever o seu LOG.

Eu criaria uma sub / function que escreve no LOG e que recebe os dados que deve escrever como parâmetros e sempre que precisasse gravar um LOG chamaria essa função.

Porém isso irá abrir o arquivo, escrever e fechar a todo momento.

Não sei o volume de info e se o processo é lento, você pode acumular em memória e gravar de tempos em tempos, isso causará um delay para o LOG estar disponivel no arquivo e se tiver um crash no programa pode perder esses registros.

Ai teria que avaliar o que é melhor para o seu caso.

Entendi, vc deve estar usando um esquema procedural.

Uma das coisas que vc pode fazer, se quiser evitar globais, é manter o log como variável local no programa principal e passa-lo como parâmetro para as funções que vc  criou.  Esse é até um exemplo de inversão de dependência: o programa principal que informa para as funções onde elas gerarão os logs.

Se é um programa rodando no console acho que mesmo o uso de variáveis globais não vai ser tão ruim. Só diria para ter cuidado se tiver alguma coisa rodando em múltiplas threads porque aí sim vc pode ter algum comportamento inesperado.

Eu evito ao máximo usar variáveis globais. Em situações assim eu tento encapsular o máximo possível de código em pequenas funções e encadear o retorno de uma função como parâmetro da função seguinte. 

Encapsular cada função do código em funções facilita a manutenção e deixa o código mais organizado. 

Não tou vendo o código que você fez mas consigo imaginar isso tudo sendo feito dentro de funções sem a necessidade de definir variáveis globais.

Inclusive pode haver uma função que recebe os dados para o log e gerar o arquivo log. 

A regra geral é nunca usar variáveis globais, isso é importante principalmente para quem está aprendendo a programar ou está na fase inicial da carreira.

Por dois motivos, primeiro porque é mais fácil programar com globais e isso faz com que o iniciante se torne um programador pior sem usar funções/métodos com passagem de parâmetro corretamente; segundo porque fica mais difícil de outra pessoa dar manutenção no seu código futuramente.

MAS

Com tempo de experiência o programador vai aprendendo que em alguns casos o uso de uma variável global faz com que o programa execute muito mais rápido do que ficar usando funções bem definidas e auto contidas.

E um dos casos é exatamente a escrita em log como é o teu caso...

O código fica mais feio, óbvio, mas acaba ficando mais rápido, ainda mais para volumes grandes de dados, como parece ser o teu caso.

E desconfio que para estar programando em VB, você não está exatamente começando :)

@DaliSalvador, é isso mesmo, o arquivo de log está em uma variável global.
@ThiagoDV, @Roloff, obrigado pelos comentários.

Eu entendi que você está lendo um arquivo de log, interpretando as linhas e gerando um relatório com o que achou.

Minhas sugestões. Elas acabaram ficando meio genéricas. São baseadas no que eu entendi da sua explicação:

- Prefira passar os dados necessários a um função como parâmetros dela. Evite variáveis globais. Mas, nesse caso, tem pouca chance de virar um problema

- Separe o código por responsabilidade: separa a lógica que lê o arquivo, da que interpreta as linhas, da que totaliza RODATNOCes.

- Trate linhas, ao invés do arquivo inteiro: pensa que o arquivo não cabe na memória. Daí tu cria as funções para tratar apenas uma linha. E, retorna o resultado.