Remix.run Logo
paulmooreparks 4 days ago

Sorry, I would flag this in a code review. It's too easy to skip past this visually. Explicit if statements make it a lot more obvious what's going on. This is too much syntactic sugar.

jobigoud 4 days ago | parent [-]

You must not do a lot of C# code reviews? This syntax is used in every codebase. In fact if you don't use it Visual Studio will underline it as "code can be simplified" suggestion.

You've never seen something like this?

    MyEvent?.Invoke(sender, arg);
paulmooreparks 4 days ago | parent [-]

I work almost exclusively in C#, and I do a lot of reviews. I'm familiar with the syntax, and I use it. It's the assignment version that I'm talking about here.