Valider une valeur dans la propriété

J'ai donc entendu que la validation d'une valeur à une propriété comme ceci:

//dummy example, let's assume that I want my value without dots
public string MyProp
{
    set
    {
        if(value.Contains('.'))
            throw new ArgumentException("Must not contain '.'", "value");
    }
}

est faux et je doit l'éviter.

Cependant, dans les premiers jours on m'a dit que c'est la bonne façon. Nous pourrions utiliser l'encapsulation, il est juste un endroit à vérifier, SEC, etc.

Quel est le problème avec mon petit exemple?

rien de mal avec elle, mais a) je préfère jeter un ArgumentException, b) vous avez oublié de définir réellement la valeur!
Vous ne pouvez pas simplement jeter une chaîne de caractères. Vous avez besoin de lancer une exception.
double possible de C# ajouter la validation sur une méthode de définition
pas doublon. Dans cette question, ils ne discutent pas des exceptions
C'est sur la bonne voie. Plusieurs parties de l' .NET framework faire la même chose.

OriginalL'auteur ramires.cabral | 2013-04-08

  1. 11

    Il n'y a rien de mal à lever des exceptions dans le setter de la propriété. Mais vous devriez jeter un ArgumentException, et aussi la valeur de la propriété!

    private string _myprop;
public string MyProp{
    set{
       if(value.Contains('.')) throw new ArgumentException("Must not contain .");
       this._myprop=value;
    }
    get { return this._myprop; }
}

    À partir d'un l'article sur les meilleures pratiques dans MSDN:

    Accesseurs de propriété doivent être simples opérations sans conditions préalables. Si un getter peut lever une exception, envisager de modifier la propriété d'être une méthode. Cette recommandation ne s'applique pas aux indexeurs. Les indexeurs peuvent lever des exceptions en raison des arguments non valides.

    Il est valable et acceptable pour lancer des exceptions à partir d'une propriété setter.

    Merci! En fait, tout était simple exemple pour la démonstration.

    OriginalL'auteur Julián Urbano

  2. 0

    Voici une bonne vue d'ensemble à partir d'un déjà demandé de Débordement de Pile question:

    C# ajouter la validation sur une méthode de définition

    OriginalL'auteur Cortright

  3. 0

    Il y a des questions similaires aussi sur.

    Vous properities devrait être un léger possible. Et si setters déclenche une erreur c'est ok, mais encore une fois, vous pourriez envisager de passer à une fonction. Les choses peuvent déraper facilement.

    D'ÉVITER de jeter des exceptions de la propriété des getters. Bien que les accesseurs
    doivent être des opérations simples et ne devraient pas avoir de conditions préalables. Si un
    getter peut lancer une exception, il devrait probablement être repensé pour être
    une méthode.

    Meilleures pratiques: le fait de lancer des exceptions de propriétés

    Quel exception à générer à partir d'une propriété setter?

    OriginalL'auteur adt

  4. -2

    Voir: Meilleures pratiques: le fait de lancer des exceptions de propriétés pour une explication et discussion sur les raisons de lever des exceptions de propriétés est mauvais.

    Il est vrai que ce post parle bien des getters.

    Setters sont généralement considérés par les consommateurs comme étant simplement la fixation d'un champ privé caché par la propriété et peut-être faire quelques autres choses comme nécessaires, les exceptions sont des comportements inattendus et pourriez-vous imaginer avoir à joindre chaque instruction set à l'intérieur d'un bloc try?

    Alors qu'il peut être un comportement accepté par l'orientation, c'est un cauchemar pour les devinettes pour les développeurs, et une logique de validation doit être contenue dans une autre méthode, puis appelé à partir de la propriété en cas de besoin.

    Si vous avez besoin de la validation ou des comportements particuliers lors de la définition de propriétés, vous devez utiliser une méthode de jeu par exemple SetMyProp(string value) comme il apporte une distinction de ce que cela peut conduire à une exception etc.

    Si vous êtes en utilisant les propriétés de quelque chose comme un modèle WPF, alors vous devriez faire usage de WPF est intégré dans la validation de données.

    -1 la très article vous lien vers les états que les exceptions dans setters sont ok. À partir de MSDN: "Il est valable et acceptable pour lancer des exceptions à partir d'un accesseur de propriété."
    Voyez-vous un problème avec un setter de faire de la validation (et lancer une exception) à l'application des règles du contrat (par exemple en ne permettant pas une valeur null)?
    oui, je pense que pour les valeurs null, c'est acceptable ou pour les cas limites où la valeur de vraiment ne devez pas prendre sur une certaine forme, mais si c'est une valeur qui est "mauvais" dans le contexte de la poursuite de l'utilisation, les exceptions doivent avoir lieu quand ils sont tenté d'être utilisé.
    Pourquoi suppose que quand vous avez des docstrings? Et votre point d'avoir un try bloc autour de chaque instruction set est apparenté à l'aide d'un try bloc autour de chaque Substring parce que votre indice de départ est peut-être hors des limites du terrain. Il suffit de vérifier les données.
    Je suis en désaccord. Il est parfaitement acceptable pour les poseurs de lever des exceptions. Sinon, comment allez-vous gérer par exemple une propriété qui n'est pas autorisé à être nulle? Si vous supprimez le setter, la sérialisation d'échouer. La capacité à faire Contract.Requires(value != null); est essentiel.

    OriginalL'auteur Clint