Quand vous êtes le nouvel arrivant et que vous continuez à voir des choses bêtes - les retravaillez-vous ?

Question

Refactorez-vous lorsque vous voyez des choses comme ça? Ou vous vous contentez de boucher votre nez et de passer à autre chose?

    public Collection GetFieldValidationRules(String key)
    {
        Collection found = null;
        try
        {
            this.mRules.TryGetValue(key, out found);
        }
        catch (ArgumentException ex)
        {
            //log the error
            Log.Error(ExceptionHandling.BuildExceptionMessage(ex));
            return null;
        }
        return found;
    }

Answer 1

(Presque) Toute entreprise qui a un effectif plus petit que ce que son charge de travail nécessite aura une dette technique ou un code désordonné. Malheureusement, (presque) chaque entreprise est comme ça.

À moins que vous ne soyez dans une organisation agile ou dans un projet qui a commencé avec peu de dette technique et l'a gardé de cette façon, lutter contre cela est presque impossible. Sous pression du temps, nous écrivons tous un code de cette façon. En fait, même Oncle Bob écrit un code moche avant d'avoir le temps de le refondre avant de le publier dans un livre.

Si vous commencez à refactoriser les choses, vous courez le risque de casser quelque chose. Si votre organisation n'a pas de tests unitaires complets, ce n'est pas un risque que vous devriez prendre.

Nettoyer la dette technique est une décision de toute l'organisation ou au moins du projet. Malheureusement, cela ne commence pas avec le nouveau venu.

Answer 2

Que fait TryGetValue?

doit-on toujours suivre les erreurs? (la journalisation)

il y a beaucoup de choses qui pourraient être valides avec cette chose

Answer 3

Vous êtes un peu vague sur votre définition d'idiocy dans le code.

Si c'est parce que TryGetValue ne lance pas une ArgumentException mais lance plutôt une ArgumentNullException, vous devriez être en sécurité en le corrigeant.

Méthode TryGetValue du dictionnaire MSDN