Welcome to the Treehouse Community

Want to collaborate on code errors? Have bugs you need feedback on? Looking for an extra set of eyes on your latest project? Get support with fellow developers, designers, and programmers of all backgrounds and skill levels here with the Treehouse Community! While you're at it, check out some resources Treehouse students have shared here.

Looking to learn something new?

Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and join thousands of Treehouse students and alumni in the community today.

Start your free trial

C# C# Objects Encapsulation and Arrays Ternary If

Posted by frobe
frobe
1,368 Points

Code seems good to me, works in Repl, but "recheck work" disagrees with the feedback "Did you include the boolean expres

Code seems good to me, works in Repl, but "recheck work" disagrees with the feedback "Did you include the boolean expression to evaluate?"

int value = -1; string textColor = null; (value <0) ? textColor = "red" : textColor = "green";

// (value < 0) or value < 0 makes no difference. So yes, the boolean expression is there, before the question mark.

CodeChallenge.cs 
int value = -1;
string textColor = null;
(value < 0) ? textColor = "red" : textColor = "green";

2 Answers

Mark Kastelic
Mark Kastelic
8,147 Points

It looks like you need to write as:

textColor = (value < 0) ? "red" : "green";
frobe
frobe
1,368 Points
frobe
frobe
1,368 Points

You are right Mark; that did pass the test. Assigning a value IN a 'ternary if' is also less readable then assigning the value OF a 'ternary if' to a variable. Less error prone too. Good solution.

If that automatically means that this is 'ugly':

(value < 0) ? textColor1 =  "red" : textColor2 = "green";

and should be this:

if (value < 0)
  {
  textColor1 =  "red";
  }
else 
  {
  textColor2 =  "green";
  }

I do not know.