Imbriquée ternaire opérateurs

J'ai ce code:

_viewModel.PhoneDefault = user == null ? "" :
    (string.IsNullOrEmpty(user.PhoneDay) ?
        (string.IsNullOrEmpty(user.PhoneEvening) ?
            (string.IsNullOrEmpty(user.Mobile) ? "" : 
                user.Mobile) :
            user.PhoneEvening) :
         user.PhoneDay);

Est-il une meilleure façon d'écrire cela pour le rendre plus lisible?

  • En évitant imbriquée ternaires va certainement rendre le code plus lisible
  • Avez-vous considéré l'utilisation de "switch" ?
InformationsquelleAutor KaptajnKold | 2011-08-18
  1. 10

    Dans votre cas, vous pouvez écrire une fonction d'aide, comme ceci:

    //return the first parameter that isn't null or empty
public static string CoalesceStrings(params string[] src)
{
    foreach (var str in src)
        if (!string.IsNullOrEmpty(str))
            return str;
    return "";
}

    Puis il suffit d'appeler, comme ceci:

    _viewModel.PhoneDefault = user == null ? "" :
    CoalesceStrings(user.PhoneDay, user.PhoneEvening, user.Mobile);
    InformationsquelleAutor Gabe
  2. 6

    Écrire une autre méthode pour obtenir le numéro de téléphone, quelque chose comme ceci:

    public string GetDefaultPhone(User user)
        {
            if(user == null)
            {
                return string.Empty;
            }

            if(!string.IsNullOrEmpty(user.PhoneDay))
            {
                return user.PhoneDay;
            }

            if(!string.IsNullOrEmpty(user.PhoneEvening))
            {
                return user.PhoneEvening;
            }

            if(!string.IsNullOrEmpty(user.Mobile))
            {
                return user.Mobile;
            }

            return string.Empty;
        }

    Et puis dans votre code:

    _viewModel.PhoneDefault = GetDefaultPhone(user);
    InformationsquelleAutor Jason Evans
  3. 3

    Bien que j'aime Gabe et Vladimir réponses les plus. Voici une façon de faire de imbriquée ternaires look déchiffrable.

    _viewModel.PhoneDefault =
  user == null
    ? ""
  : (string.IsNullOrEmpty(user.PhoneDay)
    ? (string.IsNullOrEmpty(user.PhoneEvening)
      ? (string.IsNullOrEmpty(user.Mobile)
           ? ""
         : user.Mobile)
      : user.PhoneEvening)
    : user.PhoneDay);

    Accordée, puisque vous avez à les entourer avec des parenthèses, il devient plus difficile de le faire. En JavaScript, il s'agit en réalité très élégant car il
    permet imbriquée ternaires sans les parenthèses. Bien que votre situation n'a pas vraiment besoin de cette syntaxe parfois, cette syntaxe est très pratique.

    InformationsquelleAutor Jon49
  4. 2

    Maintenant, avec la puissance de LINQ, vous pouvez faire simple et lisible dans une seule ligne comme

    _viewModel.PhoneDefault = user == null ? "" :
(new [] {user.PhoneDay, user.PhoneEvening, user.Mobile}).FirstOrDefault(s => !string.IsNullOrEmpty(s)) ?? "";
    InformationsquelleAutor Dominik Szymański
  5. 2

    Juste de répondre parce que je n'ai pas vu l'approche que j'utilise dans ce genre de scénarios dans aucun des réponses.

    Ce sujet...

    _viewModel.PhoneDefault = user == null ? string.Empty :
    !string.IsNullOrEmpty(user.PhoneDay)     ? user.PhoneDay :
    !string.IsNullOrEmpty(user.PhoneEvening) ? user.PhoneEvening :
    !string.IsNullOrEmpty(user.Mobile)       ? user.Mobile : 
    string.Empty;

    C'est gênant que vous avez à vérifier par l'utilisateur nullness au début parce que les forces de répéter string.Empty, mais néanmoins, je pense que c'est assez lisible.

    Mes deux cents 😉

    InformationsquelleAutor LucasMetal
  6. 1

    Bien, aussi longtemps que les entrées peuvent être juste la valeur null ou valide:

    if (user == null)
{
    _viewModel.PhoneDefault = String.Empty;
}
else
{
    _viewModel.PhoneDefault = user.PhoneDay ?? user.PhoneEvening ?? user.Mobile ?? String.Empty;
}

    Vous pouvez même définir une méthode d'extension sur la chaîne à convertir des chaînes vides à null et l'utiliser ici, mais je suis tout simplement fou.

    Si elles peuvent être une chaîne vide, il suffit d'utiliser les autres personnes de réponses. Jason méthode est agréable et propres.

    InformationsquelleAutor Vladimir
  7. 0

    Ajouter un peu plus parenthèse pourrait le rendre plus lisible. Vous pouvez également le décomposer en une série de if/else

    if (user != null)
{
  if (!string.IsNullOrEmpty(user.PhoneDay))
    _viewModel.PhoneDefault = user.PhoneDay;
  else if (!string.IsNullOrEmpty(user.PhoneEvening))
    _viewModel.PhoneDefault = user.PhoneEvening;
  else if (!string.IsNullOrEmpty(user.Mobile))
    _viewModel.PhoneDefault = user.Mobile;
  else
    _viewModel.PhoneDefault = "";
}
    InformationsquelleAutor HypnoToad