There are a few different issues in play here, and I don't think this is quite as clear-cut as some of the comments here suggest.
Streamlining posts by, for instance, removing initial paragraphs that don't address the matter at hand generally counts as useful copyediting. While personally, had I come across your question organically, I guess I wouldn't have found it enough of a problem to bother with editing just for that, I can see why the edit was made, and the motivation is broadly valid.
This general point aside, I think your objection #2 is justified. In your original text, the remarks about ì!
conveyed that your wish to avoid it was motivated by code neatness. That is relevant context to answer authors when evaluating which alternatives would be more appropriate in a solution, and removing it does change the meaning of the post. On a related note, there were some suggestions in the comments here that the word "ugly" had to be removed because it was inherently subjective. That feels completely off-base to me: we aren't supposed to police post content to this level of granularity.
A subtler aspect of the edit is that, in your original text, the core question was not formulated in one single passage, but spread across several paragraphs. That being so, the streamlining done by the edit had the side effect of making the remaining text feel a little disjointed, by removing the (supposedly) inessential turns of phrase that arguably acted as glue. The compromise edit I did to your question attempted to deal with that by bringing the different parts of the question together in the second paragraph -- and I now see you have further improved on it today, by integrating the end note about the generality of the question to the main body of the text.
i!
could be avoided."i
despite the fact thati
is nullable and yet it has not been checked againstnull
in the current scope. (It has been checked in a previous scope, but the compiler cannot possibly know that.)Where
clause). Well, thisi!
is not so "ugly" asgoto
in C, but the reason to avoid it is definitely objective. Most likely, that reason is well-known for everyone who programs in C#/LINQ more than a day, so epithets fori!
are not so vital. But I think that single-word "ugly" won't harm the question post.i!
construction (and MikeNakes explained me that meaning). As for "ugly" word, I find its general sense being sufficient to describe some programming language constructions as unnatural and confusing. E.g. as I noted in the previous comment, agoto
construction in C is ugly. (But sometimes usinggoto
is a lesser evil than alternatives).i!
is still necessary?i!
paragraph mentioned here) without actually clarifying its main point. (2) The suggested duplicate target (a completely general feature request) isn't a good fit for a request for comment on a specific edit.