Jak zrobić ToString dla potencjalnie null obiektu?

Czy istnieje prosty sposób na wykonanie następujących czynności:

String s = myObj == null ? "" : myObj.ToString();

Wiem, że mogę wykonać następujące czynności, ale naprawdę uważam to za hack:

String s = "" + myObj;

Byłoby wspaniale, gdyby Convert.ToString () miał do tego odpowiednie przeciążenie.

C # 6.0 Edycja:

Dzięki C # 6.0 możemy teraz mieć zwięzłą, wolną od rzutowania wersję oryginalnej metody:

string s = myObj?.ToString() ?? "";

Lub nawet używając interpolacji:

string s = $"{myObj}";

Oryginalna odpowiedź:

string s = (myObj ?? String.Empty).ToString();

lub

string s = (myObjc ?? "").ToString()

by być jeszcze bardziej zwięzłym.

Niestety, jak już wskazano, często będziesz potrzebować rzutowania po obu stronach, aby działało to z typami innymi niż String lub Object:

string s = (myObjc ?? (Object)"").ToString()
string s = ((Object)myObjc ?? "").ToString()

Dlatego, chociaż może się wydawać elegancka, obsada jest prawie zawsze konieczna i nie jest tak zwięzła w praktyce.

Jak zasugerowano w innym miejscu, polecam użycie metody rozszerzenia, aby uczynić to czystszym:

public static string ToStringNullSafe(this object value)
{
    return (value ?? string.Empty).ToString();
}

string.Format("{0}", myObj);

string.Format sformatuje null jako pusty ciąg i wywoła ToString () na obiektach innych niż null. Jak rozumiem, właśnie tego szukałeś.

Byłoby wspaniale, gdyby Convert.ToString () miał do tego odpowiednie przeciążenie.

Od tamtej Convert.ToString(Object value)pory .Net 2.0 (około 5 lat przed zapytaniem o to pytanie) wydaje się robić dokładnie to, czego chcesz:

http://msdn.microsoft.com/en-us/library/astxcyeh(v=vs.80).aspx

Czy brakuje mi / źle interpretuję coś naprawdę oczywistego?

Za pomocą metody rozszerzenia można to osiągnąć:

public static class Extension
{
    public static string ToStringOrEmpty(this Object value)
    {
        return value == null ? "" : value.ToString();
    }
}

Poniższe nie zapisałyby nic na ekranie i nie zgłosiłyby wyjątku:

        string value = null;

        Console.WriteLine(value.ToStringOrEmpty());

Nie zgadzam się z tym:

String s = myObj == null ? "" : myObj.ToString();

jest w jakikolwiek sposób hackerem. Myślę, że to dobry przykład przejrzystego kodu. Jest absolutnie oczywiste, co chcesz osiągnąć i że oczekujesz zerowej wartości.

AKTUALIZACJA:

Teraz widzę, że nie powiedziałeś, że to był hack. Ale z pytania wynika, że ​​myślisz, że ten sposób nie jest właściwy. Moim zdaniem to zdecydowanie najwyraźniejsze rozwiązanie.

string s = String.Concat(myObj);

byłby najkrótszą drogą, jak sądzę, i miałby również znikomy narzut wydajności. Pamiętaj jednak, że dla czytelnika kodu nie byłoby całkiem jasne, jaki jest zamiar.

właściwie nie rozumiałem, co chcesz robić. Jak rozumiem, możesz napisać ten kod w inny sposób. Pytasz o to czy nie? Czy możesz to bardziej wyjaśnić?

string s = string.Empty;
    if(!string.IsNullOrEmpty(myObj))
    {
    s = myObj.ToString();
    }

Mogę zostać pobity za moją odpowiedź, ale tak czy inaczej:

Po prostu napisałbym

string s = "" if (myObj! = null) {x = myObj.toString (); }

Czy korzystanie z operatora trójskładnikowego się opłaca? Nie wiem z góry mojej głowy.

I oczywiście, jak wspomniano powyżej, możesz umieścić to zachowanie w metodzie takiej jak safeString (myObj), która pozwala na ponowne użycie.

Miałem ten sam problem i rozwiązałem go po prostu rzucając obiekt na łańcuch. Działa to również w przypadku obiektów o wartości null, ponieważ ciągi znaków mogą mieć wartość null. O ile absolutnie nie chcesz mieć łańcucha zerowego, powinno to działać dobrze:

string myStr = (string)myObj; // string in a object disguise or a null

Niektóre (szybkie) testy wydajności podsumowujące różne opcje, ale nie to, że to naprawdę ma znaczenie # mikrooptymalizacja (przy użyciu rozszerzenia linqpad )

Opcje

void Main()
{
    object objValue = null;
    test(objValue);
    string strValue = null;
    test(strValue);
}

// Define other methods and classes here
void test(string value) {
    new Perf<string> {
        { "coallesce", n => (value ?? string.Empty).ToString() },
        { "nullcheck", n => value == null ? string.Empty : value.ToString() },
        { "str.Format", n => string.Format("{0}", value) },
        { "str.Concat", n => string.Concat(value) },
        { "string +", n => "" + value },
        { "Convert", n => Convert.ToString(value) },
    }.Vs();
}

void test(object value) {
    new Perf<string> {
        { "coallesce", n => (value ?? string.Empty).ToString() },
        { "nullcheck", n => value == null ? string.Empty : value.ToString() },
        { "str.Format", n => string.Format("{0}", value) },
        { "str.Concat", n => string.Concat(value) },
        { "string +", n => "" + value },
        { "Convert", n => Convert.ToString(value) },
    }.Vs();
}

Prawdopodobnie ważne jest, aby wskazać, że Convert.ToString(...)zachowa ciąg pusty.

Wyniki

Obiekt

• nullcheck 1,00x 1221 taktów minęło (0,1221 ms) [przy 10 tys. powtórzeń, 1,221E-05 ms na]
• coallesce 1,14x 1387 taktów minęło (0,1387 ms) [przy 10 tys. powtórzeń, 1,387E-05 ms na]
• ciąg + 1,16x 1415 taktów minęło (0,1415 ms) [przy 10 tys. powtórzeń, 1,415E-05 ms na]
• str.Concat 1,16x 1420 taktów minęło (0,142 ms) [przy 10 tys. powtórzeń, 1,42E-05 ms na]
• Konwersja 1,58x 1931 taktów, które upłynęły (0,1931 ms) [w 10 tys. Powtórzeń, 1,931E-05 ms na]
• Str. format 5,45x 6655 taktów minęło (0,6655 ms) [przy 10 tys. powtórzeń, 6,655E-05 ms na]

Strunowy

• nullcheck 1,00x 1190 taktów minęło (0,119 ms) [przy 10 tys. powtórzeń, 1,19E-05 ms na]
• Konwertuj 1,01x 1200 taktów, które upłynęły (0,12 ms) [przy 10 tys. Powtórzeń, 1,2 E-05 ms na]
• ciąg + 1,04x 1239 taktów minęło (0,1239 ms) [przy 10 tys. powtórzeń, 1,239E-05 ms na]
• coallesce 1,20x 1423 taktów minęło (0,1423 ms) [przy 10 tys. powtórzeń, 1,423E-05 ms na]
• str.Concat 4,57x 5444 taktów minęło (0,5444 ms) [przy 10 tys. powtórzeń, 5,444E-05 ms na]
• Str. format 5,67x 6750 taktów minęło (0,675 ms) [przy 10 tys. powtórzeń, 6,75E-05 ms na]

Komentarz Holstebroe byłby najlepszą odpowiedzią:

string s = string.Format("{0}", myObj);

Jeśli myObjjest null, Format umieszcza tam wartość pustego ciągu.

Spełnia również wymagania dotyczące jednej linii i jest łatwy do odczytania.

Mimo że jest to stare pytanie, a OP zapytany o C #, chciałbym udostępnić rozwiązanie VB.Net dla tych, którzy pracują z VB.Net zamiast C #:

Dim myObj As Object = Nothing
Dim s As String = If(myObj, "").ToString()

myObj = 42
s = If(myObj, "").ToString()

Niestety VB.Net nie pozwala na? -Operator po zmiennej, więc myObj? .ToString nie jest poprawne (przynajmniej nie w .Net 4.5, którego użyłem do testowania rozwiązania). Zamiast tego używam If, aby zwrócić pusty ciąg w przypadku, gdy myObj ist Nothing. Zatem pierwszy Tostring-Call zwraca pusty ciąg, podczas gdy drugi (gdzie myObj nie jest Nothing) zwraca „42”.