Bonnes pratiques de programmation/bonnes pratiques de programmation

Question

J'essaie de développer de bonnes pratiques de programmation et aujourd'hui je travaille sur un programme qui va prendre des informations à partir de zones de texte, boutons radio et cases à cocher. Je sais que je pourrais le faire comme ceci, par exemple:

TextBoxResult.Text = Val(TextBoxNumbA.Text) + Val(TextBoxNumbB.Text) 

et cela fonctionnera. Ou je pourrais aussi le faire comme cela et il fonctionnera aussi bien:

Dim result, numberA, numberB as Integer 
numberA = Val(TextBoxNumbA.Text) 
numberB = Val(TextBoxNumbB.Text) 
result = numberA + numberB 
TextBoxResult.Text = result 

Ma question est: qui est la bonne façon de le faire? Le chemin court sans déclarer les variables ou le long chemin déclarant les variables? Comment les sociétés de logiciels gèrent-elles cela?

Answer 1

Ignorant la partie val horrible. Devriez-vous utiliser des variables intermédiaires? Oui si cela rend le code plus compréhensible. Dans le cas où vous avez posté je ne le crois pas, mais c'est seulement mon opinion.

Étant donné que vous utilisez VB.net, ne vous inquiétez pas de la quantité de mémoire supplémentaire requise, le compilateur s'occupera de cela. Dans un langage interprété, cela pourrait être un problème, mais le code n'est pas écrit pour que les ordinateurs «comprennent», c'est écrit pour les programmeurs. Toujours aller pour d'abord comprendre. Vous pouvez le rendre incompréhensible une fois que vous êtes sûr qu'il est correct ...

Answer 2

Val est une fonction héritée restante. Je recommanderais de passer à des convertisseurs spécifiques. Cela fournit également une protection pour les valeurs qui ne sont pas convertibles comme des lettres. N'oubliez pas de convertir les types. Tournez Option Strict On et vous verrez.

Integer.TryParse

Dim result As integer = 0 
Dim numberA, numberB as Integer 
If Integer.TryParse(TextBoxNumbA.Text, numberA) Then 
    If Integer.TryParse(TextBoxNumbB.Text, numberB) Then 
    result = numberA + numberB 
    End If 
End If 
TextBoxResult.Text = result.ToString("n2")