Skip to main content
adicionou 104 caracteres ao conteúdo
Fonte Link
Maniero
  • 484,4mil
  • 94
  • 1,3mil
  • 2,2mil

Ia só comentar mas ficou grande.

Essencialmente não, afinal ele faz muito pouca coisa.

A não ser que você queira aplicar uma série de coisas que se costuma recomendar em termos de arquitetura, mas seria um absurdo fazer isto em um exemplo tão simples. Tudo é contexto, e neste contexto fazer o melhor não é o mais importante. Eu só recomendaria não cometer erros absurdos, não fazer de qualquer jeito. Neste contexto de um código de aprendizado é mais importante cuidar da legibilidade, do estilo de codificação, de estar correto, do que da boa arquitetura, performance, etc.

Claro que tem pequenos detalhes que podem ser mudados, mas provavelmente mais por gosto do que por necessidade. Então eu:

  • padronizaria os espaçamento entre operadores
  • eventualmente daria nomes melhores para as variáveis
  • cuidaria para garantir o fechamento do arquivo mesmo se der erro, que sequer está sendo feito em condições normais
  • faria uma verificação da entrada de dados
  • mudaria pelo menos a escrita na tela de criação do arquivo para o local onde efetivamente cria o arquivo
  • não atribuiria valores para a mesma variável sem necessidade (op = 0)
  • declararia as variáveis mais próximas de onde são usadas
  • talvez usaria um op > 5 no lugar de op >= 6
  • mudaria o op==0 para op <= 0 e mudaria a mensagem para "Os valores devem ser entre 1 e 5."

Tem outras perfumarias que podem ser feitas, mas isto é o mais importante para um caso assim.

Se lembrarPercebeu como ainda está pecando no básico? Não tente fazer perfeito antes de mais alguma coisa, eu editoacertar o fundamento.

Ia só comentar mas ficou grande.

Essencialmente não, afinal ele faz muito pouca coisa.

A não ser que você queira aplicar uma série de coisas que se costuma recomendar em termos de arquitetura, mas seria um absurdo fazer isto em um exemplo tão simples. Tudo é contexto, e neste contexto fazer o melhor não é o mais importante. Eu só recomendaria não cometer erros absurdos. Neste contexto de um código de aprendizado é mais importante cuidar da legibilidade, do estilo de codificação, do que da boa arquitetura, performance, etc.

Claro que tem pequenos detalhes que podem ser mudados, mas provavelmente mais por gosto do que por necessidade. Então eu:

  • padronizaria os espaçamento entre operadores
  • eventualmente daria nomes melhores para as variáveis
  • cuidaria para garantir o fechamento do arquivo mesmo se der erro, que sequer está sendo feito em condições normais
  • faria uma verificação da entrada de dados
  • mudaria pelo menos a escrita na tela de criação do arquivo para o local onde efetivamente cria o arquivo
  • não atribuiria valores para a mesma variável sem necessidade (op = 0)
  • declararia as variáveis mais próximas de onde são usadas
  • talvez usaria um op > 5 no lugar de op >= 6
  • mudaria o op==0 para op <= 0 e mudaria a mensagem para "Os valores devem ser entre 1 e 5."

Tem outras perfumarias que podem ser feitas, mas isto é o mais importante para um caso assim.

Se lembrar de mais alguma coisa, eu edito.

Ia só comentar mas ficou grande.

Essencialmente não, afinal ele faz muito pouca coisa.

A não ser que você queira aplicar uma série de coisas que se costuma recomendar em termos de arquitetura, mas seria um absurdo fazer isto em um exemplo tão simples. Tudo é contexto, e neste contexto fazer o melhor não é o mais importante. Eu só recomendaria não cometer erros absurdos, não fazer de qualquer jeito. Neste contexto de um código de aprendizado é mais importante cuidar da legibilidade, do estilo de codificação, de estar correto, do que da boa arquitetura, performance, etc.

Claro que tem pequenos detalhes que podem ser mudados, mas provavelmente mais por gosto do que por necessidade. Então eu:

  • padronizaria os espaçamento entre operadores
  • eventualmente daria nomes melhores para as variáveis
  • cuidaria para garantir o fechamento do arquivo mesmo se der erro, que sequer está sendo feito em condições normais
  • faria uma verificação da entrada de dados
  • mudaria pelo menos a escrita na tela de criação do arquivo para o local onde efetivamente cria o arquivo
  • não atribuiria valores para a mesma variável sem necessidade (op = 0)
  • declararia as variáveis mais próximas de onde são usadas
  • talvez usaria um op > 5 no lugar de op >= 6
  • mudaria o op==0 para op <= 0 e mudaria a mensagem para "Os valores devem ser entre 1 e 5."

Tem outras perfumarias que podem ser feitas, mas isto é o mais importante para um caso assim.

Percebeu como ainda está pecando no básico? Não tente fazer perfeito antes de acertar o fundamento.

adicionou 52 caracteres ao conteúdo
Fonte Link
Maniero
  • 484,4mil
  • 94
  • 1,3mil
  • 2,2mil

Ia só comentar mas ficou grande.

Essencialmente não, afinal ele faz muito pouca coisa.

A não ser que você queira aplicar uma série de coisas que se costuma recomendar em termos de arquitetura, mas seria um absurdo fazer isto em um exemplo tão simples. Tudo é contexto, e neste contexto fazer o melhor não é o mais importante. Eu só recomendaria não cometer erros absurdos. Neste contexto de um código de aprendizado é mais importante cuidar da legibilidade, do estilo de codificação, do que da boa arquitetura, performance, etc.

Claro que tem pequenos detalhes que podem ser mudados, mas provavelmente mais por gosto do que por necessidade. Então eu:

  • Eu padronizaria os espaçamento entre operadores
  • eventualmente daria nomes melhores para as variáveis
  • cuidaria para garantir o fechamento do arquivo mesmo se der erro, que sequer está sendo feito em condições normais
  • faria uma verificação da entrada de dados
  • mudaria pelo menos a escrita na tela de criação do arquivo para o local onde efetivamente cria o arquivo
  • não atribuiria valores para a mesma variável sem necessidade (op = 0)
  • declararia as variáveis mais próximas de onde são usadas
  • talvez usaria um op > 5 no lugar de op >= 6
  • mudaria o op==0 para op <= 0 e mudaria a mensagem para "Os valores devem ser entre 1 e 5."

temTem outras perfumarias que podem ser feitas, mas isto é o mais importante para um caso assim.

Se lembrar de mais alguma coisa, eu edito.

Ia só comentar mas ficou grande.

Essencialmente não, afinal ele faz muito pouca coisa.

A não ser que você queira aplicar uma série de coisas que se costuma recomendar em termos de arquitetura, mas seria um absurdo fazer isto em um exemplo tão simples. Tudo é contexto, e neste contexto fazer o melhor não é o mais importante. Eu só recomendaria não cometer erros absurdos. Neste contexto de um código de aprendizado é mais importante cuidar da legibilidade, do estilo de codificação, do que da boa arquitetura, performance, etc.

Claro que tem pequenos detalhes que podem ser mudados, mas provavelmente mais por gosto do que por necessidade.

  • Eu padronizaria os espaçamento entre operadores
  • eventualmente daria nomes melhores para as variáveis
  • cuidaria para garantir o fechamento do arquivo mesmo se der erro, que sequer está sendo feito em condições normais
  • faria uma verificação da entrada de dados
  • mudaria pelo menos a escrita na tela de criação do arquivo para o local onde efetivamente cria o arquivo
  • não atribuiria valores para a mesma variável sem necessidade (op = 0)
  • declararia as variáveis mais próximas de onde são usadas
  • talvez usaria um op > 5 no lugar de op >= 6
  • mudaria o op==0 para op <= 0 e mudaria a mensagem para "Os valores devem ser entre 1 e 5."

tem outras perfumarias que podem ser feitas, mas

Se lembrar de mais alguma coisa, eu edito.

Ia só comentar mas ficou grande.

Essencialmente não, afinal ele faz muito pouca coisa.

A não ser que você queira aplicar uma série de coisas que se costuma recomendar em termos de arquitetura, mas seria um absurdo fazer isto em um exemplo tão simples. Tudo é contexto, e neste contexto fazer o melhor não é o mais importante. Eu só recomendaria não cometer erros absurdos. Neste contexto de um código de aprendizado é mais importante cuidar da legibilidade, do estilo de codificação, do que da boa arquitetura, performance, etc.

Claro que tem pequenos detalhes que podem ser mudados, mas provavelmente mais por gosto do que por necessidade. Então eu:

  • padronizaria os espaçamento entre operadores
  • eventualmente daria nomes melhores para as variáveis
  • cuidaria para garantir o fechamento do arquivo mesmo se der erro, que sequer está sendo feito em condições normais
  • faria uma verificação da entrada de dados
  • mudaria pelo menos a escrita na tela de criação do arquivo para o local onde efetivamente cria o arquivo
  • não atribuiria valores para a mesma variável sem necessidade (op = 0)
  • declararia as variáveis mais próximas de onde são usadas
  • talvez usaria um op > 5 no lugar de op >= 6
  • mudaria o op==0 para op <= 0 e mudaria a mensagem para "Os valores devem ser entre 1 e 5."

Tem outras perfumarias que podem ser feitas, mas isto é o mais importante para um caso assim.

Se lembrar de mais alguma coisa, eu edito.

adicionou 45 caracteres ao conteúdo
Fonte Link
Maniero
  • 484,4mil
  • 94
  • 1,3mil
  • 2,2mil

Ia só comentar mas ficou grande.

Essencialmente não, afinal ele faz muito pouca coisa.

A não ser que você queira aplicar uma série de coisas que se costuma recomendar em termos de arquitetura, mas seria um absurdo fazer isto em um exemplo tão simples. Tudo é contexto, e neste contexto fazer o melhor não é o mais importante. Eu só recomendaria não cometer erros absurdos. Neste contexto de um código de aprendizado é mais importante cuidar da legibilidade, do estilo de codificação, do que da boa arquitetura, performance, etc.

Claro que tem pequenos detalhes que podem ser mudados, mas provavelmente mais por gosto do que por necessidade.

  • Eu padronizaria os espaçamento entre operadores
  • eventualmente daria nomes melhores para as variáveis
  • cuidaria para garantir o fechamento do arquivo mesmo se der erro, que sequer está sendo feito em condições normais
  • Nãofaria uma verificação da entrada de dados
  • mudaria pelo menos a escrita na tela de criação do arquivo para o local onde efetivamente cria o arquivo
  • não atribuiria valores para a mesma variável sem necessidade (op = 0)
  • declararia as variáveis mais próximas de onde são usadas
  • talvez usaria um op > 5 no lugar de op >= 6
  • mudaria o op==0 para op <= 0 e mudaria a mensagem para "Os valores devem ser entre 1 e 5."

tem outras perfumarias que podem ser feitas, mas

Se lembrar de mais alguma coisa, eu edito.

Ia só comentar mas ficou grande.

Essencialmente não, afinal ele faz muito pouca coisa.

A não ser que você queira aplicar uma série de coisas que se costuma recomendar em termos de arquitetura, mas seria um absurdo fazer isto em um exemplo tão simples.

Claro que tem pequenos detalhes que podem ser mudados, mas provavelmente mais por gosto do que por necessidade.

  • Eu padronizaria os espaçamento entre operadores
  • eventualmente daria nomes melhores para as variáveis
  • cuidaria para garantir o fechamento do arquivo mesmo se der erro, que sequer está sendo feito em condições normais
  • Não atribuiria valores para a mesma variável sem necessidade (op = 0)
  • declararia as variáveis mais próximas de onde são usadas
  • talvez usaria um op > 5 no lugar de op >= 6
  • mudaria o op==0 para op <= 0 e mudaria a mensagem para "Os valores devem ser entre 1 e 5."

tem outras perfumarias que podem ser feitas, mas

Se lembrar de mais alguma coisa, eu edito.

Ia só comentar mas ficou grande.

Essencialmente não, afinal ele faz muito pouca coisa.

A não ser que você queira aplicar uma série de coisas que se costuma recomendar em termos de arquitetura, mas seria um absurdo fazer isto em um exemplo tão simples. Tudo é contexto, e neste contexto fazer o melhor não é o mais importante. Eu só recomendaria não cometer erros absurdos. Neste contexto de um código de aprendizado é mais importante cuidar da legibilidade, do estilo de codificação, do que da boa arquitetura, performance, etc.

Claro que tem pequenos detalhes que podem ser mudados, mas provavelmente mais por gosto do que por necessidade.

  • Eu padronizaria os espaçamento entre operadores
  • eventualmente daria nomes melhores para as variáveis
  • cuidaria para garantir o fechamento do arquivo mesmo se der erro, que sequer está sendo feito em condições normais
  • faria uma verificação da entrada de dados
  • mudaria pelo menos a escrita na tela de criação do arquivo para o local onde efetivamente cria o arquivo
  • não atribuiria valores para a mesma variável sem necessidade (op = 0)
  • declararia as variáveis mais próximas de onde são usadas
  • talvez usaria um op > 5 no lugar de op >= 6
  • mudaria o op==0 para op <= 0 e mudaria a mensagem para "Os valores devem ser entre 1 e 5."

tem outras perfumarias que podem ser feitas, mas

Se lembrar de mais alguma coisa, eu edito.

adicionou 152 caracteres ao conteúdo
Fonte Link
Maniero
  • 484,4mil
  • 94
  • 1,3mil
  • 2,2mil
Carregando
Fonte Link
Maniero
  • 484,4mil
  • 94
  • 1,3mil
  • 2,2mil
Carregando