The thing I noticed for now are only the short name (r, g, enumeration) these should be changed to the meaningfull names.
I've done the change, imo I feel short-range variables much more readable when short-written :)
Ex: records.Select(r => r.Id); we know that r is record we don't have to go far back in the code to understand what the variable means and the action of the lambda is clearer we directly see that we're extracting Id field. But opinion 👍
KYPremco
commented
10 months ago
I've done the change, imo I feel short-range variables much more readable when short-written :)
Ex:
records.Select(r => r.Id);we know thatrisrecordwe don't have to go far back in the code to understand what the variable means and the action of the lambda is clearer we directly see that we're extractingIdfield. But opinion 👍