103

I am pulling varchar values out of a DB and want to set the string I am assigning them to as "" if they are null. I'm currently doing it like this:

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

There seems like there should be a way to do this in a single line something like:

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

However I can't find an optimal way to do this. Is there a better way or is what I have the best way to do it?

Alexander Abakumov
  • 12,301
  • 14
  • 79
  • 125
Splashlin
  • 6,705
  • 12
  • 45
  • 50

12 Answers12

127

Try this:

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

You can also use the null-coalescing operator as other have said - since no one has given an example that works with your code here is one:

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

But this example only works since a possible value for this.approved_by is the same as one of the potential values that you wish to set it to. For all other cases you will need to use the conditional operator as I showed in my first example.

Andrew Hare
  • 333,516
  • 69
  • 632
  • 626
  • 10
    Doesn't this null coalescing operator throw a nullreference anyways? The way I see this code does the following: `if (planRec.approved_by == null) { this.approved_by = planRec.approved_by.toString(); //<= nullref } else { this.approved_by = planRec.approved_by;}`. If I'm wrong please point out the error. – Destrictor Feb 19 '13 at 08:18
  • 13
    @Destrictor is right, the code is broken. Here's a fix using another fun operator: `this.approved_by = planRec.approved_by?.toString() ?? "";` – Patrick Jul 20 '16 at 05:56
  • I could swear I've seen some kind of syntax that lets you assign to an attribute of an object if that object isn't null, like someObject?.someAttribute = value; but I can't seem to find anything like that. – Shavais Oct 20 '21 at 22:28
46

You are looking for the C# coalesce operator: ??. This operator takes a left and right argument. If the left hand side of the operator is null or a nullable with no value it will return the right argument. Otherwise it will return the left.

var x = somePossiblyNullValue ?? valueIfNull;
JaredPar
  • 703,665
  • 143
  • 1,211
  • 1,438
  • This answer does not cover the all cases from the question (only partially). Notice, that there is "string.IsNullOrEmpty". So Empty case is not handled here. – Bronek Jan 23 '20 at 10:43
  • upvoted because this was the answer I was looking for. (I just needed the null check) – edank Jan 26 '21 at 06:16
42

Starting with C# 8.0, you can use the ??= operator to replace the code of the form

if (variable is null)
{
    variable = expression;
}

with the following code:

variable ??= expression;

More information is here

nzrytmn
  • 4,937
  • 1
  • 37
  • 33
40

The coalesce operator (??) is what you want, I believe.

Dave Ward
  • 58,451
  • 13
  • 115
  • 134
27

My guess is the best you can come up with is

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

Of course since you're hinting at the fact that approved_by is an object (which cannot equal ""), this would be rewritten as

this.approved_by = (planRec.approved_by ?? string.Empty).ToString();
Dmitri Nesteruk
  • 21,994
  • 22
  • 93
  • 157
25

With C#6 there is a slightly shorter way for the case where planRec.approved_by is not a string:

this.approved_by = planRec.approved_by?.ToString() ?? "";
Malcolm
  • 1,223
  • 1
  • 14
  • 24
20

Use the C# coalesce operator: ??

// if Value is not null, newValue = Value else if Value is null newValue is YournullValue
var newValue = Value ?? YourNullReplacement;
Nova Ardent
  • 161
  • 16
Ramgy Borja
  • 2,064
  • 2
  • 17
  • 37
11

To extend @Dave's answer...if planRec.approved_by is already a string

this.approved_by = planRec.approved_by ?? "";
Community
  • 1
  • 1
Paul Alexander
  • 31,392
  • 14
  • 94
  • 149
2

To assign a non-empty variable without repeating the actual variable name (and without assigning anything if variable is null!), you can use a little helper method with a Action parameter:

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

And then just use it:

CallIfNonEmpty(this.approved_by, (s) => planRec.approved_by = s);
Jake Chasan
  • 5,918
  • 8
  • 40
  • 84
Jack Miller
  • 5,677
  • 2
  • 39
  • 52
2

The accepted answer was correct in time, when it was given.

For people still finding this question: Today you can use the ??= Operator.

e.g:

private string _test = null;

private void InitIfNull(){
 _test ??= "Init";
}
Steav
  • 1,469
  • 13
  • 28
1

You can also do it in your query, for instance in sql server, google ISNULL and CASE built-in functions.

Taryn
  • 234,956
  • 54
  • 359
  • 399
user133371
  • 331
  • 2
  • 6
  • 18
-4

I use extention method SelfChk

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

Then use like

string a = null;
"A".SC(ref a);
Ali Humayun
  • 1,658
  • 1
  • 15
  • 11