Plus court chemin pour vérifier la valeur null et attribuer une autre valeur si ce n'

Je suis en tirant varchar des valeurs d'une base de données et que vous voulez définir la string je suis en les assignant à que "" si elles sont null. Je suis actuellement en train de faire comme ceci:

if (string.IsNullOrEmpty(planRec.approved_by) == true)
  this.approved_by = "";
else
  this.approved_by = planRec.approved_by.toString();

Il semble que il doit y avoir un moyen de le faire dans une seule ligne quelque chose comme:

this.approved_by = "" || planRec.approved_by.toString();

Cependant je ne peux pas trouver un meilleur moyen pour ce faire. Est-il une meilleure façon ou est-ce que j'ai la meilleure façon de le faire?

  • L' == true est superflu ici...
InformationsquelleAutor Splashlin | 2009-07-13
  1. 84

    Essayez ceci:

    this.approved_by = IsNullOrEmpty(planRec.approved_by) ? "" : planRec.approved_by.toString();

    Vous pouvez également utiliser le null-coalescence de l'opérateur comme les autres l'ont dit - puisque personne n'a donné un exemple qui fonctionne avec votre code ici est un:

    this.approved_by = planRec.approved_by ?? planRec.approved_by.toString();

    Mais cet exemple ne fonctionne que depuis une valeur possible pour this.approved_by est la même que l'une des valeurs possibles que vous souhaitez le configurer. Pour tous les autres cas, vous aurez besoin d'utiliser l'opérateur conditionnel comme je l'ai montré dans mon premier exemple.

    • N'est-ce pas null coalescence opérateur de jeter un nullreference de toute façon? La façon dont je vois ce code est le suivant: if (planRec.approved_by == null) { this.approved_by = planRec.approved_by.toString(); //<= nullref } else { this.approved_by = planRec.approved_by;}. Si je me trompe s'il vous plaît signaler l'erreur.
    • bon, le code est cassé. Voici un correctif à l'aide d'un autre plaisir de l'opérateur: this.approved_by = planRec.approved_by?.toString() ?? ""; <- C'est le null-opérateur conditionnel, et je ne pense pas que c'était autour de quand cette question a d'abord été posées et répondues. Espérons OP voit, donc il peut aller refactoriser ses 7 ans de code, de deux emplois: il y a) .
    InformationsquelleAutor Andrew Hare
  2. 29

    La fusionner opérateur (??) est ce que vous voulez, je crois.

    • Yep...mais seulement parce que la valeur par défaut est "".
    InformationsquelleAutor Dave Ward
  3. 25

    Ma conjecture est le meilleur que vous pouvez venir avec est

    this.approved_by = IsNullOrEmpty(planRec.approved_by) ? string.Empty
                                                      : planRec.approved_by.ToString();

    Bien sûr, puisque vous êtes allusion au fait que approved_by est un object (qui ne peut pas égal à ""), ce serait réécrit comme

    this.approved_by = (planRec.approved_by ?? string.Empty).ToString();
    InformationsquelleAutor Dmitri Nesteruk
  4. 21

    Vous êtes à la recherche pour le C# fusionnent opérateur: ??. Cet opérateur prend à gauche et à droite argument. Si le côté gauche de l'opérateur est null ou une nullable sans valeur, il sera de retour le bon argument. Sinon, ce sera le retour de la gauche. 

    var x = somePossiblyNullValue ?? valueIfNull;
    InformationsquelleAutor JaredPar
  5. 13

    Avec C#6 est légèrement plus courte pour le cas où planRec.approved_by n'est pas une chaîne de caractères:

    this.approved_by = planRec.approved_by?.ToString() ?? "";
    InformationsquelleAutor Malcolm
  6. 10

    Utiliser le C# fusionnent opérateur: ??

    //if Value is not null, newValue = Value else if Value is null newValue is YournullValue
var newValue = Value ?? YourNullReplacement;
    InformationsquelleAutor Ramgy Borja
  7. 9

    D'étendre @Dave répondre...si planRec.approved_by est déjà une chaîne

    this.approved_by = planRec.approved_by ?? "";
    InformationsquelleAutor Paul Alexander
  8. 2

    Pour attribuer un non-variable vide, sans répéter le nom de variable (et sans affecter n'importe quoi, si la variable est null!), vous pouvez utiliser un peu de méthode d'assistance avec un Action paramètre:

    public static void CallIfNonEmpty(string value, Action<string> action)
{
    if (!string.IsNullOrEmpty(value))
        action(value);
}

    Et puis il suffit de l'utiliser:

    CallIfNonEmpty(this.approved_by, (s) => planRec.approved_by = s);
    InformationsquelleAutor Jack Miller
  9. 1

    Vous pouvez aussi le faire dans votre requête, par exemple dans sql server, google ISNULL et CASE fonctions intégrées.

    InformationsquelleAutor user133371
  10. -4

    - Je utiliser l'extension de la méthode SelfChk 

    static class MyExt {
//Self Check 
 public static void SC(this string you,ref string me)
    {
        me = me ?? you;
    }
}

    Ensuite utiliser comme 

    string a = null;
"A".SC(ref a);
    • Très court, mais très facile à lire et à comprendre.
    • Je suppose que tu veux dire mais pas facile à lire et à comprendre, est-ce vrai?
    • Ups! Vous avez absolument raison: Très court, mais PAS facile à lire et à comprendre (du moins si vous n'êtes pas habitué à cette notation)
    • bien dans ce cas, essayez de lire un livre. Vous l'avais trouvé vraiment élaborative et facile à comprendre :p
    InformationsquelleAutor Ali Humayun